ソフトウェアデザイン工学 EMの適用結果報告.pptVIP

ソフトウェアデザイン工学EPMの適用結果報告 2007年2月2日 （株）日立システムアンドサービス 十九川博幸 Group1（規模推移） Group2（規模推移） Group3（規模推移） Group4（規模推移） Group5（規模推移） Group1（CVSバージョン） Group2（CVSバージョン） Group3（CVSバージョン） Group4（CVSバージョン） Group5（CVSバージョン） EPMのモニタ結果 ソースコードの静的解析 雛型の指摘（1） 雛型の指摘（2） Group1の指摘 Group2の指摘 Group3の指摘（1） Group3の指摘（2） Group4の指摘 Group5の指摘（1） Group5の指摘（2） ソースコードの静的解析２ * * 初期登録12/8 テストコードの削除 初期登録12/8 初期登録12/8 初期登録12/8 テストコードの削除 初期登録12/8 規模推移 　?テストコードの削除などで一時的に減少しているグループがあるが、特に異常な状態は見受けられなかった 　?急激なコード増の時期があるが、CVS詳細情報と見比べても、特に異常な状態は見受けられなかった （バグ情報と比較できれば、効果的な評価ができると思われる） CVSバージョン情報（バージョン毎のCVS操作回数） 　?バージョン1.1（初期登録のまま修正なし）が110件弱存在する 　?Group2と3は、バージョン1.2（1回だけ修正）の比率が高く、全体の修正回数が少ない（作り込み品質が良かったか、バグをたたき出せていない） （バグ情報と比較できれば、効果的な評価ができると思われる） JAVAのコーディング規約違反などの好ましくないコーディングを、ツールを使ってチェックすることができる ここでいう「好ましくないコード」は、バグでは無く、勘違いやミスを誘発しやすいコードを指す 今回は、PMDというJAVAソースコード解析ツールを使用して、 2007/01/29 11：00時点のソースコード（StockStateDAO）に対して静的解析を行った 各班の解析結果を報告します。 PMDの詳細は、下記URLを参照ください / /jp/developerworks/java/050121/j_j-pmd.html ?Private field conn could be made final; it is only initialized in the declaration or constructor. 重要度：軽微 原因：コネクション用変数のfinal指定忘れ 該当のコーディング private Connection conn; ?Avoid variables with short names like rs 重要度：軽微 原因：命名規則の問題（変数名が短すぎる） 該当のコーディング ResultSet rs = null; ?Avoid empty if statements 重要度：実装途中の可能性あり 原因：空のIfステートメントあり 該当のコーディング if(count == 0){ } 特になし ?Avoid using if statements without curly braces 重要度：軽微 原因：If文の｛｝忘れ 該当のコーディング if(rs != null) rs.close(); ?Avoid variables with short names like sd 重要度：軽微 原因：命名規則の問題（変数名が短すぎる） 該当のコーディング StockData sd = null; ?Ensure that resources like this ResultSet object are closed after use 重要度：重大 原因：リソースのclose漏れ(finallyブロックでcloseできていない) 該当のコーディング ResultSet rs = null; finallyブロックが無い ?Avoid unused imports such as org.apache.xalan.lib.sql.SQLErrorDocument 重要度：実装途中の可能性あり 原因：使用されていないimport文 該当のコーディング import org.apache.xalan.lib.sql.SQLErrorDocument; *