0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 3 years have passed since last update.

コードレビューの観点メモ

0
Last updated at Posted at 2023-04-04

※自分用メモです。随時、変更・追記してきます。

基本的に以下の書籍を参考にしています。

  • リーダブルコード
  • CleanCode

大前提

  • 理解しやすいコードであること
  • 一度に一つのことを行うこと

名前

  • 気取った名前より、意味の伝わる名前であること

コメント

  • コメントすべきではないこと
    • コードからすぐに推測できるようなこと
    • 変更内容の履歴のような日誌コメント
  • コメントすべきこと
    • 情報を与えるコメント
    • なぜそうしたのか、意図を伝えるコメント
    • 結果に対する警告コメント
    • TODOコメント

制御フロー

  • ネストは浅くすること
  • 説明変数を利用すること
  • 要約変数を利用すること

変数

  • スコープは小さいこと

関数

  • 小さいこと
  • 一つのことを行なっていること
  • 引数は少ないこと
  • フラグ引数は極力使わないこと
  • 副作用を避けること
  • DRY原則

クラス

  • 小さいこと
  • 単一責任の原則
  • 凝集性が高いこと
  • オープン/クロースドの原則
  • 依存性逆転の原則
0
0
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
0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?