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.5k
PHP Insights - リファクタリングが100倍楽しくなるツール -
akki
June 20, 2019
Tweet
Share
More Decks by akki
See All by akki
Open AI APIを使う前に知っておきたいアカウントTier の話
akki_megane
0
1.9k
データの民主化はじめました 俺たちの民主化はこれからだ
akki_megane
1
1.4k
フィーチャートグルを 使って素早く価値を検証する 早く安全に失敗し学ぶために
akki_megane
0
2.5k
技術的負債を返し続ける取り組み
akki_megane
0
560
「明日からフロントもよろしく」と言われたときに備える Atomic Design
akki_megane
0
3.6k
Editor 調査
akki_megane
0
160
Laravel Vapor Serverless Laravel
akki_megane
2
320
アノテーションコメントについて調べてみた
akki_megane
2
630
入門 無限LT
akki_megane
0
4.3k
Featured
See All Featured
Practical Tips for Bootstrapping Information Extraction Pipelines
honnibal
PRO
10
720
The Straight Up "How To Draw Better" Workshop
denniskardys
232
140k
Easily Structure & Communicate Ideas using Wireframe
afnizarnur
191
16k
Sharpening the Axe: The Primacy of Toolmaking
bcantrill
38
1.8k
Distributed Sagas: A Protocol for Coordinating Microservices
caitiem20
329
21k
Reflections from 52 weeks, 52 projects
jeffersonlam
346
20k
個人開発の失敗を避けるイケてる考え方 / tips for indie hackers
panda_program
93
16k
Refactoring Trust on Your Teams (GOTO; Chicago 2020)
rmw
31
2.7k
Fireside Chat
paigeccino
34
3k
A designer walks into a library…
pauljervisheath
204
24k
Build your cross-platform service in a week with App Engine
jlugia
229
18k
Code Reviewing Like a Champion
maltzj
520
39k
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(日曜) 満員御礼 次回ぜひ まだ枠あるよ