2012-01-05 11 views
2

ここで手足に出る。入力を提供するために間違って行ったことを判断するのに十分な数のクラスとコード単位をデベロッパーがサンプリングしてコードレビューを行っていますか?これはちょうどあなたが時間を押され、それらによって書かれたすべての一行を見直すことはできません。私は、開発者がどこでも同じタイプの間違いを複製するかもしれないという前提で取り組んでいます。これは効果的なアプローチですか?または部分的に有効であるか、まったく効果がないか?コードレビューのアプローチ

他にどのような方法が適していますか?コードの行を書いた各開発者と一緒に座って、なぜ彼らがそれをどうしてやったのか尋ねるのですか?またはあなた自身でそれを行い、あなたの調査結果の概要を電子メールで送ってください。

一般的な15日間のスプリントですべてのコードベースを確認する必要がある場合、ジュニア開発者/同僚によって作成されたレビューコードにどのようにアプローチしますか?

答えて

3

コードが受け入れられることを知るには、いくつかのクラスとユニットを選ぶだけで十分だとは思いません。完全に書かれたコードのいくつかの部分を選んで大きな問題を逃すことは可能です。

時間が限られている場合は、レビューの仕方を最適化する必要があります。まず、一般的なエラーやコード標準偏差の自動レビューを行うために実行できるツール(StyleCop、FxCopなど)があることです。

その後、私はあなたが重要な機能が何であるかを考え、行ごとにレビューする必要があると思います。シノプシスを送信し、必要に応じて開発者と一緒に座っているのは分かりやすいアプローチです。

これは、完全なレビューを必要とする最初のコードセットに過ぎません。変更した内容を見てもそれほど悪くはないが、チームサイズによって異なります。

1

ジュニア開発者として、私はあなたが時間を持っている場合座っていることが最良であることがわかります。何かが明確でないか、まっすぐに間違っているときに、自分が何をしようとしているかを説明することは簡単です。

書かれたコードのすべての行が過剰です - 何らかの理由でQAが存在します。あなたは、会社のコードコンベンションに従っていることを確認し、われわれが認識するのに十分な経験がないばかげたものが愚かでないことを防ぐために探している。

関連する問題