質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.61%

  • PHP

    19882questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

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

受付中

回答 6

投稿

  • 評価
  • クリップ 2
  • VIEW 2,402

jdvqgf

score 5

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

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

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

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

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

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 6

+6

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+4

はじめまして。

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

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

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+2

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    19882questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。