コードレビュー(仕様的な点ではなくて規約的なところのチェック)などで気をつけることなど。
ポイントとしては検知ツールで確認するコストを減らすことが一番大事(カスタマイズの方法について調べておく)、あるいはコミット時に静的解析ツールが走って、ルールを守らないとコミットできないなど。
人チェックをしないのがコツだが、する場合、10以上のルールを同時にチェックすると漏れるため、メソッド単体ぐらいに切ってゆっくり見ていくほうが良いでしょう。普段からチェックし、チェックリストを作り、一気にチェックをしないことが大切だと思います。
ありがちなルール
Contents
文字のキャメル、スネークの統一、その他ルール
おそらくツールで検知できるのでこれで回避。稀にerrmsg(errMsg)のように検知できにくいものもあるので注意。
スコープの長さと変数名を対応させるなども、コードを書いていて、癖がつく部分もあるが人チェックが必要。
if文やfor文のスペースや括弧の位置
癖をつけておけば大丈夫(目が違和感を覚える)だと思うが、100%になることはなかなか難しいので、ツールでチェックできるようにしましょう。
コメントの書き方
Document作成ツールでチェックするのが一番わかりやすいが、内容のチェックまではおそらく人チェックがいると思います。
入れる場所などは完全なシステム的なチェックは難しいので人チェックで。
行の長さ、メソッドの長さ、クラスの長さチェック
普段からコードを書いて癖づけするのが一番(ここだけに限りませんが・・・)
１行あたりの場合はチェックがつくことが多い(100文字以下など)ですが、メソッド、クラスの長さなどはチェックができないことが多いでしょう。
この点は分割することを癖にするのが一番良いく、規約云々ではなく、このほうが書きやすくなることが多いです。
引数チェック
５以上を超えると見にくくなります。規約もそうですが、こちらも保守性が高いコードを書いておけば必然的に身につくはずです。
特定メソッドの推奨
一番見落としがち。例えばJavaではStringBuilderで文字を追加する、Listの定義ではArrayListやHashListを使わないなど。同時に複数チェックを入れると見逃しやすいので、この点だけに注意してチェックする。
言語自体への慣れもそうですし、普段から模範的なコードを書いておく必要があると思います。
