LoginSignup
2
3

More than 3 years have passed since last update.

リーダブルコードまとめ

Last updated at Posted at 2021-05-06

備忘のためのメモです。

第一部 表面上の改善

第二章 名前に詰め込むべき情報

  1. 明確な単語を選ぶ

    例えば、Getではなく、状況に応じてFetchやDownloadなどを使う。

  2. 汎用的な名前をできれば避ける(tmpやretval等)

  3. 具体的な名前を使って、物語を詳細に説明する

    例えば、ServerCanStart()よりもCanListenOnPort()のほうが明確だ。

  4. 変数名に大切な情報を追加する

    単位がつけるべき場合は、例えば、ミリ秒を表す変数名には後ろにmsをつける、またこれからエスケープが必要な変数名には、前にrawをつける。

  5. スコープの大きな変数には長い名前をつける

    スコープが数画面に及ぶ変数には1~2文字の短い暗号めいた名前をつけてはいけない。短い名前はスコープが数行の変数につけるべきだ。

  6. 大文字やアンダースコアなどに意味を含める

    例えば、HTMLのidにはアンダースコア、 classにはハイフン。また、クラスのメンバー変数にはアンダースコア、ローカル変数にはハイフン。

第三章 誤解されない名前

最善の名前とは、誤解されない名前であり、他人が意図を正しく理解できるもの。例えば上下の限界値であればhmax_/min_、包括的範囲であればfirst/last、包含/排他的範囲であればbegin/endを使う。get()やsiza()には、軽量メソッドが期待されるので、あまり使わない(getが入ってる関数が重い場合、イメージと合わない)。

第四章 美しさ

一貫性と意味のあるやり方でコードを整理し、簡単にコードを読める様にする。以下は例。

  1. 複数のコードブロックで同じ処理をしているなら、シルエットも同じようなものにする。
  2. コードの列(左端とか)を整理すれば、概要が把握しやすくなる
  3. 処理などの順番は、意味のあるものにして、常にその順番を守る。
  4. 大きなブロックは空行を使って、論理的な段落に分ける。

第五章 コメントすべきこと

コメントの目的は、コードの意図を読み手に理解してもらうこと。

  1. コメントすべきではないこと
    • コードからすぐに抽出できること
    • 酷いコードを補助する補助的なコメント。この場合、補助コメントを書くのではなく、コードを修正する。
  2. コメントすべき、自分の考え
    • なぜコードが他のやり方でなく、こうなっているのか
    • コードの欠陥を下記の様なコメントで補助する
      • TODO: 後でやる
      • FIXME: 既知の不具合があるコード
      • HACK: あまりきれいじゃない解決策
      • XXX: 危険、大きな問題がある
    • 定数の値が持つ「背景」
  3. コメントすべき、読み手への配慮
    • 読み手が「?」となる部分を予測しコメントをする
    • 平均的な読み手が驚く様な動作は文章化しておく
    • ファイルやクラスには「全体像」のコメントを書く。
    • 読み手が細部に捕われないように、コードブロックに概要コメントを記載する

第六章 コメントは正確で簡潔に

小さな領域にできるだけ多くの情報を詰め込んだコメントを書くコツは下記。

  1. 複数のものを指す可能性がある「それ」や「これ」などの代名詞を避ける。
  2. 関数の動作はできるだけ正確に説明する。
  3. コメントに含める入出力の実例を慎重に選ぶ
  4. コードの意図は、詳細レベルでなく、高レベルで記述する。
  5. よく分からない引数にはインラインコメントを使う。
  6. 多くの意味が込められている言葉や表現を使って、コメントを簡潔に保つ。

第二部 ループとロジックの単純化

第七章 制御フローを読みやすく書く

制御フローを読みやすくする為に、下記を意識する。

  1. 比較の条件を書く場合は、変化する値を左に、より安定した値を右に配置する。
  2. if/else文のブロックは、適切に並び替える。一般的には、肯定形・単純・目立つものを先に処理する。この基準は衝突する場合があるが、衝突しなければ基準を守る。
  3. 三項演算子、do/whileループ、gotoなどは、コードが読みにくくなるので、なるべく使わない。
  4. 深いネスト避ける。早めにreturnしてあげる。こうすることでネストを削除したり、コードをクリーンにできる。また、「ガード説=(関数の上部で単純な条件を先に処理をする)」を意識する。

第八章 巨大な式を分割する

巨大な式を一度に理解しようとしても難しいので、小さく分解すべし。説明変数(式の値を保有する変数)を利用することが簡単な分解方法で、下記3点が説明変数の利点。

  1. 巨大な式を分解できる
  2. 簡潔な名前で式を説明することで、コードを文書化できる
  3. コードの主要な「概念」を読み手が認識しやすくなる

第九章 変数と読みやすさ

プログラムの変数はすぐに増えるので、いずれ追跡できなくなる。変数を減らして、できるだけ軽量にすればコードは読みやすくなる。具体的な削減方法は下記。

 1. 邪魔な変数を削除する
 結果をすぐに使って、「中間結果」の変数を削除する
 
 2. 変数のスコープをできるだけ小さくする
 変数を数行のコードからしか見えない位置に移動する
 
 3. 一度だけ書き込む変数を使う
 変数に一度だけ値を設定すれば、コードが理解しやすくなる(const等)

第三部 コードの再構成

第十章 無関係の下位問題を抽出する

プロジェクト固有のコードから汎用コードを分離すべし。ほとんどのコードは汎用化できる。一般的な問題を解決するライブラリやヘルパー関数を作っていけば、プログラム固有の小さな核だけが残る。こうすることで、プロジェクトの他の部分から分離された、境界線の明確な小さな問題に集中できる。

第十一章 一度に一つのことを

読みにくいコードがあれば、そこで行われている処理を列挙し、外出し可能なものは、切り離すべき。

第十二章 コードに思いを込める

書いたプログラムは簡単に言葉で説明できる様にする。実際に声に出して、説明できなければならない(できないのであれば理解していない)。

第十三章 短いコードを書く

  • 不必要な機能をプロダクトから削除し、過剰な機能を持たせない。
  • 最も簡単に問題を解決できるような要求を考える
  • 定期的に全てのAPIを読んで、標準ライブラリに慣れ親しんでおく(こういうものがあると知っておくことが非常に重要)

第四部 選抜テーマ

第十四章 テストと読みやすさ

テストコードも読みやすさが重要。テストコードの改善ポイントは下記。

  • テストのトップレベルはできるだけ簡潔に。入出力のテストはコード1行で記載できた方がいい
  • テストが失敗したらバグの発見や修正がしやすくなるようなエラーメッセージを表示する
  • テストに有効な最も単純な入力値を使う
  • テスト関数に説明的な名前をつけて、何をテストしているのかを明らかにする。
2
3
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
2
3