nprogram’s blog

気ままに、プログラミングのトピックについて書いていきます

コードレビューで指摘していただいた箇所まとめ (C++編)

コードレビューで指摘していただいた箇所をまとめて記載したいと思います。

  • ヘッダ内では、極力他のヘッダをインクルードしないようにしましょう
    • 2重インクルードや、循環参照(*1)を引き起こします
  • クラス名、メソッド名、変数名をしっかり考えよう
    • 例えば、犬クラスは、犬という情報を表現するクラスになっていますか?犬クラスなのに、犬クラスの中にスマートフォンを操作するメソッドがあってはいけません。( ´∀` )
  • コーディングスタイルが統一しましょう
    • スペース
    • 中かっこの位置
  • for文(繰り返し文)の中に、繰り返すたびに呼び出す必要がないのに、呼び出している処理はありませんか。
    • for文の外に処理を映しましょう
  • constが使えるのに、constを使っていない変数、メソッド引数はありませんか。
    • constが使える場合には、constを使いましょう
  • ポインタ渡しと参照渡し、どちらでも実現できる場合は、参照渡しにしたほうが良いと思います
    • 参照渡しは、実体がなければ実施できないため、呼び出し先のメソッドで、NULLポインタチェックが必要にならない

(*1) 循環参照問題とは、ヘッダーファイルAがヘッダーファイルBをインクルードし、ヘッダーファイルBがヘッダーファイルAをインクルードする事