Le document présente une liste des principaux pièges à éviter en PHP, discutés lors d'un événement AFUP en 2019. Il aborde des erreurs courantes de syntaxe, d'efficacité de code et d'utilisation de fonctions, tout en fournissant des exemples de code et des solutions. En outre, il met en avant l'importance de l'analyse statique du code pour améliorer la performance et la modernisation du code PHP.

1. Top 10
chausse-trappes
en PHP
AFUP Aix-Marseille 2019

2. Agenda
1 heure
Top 10+ chausses-trappes PHP
Améliorez votre code dès maintenant
Ca peut attendre demain

3. QUI PARLE?
➤ Damien Seguy
➤ Directeur technique Exakat
➤ Analyse statique de code PHP
➤ Maison de retraite pour elephpant

4. Erreur tapie dans le code
Un puit de performances
Un outil pratique
La voie de l'éléphpant
🐞
🛠
🐘
🚀

5. La légende de strpos()
<?php
if (strpos($string, 'a')) { }
if (strpos($string, 'a') == 0) { }
if ($x = strpos($string, 'a')) { }
🐞

6. Comparaison strpos
<?php
// La seule comparaison possible pour 0/false
if (strpos($string, 'a') === false) { }
// Pas de 0, pas de confusion
if (strpos($string, 'a') == 2) { }
// strpos n'est pas la seule!
if (preg_match($regex, $string)) { }
🐘

7. comparaison strpos
array_search()
collator_compare()
collator_get_sort_key()
current()
fgetc()
file_get_contents()
file_put_contents()
fread()
iconv_strpos()
iconv_strrpos()
imagecolorallocate()
imagecolorallocatealpha()
mb_strlen()
next()
pcntl_getpriority()
preg_match()
prev()
readdir()
stripos()
strpos()
strripos()
strrpos()
strtok()
curl_exec()

8. Define()
<?php
define('A', true);
?>
🚀

9. Define() vers const
<?php
const A = true;
?>
🐘

10. Define() en déclin
<?php
define('A', 3, true);
define($x, $y);
?>
🛠

11. define() vers const
<?php
const A = true;
const B = 33 + 12 * (23 - 34);
const C = array(A, B, D::E);
const D = A ? B : C;
?>
🛠

12. Affichages répétés
<?php
print 'a';
print $b ;
print 'c';
?>
🚀

13. Affichages répétés
<?php
print 'a' . $b . 'c';
?>
🚀

14. Affichages répétés
<?php
echo 'a' , $b , 'c';
?>
🐘

15. Affichages répétés
<?php
echo 'a',
$b ,
'c';
?>

16. echo ne fonctionne pas
<?php
echo( 'a',
$b ,
'c',
);
?>

17. <?php
$b = 3; $c = 1;
$a1 = $b and $c;
$a2 = $b && $c;
?>
🐞
La logique en toutes lettres

18. La logique en toutes lettres
<?php
$b = 3; $c = 1;
$a1 = ($b and $c);
$a2 = $b && $c;
?>
🐘

19. L'ordre en toutes lettres
<?php
echo -3 ** 2;
$x = new stdClass();
var_dump(!$x instanceof stdClass);
?>
🐞

20. Loin du réel
<?php
// Ne plus utiliser
if (!is_real($a)) {
$a = (real) $a;
}
?>
🐞

21. Ca flotte…
<?php
if (!is_float($a)) {
$a = (float) $a;
}
?>
🐘
// Toujours valide en 7.4

22. Se mesurer au réel
<?php
$a = 1/7;
$b = 2.0;
var_dump( 7 * $a === 1);
var_dump( (0.1 + 0.7) * 10 == 8);
var_dump( (0.1 + 0.7) * 10);
?>
🐞

23. Se mesurer au réel
<?php
$a = 1/7;
$c = ((0.1 + 0.7) * 10) - 8;
// calcul de précision
var_dump( abs($c) < 0.0001);
?>
🐘

24. Substr(,,1) ?
<?php
$string = "abcde";
echo substr($string, $pos, 1);
?>
🚀

25. Substr(,,1) ?
<?php
$string = "abcde";
echo substr($string, $pos, 1);
echo $string[$pos];
?>
🐘

26. Substr(,,1) ?
<?php
$string = "abcde";
echo substr($string, -1, 1);
echo $string[-1];
?>
🐘

27. Substr(,,1) ?
<?php
$string = "ab⼈人cde";
echo substr($string, $pos, 1);
echo $string[$pos];
echo mb_substr($string, $pos, 1);
// $pos = 1 => bbb
// $pos = 2 => ??⼈人
?>
🐘

28. Toujours substr()
<?php
$result = substr(strtolower($s), $o, $l);
?>
🚀

29. Substr() d'abord!
<?php
$result = strtolower(substr($s, $o, $l));
?>
🐘

30. array_slice() d'abord!
<?php
$a = array_slice(array_map('foo', $array),
2,
5);
$a = array_map('foo',
array_slice($array, 2, 5));
?>
🚀
🐘

31. array_merge() en boucle
<?php
$res = $sqlite3->query(
'SELECT lists FROM table');
while ($row = $res->fetchArray(PDO_ASSOC)) {
$l = explode(',', $row['value']);
$final = array_merge($final, $l);
}
?>
🚀

32. array_merge() en boucle
<?php
$res = $sqlite3->query(
'SELECT lists FROM table');
while ($row = $res->fetchArray(PDO_ASSOC)) {
$l = explode(',', $row['value']);
$a []= $l;
}
$final = array_merge(...$a);
?>
🐘

33. concat en boucle
<?php
$res = $sqlite3->query(
'SELECT value FROM table');
while ($row = $res->fetchArray(PDO_ASSOC)) {
$a .= $row['value'];
}
?>
🚀

34. concat en boucle
<?php
$res = $sqlite3->query(
'SELECT value FROM table');
$a = array();
while ($row = $res->fetchArray(PDO_ASSOC)) {
$a []= $row['value'];
}
$final = implode('', $a);
?>
🐘

35. addition en boucle
<?php
$res = $sqlite3->
query('SELECT quantite FROM table');
while ($row = $res->fetchArray(PDO_ASSOC)) {
$a += $row['quantite'];
}
?>
🐘
Vu
A
la
tElE

36. <?php
function g1() : Generator {
for ($i = 0; $i < 4; ++$i ) { yield $i; }
}
function g2() : Generator {
for ($i = 5; $i < 10; ++$i ) { yield $i; }
}
function aggregator() : Generator {
yield from g1();
yield from g2();
}
print_r(iterator_to_array());
Yield la clé 🐞

37. /*
Array
(
[0] => 6
[1] => 7
[2] => 8
[3] => 9
[4] => 4
[5] => 5
)
*/
Yield la clé

38. <?php
function g1() : Generator {
for ($i = 0; $i < 4; ++$i) { yield $i => $i; }
}
function g2() : Generator {
for ($i = 5; $i < 10; ++$i) { yield $i => $i;}
}
function aggregator() : Generator {
yield from g1();
yield from g2();
}
print_r(iterator_to_array());
Yield la clé 🐘

39. /*
0
1
2
3
4
5
6
7
8
9
*/
Yield la clé

40. Le sous-motif qui manque
<?php
preg_match('/(a)(b)?/', 'abc', $r);
/*
Array
(
[0] => ab
[1] => a
[2] => b
)
*/
?>
🐞

41. Le sous-motif qui manque
<?php
preg_match('/(a)(b)?/', 'adc', $r);
/*
Array
(
[0] => a
[1] => a
)
*/
🐞

42. Le sous-motif qui manque
<?php
preg_match('/(a)(b)?(.)/', 'adc', $r);
/*
Array
(
[0] => ad
[1] => a
[2] =>
[3] => d
)
*/
🐘

43. La marque sans-nom™
<?php
preg_match('/(?<ici>a)(b)?(.)/', 'adc', $r);
preg_match("/(?'ici'a)(b)?(.)/", 'adc', $r);
print_r($r);
/*
Array
(
[0] => ad
[ici] => a
[1] => a
[2] =>
[3] => d
)
*/
🐘

44. Ne pas unset() les propriétés
<?php
class x {
public $y = 1;
}
$x = new x();
unset($x->y);
print_r($x);
?>
x Object
(
)
🐞

45. Ne pas unset() les propriétés
<?php
class x {
public $y = 1;
}
$x = new x();
$x->y = null;
var_dump(isset($x->y));
print_r($x);
?>
bool(false)
x Object
(
[y] =>
)
🐘

46. Ne pas unset() les propriétés
<?php
class x {
public $y = 1;
}
$x = new x();
$x->z = 3;
print_r($x);
?>
x Object
(
[y] => 1
[z] => 3
)
🛠

47. Propriété non-initialisée
<?php
class x {
function foo() {
if (!isset($this->y)) {
$this->y = 0;
}
return $this->y++;
}
?>
🚀

48. Propriété non-initialisée
<?php
class x {
public $y = 0;
function foo() {
if ($this->y === null) {
$this->y = 0;
}
}
return $this->y++;
}
?>
🐘

49. Le mois prochain
<?php
echo date('F',
strtotime('+1 month',
mktime(0,0,0,$i,31,2019)));
?>
// 1er janvier => 1er février
// 31 octobre => 1er décembre
// 31 janvier => 2 ou 3 mars
🐞

50. Le mois prochain
<?php
echo date('F',
strtotime('first day of next month',
mktime(0,0,0,$i,31,2019)));
?>
// 1er janvier => 1er février
// 31 octobre => 1 novembre
// 31 janvier => 1er février
🐘

51. C'est quand, demain?
<?php
$demain = time() + 86400;
?>
🐞

52. C'est quand, demain?
<?php
$demain = new DateTime('tomorrow');
?>
🐘

53. Combien dure mon script?
<?php
$debut = microtime(true);
// grosse masse de code magnifique
$fin = microtime(true);
print number_format(($fin - $debut),
2)
.'ms';
?>
🐞

54. Combien dure mon script?
<?php
$debut = hrtime(true);
// grosse masse de code magnifique
$fin = hrtime(true);
print number_format(($fin - $debut) / 1000000,
2)
.'ms';
?>
🐘

55. Et maintenant ?
<?php
// version rapide
$a = time();
// version lente
$b = strtotime('now');
?>
🚀

56. La vengence de la
référence
<?php
$a = range(0, 3);
foreach($a as &$b) { }
foreach($a as $b) { }
print_r($a);
?>
Array
(
[0] => 0
[1] => 1
[2] => 2
[3] => 2
)
🐞

57. La vengence de la
référence
<?php
$a = range(0, 3);
foreach($a as &$b) { }
unset($b);
foreach($a as $b) { }
print_r($a);
?>
Array
(
[0] => 0
[1] => 1
[2] => 2
[3] => 3
)
🐘

58. list() en PHP 4
<?php
list($a, $b) = array( 1, 2, 3);
?>
🛠

59. list() en PHP 5
<?php
[$a, $b] = [ 1, 2, 3];
?>
🛠
🐘

60. list() en PHP 7
<?php
['c' => $a, 'e' => ['d'=> $b]] =
['c' => 1,
'e' =>['d' => 2,
'f' => 3,]
,];
?>
🛠
🐘

61. list() en PHP 7
<?php
['e' => $a, 'e' => ['d'=> $b]] =
['c' => 1,
'e' =>['d' => 2,
'f' => 3,]
,];
?>
🛠
🐘

62. List() pour le futur!
<?php
$res = $sqlite3->query(
'SELECT lists FROM table');
foreach($res->fetchArray(SQLITE3_ASSOC)
as ['lists' => $list]) {
$l = explode(',', $list);
$a[] = $l;
}
$final = array_merge(…$a);
?>
🛠
🐘

63. Encore plus ?
🛠
🐘Exakat
Analyse statique de votre code
Prépare pour PHP 7.4 / 8.0 (parbleu!)
Modernise votre code
https://www.exakat.io/

64. Gramaci!
@exakat