Čtvrtkon #1 - Refactoring

Transcript

1. Refactoring Václav Novotný, 6. září 2012, Čtvrtkon #1 (http://www.ctvrtkon.cz)
2. Problém Máme špatný kód. Jak z toho ven?
3. Řešením je refactoring zlepšování kvality bez změny funkčnosti
4. Příklad špatného kódu <?php $articles = $articleRepository->findAll(); foreach ($articles as $article) { echo $article->getTitle(); echo $article->getText(); }
5. Trochu vylepšený kód <?php $articles = $articleRepository->findAll(); foreach ($articles as $article) { printArticle($article); } function printArticle(Article $article) { echo $article->getTitle(); echo $article->getText(); }
6. Co jsme udělali? Snížili jsme složitost programu, aniž bychom změnili jeho funkcionalitu.
7. Další příklad <?php // ArticleController.php if ($article->getPublishingDate() == date('Y-m-d') && $article->getStatus() == 2 && count($article->getComments()) > 0) { $article->setScore(10); }
8. Další příklad <?php // ArticleController.php if ($article->isPublishedToday() && $article->getStatus() == 2 && count($article->getComments()) > 0) { $article->setScore(10); }
9. Další příklad <?php // ArticleController.php if ($article->isPublishedToday() && $article->isPublished() && count($article->getComments()) > 0) { $article->setScore(10); }
10. Další příklad <?php // ArticleController.php if ($article->isPublishedToday() && $article->isPublished() && $article->hasComments()) { $article->setScore(10); }
11. Další příklad <?php // ArticleController.php if ($article->isPublishedToday() && $article->isPublished() && $article->hasComments()) { $article->promoteArticle(); }
12. Další příklad <?php // ArticleController.php $articlePromoter->promoteArticle($article); // ArticlePromoter.php const TOP_ARTICLE = 10; public function promoteArticle(Article $article) { if ($article->isPublishedToday() && $article->isPublished() && $article->hasComments()) { $article->setScore(self::TOP_ARTICLE); }
13. Kam až mám jít? Je to hodně o citu a o domluvě v týmu.
14. Existuje chytrá kniha!
16. Je ale trochu nudná
17. Existuje chytrý web (trochu méně nudný) http://www.sourcemaking.com
18. Jak poznat, kde to hnije?
19. Pear rozšíření PHPMD PHP Mess Detector
21. Co PHPMD dělá? Pomocí statistické analýzy kódu ukazuje místa vhodná pro refactoring.
22. Code Size Rules • Cyclomatic complexity • N-path complexity • ExcessiveMethodLength • ExcessiveClassLength • ExcessiveParameterList • ExcessivePublicCount • TooManyFields • TooManyMethods • ExcessiveClassComplexity
23. Design Rules • ExitExpression • EvalExpression • GotoStatement • NumberOfChildren • DepthOfInheritance • CouplingBetweenObjects
24. Naming Rules • ShortVariable • LongVariable • ShortMethodName • ConstructorWithNameAsEnclosingClass • ConstantNamingConventions • BooleanGetMethodName
25. Vlastní pravidla Můžete si napsat vlastní pravidla. Třída implementující rozhraní PHP_PMD_Rule.
26. Refactoring
27. Pamatujte • nesmíte přidávat novou funkcionalitu • dělejte malé kroky • opravdu dělejte malé kroky!
28. Pamatujte #2 • bez testů se refactoruje těžko • jde o nedílnou součást vývoje • refactorovaný kód co nejčastěji začleňujte zpět do hlavní vývojové větve
29. Dotazy?
30. Děkuji za pozornost @VaclavNovotny