アカウント名:
パスワード:
>「キャメルケースを使うか、アンダースコアで区切るか」「中かっこの配置」「タブを使うか、スペースを使うか」といったコーディングスタイルのドキュメントが用意されていた。じゃ、読みましょう。適用しましょう。
前提となるドキュメントを読んでいないようなのに、レビューしてもしょうがありません。あなたの時間はたしかに無駄になりますが、レビューした場合あなた以外の人が無駄になる可能性があります。
私はEclipse派ですが、自動生成する際のコーディングスタイルを設定できます。自動生成できない部分に関しては、標準化する際に議論を行い、できるだけ自動化する、または採算が合わないようなら標準化適用外にしてます。
#標準化って生産性よりも、保守性をあげてるんじゃないの?
その通りですね.理由なく標準化を守らないのであれば,それはレビューに対する障害でしょう.
標準化されてないコード箇所があれば,そこは「怪しいコード」である可能性が高く,「ここについてフォーカスしてくれ」という指定でもなければ,レビュー時に最も力を入れるべき箇所です.
また, 標準化を無視するのであれば,その理由をレビュー時に明確にすればいいだけです.それは「この標準は生産性を下げる」と問題提起する機会にもなりますし.
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
クラックを法規制強化で止められると思ってる奴は頭がおかしい -- あるアレゲ人
コーディング標準はツールが自動的に適用してくれます。 (スコア:5, 参考になる)
>「キャメルケースを使うか、アンダースコアで区切るか」「中かっこの配置」「タブを使うか、スペースを使うか」といったコーディングスタイルのドキュメントが用意されていた。
じゃ、読みましょう。適用しましょう。
前提となるドキュメントを読んでいないようなのに、レビューしてもしょうがありません。
あなたの時間はたしかに無駄になりますが、レビューした場合あなた以外の人が無駄になる可能性があります。
私はEclipse派ですが、自動生成する際のコーディングスタイルを設定できます。
自動生成できない部分に関しては、標準化する際に議論を行い、できるだけ自動化する、または採算が合わないようなら標準化適用外にしてます。
#標準化って生産性よりも、保守性をあげてるんじゃないの?
Re:コーディング標準はツールが自動的に適用してくれます。 (スコア:2)
その通りですね.
理由なく標準化を守らないのであれば,
それはレビューに対する障害でしょう.
標準化されてないコード箇所があれば,
そこは「怪しいコード」である可能性が高く,
「ここについてフォーカスしてくれ」という指定でもなければ,
レビュー時に最も力を入れるべき箇所です.
また, 標準化を無視するのであれば,
その理由をレビュー時に明確にすればいいだけです.
それは「この標準は生産性を下げる」と問題提起する機会にもなりますし.