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.7k
データの民主化はじめました 俺たちの民主化はこれからだ
akki_megane
2
1.7k
フィーチャートグルを 使って素早く価値を検証する 早く安全に失敗し学ぶために
akki_megane
0
2.9k
技術的負債を返し続ける取り組み
akki_megane
0
630
「明日からフロントもよろしく」と言われたときに備える Atomic Design
akki_megane
0
3.7k
Editor 調査
akki_megane
0
200
Laravel Vapor Serverless Laravel
akki_megane
2
370
アノテーションコメントについて調べてみた
akki_megane
2
770
入門 無限LT
akki_megane
0
4.8k
Featured
See All Featured
個人開発の失敗を避けるイケてる考え方 / tips for indie hackers
panda_program
110
20k
The Cult of Friendly URLs
andyhume
79
6.5k
Fireside Chat
paigeccino
39
3.6k
Building Flexible Design Systems
yeseniaperezcruz
328
39k
Practical Orchestrator
shlominoach
190
11k
YesSQL, Process and Tooling at Scale
rocio
173
14k
Automating Front-end Workflow
addyosmani
1370
200k
Into the Great Unknown - MozCon
thekraken
40
2k
The Invisible Side of Design
smashingmag
301
51k
The Myth of the Modular Monolith - Day 2 Keynote - Rails World 2024
eileencodes
26
3k
Code Review Best Practice
trishagee
70
19k
Code Reviewing Like a Champion
maltzj
525
40k
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(日曜) 満員御礼 次回ぜひ まだ枠あるよ