静的解析、しましょうか。

自己紹介株式会社ミクシィたんぽぽチームリーダ広木大地@hiroki_daichiやっていること＊刺身にたんぽぽ＊サービスアーキテクト＊非機能要件全般＊サービスのトラブル＊安くて速くて安心

静的解析、しましょうか。せいてきかいせき

静的解析ってなんなんだよ。

コードレビュー自動化刺身にたんぽぽおかせないソフトウェア測定

Technical Debt技術的負債最初のコードを出荷することは、借金をしに行くことと同じである。小さな負債は、代価を得て即座に書き直す機会を得るまでの開発を加速する。危険なのは、借金が返済されなかった場合である。

複雑な依存関係最長不倒関数王様モジュール

01●　　　　　　　　●　　　　　　　　　●　　　　　　　　　●　　　　　　●　　　　　　　　●　　　　　　　　●　　　　　技術的負債の見える化Inspect-package

95計測できないものは改善できない。

最長不倒関数Node(2)Node(3)If( is_foo())do_var()Edge(4)Node(1)Node(5)Edge(2)startdo_hoge()endEdge(1)Edge(3)Edge(5)Node(4)Edge(6)分岐、ループなど関数をグラフ構造としてとらえて、その複雑さを測定する。(Cyclomatic Complexity)

王様モジュールgoodbadModuleModuleより大きく、より多くの人によって差分が加えられているコードを探して計測する。

複雑な依存goodgoodbadBBBEAAACCCFDDDGより多くに依存しながら、より多くに依存されているコンポーネントを悪い依存として測定する。

計測measurement２０１１年2月某日サービスの結合性に関するポリシーは決めたこれから順次リファクタリングが必要あの日以来、すべてがスコアリングされた。

依存されている度サービス全体への負債依存している度$ inspect-package Mixi::TheServiceMayUKnow -type ALLSCORE Ca Ce CC SRP couple cohesion240 173 261 527 41520 138335最長不倒関数度コンポーネント内の負債王様モジュール指数

王様モジュールをレビュー$inspect-package Mixi/TheServiceUMayKnow -type SRPscore( 123):rev( 98):user( 16):lines(1331) Mixi/TheServiceUMayKnow.pmscore( 107):rev( 80):user( 18):lines(1323) Mixi/TheServiceUMayKnow/Object.pmscore( 47):rev( 40):user( 5):lines( 659) Mixi/TheServiceUMayKnow/Admin/Ad.pm1300行で16人がいじってるけど、そろそろ責務がいりまじってるんじゃない？

最長不倒関数をレビュー$inspect-package Mixi::TheServiceUMayKnow -type CC 59 get_status　　 lib/Mixi/TheServiceUMayKnow/Analytics.pm 46 summarize 　 lib/Mixi/TheServiceUMayKnow/Counter/Summarizer.pm44 _select_own_page lib/Mixi/TheServiceUMayKnow/Feed.pm このモジュールのget_statusが複雑になりすぎてて、テストケース網羅できないほどだよ。

嫌だわ、早くすりつぶさないと

せいてきかいせき静的解析、しましょうか。

くわしいことはWEB+DB Press 62のPerl Hackers Hubにも載ってますhttp://gihyo.jp/dev/serial/01/perl-hackers-hub/000801

静的解析、しましょうか。

More Related Content

Similar to 静的解析、しましょうか。 (20)

Recently uploaded (10)

静的解析、しましょうか。