Dans mon nouveau boulot, j’ai choisi de faire une V3 de la plateforme de site, basée sur l’existant.
Je rencontre énormément de code inutile, complexe ou …. inutilement complexe !
Dans cette dernière catégorie, voici deux exemples tirés du fichier VOD.class.php
Une fonction établit une note aux VOD en fonction des taux de clics.
Code Pourri :
$rating += 2;
if($rating > 5) {
$rating = 5;
}
Bon code : $rating = min(5, $rating +2);
Dans la même fonction :
Code pourri :
//Retrouve le rating a partir de ce chiffre
if($val < 0.2) {
$rating = 1;
}
elseif($val >= 0.2 && $val < 0.4) {
$rating = 2;
}
elseif($val >= 0.4 && $val < 0.6) {
$rating = 3;
}
elseif($val >= 0.6 && $val < 0.8) {
$rating = 4;
}
elseif($val >= 0.8) {
$rating = 5;
}
Code correct : $rating = ceil(5*$val);
La différence est édifiante…..
Laisser un commentaire