トランクに戻る前にコードレビューを実行することを主張する必要がありますか？

Question

StackOverFlowから再投稿を要求しました：

私は開発のための非常に限られた時間で小さな開発時間で働いています。私たちは仕事の結果にとって重要なツールを開発しますが、毎日使用されていません。私は、プログラマーとしてバックグラウンドを持つチームで唯一の人です。

私の問題は、1年以上トランクに戻る前にコードレビューを推進していることです。誰もがこれに同意しましたが、それでもレビューされたのは私のコードだけです。長い休暇から戻って、私は「これはugい解決策です - できるだけ早く削除」と「クイックフィックス」としてコードコメントでトランクに戻ります。また、新しいのは、男がツールの責任を任命されたことです。 （最初に私に提供されましたが、私は仕事に関連していない理由のために断りました。）そして、彼はこれが仕事のための問題であると考えています。

私の懸念は、他の開発者がugいコードを書いていることです。多くの場合、カプセル化を破り、巨大なクラスを書き、奇妙な場所で内部クラスを追加し、ユニットテストがほとんどまたはまったくないなどです。最終的にツールをさらに開発することは不可能になります。

トランクに戻る前にコードレビューを実行するか、コード品質の雌犬だと主張する必要がありますか？

Answer 1

私は以前も同様の状況にありましたが、「コードを維持する必要がありますか」に依存しています。

コードを維持する必要がある場合、高品質のコードが必要な場合、私は個人的にすべてのコミットについてコードレビューを必要としません（つまり、プログラマーは特定のコードがレビューが必要かどうかを自分で決定できます）それは整理されているかもしれません。

これを読むとき：

私の懸念は、他の開発者がugいコードを書いていることです。多くの場合、カプセル化を破り、巨大なクラスを書き、奇妙な場所で内部クラスを追加し、ユニットテストがほとんどまたはまったくないなどです。最終的にツールをさらに開発することは不可能になります。

あなたの問題は単なるコードレビューよりも大きいと思います。いくつかのガイドラインが欠落しているように見えますが、それらは実装されていません。ユニットテストがない/少ない場合は悪い考えかもしれませんが、特定のケースに依存します。でも、 breaking encapsulation, writing huge classes, ... 実際にバグが発生しやすいコードを作成して、間違いなく修正する必要があります。

Answer 2

コードがレビューし、いくつかのコーディングガイドラインを維持することは良い考えだと思いますが、 すべてのチェックイン 時間の無駄です。チームを設立し、若いプログラマーと一緒にいるときは良い考えですが、経験豊富なプログラマーは自分で考えることができ、最終的には彼らを信頼しなければなりません。そうは言っても、定期的なコードレビューを行うことができます。

そして、あなたの同僚の修正に関する小さなコメント - 時々醜い修正をする 正しいソリューションです. 。この特定のコードは、多くの時間を投資するほど重要ではない可能性があります。単純なソリューションで十分であり、他のことに時間を費やす方が良いかもしれません。コードを「きれい」にすることは、プログラマーとしての主な目標ではありません。 あなたの主な目標は、提供することです そして、コードのすべての行に夢中になると、単にそこに到達することはありません。

私が言いたいのは、あなたはあなたの戦いを選ぶ必要があります。配達戦争に勝つために、わずかなユーティリティクラスでの戦いに負けても構いません（または、この非常に重要なサブシステム戦争、そのため）。

Answer 3

問題のプログラムが1回限りのスローアウェイプロトタイプではない場合、チェックインごとにコードレビューが必須であると思います。

上級開発者は、必要に応じてレビューを要求するほど良心的であることが知られていれば、レビューされていないチェックインの特権を持つことができます。

Answer 4

誰かがより良いコーディング基準を実施し始めるまで、コードレビューが答えであるかどうかはわかりません。誰かがくだらないコードを書いて、それをコメント/認めて、とにかくそれをチェックします。誰かがコードを拒否したい場合、レビューは何をするのでしょうか？

必要なのは、標準を設定し、主要な犯人をより密接に監督し、良いコードが常に時間をかけるとは限らないと頭の中に叩きます。彼らは、悪い習慣を変えることを拒否する言い訳としてタイムラインの使用をやめる必要があります。