コーディング規約を守らない人がいる

Question

同じチームにコーディング規約を守らない人が数名います。

例えばインデントにはタブスペースを使うと決めているのに半角スペースを使っている。)の後の{は改行を入れずに使うと決めているのにいつも改行しているなど。
一度の忠告では聞いてくれなくて、理由を聞いてみると「めんどくさい」とか「規約が生理的に受け付けない」など最初から守る気がないようなのです。

規約を無理に守らせて開発にストレスを感じるくらいなら別にそのままでいいと思うのですが、エゴを通されるのはこの先衝突が起こりそうで不安です。

私は気が弱いのできつく言うことができないので毎回諦めてしまいます。
規約が存在してそれを守る必要がある理由、組織の一員としての振る舞いや考え方を伝えるにはどうしたらよいでしょうか。

ちなみに私はテキストエディタに保存するときに自動で整形するように設定しているのでコーディング規約に関しては全くストレスは感じないのですが、「生理的に受け付けない」と言われるとその方法もすすめられないなーと思っています。
なにかよい方法があるといいのですが。

Answer 1

インデントのタブスペースは難しいですね。タグにPHPが付いているのでPHPだけを前提にしても、PSR-2ではスペース4文字ですが、フレームワーク等の定義している標準的なコーディング規約ではタブであったりとまちまちです。
一方的に押し付けるのではなくどちらのルールを採用するかチームで議論してみてはいかがでしょうか？

Answer 2

はじめまして。

コーディング規約の中身は通常、命名ルール、コーディングルール、禁止事項などに大別されると思いますが、いずれも保守性や可読性を高め、プログラムの快適な動作などの品質を保持するためにそれが必要だとされています。

ただ、コーディング規約をあまりこまかく策定しすぎると、アップデートや言語仕様が変更された場合など、頻繁に見直すことをしないと規約そのものがレガシーなものになってしまい使えなくなるおそれがあります。

jdvqgfさんがチームの中でどのような立場にあるのかわかりませんが、じっさいには、管理者の立場からみたコーディング規約のあり方と、エンジニアからみたコーディング規約のあり方とは少なからず異なってくると思います。
チームで開発を行う場合、マネージャークラスのメンバーがコーディングチェックしたり、メンバーが互いにレビューしたりできない（しない）のであれば、（生理的に受け入れられる）最低限のお約束だけでも話し合いで決定したほうがよいのではないでしょうか。

納期に追われたり、経験不足であったり、さまざまな原因でコーディング規約がおろそかになってしまうことがあります。しかし、そのようなときでも、チームで開発を行う際には、「書かれたコードは私物化しない」という考え方だけは徹底するべきだと思います。
エンジニアにとって大切なことは、チームの一員として、より大きな（*）、より精度の高いプロダクトをつくりあげていくことだと僕は考えています。

さいごになりましたが、タグに PHPとありましたので、付録として PHPコードをPSRのコーディング規約にそって整えてくれる「PHP Coding Standard Fixer（リンク、紹介ブログ）」というツールを紹介しておきます。
チームのメンバーに使うことを拒否されてしまえばそれまでですが、こうしたツールを使用するのもひとつの方法です。よろしければ、お試しください。

*大きな：プロダクトのスケールや露出量ではなく、そのアイデアや社会にあたえるインパクトの大きさを指しています。

Answer 3

めんどくさい、なんてのは開発ツールで自動整形できますから言い訳になりません。
生理的に受け付けない、も、理由にできません。それだったら生理的に受け付けないから一緒に仕事できない、という言い分も通ってしまいます。極端な話、それを言い出した瞬間に、その人は職を失っても文句を言えない（だって業務上の命令を拒否しているんですから）ことを自覚してないんでしょうねえ。

例えば「この規則のこれこれは実情に合っていないから再考を検討してほしい」という要望なら受け付けられますが、代案も何もなくただ嫌だからやりたくない、では仕事に対する態度が悪い、と評価されてしかるべきでしょう。

Answer 4

自動で整形できるような規約であれば、別に守る必要はないのでは？
自動で整形できるんですから。

Answer 5

時々こっそり一括変換してしまいますけどね。
まぁ開発ツールを統一した上で設定も変えないようにすれば、自ずとこの種の問題は解決するような気がします。

正直、もっとひどいレベルの"守らない"をする人とかざらにいたりしますので、
可読性を損なうようなひどい状況であれば、なぜ守らなければならないのかを話せばよいかと思います。

ルールのためのルールではなく、誰が触っても一定の方針に従って書かれていることにより、
保守しやすくなるというのがコーディングルールの本質だと思いますので。

経験上、SQL直書き禁止と言ってるのに書いているようなことすらありますからね。
それに比べたらかわいいものだと思ってしまいます。

Answer 6

チームで開発しているのであれば、開発ツールは統一した方が良いと思います。
例えばeclipseだと、コードの規約を設定したりワンアクションでフォーマットをかける機能がありますよ。