Recommended
More Related Content
What's hot
What's hot (20)
Similar to レガシーなWebアプリケーションと向き合う
Similar to レガシーなWebアプリケーションと向き合う (20)
More from Yuta Ohashi
More from Yuta Ohashi (17)
Recently uploaded
Recently uploaded (12)
レガシーなWebアプリケーションと向き合う
- 2. 22 ⽬次 • レガシーなWebアプリケーションとは？ • レガシーコードへのアプローチ⽅法 • コードのレガシーさの計測 • 計測できるレガシーさ • 計測できないレガシーさ • コードをさわらずになんとかする • まとめ
- 3. blue_goheimochi ⼤橋 佑太 株式会社オウケイウェイヴ PHP(Laravel), Javascript(Vue.js/Nuxt.js), Laravel JP Conference 2020, PHPerKaigi 2020, サッカー, フットサル, 浜松まつり, ポケカ, さわやかのげんこつハンバーグ おおはし ゆうた
- 5. 55 レガシーなWebアプリケーションとは？ • 古い • テストコードがない • テストが書けない(書くのが難しい) • アプリケーション⾃体が⼤きい • 初期のコードを書いた⼈がもういない • etc... etc ... etc ...
- 9. 99 レガシーなWebアプリケーションとは？ • レガシーコード • レガシー基盤 • レガシーカルチャー
- 10. 1010 レガシーなWebアプリケーションとは？ • レガシーコード • テストされていない∕テストできないコード • 柔軟性のないコード • 技術的負債を抱え込んでいるコード • レガシー基盤 • レガシーカルチャー
- 11. 1111 レガシーなWebアプリケーションとは？ • レガシーコード • テストされていない∕テストできないコード • 柔軟性のないコード • 技術的負債を抱え込んでいるコード • レガシー基盤 • 開発環境、過去の依存関係、複数の異なる環境 • レガシーカルチャー
- 12. 1212 レガシーなWebアプリケーションとは？ • レガシーコード • テストされていない∕テストできないコード • 柔軟性のないコード • 技術的負債を抱え込んでいるコード • レガシー基盤 • 開発環境、過去の依存関係、複数の異なる環境 • レガシーカルチャー • 変化が怖い、知識の孤⽴
- 13. 1313 レガシーなWebアプリケーションとは？ • レガシーコード • テストされていない∕テストできないコード • 柔軟性のないコード • 技術的負債を抱え込んでいるコード
- 14. レガシーコードへのアプローチ⽅法
- 15. 1515 レガシーコードへのアプローチ⽅法 • リファクタリング • リアーキテクティング • ビッグ‧リライト
- 16. 1616 レガシーコードへのアプローチ⽅法 • リファクタリング • メソッド‧クラスレベルでコードを変更 • リアーキテクティング • ビッグ‧リライト
- 17. 1717 レガシーコードへのアプローチ⽅法 • リファクタリング • メソッド‧クラスレベルでコードを変更 • リアーキテクティング • モジュール‧コンポーネントレベルでコードを変更 • ビッグ‧リライト
- 18. 1818 レガシーコードへのアプローチ⽅法 • リファクタリング • メソッド‧クラスレベルでコードを変更 • リアーキテクティング • モジュール‧コンポーネントレベルでコードを変更 • ビッグ‧リライト • ≒（⾔語の置き換えなども含む）リプレイス
- 19. 1919 レガシーコードへのアプローチ⽅法 • リファクタリング • メソッド‧クラスレベルでコードを変更 • 計測できるレガシーをリファクタリングする • リアーキテクティング • モジュール‧コンポーネントレベルでコードを変更 • 計測できないレガシーをリファクタリングする • ビッグ‧リライト • ≒（⾔語の置き換えなども含む）リプレイス
- 20. 2020 レガシーコードへのアプローチ⽅法 • リファクタリング • メソッド‧クラスレベルでコードを変更 • 計測できるレガシーをリファクタリングする • リアーキテクティング • モジュール‧コンポーネントレベルでコードを変更 • 計測できないレガシーをリファクタリングする • ビッグ‧リライト • ≒（⾔語の置き換えなども含む）リプレイス というような、 3つの観点に置き換えて考えています
- 21. コードのレガシーさの計測
- 23. 2323 計測できるレガシーさ • コードはどれくらい⼤きいか？(phplocなど) • 重複コードはあるか？(phpcpdなど) • コーディング規約が守られているか？(phpcsなど) • 循環的複雑度は？(phpmd, lizardなど) • コードにエラーはないか？(PHPStan, Phan, PHPUnitなど) • アプリケーションに脆弱性はないか？(VAddyなど)
- 24. 2424 計測できるレガシーさ • コードはどれくらい⼤きいか？(phplocなど) • 重複コードはあるか？(phpcpdなど) • コーディング規約が守られているか？(phpcsなど) • 循環的複雑度は？(phpmd, lizardなど) • コードにエラーはないか？(PHPStan, Phan, PHPUnitなど) • アプリケーションに脆弱性はないか？(VAddyなど) コードを解析するためのツールは⾊々ある （できることがかぶっているツールもあるので、⽬的にあったものを使う）
- 25. 2525 計測できないレガシーさ • なんかここはあやしい‧‧‧ • 不吉なにおいがする‧‧‧ • // ここのコードは変更しないこと • ここは〇〇パターンを使ったらキレイで柔軟な コードになるよ • こいつはエンティティでこいつはバリューオ… 知識や経験が活きてくるであろう領域
- 26. 2626 計測できないレガシーさ • なんかここはあやしい‧‧‧ • 不吉なにおいがする‧‧‧ • // ここのコードは変更しないこと • ここは〇〇パターンを使ったらキレイで柔軟な コードになるよ • こいつはエンティティでこいつはバリューオ… 知識や経験が活きてくるであろう領域知識や経験が活きてくる(であろう)領域 OOP, デザインパターン, 設計原則, 独⽴コアレイヤパターン, レイヤ化, DDD etc...
- 30. 3030 • PHPバージョン： PHP7.2 • フレームワーク： CakePHP 3.5 • ローカル環境： Docker / Docker Compose • テスト： PHPUnit, Cypress(E2Eテスト) • バージョン管理： Git forteeのレガシーさは？
- 31. 3131 • PHPバージョン： PHP7.2 • フレームワーク： CakePHP 3.5 • ローカル環境： Docker / Docker Compose • テスト： PHPUnit, Cypress(E2Eテスト) • バージョン管理： Git forteeのレガシーさは？ ‧‧‧昨⽇書いたコードが レガシーに感じるときもありすよね！？ エンジニアたるもの昨⽇より今⽇、今⽇より明⽇、明⽇より明後⽇、常に前に進む気持ちが⼤事！ いつでも新しいことにチャレンジできる気持ちでありたい！
- 33. 3333 • PHPバージョン： PHP 4.3.11 • フレームワーク： なし • ローカル環境： XAMPP • テスト： なし • バージョン管理： SVN Web App 2のレガシーさは？
- 34. 3434 計測できるレガシーさ • コードはどれくらい⼤きいか？(phplocなど) • 重複コードはあるか？(phpcpdなど) • コーディング規約が守られているか？(phpcsなど) • 循環的複雑度は？(phpmd, lizardなど) • コードにエラーはないか？(PHPStan, Phan, PHPUnitなど) • アプリケーションに脆弱性はないか？(VAddyなど)
- 35. 3535 計測できるレガシーさ • コードはどれくらい⼤きいか？(phplocなど) • 重複コードはあるか？(phpcpdなど) • コーディング規約が守られているか？(phpcsなど) • 循環的複雑度は？(phpmd, lizardなど) • コードにエラーはないか？(PHPStan, Phan, PHPUnitなど) • アプリケーションに脆弱性はないか？(VAddyなど)
- 36. 3636 • sebastianbergmann/phpcpd • Copy/Paste Detector (CPD) for PHP code. • コピペコードを検出してくれる • Don’t Repeat Yourselfとはよく⾔われるが‧‧ • コピペコードが必ず悪ではないので、検出された対象の コードは⽬視で確認してみましょう 重複コードはあるか？
- 37. 3737 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd src phpcpd 4.1.0 by Sebastian Bergmann. Found 14 clones with 548 duplicated lines in 24 files: - /app/src/Model/Table/ConferencePagesTable.php:51-105 (54 lines) /app/src/Model/Table/PagesTable.php:44-98 ~~~~~~~~~~~ - /app/src/Controller/ConferenceAttendeeTypesController.php:104-131 (27 lines) /app/src/Controller/ConferenceAttendeeTypesController.php:143-170 ~~~~~~~~~~~ - /app/src/Classes/ForteeUtil.php:73-109 (36 lines) /app/src/Classes/ImageFetcher.php:277-313 ~~~~~~~~~~~ 1.08% duplicated lines out of 50840 total lines of code. Average size of duplication is 39 lines, largest clone has 54 of lines fortee forteeのチェック結果
- 38. 3838 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd src phpcpd 4.1.0 by Sebastian Bergmann. Found 14 clones with 548 duplicated lines in 24 files: - /app/src/Model/Table/ConferencePagesTable.php:51-105 (54 lines) /app/src/Model/Table/PagesTable.php:44-98 ~~~~~~~~~~~ - /app/src/Controller/ConferenceAttendeeTypesController.php:104-131 (27 lines) /app/src/Controller/ConferenceAttendeeTypesController.php:143-170 ~~~~~~~~~~~ - /app/src/Classes/ForteeUtil.php:73-109 (36 lines) /app/src/Classes/ImageFetcher.php:277-313 ~~~~~~~~~~~ 1.08% duplicated lines out of 50840 total lines of code. Average size of duplication is 39 lines, largest clone has 54 of lines fortee コード解析するためのツールを 詰め込んだコンテナ
- 39. 3939 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd src phpcpd 4.1.0 by Sebastian Bergmann. Found 14 clones with 548 duplicated lines in 24 files: - /app/src/Model/Table/ConferencePagesTable.php:51-105 (54 lines) /app/src/Model/Table/PagesTable.php:44-98 ~~~~~~~~~~~ - /app/src/Controller/ConferenceAttendeeTypesController.php:104-131 (27 lines) /app/src/Controller/ConferenceAttendeeTypesController.php:143-170 ~~~~~~~~~~~ - /app/src/Classes/ForteeUtil.php:73-109 (36 lines) /app/src/Classes/ImageFetcher.php:277-313 ~~~~~~~~~~~ 1.08% duplicated lines out of 50840 total lines of code. Average size of duplication is 39 lines, largest clone has 54 of lines fortee 14個の重複コードを検出。24ファイル、合計548⾏が対象
- 40. 4040 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd src phpcpd 4.1.0 by Sebastian Bergmann. Found 14 clones with 548 duplicated lines in 24 files: - /app/src/Model/Table/ConferencePagesTable.php:51-105 (54 lines) /app/src/Model/Table/PagesTable.php:44-98 ~~~~~~~~~~~ - /app/src/Controller/ConferenceAttendeeTypesController.php:104-131 (27 lines) /app/src/Controller/ConferenceAttendeeTypesController.php:143-170 ~~~~~~~~~~~ - /app/src/Classes/ForteeUtil.php:73-109 (36 lines) /app/src/Classes/ImageFetcher.php:277-313 ~~~~~~~~~~~ 1.08% duplicated lines out of 50840 total lines of code. Average size of duplication is 39 lines, largest clone has 54 of lines fortee テーブルオブジェクトなので ある程度はしょうがないかな
- 41. 4141 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd src phpcpd 4.1.0 by Sebastian Bergmann. Found 14 clones with 548 duplicated lines in 24 files: - /app/src/Model/Table/ConferencePagesTable.php:51-105 (54 lines) /app/src/Model/Table/PagesTable.php:44-98 ~~~~~~~~~~~ - /app/src/Controller/ConferenceAttendeeTypesController.php:104-131 (27 lines) /app/src/Controller/ConferenceAttendeeTypesController.php:143-170 ~~~~~~~~~~~ - /app/src/Classes/ForteeUtil.php:73-109 (36 lines) /app/src/Classes/ImageFetcher.php:277-313 ~~~~~~~~~~~ 1.08% duplicated lines out of 50840 total lines of code. Average size of duplication is 39 lines, largest clone has 54 of lines fortee 同じコントローラーの中に 27⾏の重複があるだと！？
- 42. 4242 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd src phpcpd 4.1.0 by Sebastian Bergmann. Found 14 clones with 548 duplicated lines in 24 files: - /app/src/Model/Table/ConferencePagesTable.php:51-105 (54 lines) /app/src/Model/Table/PagesTable.php:44-98 ~~~~~~~~~~~ - /app/src/Controller/ConferenceAttendeeTypesController.php:104-131 (27 lines) /app/src/Controller/ConferenceAttendeeTypesController.php:143-170 ~~~~~~~~~~~ - /app/src/Classes/ForteeUtil.php:73-109 (36 lines) /app/src/Classes/ImageFetcher.php:277-313 ~~~~~~~~~~~ 1.08% duplicated lines out of 50840 total lines of code. Average size of duplication is 39 lines, largest clone has 54 of lines fortee こちらは違うファイルで36⾏の重複が！
- 46. 4646 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd src phpcpd 4.1.0 by Sebastian Bergmann. Found 14 clones with 548 duplicated lines in 24 files: - /app/src/Model/Table/ConferencePagesTable.php:51-105 (54 lines) /app/src/Model/Table/PagesTable.php:44-98 ~~~~~~~~~~~ - /app/src/Controller/ConferenceAttendeeTypesController.php:104-131 (27 lines) /app/src/Controller/ConferenceAttendeeTypesController.php:143-170 ~~~~~~~~~~~ - /app/src/Classes/ForteeUtil.php:73-109 (36 lines) /app/src/Classes/ImageFetcher.php:277-313 ~~~~~~~~~~~ 1.08% duplicated lines out of 50840 total lines of code. Average size of duplication is 39 lines, largest clone has 54 of lines fortee 50,840⾏の中、重複があったコードは1.08% 重複コードの平均⾏数は39⾏で、最⼤54⾏が重複している
- 47. 4747 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd . phpcpd 4.1.0 by Sebastian Bergmann. Found 184 clones with 9893 duplicated lines in 142 files: - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:166-524 (358 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:170-528 ~~~~~~~~~~ - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:321-561 (240 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:334-574 ~~~~~~~~~~~ - /app/xxxxxx/controllers/xxxxxx.php:909-1084 (175 lines) /app/xxxxxx/controllers/xxxxxx.php:1247-1422 ~~~~~~~~~~ 8.41% duplicated lines out of 112932 total lines of code. Average size of duplication is 54 lines, largest clone has 358 of lines Web App 2 Web App 2のチェック結果
- 48. 4848 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd . phpcpd 4.1.0 by Sebastian Bergmann. Found 184 clones with 9893 duplicated lines in 142 files: - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:166-524 (358 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:170-528 ~~~~~~~~~~ - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:321-561 (240 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:334-574 ~~~~~~~~~~~ - /app/xxxxxx/controllers/xxxxxx.php:909-1084 (175 lines) /app/xxxxxx/controllers/xxxxxx.php:1247-1422 ~~~~~~~~~~ 8.41% duplicated lines out of 112932 total lines of code. Average size of duplication is 54 lines, largest clone has 358 of lines Web App 2 184個の重複コードを検出。142ファイル、合計9,893⾏が対象
- 49. 4949 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd . phpcpd 4.1.0 by Sebastian Bergmann. Found 184 clones with 9893 duplicated lines in 142 files: - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:166-524 (358 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:170-528 ~~~~~~~~~~ - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:321-561 (240 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:334-574 ~~~~~~~~~~~ - /app/xxxxxx/controllers/xxxxxx.php:909-1084 (175 lines) /app/xxxxxx/controllers/xxxxxx.php:1247-1422 ~~~~~~~~~~ 8.41% duplicated lines out of 112932 total lines of code. Average size of duplication is 54 lines, largest clone has 358 of lines Web App 2 Viewのファイルなんかも 重複に引っかかってはいるが‧‧‧
- 50. 5050 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd . phpcpd 4.1.0 by Sebastian Bergmann. Found 184 clones with 9893 duplicated lines in 142 files: - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:166-524 (358 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:170-528 ~~~~~~~~~~ - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:321-561 (240 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:334-574 ~~~~~~~~~~~ - /app/xxxxxx/controllers/xxxxxx.php:909-1084 (175 lines) /app/xxxxxx/controllers/xxxxxx.php:1247-1422 ~~~~~~~~~~ 8.41% duplicated lines out of 112932 total lines of code. Average size of duplication is 54 lines, largest clone has 358 of lines Web App 2 Fatかつ重複したコントローラ‧‧‧
- 51. 5151 重複コードはあるか？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer phpcpd . phpcpd 4.1.0 by Sebastian Bergmann. Found 184 clones with 9893 duplicated lines in 142 files: - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:166-524 (358 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:170-528 ~~~~~~~~~~ - /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:321-561 (240 lines) /app/xxxxxx/xxxxxx/xxxxxx/xxxxxx.php:334-574 ~~~~~~~~~~~ - /app/xxxxxx/controllers/xxxxxx.php:909-1084 (175 lines) /app/xxxxxx/controllers/xxxxxx.php:1247-1422 ~~~~~~~~~~ 8.41% duplicated lines out of 112932 total lines of code. Average size of duplication is 54 lines, largest clone has 358 of lines Web App 2 112,932⾏の中、重複があったコードは8.41% 重複コードの平均⾏数は54⾏で、最⼤358⾏が重複している
- 52. 5252 重複コードはあるか？ fortee Web App 2 総コード⾏数 50,840 112,932 重複コード率 1.08% 8.41% 重複コード⾏数 548 9,893 重複コードの個数 14 184 重複コードが含まれる ファイル数 24 142
- 53. 5353 terryyin/lizard • Lizard is an extensible Cyclomatic Complexity Analyzer for many programming languages • PHP以外の⾔語も解析できるPythonのツール • 循環的複雑度を計測できる • 循環的複雑度 • if/else, for, switchなどの数を計測 • コードがどれだけ複雑か？の指標 循環的複雑度は？
- 54. 5454 循環的複雑度は？ 循環的複雑度 複雑さの状態 バグ混⼊確率 10以下 ⾮常に良い構造 25% 30以上 構造的なリスクあり 40% 50以上 テスト不可能 70% 75以上 いかなる変更も誤修正を⽣む 98%
- 55. 5555 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard src/ -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 162 28 1001 3 187 importAttendee@298- 484@src/Classes/TicketVendors/EventbriteCore.php 116 27 1001 1 148 importPop3@66-213@src/Classes/PopAccountCore.php 159 21 1174 0 185 form@199-383@src/Controller/SpeakerProposalsController.php 89 21 587 0 116 twitter@197-312@src/Controller/AvatarsApiController.php 376 20 2689 0 454 main@54-507@src/Shell/DataMigrationShell.php 101 19 829 0 135 createArchive@71-205@src/Classes/AttendeeCore.php 73 19 498 0 93 upload@43-135@src/Controller/AvatarsApiController.php 49 18 412 1 65 _parse@83-147@src/Classes/OpenGraph.php 74 18 707 1 112 importPop3@43-154@src/Classes/SponsorsLib.php 78 18 540 5 88 linkButton@55-142@src/View/Helper/FaHelper.php 76 17 487 2 80 repairText@246- 325@src/Controller/ReceivedEmailsController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 fortee forteeのチェック結果
- 56. 5656 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard src/ -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 162 28 1001 3 187 importAttendee@298- 484@src/Classes/TicketVendors/EventbriteCore.php 116 27 1001 1 148 importPop3@66-213@src/Classes/PopAccountCore.php 159 21 1174 0 185 form@199-383@src/Controller/SpeakerProposalsController.php 89 21 587 0 116 twitter@197-312@src/Controller/AvatarsApiController.php 376 20 2689 0 454 main@54-507@src/Shell/DataMigrationShell.php 101 19 829 0 135 createArchive@71-205@src/Classes/AttendeeCore.php 73 19 498 0 93 upload@43-135@src/Controller/AvatarsApiController.php 49 18 412 1 65 _parse@83-147@src/Classes/OpenGraph.php 74 18 707 1 112 importPop3@43-154@src/Classes/SponsorsLib.php 78 18 540 5 88 linkButton@55-142@src/View/Helper/FaHelper.php 76 17 487 2 80 repairText@246- 325@src/Controller/ReceivedEmailsController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 fortee 循環的複雑度の値が⼤きい順でソート条件を指定している
- 57. 5757 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard src/ -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 162 28 1001 3 187 importAttendee@298- 484@src/Classes/TicketVendors/EventbriteCore.php 116 27 1001 1 148 importPop3@66-213@src/Classes/PopAccountCore.php 159 21 1174 0 185 form@199-383@src/Controller/SpeakerProposalsController.php 89 21 587 0 116 twitter@197-312@src/Controller/AvatarsApiController.php 376 20 2689 0 454 main@54-507@src/Shell/DataMigrationShell.php 101 19 829 0 135 createArchive@71-205@src/Classes/AttendeeCore.php 73 19 498 0 93 upload@43-135@src/Controller/AvatarsApiController.php 49 18 412 1 65 _parse@83-147@src/Classes/OpenGraph.php 74 18 707 1 112 importPop3@43-154@src/Classes/SponsorsLib.php 78 18 540 5 88 linkButton@55-142@src/View/Helper/FaHelper.php 76 17 487 2 80 repairText@246- 325@src/Controller/ReceivedEmailsController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 fortee CCN = Cyclomatic Complexity Number = 循環的複雑度
- 58. 5858 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard src/ -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 162 28 1001 3 187 importAttendee@298- 484@src/Classes/TicketVendors/EventbriteCore.php 116 27 1001 1 148 importPop3@66-213@src/Classes/PopAccountCore.php 159 21 1174 0 185 form@199-383@src/Controller/SpeakerProposalsController.php 89 21 587 0 116 twitter@197-312@src/Controller/AvatarsApiController.php 376 20 2689 0 454 main@54-507@src/Shell/DataMigrationShell.php 101 19 829 0 135 createArchive@71-205@src/Classes/AttendeeCore.php 73 19 498 0 93 upload@43-135@src/Controller/AvatarsApiController.php 49 18 412 1 65 _parse@83-147@src/Classes/OpenGraph.php 74 18 707 1 112 importPop3@43-154@src/Classes/SponsorsLib.php 78 18 540 5 88 linkButton@55-142@src/View/Helper/FaHelper.php 76 17 487 2 80 repairText@246- 325@src/Controller/ReceivedEmailsController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 fortee 循環的複雑度が30を超えるファイルはない様⼦！
- 59. 5959 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard src/ -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 162 28 1001 3 187 importAttendee@298- 484@src/Classes/TicketVendors/EventbriteCore.php 116 27 1001 1 148 importPop3@66-213@src/Classes/PopAccountCore.php 159 21 1174 0 185 form@199-383@src/Controller/SpeakerProposalsController.php 89 21 587 0 116 twitter@197-312@src/Controller/AvatarsApiController.php 376 20 2689 0 454 main@54-507@src/Shell/DataMigrationShell.php 101 19 829 0 135 createArchive@71-205@src/Classes/AttendeeCore.php 73 19 498 0 93 upload@43-135@src/Controller/AvatarsApiController.php 49 18 412 1 65 _parse@83-147@src/Classes/OpenGraph.php 74 18 707 1 112 importPop3@43-154@src/Classes/SponsorsLib.php 78 18 540 5 88 linkButton@55-142@src/View/Helper/FaHelper.php 76 17 487 2 80 repairText@246- 325@src/Controller/ReceivedEmailsController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 fortee メソッドと⾏番号が分かる
- 62. 6262 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard . -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 247 63 2436 0 353 aaaaAction@41-393@./xxxxx/controllers/xxxxxController.php 139 41 1272 0 202 bbbbAction@625-826@./xxxxx/controllers/xxxxxController.php 124 39 1018 0 161 ccccAction@77-237@./xxxxx/controllers/xxxxxController.php 81 37 351 1 84 xxxx@840-923@./xxxx/xxxx/Controller/xxxx/Abstract.php 152 37 1086 0 197 xxxxAction@21-217@./xxxx/xxxx/xxxx/xxxxController.php 143 36 1110 0 197 xxxxAction@740-936@./xxxx/xxxx/xxxxController.php 150 35 1516 0 223 xxxxAction@33-255@./xxxx/controllers/xxxxController.php 63 34 741 1 72 xxxx@16-87@./xxxx/xxxx/xxxx/xxxx/xxxx/xxxx.php 81 34 917 0 106 aaaaAction@229-334@./yyyy/controllers/yyyyController.php 110 31 1071 0 143 xxxxAction@20-162@./xxxx/controllers/xxxxController.php 75 30 753 0 108 bbbb@79-186@./xxxx/controllers/yyyyController.php 97 29 926 0 127 ccccAction@609-735@./xxxx/controllers/yyyyController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 Web App 2 Web App 2のチェック結果
- 63. 6363 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard . -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 247 63 2436 0 353 aaaaAction@41-393@./xxxxx/controllers/zzzzController.php 139 41 1272 0 202 bbbbAction@625-826@./xxxxx/controllers/zzzzController.php 124 39 1018 0 161 ccccAction@77-237@./xxxxx/controllers/zzzzController.php 81 37 351 1 84 xxxx@840-923@./xxxx/xxxx/Controller/xxxx/Abstract.php 152 37 1086 0 197 xxxxAction@21-217@./xxxx/xxxx/xxxx/xxxxController.php 143 36 1110 0 197 xxxxAction@740-936@./xxxx/xxxx/xxxxController.php 150 35 1516 0 223 xxxxAction@33-255@./xxxx/controllers/xxxxController.php 63 34 741 1 72 xxxx@16-87@./xxxx/xxxx/xxxx/xxxx/xxxx/xxxx.php 81 34 917 0 106 aaaaAction@229-334@./yyyy/controllers/yyyyController.php 110 31 1071 0 143 xxxxAction@20-162@./xxxx/controllers/xxxxController.php 75 30 753 0 108 bbbb@79-186@./xxxx/controllers/yyyyController.php 97 29 926 0 127 ccccAction@609-735@./xxxx/controllers/yyyyController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 Web App 2 循環的複雑度が63… バグ混⼊確率70%以上…
- 64. 6464 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard . -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 247 63 2436 0 353 aaaaAction@41-393@./xxxxx/controllers/zzzzController.php 139 41 1272 0 202 bbbbAction@625-826@./xxxxx/controllers/zzzzController.php 124 39 1018 0 161 ccccAction@77-237@./xxxxx/controllers/zzzzController.php 81 37 351 1 84 xxxx@840-923@./xxxx/xxxx/Controller/xxxx/Abstract.php 152 37 1086 0 197 xxxxAction@21-217@./xxxx/xxxx/xxxx/xxxxController.php 143 36 1110 0 197 xxxxAction@740-936@./xxxx/xxxx/xxxxController.php 150 35 1516 0 223 xxxxAction@33-255@./xxxx/controllers/xxxxController.php 63 34 741 1 72 xxxx@16-87@./xxxx/xxxx/xxxx/xxxx/xxxx/xxxx.php 81 34 917 0 106 aaaaAction@229-334@./yyyy/controllers/yyyyController.php 110 31 1071 0 143 xxxxAction@20-162@./xxxx/controllers/xxxxController.php 75 30 753 0 108 bbbb@79-186@./xxxx/controllers/yyyyController.php 97 29 926 0 127 ccccAction@609-735@./xxxx/controllers/yyyyController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 Web App 2 メソッドごとのデータ
- 65. 6565 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard . -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 247 63 2436 0 353 aaaaAction@41-393@./xxxxx/controllers/zzzzController.php 139 41 1272 0 202 bbbbAction@625-826@./xxxxx/controllers/zzzzController.php 124 39 1018 0 161 ccccAction@77-237@./xxxxx/controllers/zzzzController.php 81 37 351 1 84 xxxx@840-923@./xxxx/xxxx/Controller/xxxx/Abstract.php 152 37 1086 0 197 xxxxAction@21-217@./xxxx/xxxx/xxxx/xxxxController.php 143 36 1110 0 197 xxxxAction@740-936@./xxxx/xxxx/xxxxController.php 150 35 1516 0 223 xxxxAction@33-255@./xxxx/controllers/xxxxController.php 63 34 741 1 72 xxxx@16-87@./xxxx/xxxx/xxxx/xxxx/xxxx/xxxx.php 81 34 917 0 106 aaaaAction@229-334@./yyyy/controllers/yyyyController.php 110 31 1071 0 143 xxxxAction@20-162@./xxxx/controllers/xxxxController.php 75 30 753 0 108 bbbb@79-186@./xxxx/controllers/yyyyController.php 97 29 926 0 127 ccccAction@609-735@./xxxx/controllers/yyyyController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 Web App 2 ほぼほぼコントローラーが占めている ロジックがコントローラに‧‧‧ Fat‧‧‧
- 66. 6666 循環的複雑度は？ % docker run --rm -v ${PWD}:/app blue-goheimochi/php-code-analyzer lizard . -s cyclomatic_complexity ========================================================================================= !!!! Warnings (cyclomatic_complexity > 15 or length > 1000 or parameter_count > 100) !!!! ================================================ NLOC CCN token PARAM length location ------------------------------------------------ 247 63 2436 0 353 aaaaAction@41-393@./xxxxx/controllers/zzzzController.php 139 41 1272 0 202 bbbbAction@625-826@./xxxxx/controllers/zzzzController.php 124 39 1018 0 161 ccccAction@77-237@./xxxxx/controllers/zzzzController.php 81 37 351 1 84 xxxx@840-923@./xxxx/xxxx/Controller/xxxx/Abstract.php 152 37 1086 0 197 xxxxAction@21-217@./xxxx/xxxx/xxxx/xxxxController.php 143 36 1110 0 197 xxxxAction@740-936@./xxxx/xxxx/xxxxController.php 150 35 1516 0 223 xxxxAction@33-255@./xxxx/controllers/xxxxController.php 63 34 741 1 72 xxxx@16-87@./xxxx/xxxx/xxxx/xxxx/xxxx/xxxx.php 81 34 917 0 106 aaaaAction@229-334@./yyyy/controllers/yyyyController.php 110 31 1071 0 143 xxxxAction@20-162@./xxxx/controllers/xxxxController.php 75 30 753 0 108 bbbb@79-186@./xxxx/controllers/yyyyController.php 97 29 926 0 127 ccccAction@609-735@./xxxx/controllers/yyyyController.php ========================================================================================== Total nloc Avg.NLOC AvgCCN Avg.token Fun Cnt Warning cnt Fun Rt nloc Rt ------------------------------------------------------------------------------------------ 33248 16.3 2.3 104.2 1743 11 0.01 0.05 Web App 2 上位3つは「同じ」コントローラー‧‧‧
- 67. 6767 循環的複雑度は？ fortee Web App 2 総コード⾏数 (コメントなし) 33,248 69,946 平均CCN 2.3 2.8 総メソッド数 1,743 4,518
- 68. 6868 計測できるレガシーさ • コードはどれくらい⼤きいか？(phplocなど) • 重複コードはあるか？(phpcpdなど) • コーディング規約が守られているか？(phpcsなど) • 循環的複雑度は？(phpmd, lizardなど) • コードにエラーはないか？(PHPStan, Phan, PHPUnitなど) • アプリケーションに脆弱性はないか？(VAddyなど)
- 69. 6969 計測できるレガシーさ • コードはどれくらい⼤きいか？(phplocなど) • 重複コードはあるか？(phpcpdなど) • コーディング規約が守られているか？(phpcsなど) • 循環的複雑度は？(phpmd, lizardなど) • コードにエラーはないか？(PHPStan, Phan, PHPUnitなど) • アプリケーションに脆弱性はないか？(VAddyなど) 計測できるレガシーさは ツールで確認しつつ、改善の指針にできる
- 72. 7272 • PHPバージョン： PHP4.3.11 • フレームワーク： なし • ローカル環境： XAMPP • テスト： なし • バージョン管理： SVN Web App 2のレガシー具合は？ リファクタ‧‧？コードに触れる‧‧‧？ ウッ‧‧‧あたまが‧‧‧
- 73. コードをさわらずになんとかする
- 75. 7575 • レガシー過ぎて触りたくない触れない‧‧‧ • レガシーソフトウェア改善ガイド「完全な書き直し は、ほとんど常によくないアイデア」とは⾔うが… • でも‧‧‧ねぇ‧‧‧ • やっぱり触りたくない‧‧‧ • チームの状況など加味してよくよく考える必要はある • よし、ビッグ‧リライトだ！ • どうやる‧‧？ コードをさわらずになんとかする
- 76. 7676 • レガシー過ぎて触りたくない触れない‧‧‧ • レガシーソフトウェア改善ガイド「完全な書き直し は、ほとんど常によくないアイデア」とは⾔うが… • でも‧‧‧ねぇ‧‧‧ • やっぱり触りたくない‧‧‧ • チームの状況など加味してよくよく考える必要はある • よし、ビッグ‧リライトだ！ • どうやる‧‧？ コードをさわらずになんとかする 実際に私の現場で やろうとしているアプローチを forteeで再現してみます
- 84. 8484 • URIのパスごとに振り分ける • 少しずつ、新しいアプリケーションに変えていける • forteeにあるようなE2Eのテスト(Selenium, Cypress など)を作成して動作の担保 • ⾃動テストが絶対必要というわけではないとは思う • データの更新などがある場合 • MySQLが発⾏するSQLなどを確認してデグレが起 こっていないことを確認する コードをさわらずになんとかする
- 85. 8585 • URIのパスごとに振り分ける • 少しずつ、新しいアプリケーションに変えていける • forteeにあるようなE2Eのテスト(Selenium, Cypress など)を作成する • ⾃動テストが絶対必要というわけではないとは思う • データの更新などがある場合 • MySQLが発⾏するSQLなどを確認してデグレが起 こっていないことを確認する コードをさわらずになんとかする 他にも⽅法はあると思いますが、 狭い範囲にフォーカスできる⽅法を選ぶ
- 86. まとめ
- 87. 8787 まとめ • レガシーなWebアプリケーションとは？ • レガシーコードに注⽬してアプローチの観点を考察 • 計測できるレガシーのリファクタリング • 計測できないレガシーのリファクタリング • リプレイス • レガシーさが計測できる場合 • ツールで確認しながらコードを改善する • ビッグ‧リライト（リプレイス）を選択する場合 • なるべく狭い範囲を対象に進められる⽅法を選ぶ
- 88. 8888 まとめ • レガシーなWebアプリケーションとは？ • レガシーコードに注⽬してアプローチの観点を提⽰ • 計測できるレガシーのリファクタリング • 計測できないレガシーのリファクタリング • リプレイス • レガシーさが計測できる場合 • ツールで確認しながらコードを改善する • ビッグ‧リライト（リプレイス）を選択する場合 • なるべく狭い範囲を対象に進められる⽅法を選ぶ みなさまの現場における レガシーなWebアプリケーションと 向き合うための⼀助となれば幸いです
- 89. 8989 まとめ • レガシーなWebアプリケーションとは？ • レガシーコードに注⽬してアプローチの観点を提⽰ • 計測できるレガシーのリファクタリング • 計測できないレガシーのリファクタリング • リプレイス • レガシーさが計測できる場合 • ツールで確認しながらコードを改善する • ビッグ‧リライト（リプレイス）を選択する場合 • なるべく狭い範囲を対象に進められる⽅法を選ぶ ご清聴ありがとうございました！