コードレビュー

作成されたソースコードを開発者以外の第三者が確認し、品質を検証する活動。バグの早期発見、コーディング規約の遵守確認、保守性の向上などを目的として実施される。

ソースコードを他の開発者が読んで、コーディング標準への準拠、効率性、保守性、欠陥の有無を確認する活動。ピアコードレビュー、コードインスペクション、ウォークスルーなどの形式がある。

ソースコードを開発者以外の人がチェックし、バグ、設計上の問題、コーディング規約違反などを検出する手法。ペアレビュー、プルリクエストベースのレビューなどの形態がある。品質向上と知識共有に寄与する。

他の開発者がソースコードを検査し、バグ、設計問題、コーディング規約違反などを検出するレビュー手法。ウォークスルー、インスペクション、ペアプログラミングなどの形態がある。品質向上と知識共有に有効。

作成したソースコードを、作成者以外の者が読んで、誤り、規約への違反、保守しにくい箇所などを見つけるレビュー。インスペクションやウォークスルーの形で行うほか、変更のたびにツール上で確認するプルリクエストのレビューが広く使われる。テストより前の段階で欠陥を見つけられ、知識の共有にも役立つ。

作成したソースコードを作成者以外が読み、誤り、コーディング標準への違反、保守性や性能・セキュリティ上の問題を指摘する活動。プルリクエストを使ったレビューのほか、インスペクションやウォークスルーの形で行う。

プログラムの記述スタイルや命名規則、コメントの書き方、インデンテーション、ネストの深さ制限、使用禁止命令などを統一するための規約。可読性・保守性の向上とバグ混入の防止を目的とし、チーム開発での一貫性を確保する。

他の値から大きく離れた値。入力ミスの場合もあれば、実際に起きた重要な事象の場合もあるため、機械的に除外せず背景を確認してから判断する。

本来値があるべき箇所が空になっているデータ。除外すると件数が減り、平均値などで補完すると分布が歪むため、発生原因を踏まえて扱いを決める必要がある。

製品やシステムが、特定の利用者にとって特定の利用状況において、有効性・効率性・満足度をもって使用できる度合い。ISO 9241-11で定義され、学習しやすさ、操作効率、記憶しやすさ、エラーの少なさ、主観的満足度の5つの指標で評価される。

脆弱性を作り込まないことを意識した実装手法。入力値の検証、出力時のエスケープ、プレースホルダの使用、エラー情報を過剰に返さないことなどを徹底し、設計・実装段階で脆弱性の発生を抑える。

コンテナ環境に特有のリスクに対処する取組み。イメージに含まれる脆弱な部品の検査、信頼できるレジストリの利用、特権コンテナの禁止、ホストカーネルを共有する前提での分離設定などが要点となる。