Discussion :

[laurentSc]

Bonjour,

je justifie mon statut de (vieux) débutant : je souhaite créer une fonction qui ajoute une donnée à un tableau.
Voici mon code actuel :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
public function FeedIDApplication( $appli_name, $ID)
   {
       $ID[]=$appli_name;
   }
 
//l'appel à la fonction :
$LCSVInBDD->FeedIDApplication( $applicationname, $this->id_application);

Avec ce code, le tableau $this->id_application n'est pas alimenté. Comment faut faire, SVP ?

[laurentSc]

Bonsoir,

j'ai trouvé une solution qui vaut ce qu'elle vaut (vous avez le droit de critiquer), mais elle fait le boulot :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
public function FeedIDApplication( $appli_name, $ID)
{
    $size_array=count($ID);
 
    $result = array_pad($ID,$size_array+1 ,$appli_name);
 
    return($result);}
 
//l'appel à la fonction :
$this->id_application=$LCSVInBDD->FeedIDApplication( $applicationname, $this->id_application);

[cavo789]

Dans ton premier post, tu définis deux paramètres "par valeur": publicfunction FeedIDApplication($appli_name, $ID). Par valeur et donc ce qui est fait "dans la fonction" reste dans la fonction. Ce que tu cherchais à faire, c'est possible mais par référence : public function FeedIDApplication( $appli_name, &$ID) (note le ampersand devant la seconde variable)

Là, cela aurait pû fonctionne (**je n'ai pas testé**).

Toutefois, ce type d'appel devrait être limité au maximum. Dans ton premier post, tu as une fonction qui ne retourne rien; il suffirait que tu fasses un return; ce que tu fais dans ton deuxième exemple.

Au-delà de tes exemples: tu cherches donc à ajouter une entrée à un tableau; tu peux faire ça simplement : $this->id_application[] = 'PHP Doc';Quelques remarques :

* Ton code est "PHP 4" (ou pire); permets-moi d'être direct, c'est pour insister sur le fait que nous avons PHP 7 depuis un paquet d'années maintenant et nous avons les types de données p.ex. (string, array, ...). Ta fonction serait plus propre comme ceci : public function feedIDApplication(string $appli_name, array $ID): array; tu définis le type des paramètres et tu définis ce que tu renvoies,
* Une fois tu utilises $appli-name et une autre fois, pour strictement la même variable $applicationname,
* Ta fonction semble être plus un helper qu'une fonction d'une classe puisqu'il n'y a aucune dépendance à une donnée de la classe; du coup $LCSVInBDD->FeedIDApplication pourrait devenir MonHelper::FeedIDApplication (il faut définir une nouvelle classe MonHelper avec une fonction publique et statique FeedIDApplication* Ta variable $size_array=count($ID); ne sert pas à grand chose, juste un appel dans la fonction array_pad($ID,$size_array+1 ,$appli_name);; du coup, tu peux optimiser avec array_pad($ID,count($ID)+1 ,$appli_name);; cette syntaxe n'est pas plus compliquée à lire, tu te passes d'une variable qui finalement ne servait à rien,
* ...

Je ne vais pas te suggérer de lire encore et encore des tutoriels PHP 7, je suppose que tu l'as déjà fait.

Un conseil différent du coup : beaucoup plus compliqué à mettre en oeuvre mais si tu réussis à le faire, le logiciel https://github.com/rectorphp/rector est un must have (un coach hors pair !) pour apprendre à mieux coder. Mais, oui, il faut déjà une certaine maîtrise; à garder sous le coude.

Bonne soirée.