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.3k
PHP Insights - リファクタリングが100倍楽しくなるツール -
akki
June 20, 2019
Tweet
Share
More Decks by akki
See All by akki
データの民主化はじめました 俺たちの民主化はこれからだ
akki_megane
0
1.2k
フィーチャートグルを 使って素早く価値を検証する 早く安全に失敗し学ぶために
akki_megane
0
2.1k
技術的負債を返し続ける取り組み
akki_megane
0
480
「明日からフロントもよろしく」と言われたときに備える Atomic Design
akki_megane
0
3.4k
Editor 調査
akki_megane
0
160
Laravel Vapor Serverless Laravel
akki_megane
2
280
アノテーションコメントについて調べてみた
akki_megane
2
480
入門 無限LT
akki_megane
0
3.7k
バグを直す前にまずテストを書く
akki_megane
0
1.6k
Featured
See All Featured
Navigating Team Friction
lara
177
13k
10 Git Anti Patterns You Should be Aware of
lemiorhan
644
57k
Side Projects
sachag
451
41k
ParisWeb 2013: Learning to Love: Crash Course in Emotional UX Design
dotmariusz
101
6.6k
Let's Do A Bunch of Simple Stuff to Make Websites Faster
chriscoyier
501
140k
Ruby is Unlike a Banana
tanoku
95
10k
Teambox: Starting and Learning
jrom
126
8.4k
[RailsConf 2023] Rails as a piece of cake
palkan
21
3.8k
RailsConf 2023
tenderlove
0
510
Imperfection Machines: The Place of Print at Facebook
scottboms
257
12k
Designing Experiences People Love
moore
135
23k
JavaScript: Past, Present, and Future - NDC Porto 2020
reverentgeek
39
4.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(日曜) 満員御礼 次回ぜひ まだ枠あるよ