AS400 / IBM i のRPG/CL改修では、コンパイルが通っただけでは安心できません。業務データの更新、例外時の動き、MSGW、ライブラリリスト、後続処理、帳票、戻し手順まで見て初めて本番に出せます。コードレビューは、若手育成にも障害予防にも効きます。
レビューで見る観点
| 観点 | 見ること | リスク |
|---|---|---|
| ファイル更新 | 更新条件、キー、コミット単位 | 誤更新や二重更新 |
| エラー処理 | CPF/RNX、例外時の戻り | 途中停止や後続不整合 |
| MSGW | 返信が必要な処理か | 夜間バッチ停止 |
| LIBL | 開発・テスト・本番の差 | 別オブジェクト参照 |
| テスト | 正常系、異常系、戻し | 本番で初めて分かる不具合 |
業務影響をレビューに入れる
コードだけを見ても、請求締め、在庫更新、出荷確定、帳票出力への影響は分かりません。レビューでは、修正内容がどの業務に効くのか、利用部門がどう確認するのかも見ます。
AIはレビュー観点の抜け漏れ確認に使う
Codexには、匿名化した修正概要、エラー文、テスト観点を渡し、レビュー観点を洗い出す使い方ができます。実ソースや本番データをそのまま入れず、最終レビューは人が行います。
コンパイルエラーは RPG/CLコンパイルエラーの調べ方、ライブラリ差異は ライブラリリスト違いによる障害、テストは テスト・エビデンスチェックリスト を参照してください。
