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
2.2k
データの民主化はじめました 俺たちの民主化はこれからだ
akki_megane
1
1.4k
フィーチャートグルを 使って素早く価値を検証する 早く安全に失敗し学ぶために
akki_megane
0
2.5k
技術的負債を返し続ける取り組み
akki_megane
0
570
「明日からフロントもよろしく」と言われたときに備える Atomic Design
akki_megane
0
3.6k
Editor 調査
akki_megane
0
170
Laravel Vapor Serverless Laravel
akki_megane
2
330
アノテーションコメントについて調べてみた
akki_megane
2
650
入門 無限LT
akki_megane
0
4.3k
Featured
See All Featured
jQuery: Nuts, Bolts and Bling
dougneiner
61
7.5k
Code Reviewing Like a Champion
maltzj
520
39k
Visualizing Your Data: Incorporating Mongo into Loggly Infrastructure
mongodb
44
9.3k
Dealing with People You Can't Stand - Big Design 2015
cassininazir
365
25k
The Cult of Friendly URLs
andyhume
78
6.1k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
28
9.1k
Fontdeck: Realign not Redesign
paulrobertlloyd
82
5.3k
The Illustrated Children's Guide to Kubernetes
chrisshort
48
48k
The Psychology of Web Performance [Beyond Tellerrand 2023]
tammyeverts
45
2.2k
GraphQLの誤解/rethinking-graphql
sonatard
67
10k
Docker and Python
trallard
42
3.1k
No one is an island. Learnings from fostering a developers community.
thoeni
19
3k
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(日曜) 満員御礼 次回ぜひ まだ枠あるよ