Suivi qualité avec sonar pour php

Suivi qualité avec Sonar pour PHPGabriele SantiniPHPTour Lille - 25/11/2011

WhoAmI• Gabriele Santini• Consultant – Architecte Logiciel• Indépendant -> Zend Technologies• Sonar pour PHP Passions professionnelles - Modélisation métier - Bonnes pratiques - Suivi qualité

(Ex) PlanDuTalk• Prendre un applicatif de mauvaise qualité• Le placer sous surveillance Sonar• Utiliser Sonar comme guide qualité• Améliorer la qualité de l’applicatif• Suivre les progrès avec Sonar

PlanDuTalk• Prendre un applicatif de mauvaise qualité• Le placer sous surveillance Sonar• Montrer Sonar• Préparer l’applicatif pour le refactoring• Paramétrer Sonar pour le refactoring• Début du refactoring• Afficher les amélioration dans Sonar

(Ex) ButsDuTalk• Introduire Sonar pour PHP• Parler bonnes pratiques XP• Montrer des techniques pour le « legacy code »

(Ex) ButsDuTalk• Introduire Sonar pour PHP• Parler bonnes pratiques XP• Montrer des techniques pour le « legacy code »• … s’amuser avec l’exemple de « revamping »

ButsDuTalk• Introduire Sonar pour PHP • Parler bonnes pratiques XP • Montrer des techniques pour le « legacy code » • … s’amuser avec l’exemple de « revamping » 

Exemple• Petit projet (3-6k LOC)• Vieux code (à la « PHP3/4 ») • Recherche Sourceforge de projets s’arretant ~2006• Intéressant pour moi• And the winner is…

PHPRiSK !

OpeningTheBoxclass class_mysql{ // PSR0 ?  function class_mysql() { global $CONFIG; $this->host = $CONFIG[mysql][host]; $this->user = $CONFIG[mysql][user]; $this->pass = $CONFIG[mysql][pass]; $this->dbase = $CONFIG[mysql][dbase]; return true; } function db_con(){ $db = mysql_connect($this->host, $this->user, $this->pass) or die ("verbindung $this->user@$this->host fehlgeschlagen");

DidYouSayDecoupled?#status 0: Offline# ...# status: 10 Running Gameclass class_risk{ function get_soapobject() { global $CONFIG; require_once SOAP/Client.php; // aucun répertoire “SOAP”… function soapUpdateNumOnline($numonline) { global $CONFIG;

DidYouSaySRP?Méthodes de class_risk :• 7 méthodes contenant« soap »• 21 méthodes contenant « display »• ~15 méthodes d’action• 4 méthodes contenant « By »• …• 57 méthodes au total• 1648 LOC au total

DidYouSaySoC?function display_win() { $cdb = new class_mysql; $cdb->select("SELECT CurrentGame FROM risk_config"); $row = $cdb->fetchArray(); if ($row[CurrentGame] != "Complete") { header("Location: index.php"); } $cdb->select("SELECT username, color FROM risk_players WHERE status=10"); $row = $cdb->fetchArray(); $r = "<h2>Game Over</h2><center><br><br><br><font color=".$row[color]." class="wintext">".$row[username]."wins!<br><br><br></font></center>"; return $r;}

DidYouSaySecure?function display_gameManager() { global $PHP_SELF, $CONFIG;[…]<form method="post" name="msg" action="$PHP_SELF">Heureusement nous sommes en PHP 5…

DidYouSayNormalizedDB?CREATE TABLE `risk_players` ( `num` smallint(5) unsigned NOT NULL auto_increment, `username` varchar(16) NOT NULL default , `password` varchar(32) NOT NULL default , ……)INSERT INTO `risk_players` VALUES(1, Allies, 2e970e822e1a8834203d06abb60f59ec, , 1, , 0, 0, 0, NULL, NULL, 1, 1, 0, 0, 0, 0, 0, notmyturn.php, 0, 0);

BoxOpened

Moralité Qualité externe != Qualité interne

EnterSonar• Install ! • 2 minutes coté Sonar (Ok, + téléchargement de Java) • http://docs.codehaus.org/display/SONAR/Install+Sonar#InstallS onar-The2minutestutorial • Localhost:9000 • 2 minutes coté PHP (à tester) > pear config-set auto_discover 1 > pear install pear.phpqatools.org/phpqatools • 30 secs Plugin Sonar pour PHP via l’UI Sonar • Update Center => Available Plugins

SonarDontImpliesMaven• Depuis la 2.6, Java Runner • Installation en 2 minutes http://docs.codehaus.org/display/SONAR/Analyse+with+a+simple+Java+ Runner• Fichier sonar-project.properties # required metadata sonar.projectKey=my:risk sonar.projectName=PHP Risk sonar.projectVersion=0.3 # path to source directories (required) sources=source # path to test source directories (optional) tests=test # The value of the property must be the key of the language sonar.language=php # PHP plugin specific parameters sonar.phpUnit.shouldRun=false sonar.dynamicAnalysis=false

FirstSonarRun• Lancer « sonar-runner » à la racine du projet• Le projet apparait dans l’interface web

FirstVerdict

AutomaticDeployment

FunctionalCasting

FastFunctionalTestingAu début le calque remplacera les TUADans le cycle TDD! Doit être RAPIDESelenium ne convient pasLes solutions spécifiques à un framework non plus

BehatMinkBehat : Framework BDD (style Cucumber)Mink : Framework pour les tests de recette • Abstraction du navigateur et du driver • Drivers disponibles : • Goutte (headless HTTP) • Sahi, Selenium (navigateur) • Zombie (headless javascript!)

GoutteScenario Feature: Register In order to play the risk game when I want As a site visitor I need to become a site user Scenario: Register as a new user Given I am on "register.php" When I fill in "username" with "blacksun" And I fill in "password" with "password" And I fill in "confirmpassword" with "password" And I fill in "email" with "toto@example.com" And I press "Register" Then I should see "added"

ZombieScenario Feature: Register In order to play the risk game when I want As a site visitor I need to become a site user @mink:zombie Scenario: Register as a new user Given I am on "register.php" When I fill in "username" with "blacksun" And I fill in "password" with "password" And I fill in "confirmpassword" with "password" And I fill in "email" with "toto@example.com" And I press "Register" Then I should see "added"

CodeCoveragePHP_CodeCoverage dans PHPUnitAucune solution prête à l’emploi en TFAIl est très important pour diriger la couverture=> Je fais exécuter Behat dans PHPUnit

PHPCodeCoverageASolution PHPCC init or deserialize Start cc Tests loop Stop cc PHPCC serialize PHPCC deserialize, merge and report

PHPRiskInitialDemo

DynamicAnalysis• Retour sur sonar-project.properties # required metadata sonar.projectKey=my:risk sonar.projectName=PHP Risk sonar.projectVersion=0.3 # path to source directories (required) sources=source # path to test source directories (optional) tests=test # The value of the property must be the key of the language sonar.language=php # PHP plugin specific parameters sonar.phpUnit.mainTestClass=test/functional/BehatTest.php sonar.dynamicAnalysis=true

Adjusting• Je fixe les EOL dans class_risk.php• Je sépare le composant xajax dans un répertoire xajax/• Je mets les contrôleurs de page et les assets dans www/

SonarImprove

SonarDelta

Components&Treeview PHPRisk PHPRisk > [root]

SourcesDetail• Coverage pour xajaxFunctionsDefinition.php

WhatNowDemoUn example : les Hotspots

SonarPHPStateVersion 0.6 : hier !• Introduction des profils par défaut• Prise en compte de toute propriété PHPCSVersion 0.7 : décembre• Prise en compte de toute règle PHPCS et PHPMD• Correction d’un problème très gênant sur le nommage des fichiers• Automatisation Import/Export des règles

BarackObamaFeaturesVersion 1.0 et +• Gestion des fichiers « plats »• Synchronisation avec Eclipse (plugin Eclipse)• Compatibilité avec d’autres plugins Sonar• Utilisation automatique avec un service ci-clould• Nemo pour php

SonarPHPPointers• Les documents d’installation déjà cités• Documentation officielle du plugin • http://docs.codehaus.org/display/SONAR/PHP+Plugin• Billets de blog et articles (certains « historiques ») • « Continuous Integration with Sonar » S. Marek php|architect Avril 2011 • http://www.phparch.com/magazine/2011-2/april/ • http://www.blacksun.fr/2009/12/sonar-for-php-is-on-its-way/ • http://akrambenaissi.wordpress.com/2011/02/23/le-plugin-sonar-php • http://criticallog.thornet.net/2010/11/22/sonar-for-php-coming/

PHPRiskState?Court terme• Continuer le refactoring • (en faire la présentation)• Publier• Améliorer l’usabilitéMoyen terme?• Plus de paramètres (règles, cartes, …)• Nouvelles mappes• Plusieurs AI• Les AI peuvent jouer à distance via REST

Thank you

http://lanyrd.com	62
http://tweetedtimes.com	6
http://a0.twimg.com	4
http://paper.li	1

Suivi qualité avec sonar pour php

by Gabriele Santini on Nov 28, 2011

Accessibility

Categories

Upload Details

Usage Rights

4 Embeds 73

Statistics

Suivi qualité avec sonar pour php Presentation Transcript