Discussion :

[MrSoGood]

Bonjour, comme dans le titre, j'ai rencontré l'erreur :

Fatal error: Call to a member function closeCursor() on a non-object in /home/basededonnees/public_html/faq.php on line 83

Je suis débutant, mais j'ai lu que apparement il fallait mettre closeCursor() à la fin de chaque requête, je croyais l'avoir fait, mais ce message d'erreur me porte à penser le contraire
PS : J'ai refermé l'accolade du "if (0 === sizeof($aListeErreurs))" juste au dessus du closeCursor() et je ne sais pas si c'est le bon endroit...

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 <?php
 
  // Le pseudo est-il rempli ?
  if (empty($_POST['pseudo']))
  {
    $aListeErreurs[] = 'Veuillez indiquer votre pseudo';
  }
 
	// Le message est-il rempli ?
  if (empty($_POST['message']))
  {
    $aListeErreurs[] = 'Veuillez indiquer votre message';
  }
 
	// Si aucune erreur n'a été générée
  // On enregistre le message dans la BDD
  if (0 === sizeof($aListeErreurs))
  {
 
 
  try
  {
	$bdd = new PDO('mysql:host=mysql.hostinger.fr;dbname=basededonnees;charset=utf8', 'user', 'mdp');
  }
    catch(Exception $e)
  {
        die('Erreur : '.$e->getMessage());
  }
 
  // Insertion du message à l'aide d'une requête préparée
  $req = $bdd->prepare('INSERT INTO chat (pseudo, message) VALUES(pseudo, message)');
  $req->execute(array($_POST['pseudo'], $_POST['message']));
 
 
 
  // Récupération des 50 derniers messages
  $reponse = $bdd->query('SELECT pseudo, message FROM chat ORDER BY ID DESC LIMIT 0, 50');
 
  // Affichage de chaque message (toutes les données sont protégées par htmlspecialchars)
  while ($donnees = $reponse->fetch())
  {
	echo '<p><strong>' . htmlspecialchars($donnees['pseudo']) . '</strong> : ' . htmlspecialchars($donnees['message']) . '</p>';
  }
 
}
$reponse->closeCursor();
 
 
?>

Merci pour vos réponses !

[sabotage]

Le closecursor ne peut evidemment se faire que si la requête a été faite. Donc il doit être avant le dernier }

[MrSoGood]

Merci de ta réponse ! Mais je ne sais pas trop comment insérer des conditions sans tout faire sauter...

[Watilin]

Peut-être qu'une indentation propre de ton code avec http://phpbeautifier.com/ t'aiderait à y voir plus clair dans ton code

[MrSoGood]

Merci beaucoup mais je n'ai pas très bien compris à quoi cela me servait-il ?

[Celira]

Si tu passes ton code dans le site indiqué par Watilin, ça donne ça :

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
35
36
37
38
39
40
41
42
43
44
45
46
47
 <?php
 
// Le pseudo est-il rempli ?
 
if (empty($_POST['pseudo'])) {
	$aListeErreurs[] = 'Veuillez indiquer votre pseudo';
}
 
// Le message est-il rempli ?
 
if (empty($_POST['message'])) {
	$aListeErreurs[] = 'Veuillez indiquer votre message';
}
 
// Si aucune erreur n'a été générée
// On enregistre le message dans la BDD
 
if (0 === sizeof($aListeErreurs)) {
	try {
		$bdd = new PDO('mysql:host=mysql.hostinger.fr;dbname=basededonnees;charset=utf8', 'user', 'mdp');
	}
 
	catch(Exception $e) {
		die('Erreur : ' . $e->getMessage());
	}
 
	// Insertion du message à l'aide d'une requête préparée
 
	$req = $bdd->prepare('INSERT INTO chat (pseudo, message) VALUES(pseudo, message)');
	$req->execute(array(
		$_POST['pseudo'],
		$_POST['message']
	));
 
	// Récupération des 50 derniers messages
 
	$reponse = $bdd->query('SELECT pseudo, message FROM chat ORDER BY ID DESC LIMIT 0, 50');
 
	// Affichage de chaque message (toutes les données sont protégées par htmlspecialchars)
 
	while ($donnees = $reponse->fetch()) {
		echo '<p><strong>' . htmlspecialchars($donnees['pseudo']) . '</strong> : ' . htmlspecialchars($donnees['message']) . '</p>';
	}
}
 
$reponse->closeCursor();
?>

Ce qui te permet de voir que tu utilises $reponse à l'extérieur du bloc dans lequel il est défini. Tu as donc juste à remonter $reponse->closeCursor(); à l'intérieur du bloc ouvert par if (0 === sizeof($aListeErreurs)) {.

@Watilin : chouette site , ça va m'éviter d'avoir à ouvrir mon EDI pour comprendre le code des posts mal indentés.