Passage par référence dans un appel de méthode

Version imprimable

J'ai écris une fonction qui permet de faire du rechercher / remplacer sur plusieurs variables d'un coup :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 function replace_all_by($arr_fields, $arr_replace_by){ foreach ($arr_fields as &$field){ foreach ($arr_replace_by as $arr_replace_by_key => $arr_replace_by_value){ $field = str_replace($arr_replace_by_key, $arr_replace_by_value, $field); } } } $explanation_title = str_replace('%explanation_title%', $user->lang['INTRODUCIATOR_MOD_DEFAULT_MESSAGE_TITLE'], $params['explanation_message_title']); $explanation_text = str_replace('%explanation_text%', $user->lang['INTRODUCIATOR_MOD_DEFAULT_MESSAGE_TEXT'], $params['explanation_message_text']); $rules_title = str_replace('%rules_title%', $user->lang['INTRODUCIATOR_MOD_DEFAULT_RULES_TITLE'], $params['explanation_message_rules_title']); $rules_text = str_replace('%rules_text%',generate_text_for_display($forum_rules['rules'], $forum_rules['rules_uid'], $forum_rules['rules_bitfield'], $forum_rules['rules_options']), $params['explanation_message_rules_text']); $link_goto_forum = $user->lang['INTRODUCIATOR_MOD_DEFAULT_LINK_GOTO_FORUM']; $link_post_forum = $user->lang['INTRODUCIATOR_MOD_DEFAULT_LINK_POST_FORUM']; // Replace in each string the predefined fields replace_all_by( array( &$explanation_title, &$explanation_text, &$rules_title, &$rules_text, &$link_goto_forum, &$link_post_forum ), array( '%forum_name%' => $forum_name, '%forum_url%' => append_sid("{$phpbb_root_path}viewforum.$phpEx", 'f=' . $params['fk_forum_id']), '%forum_post%' => append_sid("{$phpbb_root_path}posting.$phpEx", 'mode=post&f=' . $params['fk_forum_id']), ) );
Ce code est une partie d'un MOD Introduciator de phpBB. Or ce code est refusé car :

Citation:

Le passage de variable par référence est déprécié a partir de php 5.4, ton mod ne sera donc pas fonctionnel sous php 5.4 tout comme l'utilisation du mot-clé var qui est propre a php 4 et déprécié (standard strict) en php5 également.

J'ai donc installé la dernière version wampserver qui permet de tester avec PHP version 5.5.12
Et ce code passe sans aucun problème. J'essaie de comprendre ce qui va ou pas, si la remarque est judicieuse ou pas...
Le problème est en fait le array que je crée, justement avec des '&' devant les variables pour passer des références et qui semblerai poser des problèmes sous php 5.4 or ça passe sans problème pour moi...

Donc, est-ce que ce code est valide pour php 5.5, si oui ou non, pourquoi ? J'aimerais comprendre.
La personne qui m'a fait cette remarque, m'a dit C'est quand même mon métier mais sans expliquer. En effet, lorsque je crée une fonction avec des références, si je passe un &$ma_variable à cette fonction php me fait une erreur, mais pour un array, ça a l'air de passer... Peut-être parce que array c'est un mot clef du langage ? Et qu'on a le droit de créer un tableau sur des références ?

Sinon, comment modifier ce code pour qu'il soit conforme ?

06/06/2014, 09h18
Spartacusply

C'est ta manière de faire qui est dépréciée, pas du tout le passage par référence, je t'invite à lire ceci et plus particulièrement :

Citation:

Note: Il n'y a pas de signe de référence dans l'appel de la fonction, uniquement sur sa définition. La définition de la fonction en elle-même est suffisante pour passer correctement des arguments par référence. A partir de PHP 5.3.0, vous devriez recevoir une alerte disant que "Call-time pass-by-reference" est obsolète, lorsque vous utilisez un & dans foo(&$a);. Et à partir de PHP 5.4.0, call-time pass-by-reference a été supprimé, l'utiliser lèvera une erreur fatale.

qui est ce qu'à voulu dire la personne à l'origine de la citation ci-dessus.

Si tu as bien compris ce qui est écrit ci-dessus, corriger ton erreur est très simple.
06/06/2014, 10h55
Feneck91

J'ai très bien compris mon erreur, j'ai très bien compris ce qui est déprécié (ajouter un '&' lors de l'appel car ceci est déjà écrit dans le prototype de la méthode).
Par contre, non, corriger ce n'est pas simple car si j'enlève les & dans le tableau (array), les variables seront passées en valeur et donc, pas d'effet de bord lors de la modification des valeurs dans la fonction et donc lors du retour les variables passées en arguments dans le array créée ne sont pas modifiées...
Si j'ajoute le & dans la fonction : idem. Le tableau est passé par référence mais pas les arguments qui le compose et donc même résultat.
Ou alors passer un $$ ? Un truc dans le genre ?
De plus, pourquoi dans ce cas là, mon PHP qui est en version 5.5 sous wampserver ne provoque aucun warning de dépréciation ? (alors que le fait de faire appel d'une méthode avec un & dans l'appel (et dans la signature) provoque en effet cette erreur).
Tu en penses quoi ?

Voici un code qui est fonctionn et qui est entièrement valide :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 function replace_all_by(&$array) { foreach ($array as &$field) { $field = 'zz '; } } $a = 'a'; $b = 'b'; $c = 'c'; $array = array( &$a, &$b, &$c, ); replace_all_by($array); echo $a; echo $b; echo $c; // Affiche: 'zz zz zz'
Tu ne peut passer directement 'array' en paramètre car il n'y a que les variable qui peuvent être passé par référence (ce qui paraît logique).

06/06/2014, 12h33
Feneck91
Ce que je fais ? Passer un array en copie (par valeur) dont les éléments sont des références. Ça ne pose pas de problème.
Ton code fonctionne aussi mais un appel du type suivant ne fonctionne pas :
Code:

1 2 3 4 5 replace_all_by(array( &$a, &$b, &$c, ));
En effet, cela fonctionne si l'argument array de la fonction replace_all_by est de type passage par copie ($array) et ne fonctionne pas si est de type passage par référence (&$array) car le tableau est construit en même temps que l'appel (en C++ on a un joli warning si on crée une instance d'une classe lors de l'appel à une fonction qui prend une référence en entré (&))

Citation:

Envoyé par Spartacusply

Tu ne peut passer directement 'array' en paramètre car il n'y a que les variable qui peuvent être passé par référence (ce qui paraît logique).

Donc d'après ce que tu dis on peut passer des références dans un tableau d'éléments, ok, je pense que c'est justement cette ligne qui pose problème à mon interlocuteur.
On se rapproche de la vérité ?

En fait en te relisant, en me relisant et relisant sa remarque que je me rend compte que le problème, c'est la remarque, qui en effet n'est pas judicieuse (et même pire, est tout simplement fausse).

Ton code, tout comme un code comme celui-ci est totalement valide php 5.x :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 function replace_all_by($array) { foreach ($array as &$field) { $field = 'zz '; } } $a = 'a'; $b = 'b'; $c = 'c'; replace_all_by(array( &$a, &$b, &$c, )); echo $a; echo $b; echo $c; //Affiche zz zz zz
Cette remarque :

Citation:

Le passage de variable par référence est déprécié a partir de php 5.4, ton mod ne sera donc pas fonctionnel sous php 5.4

est fausse ! Le passage de variable par référence n'est absolument pas déprécié, c'est le passage de référence dans l'appel de la fonction qui est déprécié (qui plus est un code déprécié n'est pas pour autant non fonctionnel mais bon passons...).

En gros, ce code n'est plus valide :
Code:

1 2 3 4 5 function foo(&$var) { $var++; } $a=5; foo (&$a);
mais il suffit d'enlever la référence lors de l'appel de la fonction pour que celui-ci le redevienne.

Bref, ton code est tout à fait valide, et qui plus est ne sera vraisemblablement pas dépréciée avant un bon bout de temps.

06/06/2014, 13h52
Feneck91

J'en était arrivé aux mêmes conclusions que toi... Mais je suis expert C++, pas PHP donc quand il me dit 'C'est quand même mon métier' ... et bien je veux bien douter de mes compétences PHP...

Voici la version sans référence, sans doute un peu plus proche des "Coding Guidelines", mais beaucoup moins "user-friendly" :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
function foo($array) {
    $res = array();
    foreach ($array as $val) {
        $res[] = ++$val;
    }
    return $res;
}
 
$a = 2;
$b = 5;
$c = 10;
 
list($a, $b, $c) = foo(array($a, $b, $c));
 
echo "$a\n";
echo "$b\n";
echo "$c\n";

06/06/2014, 14h49
Feneck91

Je vais garder ma fonction user friendly qui est correcte au niveau syntaxe et non déprecated...
Je n'ai pas envie de tout changer et faire des trucs pas super propre alors que ce code est correct...
Merci beaucoup de ton aide.