Discussion :

[modus57]

Bonjour,

J’ai besoin de votre aide concernant une erreur « Call to a member function fetch() » que je rencontre depuis la installation de la mise à jour 3.2.5 de WampServer.

Je ne suis pas certain, elle doit être provoquée par la version plus récente de PHP 7.3.21 ?

Comment corriger ce problème.

Merci d’avance pour vos réponses.

[Séb.]

1. Nous donner du code
2. Checker ta connexion PDO / ta requête préparée (voir la methode PDO/PDOStatement::errorInfo()

[modus57]

Bonjour,

En fait je me viens de me rendre compte qu’après le clic sur mon bouton « Enregistrement suivant » si le ou les champs « nom, prenom, id » sont vides cela provoque « Error: Call to a member function fetch() ».

Code :

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

1
2
3
4
5
6
7
8
@$nom = $_POST['nom'];
@$prenom = $_POST['prenom'];
@$id = $_POST['id'];
 
$sql = "SELECT * FROM personnes WHERE (nom,prenom,id) > ('$nom','$prenom',$id) ORDER BY nom,prenom,id LIMIT 1";
 
$result=$bdd->query($sql);		
$data = $result->fetch(PDO::FETCH_ASSOC);

Et là quelle est la syntaxe si $data > 1 alors …

Merci d’avance pour vos réponses.

[Séb.]

Il faut généralement éviter les @ (surtout pendant le développement), ça cache les erreurs et rend le code difficile à débuguer.

Tu peux remplacer ceci :

Code :

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

1
2
3
@$nom = $_POST['nom'];
@$prenom = $_POST['prenom'];
@$id = $_POST['id'];

Par :

Code :

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

1
2
3
$nom = isset($_POST['nom']) ? $_POST['nom'] : null;
$prenom = isset($_POST['prenom']) ? $_POST['prenom'] : null;
$id = isset($_POST['id']) ? $_POST['id'] : null;

Ou avec du PHP moins vieillot :

Code :

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

1
2
3
$nom = $_POST['nom']) ?? null;
$prenom = $_POST['prenom'] ?? null;
$id = $_POST['id'] ?? null;

Voire :

Code :

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

1
2
3
$_POST['nom'] ??= null;
$_POST['prenom'] ??= null;
$_POST['id'] ??= null;

Dans les 2 cas les isset() et ?? évitent les notices, et libre à toi de remplacer les null par toute autre valeur par défaut.

Bon ensuite théoriquement tu ne devrais pas traiter un formulaire incomplet, non ?

Donc :

Code :

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

1
2
3
4
5
6
if (!isset($_POST['nom'], $_POST['prenom'], $_POST['id'])) {
    // Formulaire non soumis ou incomplet
    // Message d'erreur, redirection, etc.
} else {
    // Requête SQL avec les paramètres valorisés
}

Et pour finir, le fond du problème, est que si $id est vide ta requête donne :

Code :

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

1
2
3
4
SELECT *
FROM personnes
WHERE (nom, prenom, id) > ('Dupont', 'Antoine', )
ORDER BY ...

Ce qui est incorrect.

Au dessus je parlais de PDO::errorInfo(), il aurait fallu l'utiliser ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$sql = "...";
 
$result=$bdd->query($sql);		
if ($result === false) { // Erreur ?
    echo '<pre>'
    echo $sql;
    print_r($bdd->errorInfo());
    echo '</pre>';
    exit;
}
$data = $result->fetch(PDO::FETCH_ASSOC);

La solution est de délimiter $id par des ' dans ta requête SQL ou de lui donner une valeurs par défaut.

(et dans un 2nd temps de ne pas oublier d'échapper tes valeurs avec $bdd->quote() sinon c'est la porte ouverte à toutes les injections SQL malicieuses)

Désolé pour le pâté,

[modus57]

Bonjour,

Merci pour toutes vos explications.

Voici mon code corrigé :

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
<?php
}elseif($_POST['suivant']){
$nom = $_POST['nom'];
$prenom = $_POST['prenom'];
$id = $_POST['id'];
 
$sql = "SELECT * FROM personnes WHERE (nom,prenom,id) > ('$nom','$prenom', '$id') ORDER BY nom,prenom,id LIMIT 1";
 
$result=$bdd->query($sql);
if ($result === false) { // Erreur ?
echo '<script>alert("Les champs du formulaire sont vides sélectionner le premier ou un enregistrement!");</script>';
	?>
	      <script> history.go(-1); </script> 				
	<?php
	exit;
}		
$data = $result->fetch(PDO::FETCH_ASSOC);	
…

J’ai donc supprimé les @ des variables, délimité $id par des ' dans ta requête SQL (grossière erreur de ma part) et ça fonctionne parfaitement.

@ +

[modus57]

Re bonjour,

J’ai un petit souci.

J’ai supprimé l’@ de }elseif(@$_POST['suivant']){ et maintenant le message suivant s’affiche :

! Notice : Unfined index : suivant in C:\wamp64\www\...

Est-ce obligatoire de le supprimer ?

@ +

[Séb.]

J'ai déjà répondu à ce point