Manuel:Conventions de codage/PHP

From MediaWiki.org
Jump to navigation Jump to search
This page is a translated version of the page Manual:Coding conventions/PHP and the translation is 99% complete.
Outdated translations are marked like this.
Other languages:
English • ‎dansk • ‎español • ‎français • ‎čeština • ‎русский • ‎中文 • ‎日本語
Raccourci : CC/PHP

Cette page décrit les conventions de codage utilisées pour les fichiers de la base de code de MediaWiki écrits en PHP. Voir aussi les conventions générales qui s'appliquent à tous les langages de programmation, y compris PHP. Si vous désirez un résumé des éléments à vérifier lorsque vous faites vos validations (submit), voyez la Pre-commit checklist .

Pour aider les développeurs à corriger leur code du point de vue répartition des espaces, il existe l'outil stylize, qui vérifie la syntaxe réellement et ajoute les espaces aux endroits nécessaires; cet outil est également disponible via un interface web. Il est recommandé de l'exécuter avant de valider les fichiers dans Git.

Les règles pour PHP_CodeSniffer sont aussi disponibles. Pour plus d'informations, voir Continuous integration/PHP CodeSniffer . Le dépôt Git https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/tools/codesniffer contient les paramètres de PHP_CodeSniffer pour le code MediaWiki.

Structure du code

Espaces

MediaWiki encourage un style bien structuré au niveau des espaces pour une lisibilité optimale.

Placez les espaces de chaque côté des opérateurs binaires, par exemple :

// No:
$a=$b+$c;

// Yes:
$a = $b + $c;

Placez les espaces à côté de l'intérieur des parenthèses, sauf si elles sont vides. Ne mettez pas d'espace après un nom de fonction.

$a = getFoo( $b );
$c = getBar();

Placez les espaces à l'intérieur des crochets lorsque vous déclarez un tableau, sauf si le tableau est vide. Ne mettez pas d'espace dans les crochets lorsque vous accédez aux éléments d'un tableau.

// Yes
$a = [ 'foo', 'bar' ];
$c = $a[0];
$x = [];

// No
$a = ['foo', 'bar'];
$c = $a[ 0 ];
$x = [ ];

Les structures de contrôle telles que if, while, for, foreach, switch, ainsi que le mot clé catch , doivent être suivies d'un espace :

// Yes
if ( isFoo() ) {
	$a = 'foo';
}

// No
if( isFoo() ) {
	$a = 'foo';
}

Lorsque le 'cast' est appliqué sur un type, ne mettez pas d'espace à l'intérieur ni après l'opérateur cast :

// Yes
(int)$foo;

// No
(int) $bar;
( int )$bar;
( int ) $bar;

Dans les commentaires, il doit y avoir un espace entre le caractère # (ou //) et le commentaire lui-même.

// Yes: Proper inline comment
//No: Missing space
/***** Do not comment like this ***/

Pour aider les développeurs à corriger le code du point de vue de l'emplacement des espaces, il existe l'outil stylize qui ajoute automatiquement des caractères d'après les conventions PHP. Cet outil est disponible via l'interface web.

Opérateur ternaire

Le ternary operator peut être utilisé à profit si les expressions sont très courtes et triviales :

$title = $page ? $page->getTitle() : Title::newMainPage();

Toutefois, si vous envisagez une expression multiligne avec un opérateur ternaire, utilisez plutôt un bloc if (). N'oubliez pas que l'espace disque n'est pas cher, que la lisibilité du code est primordiale, if est en anglais mais ?: ne l'est pas. Si vous utilisez une expression ternaire sur plusieurs lignes, le point d'interrogation et les deux points doivent figurer au début des deuxième et troisième lignes et non à la fin des première et deuxième (contrairement à la convention JavaScript de MediaWiki).

Depuis que MediaWiki nécessite PHP 7.0.13 ou ultérieur, l'utilisation du shorthand ternary operator (opérateur ternaire ?:) aussi connu sous le nom d'opérateur elvis et introduit en PHP 5.3, est autorisée.

Depuis PHP 7.0, l'opérateur de fusion nul est également disponible et peut remplacer l'opérateur ternaire dans certains cas d'utilisation. Par exemple, au lieu de :

$wiki = isset( $this->mParams['wiki'] ) ? $this->mParams['wiki'] : false;

vous pouvez écrire ceci :

$wiki = $this->mParams['wiki'] ?? false;

Chaînes de caractères

Il est recommandé d'utiliser les apostrophes simples dans tous les cas où elles sont équivalentes aux doubles. Le code utilisant des guillemets simples est moins sujet aux erreurs et plus facile à relire, car il ne peut pas contenir accidentellement de séquences d'échappement ni de variables. Par exemple, l'expression régulière "/\\n+/" nécessite une barre oblique inverse supplémentaire, ce qui la rend légèrement plus confuse et source d'erreurs que '/\n+/'. De même, pour les personnes utilisant les claviers qwerty US/UK, ils sont plus faciles à taper car ils évitent d’appuyer sur la touche Maj.

Néanmoins, n'ayez pas peur d'utiliser la fonctionalité PHP d'interpolation des chaînes à double apostrophes : $elementId = "myextension-$index"; Ceci a des caractéristiques de performance quelque peu meilleures que l'équivalent utilisant l'opérateur de concatenation '.' (dot) , et c'est aussi plus joli.

Les chaînes de style Heredoc ('Here document' ou document en ligne) sont parfois utiles :

$s = <<<EOT
<div class="mw-some-class">
$boxContents
</div>
EOT;

Certains auteurs aiment utiliser END comme balise de fin, qui est également le nom d'une fonction PHP. Cela conduit à des conversations IRC comme suit :

<Simetrical>	vim also has ridiculously good syntax highlighting.
<TimStarling>	it breaks when you write <<<END in PHP
<Simetrical>	TimStarling, but if you write <<<HTML it syntax-highlights as HTML!
<TimStarling>	I have to keep changing it to ENDS so it looks like a string again
<brion-codereview>	fix the bug in vim then!
<TimStarling>	brion-codereview: have you ever edited a vim syntax script file?
<brion-codereview>	hehehe
<TimStarling>	http://tstarling.com/stuff/php.vim
<TimStarling>	that's half of it...
<TimStarling>	here's the other half: http://tstarling.com/stuff/php-syntax.vim
<TimStarling>	1300 lines of sparsely-commented code in a vim-specific language
<TimStarling>	which turns out to depend for its operation on all kinds of subtle inter-pass effects
<werdnum>	TimStarling: it looks like some franken-basic language.

Fonctions et paramètres

Evitez de passer un trop grand nombre de paramètres aux fonctions at aux constructeurs :

// Constructor for Block.php from 1.17 to 1.26. *DON'T* do this!
function __construct( $address = '', $user = 0, $by = 0, $reason = '',
	$timestamp = 0, $auto = 0, $expiry = '', $anonOnly = 0, $createAccount = 0, $enableAutoblock = 0,
	$hideName = 0, $blockEmail = 0, $allowUsertalk = 0
) {
	...
}

Il devient rapidement impossible de se souvenir de l'ordre des paramètres et vous finirez inévitablement par coder toutes les valeurs par défaut des appelants pour personnaliser un paramètre à la fin de la liste. Si vous êtes tenté de coder une fonction comme celle-ci, envisagez plutôt de passer un tableau associatif de paramètres nommés.

En général, l'utilisation de paramètres booléens est déconseillée dans les fonctions. Dans $object->getSomething( $input, true, true, false ), sans consulter la documentation de MyClass::getSomething(), il est impossible de savoir ce que ces paramètres sont censés indiquer. Il vaut bien mieux utiliser les constantes de classe et créer un paramètre de drapeau générique :

$myResult = MyClass::getSomething( $input, MyClass::FROM_DB | MyClass::PUBLIC_ONLY );

Ou pour que votre fonction accepte un tableau de paramètres nommés :

$myResult = MyClass::getSomething( $input, [ 'fromDB', 'publicOnly' ] );

Essayez de ne pas modifier les variables au cours d'une fonction et évitez de modifier les paramètres transmis à une fonction (à moins qu'ils ne soient passés par référence et que c'est évidemment le but de la fonction).

Expressions d'assignement

Utiliser une assignation comme une expression est surprenant pour le lecteur et ressemble à une erreur. N'écrivez pas ce type de code :

if ( $a = foo() ) {
    bar();
}

L'espace n'est pas cher et vous écrivez rapidement, utilisez donc à la place :

$a = foo();
if ( $a ) {
    bar();
}

Utiliser l'assignation dans une boucle while() est légitime pour l'itération :

$res = $dbr->query( 'SELECT * FROM some_table' );
while ( $row = $dbr->fetchObject( $res ) ) {
    showRow( $row );
}

Ceci est inutile dans le nouveau code; remplacez par :

$res = $dbr->query( 'SELECT * FROM some_table' );
foreach ( $res as $row ) {
    showRow( $row );
}

Emprunts au langage C

Le langage PHP a été conçu par des personnes qui aiment le langage C et qui souhaitent importer des souvenirs de ce langage en PHP. Mais PHP a quelques différences importantes par rapport à C.

En C, les constantes sont implémentées en tant que macros de préprocesseur et sont rapides. En PHP, elles sont implémentées en effectuant une recherche dans une table de hachage à l'exécution avec le nom de la constante et sont plus lents qu'en utilisant simplement un littéral de chaîne. Dans la plupart des endroits où vous utiliseriez un ensemble de macros en C ou similaires à enum, vous pouvez utiliser des littéraux de chaîne en PHP.

PHP a trois littéraux spéciaux pour lesquels la combinaison des majuscules/minuscules n'est pas significative dans le langage (depuis PHP 5.1.3), mais pour lesquels notre convention utilise toujours les minuscules: true, false et null.

Utilisez elseif et non pas else if. Ils ont des significations différentes et subtiles:

// Ce :
if ( $foo == 'bar' ) {
	echo 'Hello world';
} else if ( $foo == 'Bar' ) {
	echo 'Hello world';
} else if ( $baz == $foo ) {
	echo 'Hello baz';
} else {
	echo 'Eh?';
}

// Est actuellement équivalent à :
if ( $foo == 'bar' ) {
	echo 'Hello world';
} else {
	if ( $foo == 'Bar' ) {
		echo 'Hello world';
	} else {
		if ( $baz == $foo ) {
			echo 'Hello baz';
		} else {
			echo 'Eh?';
		}
	}
}

Et le dernier a de piètres performances.

Syntaxe alternative pour les structures de contrôle

PHP offre une syntaxe alternative pour les structures de contrôle en utilisant des points-virgule ';' et des mots-clés tels que endif, endwhile, etc. :

if ( $foo == $bar ):
    echo "<div>Hello world</div>";
endif;

Cette syntaxe doit être évitée car elle empêche beaucoup d'éditeurs de texte de faire correspondre et de refermer automatiquement les accolades. Les accolades doivent être utilisées à la place :

if ( $foo == $bar ) {
    echo "<div>Hello world</div>";
}

Placement des parenthèses

Voir indentation et alignement.

Déclarations de type dans les paramètres de fonction

Utilisez les déclarations de type et les déclarations du type retourné (type hinting) si nécessaire.

Notez que avant 7.4 PHP ne pouvait pas gérer la restriction/relaxation du type retourné dans les sous-classes.

Scalar typehints are allowed as of MediaWiki 1.35, following the switch to PHP 7.2 (T231710).

Use PHP 7.1 syntax for nullable parameters: choose

public function foo ( ?MyClass $mc ) {}

instead of

public function foo ( MyClass $mc = null ) {}

The former conveys precisely the nullability of a parameter, without risking any ambiguity with optional parameters. IDEs and static analysis tools will also recognize it as such, and will not complain if a non-nullable parameter follows a nullable one.

Nommage

Utilisez les minuscules alternées (lowerCamelCase) pour les noms de fonction ou de variable. Par exemple :

private function doSomething( $userPrefs, $editSummary )

Utilisez des majuscules alternées (UpperCamelCase) pour les noms des classes : class ImportantClass. Utilisez les majuscules avec des caractères souligné '_' pour les constantes de classe et les constantes globales : DB_MASTER, Revision::REV_DELETED_TEXT. Les autres variables sont habituellement en minuscules ou en lowerCamelCase; évitez d'utiliser les soulignés dans les noms de variables.

Il existe aussi des préfixes utilisés à différents endroits :

Préfixes de fonctions

  • wf (fonctions wiki) – fonctions de niveau général, par exemple
    function wfFuncname() { ... }
    
  • ef (fonctions d'extensions) = les fonctions globales dans les extensions, bien que dans la plupart des cas, le style moderne déclare les fonctions d'accroche comme des méthodes statiques de classe, laissant peu ou pas de fonctions brutes de premier niveau ainsi nommées. (-- brion dans Manual_talk:Coding_conventions#ef_prefix_9510)

Préférez les phrases verbales : utilisez getReturnText() au lieu de returnText().

Préfixes de variables

  • $wg – les variables globales, par exemple $wgVersion, $wgTitle. Utilisez toujours ceci pour les nouvelles variables globales, de sorte qu'il soit facile de repérer les déclarations « global $wgFoo » manquantes. Dans les extensions, le nom de l'extension doit être utilisé comme séparateur d'espace de noms. Par exemple, $wgAbuseFilterConditionLimit, et non $wgConditionLimit.
  • Les déclarations globales doivent être au début d'une fonction pour que les dépendances puissent être déterminées sans avoir à lire l'intégralité de la fonction.

Il est courant de travailler avec une instance de la classe Database; nous avons pour cela une convention de nommage qui permet de garder une trace de la nature du serveur auquel nous sommes connectés. Cela revêt une importance particulière dans les environnements répliqués, tels que Wikimedia et d’autres grands wikis; dans les environnements de développement, il n'y a généralement pas de différence entre les deux types, ce qui peut masquer des erreurs subtiles.

  • $dbw – un objet Database pour l'écriture (une connexion maître)
  • $dbr – un objet Database pour la lecture indépendante des accès concurrents (cela peut être un esclave en mode lecture seule, discrètement derrière l'état maître, donc n'essayez jamais d'écrire dans la base de données avec, ou essayez d'obtenir une réponse d'« autorisation » pour les requêtes importantes comme les droits d'accès et les états bloqués)

Vous encore pouvez trouver ce qui suit dans du code ancien, mais pour du nouveau code, cela n'est pas recommandé :

  • $ws – Variables de session, par exemple $_SESSION['wsSessionName']
  • $wc – Variables de cookies, par exemple $_COOKIE['wcCookieName']
  • $wp – Variables postées (envoyées par les champs de formulaire), par exemple $wgRequest->getText( 'wpLoginName' )
  • $m – variables membres d'objets : $this->mPage. Ceci n'est pas du tout recommandé dans le nouveau code, mais essayez de rester cohérent à l'intérieur d'une même classe.

Pièges

empty()

La fonction empty() ne doit être utilisée pour ignorer les erreurs. Sinon utilisez simplement ! (conversion booléenne).

  • empty( $var ) essentially does !isset( $var ) || !$var.
    Cas d'utilisation courant : clés de configuration booléennes optionnelles valant 'false' par défaut. $this->enableFoo = !empty( $options['foo'] );
  • Méfiez-vous des pièges des conversions booléennes.
  • Cela supprime les erreurs concernant les propriétés et les variables non définies. Si vous souhaitez uniquement tester les non définis, utilisez !isset(). Si vous souhaitez uniquement tester les valeurs « vides » (par exemple, false, zéro, tableau vide, etc.), utilisez !.

isset()

N'utilisez pas isset pour tester null. L'utilisation de isset dans cette situation pourrait entraîner des erreurs en masquant les noms de variables mal orthographiées. Au lieu de cela, utilisez $var === null.

Conversion booléenne

if ( !$var ) {
    
}
  • N'utilisez pas ! ni empty pour tester si une chaîne ou un tableau est vide, car PHP considère '0' comme 'false' – mais '0' est un titre valide et un nom d'utilisateur valide dans MediaWiki. A la place, utilisez === '' ou === [] .
  • Etudiez les règles à propos de la conversion en booléen. Faites attention quand vous convertissez des chaînes en boléens.

Autre

  • L'opérateur '+' des tableaux ne renumérote pas les clés des tableaux indexés numériquement, donc [ 'a' ] + [ 'b' ] === [ 'a' ]. Si vous voulez renuméroter les clés, utilisez array_merge() : array_merge( [ 'a' ], [ 'b' ] ) == [ 'a', 'b' ]
  • Assurez-vous que error_reporting() vaut -1. Ceci vous notifiera s'il existe des variables non définies et des autres pièges subtils que PHP ignore. Voir aussi Manuel:Comment déboguer .
  • Lorsque vous travaillez avec un fichier PHP pur (c'est à dire pas un modèle HTML), omettez les balises ?> fermantes. Ces balises causent souvent des problèmes avec les espaces de fin et les messages d’erreur en-tête déjà envoyés (voir bugzilla:17642 et http://news.php.net/php.general/280796). La convention, dans le contrôle de version, pour les fichiers est qu'il ont généralement une nouvelle ligne à la fin du fichier (que les éditeurs peuvent ajouter automatiquement), ce qui déclencherait alors cette erreur.
  • N'utilisez pas la syntaxe goto() introduite en 5.3. PHP a peut-être introduit la fonctionnalité, mais cela ne signifie pas que nous devons l'utiliser.
  • Si vous rencontrez un tableau, ne le passez pas par référence sauf si vous le devez. Et même, faites attention des conséquences (voir http://www.king-foo.be/2010/10/php-quirks-passing-an-array-by-reference/).
  • PHP vous permet de déclarer des variables statiques même au sein d'une méthode non statique d'une classe. Cela a conduit à des bugs subtils dans certains cas, car les variables sont partagées entre les instances. Là où vous n'utilisez pas une propriété private static, n'utilisez pas non plus de variable statique.

Opérateurs d'égalité

Faites attention avec les opérateurs de comparaison à double signe égal. Le triple-égal (===) est généralement plus intuitif et doit être préféré, à moins que vous ayez une raison particulière d'utiliser le double-égal (==).

  • 'foo' == 0 est vrai (!)
  • '000' == '0' est vrai (!)
  • '000' === '0' est faux
  • Pour vérifier que deux scalaires que vous supposez numériques sont égaux, utilisez ==, par exemple 5 == "5" est vrai.
  • Pour vérifier que deux variables sont de type 'string' et représentent la même séquence de charactères, utilisez ===, par exemple "1.e6" === "1.0e6" est faux.

Précision des nombres JSON

JSON utilise le système des types JavaScript, donc tous les chiffres sont représentés comme des nombres IEEE 64bits à virgule flottante. Cela signifie que plus les nombres deviennent grands, plus ils perdent en precision, jusqu'au point de où vous ne pouvez plus les différencier : les nombres au delà de 2^52 ont une précision inférieure à ±0,5 et donc un entier très grand peut se transformer en un nombre différent. Pour éviter ce problème, représentez les nombres entiers potentiellement très grands, par des chaînes en JSON.

A faire et à ne pas faire

Ne pas utiliser la construction lors de la sérialisation

Le mécanisme de sérialisation intégré de PHP (les fonctions serialize() et unserialize()) ne doivent pas être utilisées pour l'enregistrement des données (ou leur lecture) en dehors du processus actuel. A la place, utilisez la sérialisation basée sur JSON (néanmoins, faites attention aux pièges). Ceci est la règle établie par la RFC T161647.

La raison est double : (1) les données sérialisées avec ce mécanisme ne peuvent pas être désérialisées de manière fiable avec une version ultérieure de la même classe. Et (2), des données sérialisées spécialement construites peuvent être utilisées pour exécuter du code malveillant, ce qui pose un risque grave pour la sécurité.

Quelques fois, votre code ne va pas contôler le mécanisme de sérialization, mais utilisera une bibliothèque ou un pilote qui lui l'utilisera de manière interne. Dans de tels cas, il faut procéder par étapes pour réduire les risques. Le premier problème mentionné ci-dessus peut être évité en convertissant toute donnée en des tableaux ou des objets simples anonymes avant la sérialisation. Le deuxième problème peut éventuellement être résolu en utilisant la fonctionnalité de liste blanche introduite par PHP 7 pour la désérialisation.

Commentaires et documentation

Il est essentiel que votre code soit bien documenté afin que les autres développeurs et les correcteurs de bogues puissent facilement naviguer dans la logique de votre code. Les nouvelles classes, méthodes et variables membres doivent inclure des commentaires fournissant une brève description de leurs fonctionnalité (sauf si cela est évident), même si elles sont privées. En outre, toutes les nouvelles méthodes doivent documenter leurs paramètres et renvoyer des valeurs.

Nous utilisons le style de documentation Doxygen (il est très similaire à PHPDoc pour le sous-ensemble que nous utilisons) pour produire une documentation générée automatiquement à partir des commentaires du code (voir Manual:mwdocgen.php). Commencez un bloc de commentaires Doxygen avec /**, au lieu du formatage de style Qt /*!. Les commandes structurelles de Doxygen commencent par @nom de balise. (utilisez pour le caractère d'échappement plutôt @ que \ – les deux styles sont acceptés dans Doxygen, mais pour raison de compatibilité arrière et avant, MediaWiki a choisi le style @param ). Ils organisent la documentation générée (en utilisant @ingroup) et en identifiant les auteurs (en utilisant les balises @author).

Ils décrivent une fonction ou une méthode, leurs paramètres (en utilisant @param), et ce qu'elles retournent (en utilisant @return). Le format pour les paramètres est :

@param type $paramName Description of parameter

Si un paramètre peut avoir plusieurs types, séparez-les avec le caractère pipe '|', par exemple :

@param string|Language|bool $lang Language for the toc title, defaults to user language

Continuez les phrases appartenant à une annotation sur la ligne d'après, indentée avec un espace supplémentaire.

Pour chaque interface publique (méthode, classe, variable, peu importe) que vous ajoutez ou modifiez, fournissez une balise @since VERSION , ainsi les personnes qui reprendront le code via cet interface sauront qu'ils rompent la compatibilité avec les anciennes versions du code.

class Foo {

	/**
	 * @var array $bar: Description here
	 * @example [ 'foo' => Bar, 'quux' => Bar, .. ]
	 */
	protected $bar;

	/**
	 * Description here, following by documentation of the parameters.
	 *
	 * Some example:
	 * @code
	 * ...
	 * @endcode
	 *
	 * @since 1.24
	 * @param FooContext $context context for decoding Foos
	 * @param array|string $options Optionally pass extra options. Either a string
	 *  or an array of strings.
	 * @return Foo|null New instance of Foo or null of quuxification failed.
	 */
	public function makeQuuxificatedFoo( FooContext $context = null, $options = [] ) {
		/* .. */
	}

}

FIXME signifie généralement que quelque chose est mauvais ou cassé. TODO signifie que des améliorations sont nécessaires. cela ne signifie pas nécessairement que la personne qui ajoute le commentaire va le faire. HACK signifie qu'une solution rapide, mais inélégante, maladroite ou autrement sous-optimale, à un problème immédiat a été apportée, et qu'une réécriture plus complète du code devrait éventuellement être effectuée.

Entêtes des fichiers source

Afin d'être compatible avec la plupart des licences, vous devez avoir quelque chose de similaire à ce qui suit (spécifique aux applications GPLv2) au début de chaque fichier source.

<?php
/**
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License along
 * with this program; if not, write to the Free Software Foundation, Inc.,
 * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 * http://www.gnu.org/copyleft/gpl.html
 *
 * @file
 */

Balises Doxygen

Nous utilisons les annotations suivantes reconnues par Doxygen. Utilisez-les dans cet ordre, par mesure de cohérence :

Niveau du fichier :

  • @file
  • @ingroup
  • @author

Classe, membre de classe, ou membre global :

  • @todo
  • @var
  • @deprecated
  • @class
  • @see
  • @since
  • @param
  • @return
  • @throws
  • @private

Annotations de test

Dans les tests, nous utilisons entre autres les annotations suivantes. Ce n'est pas simplement de la documentation, elles ont un sens pour PHPUnit et ont un impact sur l'exécution du test.

  • @depends
  • @group
  • @covers
  • @dataProvider
  • @expectedException
  • @expectedExceptionMessage

Intégration

Il existe dans le code MediaWiki quelques éléments de code destinés à être autonomes et facilement portables vers d'autres applications. Bien que certains d’entre eux existent maintenant sous la forme de bibliothèques séparées, d'autres restent dans l’arborescence source de MediaWiki (par exemple les fichiers de /includes/libs). En dehors de ceux-ci, le code doit être intégré au reste de l'environnement MediaWiki et doit permettre à d'autres domaines de la base de code de s'y intégrer en retour.

Visibilité

Rendez les méthodes 'public', 'protected', ou 'private' (selon le rôle de la méthode). Réfléchissez : il ne s'agit pas de rendre tout 'public' !

Objets globaux

N'accédez pas directement aux superglobales PHP $_GET, $_POST, etc. utilisez $exemple à la place; il existe diverses fonctions selon le type de valeur que vous souhaitez. Vous pouvez obtenir une requête WebRequest à partir du RequestContext le plus proche, ou RequestContext::getMain() si cela est absolument nécessaire. De même, n’accédez pas directement à $_SERVER; utilisez $request->getIP() si vous voulez obtenir l'adresse IP de l'utilisateur actuel.

Méthodes statiques et propriétés

Les méthodes statiques et les propriétés peuvent être utilisées en PHP, mais faites attention lorsque vous héritez de faire la différence entre les mots-clé self et static . self va toujours faire référence à la classe où il a été défini, tandis que static va faire référence à la sous-classe particulière qui l'a invoqué. Voir la documentation PHP sur la résolution statique à la volée (Late Static Bindings) pour plus de détails.

Classes

Encapsulez votre code dans une classe orientée objet ou ajoutez des fonctionnalités aux classes existantes; n'ajoutez pas de nouvelles fonctions ni variables globales. Essayez de faire attention à la distinction entre les classes 'backend', qui représentent des entités dans la base de données (par exemple, User, Block, Revision,etc.) et les classes 'frontend', qui représentent des pages ou des interfaces visibles pour l'utilisateur (SpecialPage, Article, ChangesList, etc. . Même si votre code n'est pas clairement orienté objet, vous pouvez le placer dans une classe statique (par exemple, IP ou Html).

En raison de l'absence de méthodes et de membres de classe privés dans PHP 4, le code anciens sera marqué avec des commentaires tels que /** @private */ pour indiquer l'intention; respecter cela comme si c'était imposé par l'interpréteur.

Marquez le nouveau code avec des modificateurs de visibilité adaptés, y compris public si c'est le cas, mais n'ajoutez jamais de visibilité à du code existant sans vérifier, tester et refactoriser si nécessaire. Il est bon d'éviter les modifications de visibilité sauf si vos modications cassent de toutes manières, le fonctionnement précédent de la fonction.

Gestion des erreurs

Ne supprimez pas les erreurs avec l'opérateur @ de PHP , pour aucune raison ever. C'est cassé dès lors que E_STRICT est activé et provoque une erreur pas tracée et non expliquée, ce qui est fatal et difficile à investiguer.

La méthode propre pour gérer les erreurs est d'abord de gérer les erreurs. Par exemple, si vous envisagez d'utiliser un opérateur de suppression d'erreur pour supprimer un avertissement d'index de tableau invalide, vous devez plutôt effectuer un test isset sur l'index du tableau avant d'essayer d'y accéder. Dans la mesure du possible, il faut toujours éviter les erreurs PHP en faveur de leur récupération et ensuite de leur traitement. Cela rend le code plus compréhensible et évite d'utiliser des méthodes lentes de suppression d'erreur.

Si et seulement si il y a une situation où vous attendez un avertissement PHP inévitable, vous pouvez utiliser Wikimedia\AtEase\AtEase::suppressWarnings() et Wikimedia\AtEase\AtEase::restoreWarnings() de la bibliothèque at-easy au lieu de l'opérateur @. Ceci est pour les cas où :

  1. Il est impossible d'anticiper l'erreur qui va arriver, et
  2. Vous envisagez de gérer l'erreur de manière appropriée une fois qu'elle est arrivée.

Un exemple de ceci est l'ouverture de fichier avec fopen(). Vous pouvez anticiper l'erreur en appelant file_exists() et is_readable(), mais il est possible que le fichier ait été supprimé entre le moment du test et l'appel actuel de fopen() .

Dans ce cas, votre code doit être prêt à gérer le cas où le fichier échoue à l'ouverture, et vous pouvez utiliser MediaWiki\suppressWarnings() pour empêcher PHP d'être trop bruyant.

Notez que l'utilisation de Wikimedia\AtEase\AtEase::suppressWarnings() et Wikimedia\AtEase\AtEase::restoreWarnings() sont lentes. Ils sont beaucoup plus lents que l’utilisation de l’opérateur @, car ils impliquent au moins quatre appels de fonctions et un comptage des références, plutôt qu’un simple code d'opération.

Si votre code rencontre une erreur spontanée, vous devez lever une exception plutôt que d'utiliser le trigger_error de PHP. Le gestionnaire d'exceptions affichera ceci aussi bien que possible à l'utilisateur final et à l'administrateur du wiki, et fournira également une trace de la pile pour les développeurs. Ne levez pas une instance d'Exception elle-même, mais utilisez plutôt l'une de ses sous-classes. Les exceptions qui indiquent des erreurs de programmation doivent correspondre à l'une des exceptions fournies avec PHP ou à une sous-classe plus spécifique, tandis que les exceptions indiquant des erreurs pertinentes pour l'utilisateur final doivent être de type ErrorPageError ou de l'une de ses sous-classes.

Voir aussi