Discussion :

[Igoussam]

Bonsoir tt le monde,
j'ai un probleme je voudrais modifier un employe mais quand j'execute il me donnee cette erreur je comprend pas pourquoi:Fatal error:

Call to a member function setcin() on a non-object in C:\wamp\www\Application\employe\update.php on line 6

j'ai utilise deux fonction la fonction read qui permet de recuperer les donne a partir de l'id et update qui fait la mise a jour voila le code:
page:manager.php

Code :

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
 public function update(employe $employe){
$this->makes();
$this->st=$this->pdo->prepare("update employe set cin=:cin,nom_em=:nom_em,id_grade=:id_grade,id_affectation=:id_affectation,adress=:adress where id_em=:id_em");
$this->st->bindvalue(':cin',$employe->getcin(),pdo::PARAM_STR);
$this->st->bindvalue(':nom_em',$employe->getnom_em(),pdo::PARAM_STR);
$this->st->bindvalue(':id_grade',$employe->getid_grade(),pdo::PARAM_INT );
$this->st->bindvalue(':id_affectation',$employe->getid_affectation(),pdo::PARAM_INT );
$this->st->bindvalue(':adress',$employe->getadress(),pdo::PARAM_STR);
$exe=$this->st->execute();
 
}
public function read($id_em){
$this->makes();
$this->st=$this->pdo->prepare("select * from employe where id_em=:id_em");
$this->st->bindvalue(':id_em',$id_em,pdo::PARAM_INT );
$exe=$this->st->execute();
}

et voici la page update.php qui contient l'appel de la fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?php
 
include'manager.php';
$manager=new manager();
$co=$manager->read($_POST["id_em"]);
$co->setcin($_POST["cin"]);
$co->setnom_em($_POST["nom_em"]);
$co->setid_grade($_POST["id_grade"]);
$co->setid_affectation($_POST["id_affectation"]);
$co->setadress($_POST["adress"]);
$manager->update($co);
?>

Et merci

[Tsilefy]

Ta fonction read() ne retourne rien. Il faut que tu récupères la valeur retournée par la requête et la retournes.

[gene69]

Salut, c'est bel effort d'avoir parametrisée tes requetes, il faut maintenant que tu valides le contenues de $_POST et $_GET avant de les utilisers.

apres dans ton design, je ne comprends pas pourquoi tu construit objet manager() sans parametres d'initialisation et tu t'obliges à tout rattraper avec des setters, souvent un objet mal initialisé est la source de tous les bugs comme le tient.

ma suggestion est d'éviter les setters comme la peste et ne les conserver que sur des parametres non vitaux de l'objet (option de debug, seuil de limitation de ressources...), de passer toutes les méthodes qui se passent des attributs de la classe en static pour pouvoir l'utiliser en dehors du contexte de l'objet, quitte à lui passer un parametre.

Dans ton cas j'aurai mis une méthode static qui prends un tableau en entrée disons $unsafe, je valide les valeurs du tableau avec filter_array, j'émets une exception si la validation est KO sinon j'instancie un objet de la classe (ici manager). à l'appel j'utilise Manager::createFromArray($_POST); et je suis sur d'avoir un bel objet pas avec des données moisies dedans parce qu'un setter aura pas bien marché (oubien j'ai une exception à gérer).

Certain disent qu'il faut rendre les objets immuables. je n'irai pas jusque là.

https://youtu.be/8d2AtAGJPno?t=7m39s

[Igoussam]

Merci les amis 😉😉