レビューで保守性のためにした コメントをふりかえってみた
•
0 j'aime•385 vues
2021年12月1日ラクス様主催のコードレビュー LT会でLTしたときの資料です。
Recommandé
Contenu connexe
Tendances
Tendances (20)
Similaire à レビューで保守性のためにした コメントをふりかえってみた
Similaire à レビューで保守性のためにした コメントをふりかえってみた (20)
レビューで保守性のためにした コメントをふりかえってみた
- 2. 廣川 高久 hiro(@etBeEP5e7dwmw7P) 技術スタック Laravel,vue.js,postgresql 最近取り組んでいること ・月1回 LT発表 ・テストコード(コードカバレッジ、laravel dusk導入) ・ノボロックウォーク21（ボルダリング） ・Go
- 4. 保守性がないとどうなるの？ どこになにが書いてあるのかわからない ☠ コードが古文書みたいで読むのに時間がか かる☠ 追加開発したらバグが発生した ☠ 後々コード見る開発者 (未来の自分)が地獄を見ます☠
- 7. Point1. 名前が変数・処理の内容を表している ☠チェックした結果が int？ bool? 取得したデータ? function checkActivePost(int $product_id) { return Post::where('id', $product_id)->where(‘active’, true)->exists(); } function searchByUserId(int $user_id) { return Post::where('user_id', $user_id)->get(); } ☠戻り値が array？ colletion?
- 8. Point1. 名前が変数・処理の内容を表している ☠チェックした結果が int？ bool? 取得したデータ? function checkActivePost(int $product_id) { return Post::where('id', $product_id)->where(‘active’, true)->exists(); } function searchByUserId(int $user_id) { return Post::where('user_id', $user_id)->get(); } ☠戻り値が array？ colletion?
- 9. Point1. 名前と型・処理内容の一致 function searchByUserId(int $user_id) { return Post::where('user_id', $user_id)->get(); } function getByUserId(int $product_id) { return Post::where('user_id', $user_id)->get(); } get()を使ったメソッドということがわかる！
- 10. Point1. 名前と型・処理内容の一致 function checkActivePost(int $product_id) { return Post::where('id', $product_id)->where(‘active’, true)->exists(); } function isActive(int $product_id) { return Product::where('id', $product_id)->where(‘active’, true)->exists(); } 結果がbool値であることが直感的にわかる
- 11. Point2. FW・言語が提供する機能の使用 戻り値が array？ colletion(ラッパークラス)? わからない。。。 $review = Review::where($request->product_id) ->where($request->user_id) ->find(); if (is_null($review)) { Review::create($request->validated()); } else { $review::update($request->validated()); } ユーザーがある商品のレビューをしたことがあれば更新、なければ新規登録 意味はわかるが、処理が冗長
- 12. Point2. FW・言語が提供する機能の使用 戻り値が array？ colletion(ラッパークラス)? わからない。。。 $review = Review::updateOrCreate( [ 'product_id', $request->product_id, 'user_id', $request->user_id ], [ 'title', $request->title, 'body', $request->body ] ); FWが提供するメソッドを使っているので、何をしているかが簡単にわかる
- 13. Point3. 処理が分割されている public function show(Request $request) { // 1.入力値のバリデーション $validator = Validator::make($request->all(), [ 'post_id' => 'required|integer|exists:posts.id|' ]); // 2.商品情報の取得 ＄post = Post::find($request->post_id); // 3.ユーザーのアクセス権限チェック if (! Gate::allows('show-post', $post)) { abort(403); } // 4.表示用に情報を整形 $post->stockStatus(); } いろいろな処理を1つのメソッドに集約している
- 14. Point3. 処理内容がクラスの責務で分割されている public function show(ShowRequest $request) { // 入力値のバリデーション・認証は ShowRequestで済 // 情報の整形はResourcesで行う return Post::find($request->post_id); } クラスごとで行う処理が明確になり、 機能を変更する時の見通しが立ちやすい！