Programmation Orienté Objet Résolu

Question

Bonjour,
Je débute en POO et je souhaiterai pouvoir afficher tous les membres, pouvez vous m'aider un peu en me donnant quelques explications un peu détaillé s'ils vous plait.

J'ai ma class User, avec quelques méthodes, le systéme de connexion est fait j'affiche les infos du membre connecté comme ceux-ci.

<td><?php echo $user->getNom(); ?></td>

Et pour afficher tous les membres vous pouvez me donner quelques explications, car je nage malgrès beaucoup de lecture de tutos sur le sujet, mais en ayant des exemple concret j'y verrai plus clair.
En vous remerciant beaucoup.

totodunet · Answer

salut

je ne connais pas la structure de ta classe mais ce que tu veux là ça sent la boucle for. tu veux parcourir un à un les utilisateurs et les afficher. le tout étant de savoir où et dans quoi sont stockés tes utilisateurs

labourette · Answer

Bonjour
En fait j'ai ma class User

class User {

    private $id;
    private $nom;
    private $prenom;
    private $email;
    private $username;
    private $password;
    private $hash;

    private static $db;

    public function __construct($id = null) {
        self::init();

        //is_int -- Détermine si une variable est de type nombre entier
        if(is_int($id)) {
            try {
            $this->load($id);
            }
            catch(Exception $e) {
            throw $e;
            }
        }
    }

    public function init() {
        if(self::$db == null) self::$db = new Database();
    }

    private function load($id) {
        $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE id = :id');
        $req->bindParam(':id', $id, PDO::PARAM_INT);
        $req->execute();
        $d = $req->fetch(PDO::FETCH_ASSOC);

        if(empty($d)) throw new Exception('Utilisateur introuvable');
        $this->setUserDatas($d);
    }

    public function setNom($nom) {
        $this->nom = trim($nom);
    }

    public function getNom() {
        return $this->nom;
    }   

    public function setPrenom($prenom) {
        $this->prenom = trim($prenom);
    }

    public function getPrenom() {
        return $this->prenom;
    }
    
    public function setEmail($e) {
        $this->email = trim($e);
    }

    public function getEmail() {
        return $this->email;
    }    

    public function setUsername($u) {
        $this->username = trim($u);
    }

    public function getUsername() {
        return $this->username;
    }

    public function setPassword($p) {
        $this->password = trim($p);
    }

    public function getPassword() {
        return $this->password;
    }
    
    public function setHash($h) {
        $this->hash = $h;
    }

    private function setUserDatas($d) {
        if(!empty($d) && is_array($d)) {
            $this->id           = $d['id'];
            $this->nom          = $d['nom'];
            $this->prenom       = $d['prenom'];
            $this->email        = $d['email'];
            $this->username     = $d['username'];
            $this->password     = $d['password'];
        }
    }
    
    public function checkHash($hash) {
        return $this->hash == $hash;
    }

    public static function get($username) {
        self::init();

        $req = self::$db->prepare('SELECT id_adherent FROM tb_adherent WHERE username = :u');
        $req->bindParam(':u', $username, PDO::PARAM_STR);
        $req->execute();

        $id = $req->fetchColumn();

        return ($id) ? new self((int) $id) : null;
    }

    public static function exists($username) {
        self::init();

        $username = trim($username);

        $req = self::$db->prepare('SELECT COUNT(*) FROM tb_adherent WHERE username = :user');
        $req->bindParam(':user', $username, PDO::PARAM_STR);
        $req->execute();
        $count = $req->fetchColumn();

        return ($count == 0) ? false : true;
    }

    public function login() {
        $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE username = :u AND password = :p');
        $req->bindParam(':u',           $this->username,        PDO::PARAM_STR);
        $req->bindParam(':p',           $this->password,        PDO::PARAM_STR);
        $req->execute();
        $d = $req->fetch(PDO::FETCH_ASSOC);

        $this->setUserDatas($d);

        return (!empty($d));
    }

    public function save() {
        $req = self::$db->prepare('INSERT INTO tb_adherent (nom, prenom, email, username, password) VALUES (:nom, :prenom, :e, :u, :p)');
        $req->bindParam(':nom',         $this->nom,             PDO::PARAM_STR);
        $req->bindParam(':prenom',      $this->prenom,          PDO::PARAM_STR);
        $req->bindParam(':e',           $this->email,           PDO::PARAM_STR);        
        $req->bindParam(':u',           $this->username,        PDO::PARAM_STR);
        $req->bindParam(':p',           $this->password,        PDO::PARAM_STR);
        $req->execute();

        $req = self::$db->prepare('SELECT id_adherent FROM tb_adherent WHERE username = :u');
        $req->bindParam(':u', $this->username, PDO::PARAM_STR);
        $req->execute();

        $this->id = $req->fetchColumn();
    }

    public function update() {
        $req = self::$db->prepare('UPDATE tb_adherent SET nom = :nom, prenom = :prenom, username = :u, email = :e, password = :p WHERE id = :id');
        $req->bindParam(':id',          $this->id,              PDO::PARAM_INT);        
        $req->bindParam(':nom',         $this->nom,             PDO::PARAM_STR);
        $req->bindParam(':prenom',      $this->prenom,          PDO::PARAM_STR);
        $req->bindParam(':e',           $this->email,           PDO::PARAM_STR);        
        $req->bindParam(':u',           $this->username,        PDO::PARAM_STR);
        $req->bindParam(':p',           $this->password,        PDO::PARAM_STR);
        $req->execute();
    }
}

ensuite j'ai mon fichiet listeMembre ou je souhaite afficher tous les membres
j'ai fait j'ai essayé de faire ça peux tu me dire comment je pourrais faire.
Merci beaucoup
.

	<div id="ListeMembre">
		<table id="user_list" cellpadding="0" cellspacing="1">
			<tr>
				<th width="30%">Nom</th>
				<th width="30%">Prénom</th>
				<th width="40%">Email</th>
			</tr>
			<?php
        while($donnees = $req -> fetch(PDO::FETCH_ASSOC)) {
        
		?>
	<tr>
		<td><?php echo $donnees->nom; ?></td>
		<td><?php echo $donnees->prenom; ?></td>
		<td><?php echo $donnees->email; ?></td>

	</tr>
<?php
		}
		?>
		</table>
	</div>

fetch(PDO::FETCH_ASSOC)) { ?>

Nom	Prénom	Email
nom; ?>	prenom; ?>	email; ?>

totodunet · Answer

j'aurais plutôt récupérer toutes les données d'un coup puis ensuite parcourir la table

$donnees=$req->fetchAll(PDO::FETCH_ASSOC);

for(i=0;i<count($donnees);i++){
 echo '<tr>';
  echo '<td><'.$donnees[i]['nom'].'></td>';
  echo '<td><'.$donnees[i]['prenom'].'></td>';
  echo '<td><'.$donnees[i]['email'].'></td>';

 echo '</tr>';
}

méfies toi, $donnees n'etait pas un objet de la class User mais un tableau associatif et donc tu ne pouvait pas y appeler des méthodes.
si tu veux que PDO te renvoie un objet de la classe User tu peux faire ça :
$donnees=$req->fetchObject('User');

après ça reste une solution parmi d'autres

Qui ne tente rien n'a rien

labourette · Answer

Bonjour
Déjà merci pour ton aide.
Malheureusement ça ne fonctionne pas.
Alors voilà j'ai créé une méthod dans la class User. A partir de mon fichier Membre j'aurai souhaité appelé cette méthod pour afficher les membres.
Ma fonction est-elle correct, ensuite pour l'appel à cette fonction je voyais un truc du genre...

$d = new User();
$d->ListeMembre($id);

public function ListeMembre($id) {
        $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE id = :id');
        $req->bindParam(':id', $id, PDO::PARAM_INT);
        $req->execute();
        while($d = $req -> fetch(PDO::FETCH_ASSOC)) {
            $this->nom          = $d['nom'];
            $this->prenom       = $d['prenom'];
            $this->email        = $d['email'];
            $this->fixe         = $d['fixe'];
            $this->mobile       = $d['mobile'];
        }
    }

Merci vraiment de ton aide

totodunet · Answer

oui ça ne marche pas car j'ai omis de mettre les $ derrière ma variable i

for($i=0;$i<count($donnees);$i++){
 echo "<tr>";
  echo "<td><".$donnees[$i]['nom']."></td>";
  echo "<td><".$donnees[$i]['prenom']."></td>";
...

ton truc pourrait marcher mais l'inconvénient et de devoir justement utiliser un objet pour afficher d'autres objets de sa même instance. La preuve, tu as créer un objet User uniquement pour pouvoir afficher la liste. ou bien tu fais une fonction dite "statique" dans la classe User, comme ça tu n'a pas à utiliser d'objet et tu l'appelle directement par :
User::ListeMember($id);

mais ce que tu venais de faire tout au début était très bien aussi. tu as juste à remplacer dans ton while

while($donnees = $req -> fetch(PDO::FETCH_ASSOC))

par

while($donnees = $req -> fetchObject('User'))

et aussi que tu accèdes aux attributs par tes getters que tu as créé dans ta classe (sinon ils servent à rien^^) car tu ne peux pas accéder directement à l'attribut nom par exemple en faisant :

$donnees->nom

car nom est private

il faut faire

$donnees->getNom()

Qui ne tente rien n'a rien

labourette · Answer

Bonjour
voici ce que j'ai fait et me retourne des erreurs
ma méthod dans la page class

    public static function ListeMembre($id) {
        $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE id = :id');
        $req->bindParam(':id', $id, PDO::PARAM_INT);
        $req->execute();
        while($d = $req -> fetchObject('User')) {
            $this->id           = $d['id'];
            $this->nom          = $d['nom'];
            $this->prenom       = $d['prenom'];
            $this->email        = $d['email'];
            $this->fixe         = $d['fixe'];
            $this->mobile       = $d['mobile'];
        }
    }

mon appel à cette fonction est dans mon fichier ListeMembre.php

<?php
User::ListeMember($id); 
?>
	<tr>
		<td><?php echo $d->getNom(); ?></td>
		<td><?php echo $d->getPrenom(); ?></td>
		<td><?php echo $d->getEmail(); ?></td>

	</tr>

Merci pour tes explications et bien entendu ton aide, parce que je commence à comprendre certaines choses mais pour le mettre en pratique c'est une autre histoire.

totodunet · Answer

ce que je te propose si tu tiens à ta fonction ListeMember :

public static function ListeMembre($id) {
        $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE id = :id');
        $req->bindParam(':id', $id, PDO::PARAM_INT);
        $req->execute();
        while($d = $req -> fetchObject('User')) {
            echo '<tr>';
            echo '<td>'.$d->getNom().'</td>';
            echo '<td>'.$d->getPrenom().'</td>';
            echo '<td>'.$d->getEmail().'</td>';
            echo '</tr>';
        }
}

puis tu appelles ta fonction dans ta page :

<?php
User::ListeMember($id);?>

au fait c'est quoi l'attribut id, parce que si c'est un numéro unique pour chaque instance, il est normal qu'il y aura qu'une ligne au résultat

labourette · Answer

Bonjour, youpi super ça marche un très grand merci. Je ne sais pas si c'est une des meilleures façon mais ça fonctionne comme ça. public static function ListeMembre() { $req = self::$db->prepare('SELECT * FROM tb_adherent'); $req->bindParam(':id', $id, PDO::PARAM_INT); $req->execute(); while($d = $req -> fetchObject('User')) { echo ''; echo ''.$d->getNom().''; echo ''.$d->getPrenom().''; echo ''.$d->getEmail().''; echo ''; } } appel de la fonction

Nom	Prénom	Email
nom; ?>	prenom; ?>	email; ?>

Merci

totodunet · Answer

salut,

c'est très bien. pour simplifier tu peux enlever la ligne inutile 
$req->bindParam car tu te fiches de l'id dans ta requête sql

et pourquoi tu crées un nouvel objet User d ? Le principe de la fonction statique c'est que justement tu n'a pas à créer d'objet pour l'utiliser. On appelle ça une méthode de classe.

labourette · Answer

Bonjour
Peux tu me donner un peu des explications afin d'éviter de créer le nouvel objet ça serai vraiment mieux. J'ai regardé quelques tutos j'ai du mal à saisir les explications données.
Merci

totodunet · Answer

en gros tu enlèves juste la ligne

$d=new User();

tu vois que $d tu ne vas pas l'utiliser après.

quelques explications :

getNom() par exemple est une méthode dans ta classe qui a pour but de renvoyer le nom de l'objet. Seulement pour renvoyer le nom de l'objet, il faut qu'il ait l'objet en question donc :

$d->getNom();

c'est une méthode qui n'ait pas statique car elle lui faut un objet pour qu'elle puisse fonctionner (normal^^)

Maintenant ta fonction ListeMembre() elle est statique. Elle est déclarée dans ta classe User, c'est un outil qu'offre ta classe. Son rôle est d'effectuer une requête SQL et de renvoyer toutes les informations des objets User stockés dans ta BDD. Seulement elle à la différence de getNom(), elle n'a pas besoin d'objet. c'est pas une méthode intrinsèque à l'objet. Elle reste toutefois définie dans ta classe, c'est une méthode de classe. D'ailleurs elle peut accéder directement aux attributs sans faire appel aux getters. donc ceci reste valable :

public static function ListeMembre() {
        $req = self::$[/contents/372-fichier-db db]->prepare('[/contents/1068-sql-commande-select SELECT] * FROM tb_adherent');
        $req->execute();
        while($d = $req -> fetchObject('User')) {
            echo '<tr>';
            echo '<td>'.$d->nom.'</td>';
            echo '<td>'.$d->prenom.'</td>';
            echo '<td>'.$d->email.'</td>';
            echo '</tr>';
        }
    }

je t'avais dit d'utiliser des getters dans le cas du tout début où tu avais fait une boucle while extérieure à ta classe.

est-ce claire ou c'est un peu flou ?
n'hésites pas si tu as des remarques et questions, c'est un plaisir de répondre.

labourette · Answer

Bonjour
Tes explications sont nettement très clair quelques petits soucis néanmoins pour la compréhension de cette ligne déjà...

$req = self::$[/contents/372-fichier-db db]->prepare('[/contents/1068-sql-commande-select SELECT] * FROM tb_adherent');
 
Ensuite j'ai une erreur

Parse error: syntax error, unexpected '[', expecting T_VARIABLE or '$' in C..

totodunet · Answer

c'est syntaxiquement incorrect

je sais pas ce qui s'est passé au copier coller, désolé j'avais pas relu. ça reste en fait
$req = self::$db->prepare('SELECT * FROM tb_adherent');

labourette · Answer

j'ai cette erreur
Fatal error: Call to a member function prepare() on a non-object in C
??

totodunet · Answer

ah étrange. précédemment tu avais dit que ça fonctionnait. elle est déclaré où ta variable $db ?

labourette · Answer

en fait si j'enlève cette ligne avec cette boucle

$d = new User();

        while($d = $req -> fetchObject('User')) {
            echo '<tr>';
            echo '<td>'.$d->nom.'</td>';
            echo '<td>'.$d->prenom.'</td>';
            echo '<td>'.$d->email.'</td>';
            echo '</tr>';

ça retourne cette erreur

Fatal error: Call to a member function prepare() on a non-object in C

sinon avec cette ligne oui effectivement ça marche puis en faisant cette boucle

        while($d = $req -> fetchObject('User')) {
            echo '<tr>';
            echo '<td>'.$d->getNom().'</td>';
            echo '<td>'.$d->getPrenom().'</td>';
            echo '<td>'.$d->getEmail().'</td>';
            echo '</tr>';

labourette · Answer

variable $db est
au début de la class User

private $id;
    private $nom;
    private $prenom;
    private $email;
    private $fixe;
    private $mobile;
    private $username;
    private $password;
    private $hash;

private static $db;

totodunet · Answer

ok donc rien à voir avec $db visiblement

essaies ceci :

$req=$req->execute();
while($d=$req->fetchObject('User')){
..
}

sinon bah dans ce cas laisses la ligne $d=new User() même si là j'avoue ne plus comprendre^^
à moins qu'il faut déclarer $d avant l'entrée dans la boucle while

labourette · Answer

Bonjour Non toujours pas désolé, je te met exactement ma méthode et mon appel si au cas où.. la méthode de mon objet User public static function ListeMembre() { $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE actif = 1'); $req->execute(); while($donnees = $req -> fetchObject('User')) { echo ''; echo ''.$donnees->nom.''; echo ''.$donnees->prenom.''; echo ''.$donnees->email.''; $tel1 = $donnees->fixe; $tel1 = wordwrap ($tel1, 2, ' ', 1); echo ''.$tel1.''; $tel2 = $donnees->mobile; $tel2 = wordwrap ($tel2, 2, ' ', 1); echo ''.$tel2.''; echo ''; } } mon appel à cette fonction

Nom	Prénom	Email
nom; ?>	prenom; ?>	email; ?>

mon erreur retourné Fatal error: Call to a member function prepare() on a non-object in C: sinon si je cré un nouvel objet ça fonctionne

Nom	Prénom	Email	Téléphone Fixe	Téléphone Mobile

Merci beaucoup

totodunet · Answer

ah ça y est j'ai compris. $donnees devait être le tout premier objet User que tu as créees dans ta page. du coup si tu mettais pas cette ligne, l'attribut $db qui est statique n'existait pas. et donc ta fonction listemembre() lui est impossible de faire un prepare() dessus.

pour combler à ça, il faudrait utiliser ta fonction init() dans ListeMembre qui regarde avant si $db existe bien ou si il doit le créer

public static function ListeMembre() {
         init();
        $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE actif = 1');
        $req=$req->execute();
        while($donnees = $req -> fetchObject('User')) {
            echo '<tr>';
            echo '<td>'.$donnees->nom.'</td>';
            echo '<td>'.$donnees->prenom.'</td>';
            echo '<td>'.$donnees->email.'</td>';
            $tel1 = $donnees->fixe; $tel1 = wordwrap ($tel1, 2, ' ', 1);
            echo '<td>'.$tel1.'</td>';
            $tel2 = $donnees->mobile; $tel2 = wordwrap ($tel2, 2, ' ', 1);
            echo '<td>'.$tel2.'</td>';
            echo '</tr>';
        }
 }

Qui ne tente rien n'a rien

labourette · Answer

Bonjour Merci ça fonctionne à mercveille public static function ListeMembre() { self::init(); $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE actif = 1'); $req->execute(); while($donnees = $req -> fetchObject('User')) { echo ''; echo ''.$donnees->nom.''; echo ''.$donnees->prenom.''; echo ''.$donnees->email.''; $tel1 = $donnees->fixe; $tel1 = wordwrap ($tel1, 2, ' ', 1); echo ''.$tel1.''; $tel2 = $donnees->mobile; $tel2 = wordwrap ($tel2, 2, ' ', 1); echo ''.$tel2.''; echo ''; } } Tu peux me redonner quelques explications complémentaire à propos de cette fonction self::init(); Je te remercie.

totodunet · Answer

pas de probleme,

$db est une variable statique déclarée dans ta classe. On appelle cela une variable de classe, comme les méthodes de classes qui sont elles des fonctions statiques. Mais $db est seulement déclarée! Elle n'est initialisée à new Database() que dès lors tu construis un objet de la classe dans laquelle elle est déclarée. Donc l'appel de la fonction init() se fait dans le constructeur de User. c'est pour cela que quand tu avais fais un new User() juste avant l'appel de la fonction ListeMembre() cela fonctionnait. Mais c'est idiot de construire un objet seulement pour initialiser une variable de classe. Car c'est pareil pour la fonction statique, $db n'est pas une variable intrinsèque à l'objet.

En fait ta fonction init() initialise ta variable $db mais regarde avant ça si elle n'était pas déjà justement initialisé. Car le problème c'est que si elle ne faisait pas de vérification, $db serait de renouveau initialisé à chaque fois que tu construis un nouvel objet User. En fait ce genre de variable statique, c'est utilisé généralement pour par exemple compter le nombre d'objet User que l'on construit. Mais là il ne s'agit pas d'un compteur donc ton code de la classe peut se simplifier à juste :

private static $db=new DataBase();
dans ta classe User

puis toujours dans la classe User, supprimer dans sa totalité la fonction init() ainsi que la ligne self::init() de ta fonction ListeMembre() et tous les appels à la fonction init() que tu peux trouver.

Mais ceci dit c'est juste pour faire plus propre et optimiser un peu le programme. Sinon ça fonctionne très bien comme ça aussi^^

labourette · Answer

Bonjour
Merci pour tes explications très clair et pour tout en fait c'est super.
Cordialement.

PS J'aurai encore une lacune toujours avec POO concernant la modification de données, je ne sais pas si tu peux encore m'aider ou si j'ouvre un autre post.
Merci

totodunet · Answer

bonsoir,

pour modifier des attributs d'un objets qui sont déclarés en private dans la classe, on utilise des setters. ce sont des méthodes tel que setNom() ou setPrenom() par exemple qui affectent à l'attribut associé, la valeur mise en paramètre de la fonction. On profite du fait qu'elles puissent accéder et modifier les attributs. C'est donc le développeur de la classe qui décide de faire des setters si il juge que l'utilisateur peut modifier des attributs. Les getters eux sont des méthodes permettant de récupérer les attributs de la classe. Ils existent parce que les attributs sont déclarés en private, ceci est préféré pour des raisons de sécurité.

labourette · Answer

Bonjour
Merci,
en fait j'ai fait ceux-ci et je ne comprend pas pour ça marche pas encore quelque chose que je n'ai pas compris?
Mes méthodes de la class User

public function setNom($nom) {
        $this->nom = trim($nom);
    }

public function update() {
         $req = self::$db->prepare('UPDATE tb_adherent SET nom = :nom WHERE id_adherent = :id_adherent');
        $req->bindParam(':id',              $this->id,              PDO::PARAM_INT);        
        $req->bindParam(':nom',             $this->nom,             PDO::PARAM_STR);
        $req->execute();
}

Ensuite dans la page de mon formulaire...
J'ai créé un nouvel objet et j'ai appelé la méthode update

$donnees = new User();
$donnees->update();

et le formulaire

value="<?php $donnees->setNom($nom);?>"

totodunet · Answer

bonjour,

dans le formulaire, l'attribut HTML value  est la valeur du champ. Donc tu vas plutôt lui mettre le nom.

value="<?php $donnees->getNom();?>"
 
Qui ne tente rien n'a rien

labourette · Answer

Bonjour,
Maintenant ça me retourne une erreur comme quoi je croisque c'est une histoire de paramètre

Warning: PDOStatement::execute() [pdostatement.execute]: SQLSTATE[HY093]: Invalid parameter number: parameter was not defined in

totodunet · Answer

ah

peux tu me montrer ton bout de code HTML et PHP associé au traitement du formulaire ?

labourette · Answer

Bonjour Mon formulaire juste une partie Nom : Prénom : Je te donne toute la class User il y a la méthode "update" load($id_adherent); } catch(Exception $e) { throw $e; } } } public function init() { if(self::$db == null) self::$db = new Database(); } private function load($id_adherent) { $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE id_adherent = :id_adherent'); $req->bindParam(':id_adherent', $id_adherent, PDO::PARAM_INT); $req->execute(); $donnees = $req->fetch(PDO::FETCH_ASSOC); if(empty($donnees)) throw new Exception('Utilisateur introuvable'); $this->setUserDatas($donnees); } ########################################################################################################################## public function setNom($nom) { $this->nom = trim($nom); } public function setPrenom($prenom) { $this->prenom = trim($prenom); } public function setEmail($email) { $this->email = trim($email); } public function setNaissance($naissance) { $this->naissance = trim($naissance); } public function setFixe($fixe) { $this->fixe = trim($fixe); } public function setMobile($mobile) { $this->mobile = trim($mobile); } public function setAdresse($adresse) { $this->adresse = trim($adresse); } public function setLieuDit($lieu_dit) { $this->lieu_dit = trim($lieu_dit); } public function setCodePostal($code_postal) { $this->code_postal = trim($code_postal); } public function setVille($ville) { $this->ville = trim($ville); } public function setUsername($username) { $this->username = trim($username); } public function setPassword($password) { $this->password = trim($password); } //public function setHashCookie($hash_cookie) { // $this->hash_cookie = ($hash_cookie); //} ########################################################################################################################## public function getNom() { return $this->nom; } public function getPrenom() { return $this->prenom; } public function getEmail() { return $this->email; } public function getNaissance() { return $this->naissance; } public function getFixe() { return $this->fixe; } public function getMobile() { return $this->mobile; } public function getAdresse() { return $this->adresse; } public function getLieuDit() { return $this->lieu_dit; } public function getCodePostal() { return $this->code_postal; } public function getVille() { return $this->ville; } public function getUsername() { return $this->username; } public function getPassword() { return $this->password; } ########################################################################################################################## private function setUserDatas($donnees) { if(!empty($donnees) && is_array($donnees)) { $this->id_adherent = $donnees['id_adherent']; $this->nom = $donnees['nom']; $this->prenom = $donnees['prenom']; $this->email = $donnees['email']; $this->naissance = $donnees['naissance']; $this->fixe = $donnees['fixe']; $this->mobile = $donnees['mobile']; $this->adresse = $donnees['adresse']; $this->lieu_dit = $donnees['lieu_dit']; $this->code_postal = $donnees['code_postal']; $this->ville = $donnees['ville']; $this->username = $donnees['username']; $this->password = $donnees['password']; } } public static function get($username) { self::init(); $req = self::$db->prepare('SELECT id_adherent FROM tb_adherent WHERE username = :username'); $req->bindParam(':username', $username, PDO::PARAM_STR); $req->execute(); $id_adherent = $req->fetchColumn(); return ($id_adherent) ? new self((int) $id_adherent) : null; } public static function exists($username) { self::init(); $username = trim($username); $req = self::$db->prepare('SELECT COUNT(*) FROM tb_adherent WHERE username = :username'); $req->bindParam(':username', $username, PDO::PARAM_STR); $req->execute(); $count = $req->fetchColumn(); return ($count == 0) ? false : true; } public function login() { $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE username = :username AND password = :password AND actif = 1'); $req->bindParam(':username', $this->username, PDO::PARAM_STR); $req->bindParam(':password', $this->password, PDO::PARAM_STR); $req->execute(); $donnees = $req->fetch(PDO::FETCH_ASSOC); $this->setUserDatas($donnees); return (!empty($donnees)); } public function insert() { $req = self::$db->prepare('INSERT INTO tb_adherent (nom, prenom, email, naissance, fixe, mobile, adresse, lieu_dit, code_postal, ville, username, password) VALUES (:nom, :prenom, :email, :naissance, :fixe, :mobile, :adresse, :lieu_dit, :code_postal, :ville, :username, :password)'); $req->bindParam(':nom', $this->nom, PDO::PARAM_STR); $req->bindParam(':prenom', $this->prenom, PDO::PARAM_STR); $req->bindParam(':email', $this->email, PDO::PARAM_STR); $req->bindParam(':naissance', $this->naissance, PDO::PARAM_STR); $req->bindParam(':fixe', $this->fixe, PDO::PARAM_STR); $req->bindParam(':mobile', $this->mobile, PDO::PARAM_STR); $req->bindParam(':adresse', $this->adresse, PDO::PARAM_STR); $req->bindParam(':lieu_dit', $this->lieu_dit, PDO::PARAM_STR); $req->bindParam(':code_postal', $this->code_postal, PDO::PARAM_STR); $req->bindParam(':ville', $this->ville, PDO::PARAM_STR); $req->bindParam(':username', $this->username, PDO::PARAM_STR); $req->bindParam(':password', $this->password, PDO::PARAM_STR); $req->execute(); $req = self::$db->prepare('SELECT id_adherent FROM tb_adherent WHERE username = :username'); $req->bindParam(':username', $this->username, PDO::PARAM_STR); $req->execute(); $this->id_adherent = $req->fetchColumn(); } public function update() { $req = self::$db->prepare('UPDATE tb_adherent SET nom = :nom, prenom = :prenom, email = :email, naissance = :naissance, fixe = :fixe, mobile = :mobile, adresse = :adresse, lieu_dit = :lieu_dit, code_postal = :code_postal, ville = :ville, username = :username, password = :password WHERE id_adherent = :id_adherent'); $req->bindParam(':id_adherent', $this->id_adherent, PDO::PARAM_INT); $req->bindParam(':nom', $this->nom, PDO::PARAM_STR); $req->bindParam(':prenom', $this->prenom, PDO::PARAM_STR); $req->bindParam(':email', $this->email, PDO::PARAM_STR); $req->bindParam(':naissance', $this->naissance, PDO::PARAM_STR); $req->bindParam(':fixe', $this->fixe, PDO::PARAM_STR); $req->bindParam(':mobile', $this->mobile, PDO::PARAM_STR); $req->bindParam(':adresse', $this->adresse, PDO::PARAM_STR); $req->bindParam(':lieu_dit', $this->lieu_dit, PDO::PARAM_STR); $req->bindParam(':code_postal', $this->code_postal, PDO::PARAM_STR); $req->bindParam(':ville', $this->ville, PDO::PARAM_STR); $req->bindParam(':username', $this->username, PDO::PARAM_STR); $req->bindParam(':password', $this->password, PDO::PARAM_STR); $req->execute(); } public static function ListeMembre() { self::init(); $req = self::$db->prepare('SELECT * FROM tb_adherent WHERE actif = 1 ORDER BY nom, prenom ASC'); $req->execute(); while($donnees = $req -> fetchObject('User')) { echo ''; echo ''.$donnees->nom.''; echo ''.$donnees->prenom.''; echo ''.$donnees->email.''; $tel1 = $donnees->fixe; $tel1 = wordwrap ($tel1, 2, ' ', 1); echo ''.$tel1.''; $tel2 = $donnees->mobile; $tel2 = wordwrap ($tel2, 2, ' ', 1); echo ''.$tel2.''; echo ''; } } }

totodunet · Answer

tu m'a pas montré ce qu'il fallait. dans un formulaire HTML tu dois avoir la méthode d'envoi et l'action. puis pour récuperer la valeur que l'utilisateur à mis dans les textfields dans le script php pour le traitement, faut utiliser la variable $_POST. ou $_GET si la méthode d'envoi est get. Ces variables sont des tableaux associatifs par l'attribut name. Qui ne tente rien n'a rien

labourette · Answer

en fait je n'ai pas mis tout le formulaire

totodunet · Answer

ok donc il n'y a pas de problème côté HTML. est-ce que tu vois bien les champs remplies par défaut des  noms et prénoms de l'objet ?

maintenant ton bout de code de ta même page avec $_POST pour le traitement du formulaire. il doit se trouver idéalement en début de page avant la première balise HTML que tu envoies au client.

labourette · Answer

Bonjour,
Je ne comprends pas après avoir passé un bon moment.
Merci de ton aide.
Alors voilà ce que j'ai essayé de faire.
Pour le formulaire côté HTML les champs sont bien remplis avec

value="<?php echo $user->getNom();?>"

j'initialise les variables
cré un objet
affiche les setters pour pouvoir modifier
j'accéde à la méthode update

if(!empty($_POST)) {

$nom = $_POST['nom'];
    $prenom = $_POST['prenom'];
    $email = $_POST['email'];    
    $username = $_POST['username'];

$user = new User();

$user->setNom($nom);
        $user->setPrenom($prenom);
  $user->setEmail($email);        
        $user->setUsername($username);
  
        $user->update(); 
}

totodunet · Answer

bonsoir

les champs username et email n'existent pas dans ton formulaire HTML.

est-ce que tu as une erreur qui s'affiche après l'envoi du formulaire?

ce que je te conseille de faire,
c'est rajouter echo $req; juste avant la ligne $req->execute(); dans ta fonction update() pour qu'il affiche la requête SQL et voir ce qu'il se passe. tu fais un copier-coller de ta requête dans ton SGBD et tu regardes ce qu'il te dit.

aussi pour vérifier $_POST il est bon d'utiliser la fonction isset() pour savoir si $_POST existe avant de voir si il est vide ou pas.

if(isset($_POST)&&!empty($_POST))

labourette · Answer

Voici l'erreur retournée

Catchable fatal error: Object of class PDOStatement could not be converted to string in

labourette · Answer

Bonjour Je t'envoie la totalité de mon formulaire. Il y a quelques chose que je ne comprends pas c'est fou.. Donc ça me retourne l'erreur ci-dessus après avoir rajouté echo $req; comme tu me l'a dit sinon sans ça aucune erreur setNom($nom); $user->setPrenom($prenom); $user->setEmail($email); $user->setUsername($username); $user->update(); } ?>

totodunet · Answer

salut,

visiblement ta fonction update() semble problématique. As-tu fait un echo $req; comme je t'ai dit tans ta fonction update() pour voir ce qu'elle contient comme résultat et vérifier si elle ne comporte pas d'erreur SQL

labourette · Answer

Bonjour
oui elle retourne l'erreur que je t'ai dit en fait

Catchable fatal error: Object of class PDOStatement could not be converted to string in C

Merci

labourette · Answer

Bonjour
Voilà je pense à quelque chose, il faudrait peut-être ajouter un SELECT dans la méthode UPDATE juste avant le update en fait je vais quand même essayer.
Je te redis qu'est ce qu'il en ressors....
Merci

labourette · Answer

Bonjour
Alors voilà j'ai fait ceux-ci dans la méthode update à priori ça enregistre dans la BDD

public function update() {
        $req = self::$db->prepare('SELECT id_adherent FROM tb_adherent WHERE username = :username');
        $req->bindParam(':username', $this->username, PDO::PARAM_STR);
        $req->execute();
        $this->id_adherent = $req->fetchColumn();
        
        $req = self::$db->prepare('UPDATE tb_adherent SET nom = :nom, prenom = :prenom, email = :email, naissance = :naissance, fixe = :fixe, mobile = :mobile, adresse = :adresse, lieu_dit = :lieu_dit, code_postal = :code_postal, ville = :ville, username = :username, password = :password WHERE id_adherent = :id_adherent');
        $req->bindParam(':id_adherent',     $this->id_adherent,     PDO::PARAM_INT);        
        $req->bindParam(':nom',             $this->nom,             PDO::PARAM_STR);
        $req->bindParam(':prenom',          $this->prenom,          PDO::PARAM_STR);
        $req->bindParam(':email',           $this->email,           PDO::PARAM_STR);
        $req->bindParam(':naissance',       $this->naissance,       PDO::PARAM_STR);
        $req->bindParam(':fixe',            $this->fixe,            PDO::PARAM_STR);
        $req->bindParam(':mobile',          $this->mobile,          PDO::PARAM_STR);
        $req->bindParam(':adresse',         $this->adresse,         PDO::PARAM_STR);
        $req->bindParam(':lieu_dit',        $this->lieu_dit,        PDO::PARAM_STR);
        $req->bindParam(':code_postal',     $this->code_postal,     PDO::PARAM_STR);
        $req->bindParam(':ville',           $this->ville,           PDO::PARAM_STR);
        $req->bindParam(':username',        $this->username,        PDO::PARAM_STR);
        $req->bindParam(':password',        $this->password,        PDO::PARAM_STR);
        $req->execute();
    }

Mais voilà dans ma tête c'est pas clair du tout.
Déjà pourquoi dans le WHERE il n'y a pas plutôt

WHERE id_adherent= id_adherent');

si tu peux me donner des explications et en plus au niveau de l'affichage les donner modifier ne retourne pas dans le formulaire mais juste dans la BDD.
Merci beaucoup, j'y suis allé à taton si il y a autre chose de mieux au niveau requète.

labourette · Answer

Bonjour
C'est encore moi j'aurai besoin encore de toi  s'il te plait.
Il y a aussi quelque chose de pas normal. Lorsque je fait une modif mon formulaire à l'affichage ne garde pas les valeurs, je suis obligé de me déconnecter puis me reconnecter pour que les modifs soient afficher dans le formulaire.
Merci j'espère que tu comprends ce que je veux dire.

totodunet · Answer

salut,

 Mais voilà dans ma tête c'est pas clair du tout. 
Déjà pourquoi dans le WHERE il n'y a pas plutôt 
WHERE id_adherent= id_adherent');

je ne sais pas c'est toi le programmeur qui a écrit cela^^

si tu fais where id_adherent=id_adherent cela c'est stupide car tu demandes l'id_adherent de la table tb_adherent où l'id_adherent est égal à une valeur. Autrement si la valeur est 5, tu demandes d'afficher le numéro de l'adhérent numéro 5 donc grosso modo 5.

mais maintenant j'ai pu voir où est le problème, $this->id_adherent n'a pas de valeur car visiblement après ton select tu lui en affectes une.
 

mais ce que j'avais demandé, c'était plutôt ça :

public function update() {
        $req = self::$db->prepare('UPDATE tb_adherent SET nom = :nom, prenom = :prenom, email = :email, naissance = :naissance, fixe = :fixe, mobile = :mobile, adresse = :adresse, lieu_dit = :lieu_dit, code_postal = :code_postal, ville = :ville, username = :username, password = :password WHERE id_adherent = :id_adherent');
        $req->bindParam(':id_adherent',     $this->id_adherent,     PDO::PARAM_INT);        
        $req->bindParam(':nom',             $this->nom,             PDO::PARAM_STR);
        $req->bindParam(':prenom',          $this->prenom,          PDO::PARAM_STR);
        $req->bindParam(':email',           $this->email,           PDO::PARAM_STR);
        $req->bindParam(':naissance',       $this->naissance,       PDO::PARAM_STR);
        $req->bindParam(':fixe',            $this->fixe,            PDO::PARAM_STR);
        $req->bindParam(':mobile',          $this->mobile,          PDO::PARAM_STR);
        $req->bindParam(':adresse',         $this->adresse,         PDO::PARAM_STR);
        $req->bindParam(':lieu_dit',        $this->lieu_dit,        PDO::PARAM_STR);
        $req->bindParam(':code_postal',     $this->code_postal,     PDO::PARAM_STR);
        $req->bindParam(':ville',           $this->ville,           PDO::PARAM_STR);
        $req->bindParam(':username',        $this->username,        PDO::PARAM_STR);
        $req->bindParam(':password',        $this->password,        PDO::PARAM_STR);
        echo $req;
        $req->execute();
    }

voir ce qu'affiche le contenu de $req et exécuter la ligne dans ton SGBD pour savoir quelles erreurs vont pas.


Qui ne tente rien n'a rien

labourette · Answer

Bonjour
Dis moi juste quelle ligne j'affiche dans ma BDD?
parce que avec 
echo $req ça retourne
 Catchable fatal error: Object of class PDOStatement could not be converted to string in C......

sinon ok j'avais pas pensé where id_adherent=id_adherent

totodunet · Answer

bonsoir,

étrange... mais au vu que ton select que tu fais dans update() fait fonctionner la méthode update(), je parie que le problème est this->adherent qui n'est pas initialisé. fais echo this->adherent au début de update pour voir si il a une valeur.

labourette · Answer

Bonjour
Je ne sais pas si c'est bien ici qu'il faut faire le echo mais si c'est le cas ça me retourne

parse error: syntax error, unexpected T_VARIABLE, expecting ',' or ';' in C:

public function UpdateMembre() {
        echo $this->id_adherent
        $req = self::$db->prepare('SELECT id_adherent FROM tb_adherent WHERE username = :username');
        $req->bindParam(':username', $this->username, PDO::PARAM_STR);
        $req->execute();
        $this->id_adherent = $req->fetchColumn();

sinon le SELECT est-il vraiment indispensable? c'est juste une question en fait comme je débute en POO.
Merci

totodunet · Answer

bonsoir,

mets un ';' à la fin du echo!

 non le select théoriquement on en a pas besoin si l'attribut id_adherent est déjà initialisé. ce que va nous dire le echo

labourette · Answer

Bonjour
Donc voilà pour récapituler, après avoir mis echo $this->id_adherent; après le UpdateMembre aucune erreur retournée.
Toutes les modifs s'effectue bien dans ma BDD sauf que pour l'affichage dans le formulaire les données n'apparaissent pas à l'instantanée  il faut toujours que je me déconnecte et me reconnecte pour les valeurs prennent effets pour l'affichage.
Merci de ton aide.

Programmation Orienté Objet

47 réponses

Discussions similaires

Newsletters