Discussion :

[laurentSc]

Bonjour,

ce code génère l'erreur du titre :

Code php :

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

1
2
3
4
$query = "SELECT * FROM `infoappli` where name=:serveur";
$marqueur=array(':serveur'=>$serveur);
$qid = $bdd->prepare($query);
$qid->execute();

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[HY000]: General error: 2031 ' in C:\wamp\www\devjupiter_serveur291214\eis\page\selmachineold.htm on line 102

alors que la requête s'exécute bien dans PHPMyAdmin. Pourquoi ?

[Kreepz]

Code PHP :

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

1
2
3
4
$query = "SELECT * FROM `infoappli` where name=:serveur";
$marqueur=array(':serveur'=>$serveur);
$qid = $bdd->prepare($query);
$qid->execute();

Tu prépare ta requête mais tu ne lui donne jamais ta variable $marqueur pour les paramètres de ta requête...

[laurentSc]

Merci ; à peine le temps d'écrire ma question que j'ai la réponse ! (Simple pour un regard extérieur)

[laurentSc]

6 mois après, j'ai exactement le même problème (et c'est en faisant une recherche Google que je l'ai retrouvé). Mais la grosse bêtise que j'avais faite à l'époque n'est pas ici l'explication...

Code php :

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

1
2
3
4
$query_info = "SELECT machines.name,infotech.* FROM machines,infotech
	  WHERE machines.id=:id AND machines.name=infotech.name";
$qid=$bdd->prepare($query_info);
$qid->execute(array(':id'=>$id)) or die($str[325]);

et le message d'erreur est le même :

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[HY000]: General error: 2031 ' in C:\wamp\www\eis2\eis\page\searchmachine.htm on line 24

Merci de votre aide.

[AS_PTT]

Code :

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

1
2
3
SELECT machines.name,infotech.* FROM machines,infotech
	  WHERE machines.id=:id AND machines.name=infotech.name";
$qid=$bdd->prepare($query_info);

Tu ne ne selectionne que machines.name et non pas machines.id

Code :

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

1
2
3
SELECT machines.name,machines.id,infotech.* FROM machines,infotech
	  WHERE machines.id=:id AND machines.name=infotech.name";
$qid=$bdd->prepare($query_info);

Suis pas sur Je debute en SQL aussi

[laurentSc]

Merci de te pencher sur le problème, mais ça ne supprime pas l'erreur et de plus, si je teste la requête dans phpMyAdmin, elle marche...

[AS_PTT]

Pourquoi machines.id=:id et une jointure
machines.id=:id retourne un résultat unique pas besoin de jointures sauf si id n'est pas la clé primaire

[laurentSc]

A dire vrai, il s'agit d'un vieux code que je me contente d'actualiser (par exemple : fonctions _mysql => PDO) et de modifier, mais les requêtes SQL, je ne les modifie pas, vu qu'elles fonctionnent. Et pour creuser ton idée, j'ai simplifié la condition : SELECT machines.name,infotech.* FROM machines,infotech WHERE machines.id=:id mais l'erreur reste...

[AS_PTT]

Code :

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

$qid->execute(array(':id'=>$id)) or die($str[325]);

je suppose que $id existe dans le code précédant.

[laurentSc]

Comme $id est en fait extrait d'une bdd, j'en ai modifié la lecture :

Code php :

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

1
2
3
4
5
6
7
8
9
	$row=$qid->fetchAll(PDO::FETCH_ASSOC);
	$id = $row[0]["id"];
	echo "id=".$id."<br/>";
	/*$query_info = "SELECT machines.name,machines.id,infotech.* FROM machines,infotech
	  WHERE machines.id=:id AND machines.name=infotech.name";*/
	$query_info = "SELECT machines.name,machines.id,infotech.* FROM machines,infotech
	  WHERE machines.id=:id";
	$qid=$bdd->prepare($query_info);
	$qid->execute(array(':id'=>$id)) or die($str[325]);

mais c'est au point (la ligne 3 le confirme)...

[AS_PTT]

tu as 2 fois $qid qui semble être deux requêtes distinctes ou je me trompe

[laurentSc]

Exact, il s'agit bien de 2 requêtes distinctes, mais comme je n'ai plus besoin de lire les résultats de la première quand j'exécute la seconde, j'utilise la même variable.
Cela dit, en cherchant à te répondre, j'ai du modifier légèrement le code et corriger l'erreur : elle n'apparaît plus !

[Dendrite]

Mouais...
Avec ça, on a progressé lol...

Je sais que le SQL qui fonctionne te suffit, mais je suis convaincue que pour la clarté, tu gagnerais à écrire tes jointures proprement, et à distinguer jointures et filtres.
Quand tes requêtes se compliquent, ça change tout, tout, tout.
Pour moi, c'est comme si tu me balançais une phrase avec les mots dans le désordre. "moi aimer toi" on comprend, mais comment dire... il y a mieux.
Quitte à utiliser PDO, autant ne pas utiliser une syntaxe SQL datant de mathusalem.

Code SQL :

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

1
2
3
4
SELECT machines.name,infotech.* 
FROM machines
inner join infotech on machines.name=infotech.name
WHERE machines.id=:id

Un (éventuel) problème que je vois, c'est que name existe deux fois dans ta requête, et que autant PHPmyadmin a sa façon de gérer cela, autant toi dans le code, tu es censé renommer les champs en double.
Ici en gros, évite le .* et prends tous les champs de infotech sauf name...

[laurentSc]

Ce code est en fait un code écrit à l'époque de Mathusalem (2003) que je reprends pour le mettre à jour (et notamment fonctions _mysqlXXX => PDO). L'écriture des requêtes SQL datent donc de la même époque. Cela dit, dans un autre post, Sabotage m'avait déjà fait cette correction : http://www.developpez.net/forums/d15...retourne-rien/