このセクションでは、コード レビューを有効に使用するためのヒントを取り上げます。

セットアップのヒント

  • コード レビュー結果と適切な Concerto プロジェクトを簡単に関連付けられるよう、テスト コンフィギュレーション ([共通] タブの [セッション タグ]) でユニークなセッション タグを指定し、Concerto プロジェクトのセットアップ時にフィルターとして使用できるようにします。一般的には、1) ツールの [一般的なプロジェクト] 設定オプション (詳細については「Parasoft Concerto との接続方法」を参照) を設定すること、また 2) テスト コンフィギュレーションのセッション タグでプロジェクト変数を使用することが推奨されます。  
    • [一般的なプロジェクト] 設定オプションを設定して ${general_project} 変数を使用するのは、多くの場合に適切です。しかし、1 つのソリューションまたはワークスペース内に複数の Concerto プロジェクトがある場合は、${general_project} ではなく ${project_name} 変数をセッション タグに使用します。変数の詳細については「設定での変数の使用」を参照してください。  

  • Parasoft ソリューションが Team Server の Named Accounts を使用している場合、Team Server ユーザー アカウントが確実に Team Server の Code Review ディレクトリにアクセスできるようにしてください。適切なパス権限を設定する方法については、『Concerto 管理者マニュアル』の「Named Accounts」のセクションを参照してください。  
  • レビューにはチーム メンバー全員が責任を負うようにします。1 人で大量のコードをレビューする場合、レビュー作業の負担が大きくなる可能性があります。  
  • 1 つのテスト コンフィギュレーションを複数の Parasoft Test 製品で使用したい場合 (たとえば、C、C++、Managed C++ で開発を行っており、1 つのコード レビュー テスト コンフィギュレーションを C++test と dotTEST の両方で使用したい場合など)、「複数の製品および言語を含むコード レビューの構成」を参照してください。  

開発者へのヒント

 

  • コミット後のレビューの場合 - コードソース管理システムのリポジトリにコードをチェックインするときには、レビューアーが変更目的をすぐ理解できるようにコメントを追加します。  

レビューアーへのヒント

  • 毎日コードをレビューして、レビュー作業がたまらないようにします。定期的にレビューを実施しないと、レビュー作業の負担が大きくなります。  
  • コミット後のレビューの場合 - リビジョン ブランチにカーソルを置くと、ツールチップとしてチェックイン コメントが表示されます。各リビジョンをレビューする前に、開発者のチェックイン コメントに目を通してどの程度コードをレビューするかを判断します。たとえば、表面的な小さな変更は簡単にレビューし、アルゴリズムの再実装のリビジョンは詳しくレビューするなど、レビューにかける時間と労力を判断します。
  • No labels