Variable '$Machines' is undefined

Version imprimable

Voici le code :

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
30
31
 
      /** Chargement de la liste des machines
         * @var Site_Class $Site : Objet portant le site
         * @var ArrayObject $Machines[] : Liste des machines
         * @var PDOStatement $result : Objet PDO
         * @var int $i : Index pour la liste des machines
         */
    public function MachinesListe() :void
    {
        global $Site;
        $Site->Bdd->Phrase_SQL = "
			SELECT machines.Id, machines.Nom, CONCAT(element_url.URL,'?',element_url.URL_Parametres) AS URL 
			FROM machines
			LEFT JOIN element_url ON (element_url.Element_Table_Id = 7 AND element_url.Element_Id = machines.Id)
			INNER JOIN machines_constructeurs ON ((machines_constructeurs.Constructeur_Id=" . $this->Id . ") AND (machines.Id=machines_constructeurs.Machine_Id)) 
			ORDER BY machines.Nom";
        $result = $Site->Bdd->Requete($Site->Bdd->Phrase_SQL);
        $i = 0;
        while ($row = $result->fetch()) {
            $Machines[$i] = new Machine_Class();
            $Machines[$i]->Id = $row['Id'];
            $Machines[$i]->Nom = utf8_encode_perso($row['Nom']);
            $Machines[$i]->URL = $row['URL'];
            $i ++;
        }
        if (isset($Machines)) {
            $this->Machines = $Machines;
        } else {
            unset($this->Machines);
        }
    }

Eclipse me dit que la variable $Machines n'est pas définie. Pourtant il me semble que je l'ai définie via :

Code:

* @var ArrayObject $Machines[] : Liste des machines

et j'ai essayé aussi :

Code:

* @var ArrayObject $Machines : Liste des machines

mais rien n'y fait. Toutes mes variables sont bien définies sauf les array.

Quelqu'un a une idée ?

24/09/2018, 14h50
hocineaminoo

pourquoi vous avez Choisis ce code pour définir votre Variable ?

@var ArrayObject

et pourquoi vous n'avez pas choisis ce code ??? $var = array() ;

Je ne vois pas d'initialisation de $Machine dans le code.

Moi perso j'initialise mes variables en début de fonction, mais vous pourriez faire ceci pour ne plus avoir l'erreur

Code:

1
2
3
4
5
6
7
8
9
10
11
 
...
 $i = 0;
 $Machines = [];
        while ($row = $result->fetch()) {
            $Machines[$i] = new Machine_Class();
            $Machines[$i]->Id = $row['Id'];
            $Machines[$i]->Nom = utf8_encode_perso($row['Nom']);
            $Machines[$i]->URL = $row['URL'];
            $i ++;
        }

24/09/2018, 17h57
berthos

Citation:

Envoyé par hocineaminoo

pourquoi vous avez Choisis ce code pour définir votre Variable ?

@var ArrayObject

et pourquoi vous n'avez pas choisis ce code ??? $var = array() ;

Tout d'abord merci pour votre réponse.

J'avais bien entendu essayé cette solution mais cela ne fonctionne pas. La variable n'est toujours pas reconnue.

* &var .... est un commentaire.

Et comme ça ?

Code:

1
2
3
4
5
6
7
8
9
 
  $Machines = array();
        while ($row = $result->fetch()) {
            $Machines[$i] = new Machine_Class();
            $Machines[$i]->Id = $row['Id'];
            $Machines[$i]->Nom = utf8_encode_perso($row['Nom']);
            $Machines[$i]->URL = $row['URL'];
            $i ++;
        }

Ooopss je n'avais pas vu la réponse de Maître Pylos tout comme berthos apparemment.

25/09/2018, 09h31
berthos

Bonjour,

Oui j'avais déjà essayé ça marche mais je trouve cela dommage de redéfinir à nouveau la variable.

Merci beaucoup pour votre aide.
Bonne journée

Bonjour MaitrePylos,

Oui cela fonctionne mais je pensais que l'initialisation faite suffisait :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
class Constructeur_Class extends Element_Class
{
    // Attributs
    public $Machines = array();
    public $Class_Table_Id = 2;
    // Constantes
    // Méthodes
        /**Construction de l'objet
         *
         */
    function __construct()
    {
        parent::__construct();
        $this->Table_Get($this->Class_Table_Id);
        unset($this->Class_Table_Id); // On supprime ce champ car on a chargé la table
    }
        /** Chargement de la liste des machines
         * @var Site_Class $Site : Objet portant le site
         * @var ArrayObject $Machines[] : Liste des machines
         * @var PDOStatement $result : Objet PDO
         * @var int $i : Index pour la liste des machines
         * @var ArrayObject $row : Liste du tableau des machines
         */

25/09/2018, 18h33
badaze
@berthos.

Avec
Code:

1 2 public $Machines = array();
Tu initialises la variable de classe $Machines accessible ensuite avec $this->Machines.

$Machine et $this->Machines ne sont pas la même chose !

He ben oui, si tu ne donnes que 50% des infos, tu auras d'office une mauvaise réponse.
Comme le dit @badaze, il faut utiliser le $this

Code:

1
2
3
4
5
6
7
8
9
10
11
 
...
 $i = 0;
 
        while ($row = $result->fetch()) {
            $this->Machines[$i] = new Machine_Class();
            $this->Machines[$i]->Id = $row['Id'];
            $this->Machines[$i]->Nom = utf8_encode_perso($row['Nom']);
            $this->Machines[$i]->URL = $row['URL'];
            $i ++;
        }