17かけだしのフロントエンドエンジニアがもらったレビューを振り返るJavaScriptレビューフロントエンドみなさん、初めまして。playgroundのアドベントカレンダー17日目です。もらったコードレビューを振り返ってみたいなって思います。変更後hogeColor全人類の悩みの種ですね。色を指定したいときに、colorValueという命名ではなんの色か全くわかりません。色であることを誰が見てもわかるようにhogeColorにすると直感的にわかるようになります。作りたい機能をもとにコードを書くことが多いと思うので、その変数が何に使われるかを意識して命名したいですね。初心者の僕はアロー関数という超便利なものを使ってませんでした。アロー関数が2つ一気に変わっていてごっちゃになってるかもです。(僕も最初はごっちゃになってました。)なので、まずは関数リテラルから調べてみて、次にアロー関数というふうに段階を踏むとわかりやすいと思います。console.log(list);}ネストは2段を超えると読みづらくなります。ネストを浅くできるところは他の表現がないか、ちょっとだけ考えてみたらいいと思います。三項演算子を使うことで、if文が1文で完結することがあるので、三項演算子も使いたいですよね。)三項演算子は書き方次第ではコードが読みづらくなります。共通部分を一回で書くようにすると、読みやすく短いコードになると思います。extendWidth}`);});自分ではなんの数値か分かっていても、レビューする人はなんのための数値かわからないので、なんのための数値かわかるような変数名に代入したらいい感じになります。なるし、無駄なif分岐が増えるのもよくありません。上手にif分岐をして、空文の使用を避けるといいと思います。最後に案件に参加させてもらって、コードは自分がわかるだけじゃダメなんだなぁと思いました。読みやすいコード書けるように頑張ろ。inc.東京理科大学発ベンチャー。提携大学内にソフトウェア研究所を組織し、学生向けのTech教育を提供しています。upLoginユーザーは見つかりませんでした。