La programmation défensive (ou l’art de ne pas (se) faire confiance)

Douze années d'expérience dans les métiers du développement PHP : beaucoup de ressenti, des succès, des échecs. Des projets internes, pour des tiers, des audits, de la formation, seul, (dans|au dessus|a coté) d'une équipe. Bref, un lot d'aventures qui forgent le caractère.

Tout cela a profondément marqué ma façon d'écrire des lignes de code, ajusté mon critère "qualité" et plus généralement, ma façon de concevoir des applications PHP. Comment est-ce que j'en suis arrivé à ne plus faire confiances aux autres (moi compris) et pourquoi ? Qu'est-ce que je cherche à garantir au travers de cette approche du développement ?

Nous verrons ensemble comment utiliser les règles de la programmation défensive et comment "protéger son code" dans le but d'en assurer la pérénnité tout en garantissant les objectifs suivants : un niveau de qualité sur le long terme, la possibilité de faire des évolutions simplement et un code sur lequel on aime intervenir.

Transcript

1. La programmation défensive ou l’art de ne pas (se) faire confiance
40. class PragmaticController extends AbstractController { public function indexAction() { $foobar = $this->get('foobar'); } }
50. GOOOOOOOO !
53. use Webmozart\Assert\Assert; class Employee { public function __construct($id) { Assert::integer($id, 'The employee ID must be an integer. Got: %s'); Assert::greaterThan($id, 0, 'The employee ID must be a positive integer. Got: %s'); } }
54. new Employee('foobar'); // => InvalidArgumentException: // The employee ID must be an integer. Got: string new Employee(-10); // => InvalidArgumentException: // The employee ID must be a positive integer. Got: -10
81. use function Safe\file_get_contents; use function Safe\json_decode; // This code is both safe and simple! $content = file_get_contents('foobar.json'); $foobar = json_decode($content);
83. PHP-CS-Fixer
84. phpstan.org
85. rector
88. // SPDX-License-Identifier: GPL-3.0 pragma solidity >=0.5.0 <0.7.0; contract C { function f(uint a, uint b) public pure returns (uint) { return a * (b + 42); } }
89. <?php /** @psalm-external-mutation-free */ class Counter { private int $count = 0; public function increment() : void { $this->count++; } public function getCount() : int { return $this->count; } }
90. <?php /** @psalm-pure */ function makeCounter() : Counter { $a = new Counter(); $a->increment(); // this is fine return $a; }
91. <?php /** @psalm-pure */ function takeCounter(Counter $a) : Counter { $a->increment(); // Counter already exists, this is a mutation return new Counter(); }