Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
PHP Insights - リファクタリングが100倍楽しくなるツール -
Search
akki
June 20, 2019
3
1.6k
PHP Insights - リファクタリングが100倍楽しくなるツール -
akki
June 20, 2019
Tweet
Share
More Decks by akki
See All by akki
Open AI APIを使う前に知っておきたいアカウントTier の話
akki_megane
0
3.4k
データの民主化はじめました 俺たちの民主化はこれからだ
akki_megane
1
1.6k
フィーチャートグルを 使って素早く価値を検証する 早く安全に失敗し学ぶために
akki_megane
0
2.8k
技術的負債を返し続ける取り組み
akki_megane
0
610
「明日からフロントもよろしく」と言われたときに備える Atomic Design
akki_megane
0
3.7k
Editor 調査
akki_megane
0
180
Laravel Vapor Serverless Laravel
akki_megane
2
350
アノテーションコメントについて調べてみた
akki_megane
2
740
入門 無限LT
akki_megane
0
4.7k
Featured
See All Featured
Making the Leap to Tech Lead
cromwellryan
133
9.3k
YesSQL, Process and Tooling at Scale
rocio
172
14k
Fireside Chat
paigeccino
37
3.4k
Why Our Code Smells
bkeepers
PRO
336
57k
Build The Right Thing And Hit Your Dates
maggiecrowley
35
2.7k
The Pragmatic Product Professional
lauravandoore
33
6.6k
Mobile First: as difficult as doing things right
swwweet
223
9.6k
The MySQL Ecosystem @ GitHub 2015
samlambert
251
12k
Being A Developer After 40
akosma
91
590k
Speed Design
sergeychernyshev
30
950
Fantastic passwords and where to find them - at NoRuKo
philnash
51
3.2k
Navigating Team Friction
lara
185
15k
Transcript
PHP Insights リファクタリングが 100倍楽しくなるツール (当人比 ) 2019/06/19 秋葉誠一
自己紹介 名前:秋葉 誠一 生年月日:1993/12/17 会社:株式会社ニジボックス 趣味:スノボ、サバゲー、野球(NPB,MLB)
Agenda • PHP Insights とは? • なにが楽しいの? • まとめ
この話しを聞いて、 PHP Insightsを使ってみようとか リファクタしようかなって思ってくれたら 嬉しいです
None
リファクタする前にテストコードを書きましょう Oysters
PHP Insights
PHP Insights PHPコードの品質を分析してくれる静的解析ツール Laravel、Symfony、Yii、WordPress、Magento2 で使えるように作られています
PHP Insights コードの信頼性 単純さ アーキテクチャ コーディングスタイル 使ってるライブラリのセキュリティチェック
PHP Insights 作者:Nuno Maduro Laravelの開発者の1人 Larastanの作者
None
None
PHP Insights Forbidden public property Do not use public properties.
Use method access instead
PHP Insights Forbidden public property: Do not use public properties.
Use method access instead プロパティがPublicになってるから、カプセル化して
PHP Insights Max nesting level
PHP Insights Max nesting level ネストしすぎ
PHP Insights Having `classes` with more than 5 cyclomatic complexity
is prohibited - Consider refactoring:
PHP Insights Having `classes` with more than 5 cyclomatic complexity
is prohibited - Consider refactoring: 循環的複雑度(条件分岐やループ多さ)が高いので リファクタリングを検討して
本題
PHP Insights リファクタリングが 100倍楽しくなるツール (当人比 ) 2019/06/19 秋葉誠一
なにが楽しいの?
リファクタリングどうやってます?
今の自分の場合はガッツリ、一気にというよりは、 ボーイスカウト精神で徐々に まぁ結構場当たり的にやっとります
大事だけど、孤独な戦い
PHP Insights 使うとなにが楽しいのか?
None
点数がでる
None
改善のためのアドバイスをくれる
直す
点数が上がる
たのしーーーーー
楽しいには理由がある • 成果の可視化 • 目標設定 • 具体的な指摘
楽しいには理由がある • 成果の可視化 • 目標設定 • 具体的な指摘
楽しいには理由がある • 成果の可視化 • 目標設定 • 具体的な指摘
楽しいには理由がある • 成果の可視化 • 目標設定 • 具体的な指摘
やればやるほど点数が上がる 楽しいから続けられる
まとめ
• 解析ツール自体使ったことなかったので、’あっすげ’ • 成果の可視化はモチベーションUP • いいコードを書く勉強になる 感想
良いコードにはなると思うが 点数を上げるのが目的ではない 指摘が絶対的に正解とは限らない 戒め
7/13(土曜) 7/14(日曜) 満員御礼 次回ぜひ まだ枠あるよ