[RÉGLÉ] Prototype (Titre) d'une fonction ?

[RobertG]

Bonjour,

Autre souci lors d'un débogage (sous PHP 8.1.11) pour supprimer les erreurs qui pourraient survenir plus tard.
J'ai ce message :

Deprecated: Optional parameter $cid declared before required parameter $order is implicitly treated as a required parameter in D:\wamp64\www\demoj4\administrator\components\com_ rsappt_pro3\models\resources_detail.php on line 338

Cela concerne cette fonction

function saveorder($cid = array(), $order)

Je dois dire que je ne comprends pas pourquoi le fait de supprimer "= array()" fait disparaître l'erreur. Et surtout, qu'est-ce qui fait que $order est considéré comme requis et pas $cid.

Quelqu'un peut-il m'expliquer ?

[pmleconte]

Bonjour Robert,

En PHP, lorsque tu ajoutes =array() à un paramètre, cela signifie que, s'il n'est pas renseigné dans l'appel, il aura une valeur par défaut. Si tu ne mets rien après ton paramètre, il devient obligatoire.

En PHP 8, on est obligé de mettre les paramètres optionnels en fin d'appel de fonction : https://php.watch/versions/8.0/depre...after-optional

En PHP 8.0 => warning
En PHP 8.1 => error

Pascal

[RobertG]

Merci Pascal !

L'ennui, c'est que je ne peux pas inverser ces deux paramètres sans aller chercher tous les appels à la fonction.

[cavo789]

Hello

Je ne l'ai pas encore testé mais sous php 8, tu as dorénavant les paramètres nommés. Si j'ai bien compris ton besoin, c'est ici une solution pour toi.

https://php.developpez.com/actu/307572/Les-arguments-nommes-vont-debarquer-dans-PHP-8-et-promettent-de-nombreux-avantages-comme-la-possibilite-d-ecraser-directement-uniquement-les-valeurs-par-defaut-que-vous-souhaitez-modifier/

Note : on parle du prototype d'une fonction, pas de son "titre" ;-)

[RobertG]

Merci Christophe,

En fait, l'extension fonctionne sans problème sous PHP 8.1 à condition de ne pas activer le rapport d'erreurs.
Dans ce cas, j'essaie de comprendre comment, à part supprimer "= array()", on peut gérer ça au niveau de la fonction sans devoir aller inverser les deux paramètres.

Il y en a d'autres aussi (j'en avais corrigé dans d'autres extensions) où un paramètre n'a pas de valeur par défaut mais où le suivant en a une. Si je me souvient bien, pour régler la question j'attribuais "null" comme valeur par défaut à celui qui n'en avait pas et tout roulait.

Pardon pour mon erreur de nomenclature !

[cavo789]

Dans ce cas, j'essaie de comprendre comment, à part supprimer "= array()", on peut gérer ça au niveau de la fonction sans devoir aller inverser les deux paramètres.

Tu ne peux pas inverser les deux paramètres car, comme tu l'as mentionné, tu devrais reprendre chaque appel pour inverser là-aussi. D'ailleurs pourquoi faire cela ?

Il y en a d'autres aussi (j'en avais corrigé dans d'autres extensions) où un paramètre n'a pas de valeur par défaut mais où le suivant en a une. Si je me souvient bien, pour régler la question j'attribuais "null" comme valeur par défaut à celui qui n'en avait pas et tout roulait.

Quand un paramètre n'a pas de valeur par défaut, c'est donc un paramètre obligatoire. Par contre, quand tu assignes une valeur par défaut, là il devient optionnel et ... il n'est pas possible de déclarer un paramètre obligatoire (pour rappel, qui n'a pas de valeur par défaut) après un paramètre optionnel or c'est exactement ce qu'on voit dans saveorder($cid = array(), $order). $order serait obligatoire et ça, c'est pas logique. Du coup, tu pourrais écrire saveorder($cid = array(), $order=null) puisque, là, on dit que le second paramètre est également optionnel.

Maintenant, quelle valeur devrait avoir $order ? Est-ce qu'il ne faudrait pas écrire saveorder($cid = array(), $order = 'asc') dans l'idée que, si pas précisé, ce serait un tri ascendant ? Ici, je ne pourrais pas t'aider; faudrait lire le code de la fonction pour savoir de quoi il retourne.

[RobertG]

Merci Christophe, je crois que j'ai compris. Reste à savoir si ma mémoire ne sera pas défaillante !

[RobertG]

Bonjour,

Après analyse du code et d'une table concernée, il semble que $order corresponde à la colonne "ordering" définissant la position d'affichage des items de la table en question. Donc si j'ai vraiment compris, on pourrait mettre "$order = 0" (ou 1), sachant que cette position pouvant être définie dans la page de paramétrage de chaque élément, plusieurs items pouvant avoir la même. (dans une table de 10 éléments, cette position allait de 1 à 9, avec deux fois 9 sans que cela pose de problème, sauf que l'affichage par défaut se fait dans l'ordre des noms).