フロー解析結果を確認すると、大きく３種類のレビュー結果に分かれます。
バグの可能性がある処理フロー
デグレードの可能性がある処理フロー
冗長的な分岐がある処理フロー
このレビュー結果に分けるとき、開発現場の方々から「デグレードの可能性がある処理フロー」と「冗長的な分岐がある処理フロー」に対し、 「今はバグじゃないから問題ない」というコメントがでないか確認しましょう。
「今はバグじゃないから問題ない」というコメントは、「今、問題がなければどんなコードでもいい」という文化が開発現場に浸透している可能性が高いことを暗に示しています。
この文化が浸透している開発現場で「デグレードの可能性がある処理フロー」が多く存在する場合、コードの担当者が変更になる度に、バグが多発してしまい、担当者の変更や引継ぎが困難なコードを開発している例が多く存在します。というコメントがでないか確認しましょう。「今はバグじゃないから問題ない」というコメントは、「今、問題がなければどんなコードでもいい」という文化が開発現場に浸透している可能性が高いことを暗に示しています。この文化が浸透している開発現場で「デグレードの可能性がある処理フロー」が多く存在する場合、コードの担当者が変更になる度に、バグが多発してしまい、担当者の変更や引継ぎが困難なコードを開発している例が多く存在します。
また、「冗長的な分岐がある処理フロー」を放っておくと言う事は、余計なデバッグ作業を行っている可能性があります。つまり、コードを修正する以上にテスト/デバッグ作業に工数を費やしていることがあります。
結果として、「今、問題がなければどんなコードでもいい」という文化が浸透している開発現場では、拡張性の低いコードを開発しており、限られた工数の中で機能追加や保守作業が困難になることが多々あります。デバッグ作業に工数を費やしていることがあります。結果として、「今、問題がなければどんなコードでもいい」という文化が浸透している開発現場では、拡張性の低いコードを開発しており、限られた工数の中で機能追加や保守作業が困難になることが多々あります。
そのためにも、フロー解析結果を確認したら、バグの修正はもちろんのこと、 「現在のコードを少しでも拡張性の高いコードにするためにはどうしたらよいか？」を開発現場の皆様で考えていく事が大切になります。