Discussion :

[Louprouge]

Bonjour,

Je veux utiliser la fonction de classe

Code php :

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

1
2
 
$work->setid(25);

Mais j'ai une erreur 500 alors que si j'assigne la valeur directement dans ma classe, ça passe. Je ne comprend pas pourquoi.

Voici le code que j'ai:

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
 
class Maclasse () {
    public function setId(int $id) {        // assigne l'identifiant de la classe
        if ($id == '-1') {              // si il est négatif, on va ajouter un membre à la collection
            $this->_id = null;
        } else {                        // sinon, on utilise l'identifiant reçu
            $this->_id = $id;
        }
 
}
 
$work = new Maclasse();
$id = $_POST['id']    // vaut '25'
 
$work->setId($id);  // erreur 500 depuis le serveur ====> Call to a member function setId() on null
 
$work->_id = $id;  // Fonctionne correctement

[laurentSc]

Bonjour,

tu ne donnes pas le code complet, vu que par exemple, il manque la déclaration de la propriété public _id;, mais un truc qui me gêne, c'est la parenthèse derrière la déclaration de la classe (tu mets class Maclasse () {... au lieu de class Maclasse {...). Aucune idée si ton erreur vient de là, mais corrige...

[Dendrite]

Laurent a raison. A ceci près que c'est protected _id.
De plus, il te manquait une parenthèse fermante.
Tu devrais activer les messages d'erreur.

Code :

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

1
2
3
4
5
6
7
8
9
10
class Maclasse {
  protected int _id;
  public function setId(int $id) {        // assigne l'identifiant de la classe
    if ($id == '-1') {              // si il est négatif, on va ajouter un membre à la collection
        $this->_id = null;
    } else {                        // sinon, on utilise l'identifiant reçu
        $this->_id = $id;
    }
  }
}

[laurentSc]

OK pour protected à la place de public, mais la parenthèse,manquante, c'est où ?

[ProgElecT]

OK .... , mais la parenthèse,manquante, c'est où ?

lige 11, fermeture du bloc class Maclasse () {

[laurentSc]

Merci, je cherchais une parenthèse, pas une accolade

[Dendrite]

Ah vi, accolade, parenthèse, pardon, je mélange tout le temps les deux noms.

[cavo789]

Bonsoir

Tu as défini ta fonction setId() comme requérant un entier comme paramètre et c'est super d'avoir fait cela !

Reste que tu as fait le boulot à moitié

Du coup, tu dois forcer le type de ta variable comme ci-dessous afin que le compilateur comprenne bien que $id est un entier.

$id = (int) $_POST['id'];Bonne soirée

[Louprouge]

Bonjour à tous,

merci pour vos réponses.
Le code que j'ai partagé avec vous était un assemblage manuel de la partie de code posant problème car mes classes sont beaucoup plus grandes et je ne voulais pas encombrer mon post avec du code inutile.
Du coup, j'ai introduit des erreurs lors de la rédaction du post. Mes parenthèses et accolades sont bien équilibrées dans mon code, mais vous ne pouviez pas le savoir.

Je vous prie de m'en excuser.

@cavo789: code corrigé à ce niveau au niveau du typage, merci.

J'ai fini par trouver l'erreur : Je dérive mes programmes en copiant un code source d'un programme précédent qui est relativement proche de celui que je vais écrire et j'en retire tout ce qui est inutile... Ou supposé tel.
Ici, j'avais retiré l'appel au script de configuration, qui contient mon autoloader.

Du coup, impossible d'instancier mes classes, ce qui a conduit à l'erreur qui me rendait chèvre.

Merci pour votre aide.

[laurentSc]

@cavo789 : du PHP compilé ? C'est nouveau ?

Tu proposes $id = (int) $_POST['id'];. Ca serait pas plutôt $id = (int) $id; ?

[cavo789]

Salut

Du code compilé et oui. C'est le opcode.

Pour le code php proposé, il est parfaitement juste. Ton code aussi mais ce que je propose sur une ligne tu le fais sur deux.