コード規約をまたまた掘り返してみる

» 2018/09/27

最近、自分でコードを書く機会がありました。それを機に以前書いコード規約について書いたコラムを読み返ました。結論としては、コード規約云々で書かれている内容は、リファクタリングでやれという結論に達しました。これに関してはそれぞれのスタイルで意見が分かれると思います。ただはっきり言えるのは、コード規約の運用を間違えているスキルの低いSIerは黙れということです。

順番としては、まず動くものを作るのが最初かと思います。一通り動くものができてから、コード規約に沿った形に整形していくのが正解かと思います。「いや、最初から規約を守っていれば手間が少なくてすむじゃないか！」という反論は受け付けません。下書きせずに長文を書くような真似をする人に、規約が必要なレベルのコードが書ける道理はありません。

当たり前の話ですが、規約を厳密に決めて守らせたければ、それに見合った工数とプランが必要です。単に「仕事だから間違えずにやってとうぜんだろ」では、全くプランになっていません。「バーンとコードを書いてレビューを通せば品質OK！」とか「コード規約が品質基準」と思っている方も多いようですが、そんなものは何の保証にもなりません。当人のスキルと、どういう根拠で何をやったかが全てです。

当たり前ですが、コードを書けない人（書けると思っているだけのマネージャクラスの人）では、いくらコードレビューをやっても無意味です。同様に、そういう人が作成したコード規約は品質を誤魔化すための儀式に使用するツールでしかありません。そういうものを基準に議論をしても時間の無駄にしかなりません。もちろん、品質なんて上がりません。

まず、コード規約を記載したファイルの拡張子が "xls" で更新されていないのであれば、PDCAが回っていません。読む価値は無いので破り捨てましょう。コード規約を守るだけでは品質なんて上がりません。プロジェクト全体のスキルが底上げされれば、それに併せて規約も変わります。規約の更新や運用にノウハウを蓄積することで、初めて品質は上がるのではないでしょうか。