Discussion :

[cyci60]

breveManager.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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
 
class breveManager {
 
	private $_db;
 
	public function __construct($db) {
 
		$this->setDb($db);
	}
 
	public function setDb(PDO $dbh) {
 
		$this->_db = $dbh;
	}
 
	public function addBreve(Breve $breve) {
 
		$sql = 'INSERT INTO breve(texte, dt_creation) VALUES(:texte, :dt_creation)';
		$stmnt = $this->_db->prepare($sql);
		$stmnt->bindParam(':texte', htmlspecialchars($breve->getText()));
		$stmnt->bindParam(':dt_creation', $breve->getDt_creation());
		$stmnt->execute();
	}
 
public function updateBreve(Breve $breve) {
 
		$sql = 'UPDATE breve SET texte = :texte, dt_creation = :dt_creation WHERE id = :id';
		$stmnt = $this->_db->prepare($sql);
		$stmnt->bindParam(':id', $breve->getId());
		$stmnt->bindParam(':texte', $breve->getText());
		$stmnt->bindParam(':dt_creation', $breve->getDt_creation());
		$stmnt->execute();
	}

Message d'erreur -> Remarque: seules les variables doivent être passées ou transmises par référence.

Merci pour votre aide.

[laurentSc]

Bonjour,
le message d'erreur indique probablement sur quelle ligne, il détecte l'erreur. Merci de préciser.

[cyci60]

ligne 21 22 et 30 31 32

[laurentSc]

Comme l'erreur porte sur tous les bindparam, tu peux utiliser une autre syntaxe : $stmnt->execute([':texte'=>htmlspecialchars(($breve)->getText()),':dt_creation'=>($breve)->getDt_creation()]);(faire pareil pour le second execute)

[Trehinos]

Dans la documentation (https://www.php.net/manual/fr/pdosta....bindparam.php), on voit le prototype :

Code :

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

public PDOStatement::bindParam ( mixed $parameter , mixed &$variable)

Le & signifie que le paramètre est passé par référence. Vous tentez de passer le résultat d'une fonction dans votre code, hors un résultat de fonction n'est qu'une valeur, il n'a pas de "référence".

Il faut passer une variable, forcément, à cette fonction.
Vous pouvez faire, par exemple :

Code :

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

1
2
$id = $breve->getId();
$stmnt->bindParam(':id', $id);

---

Mais la solution de laurentSc est meilleure. execute([$params...]) est plus simple, très lisible et permet de passer des valeurs.

[cyci60]

Merci bien, ça marche