SELECT dans une requète SQL sur une page PHP

Version imprimable

Bonjour,

Je poste mon sujet ici car après plusieurs heures de développement je ne trouve toujours pas la solution à mon soucis, voici un extrait de mon fichier php:

Code:

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
try{
	$bdd = new PDO('mysql:sql.host=sql.hellriders-esport.fr;dbname=projethr', '******', '*******',array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
}
catch (Exception $e){
	die('Erreur : ' . $e->getMessage());
}
$teams = $bdd->query('SELECT * FROM Team')  or die(print_r($bdd->errorInfo()) );
$comptes = $bdd->query('SELECT * FROM compte')  or die(print_r($bdd->errorInfo()) );
while($uneteam=$teams->fetch() ){//Affichage des teams pour rejoindre
	echo($uneteam['Nom']);
	echo' : ';
	if( !($uneteam['IDJoueur1']==0)){
		$J=$uneteam['IDJoueur1'];
		$NomJoueur = $bdd->query('SELECT Nominv FROM compte WHERE ID = \'$J\'')  or die(print_r($bdd->errorInfo()) );
		echo($NomJoueur);
	}
	else echo"Joueur inconnu";
	echo',';
	if( !($uneteam['IDJoueur2']==0)){
		$J=$uneteam['IDJoueur2'];
		$NomJoueur = $bdd->query('SELECT Nominv FROM compte WHERE ID = \'$J\'')  or die(print_r($bdd->errorInfo()) );
		echo($NomJoueur);
	}
	else echo"Joueur inconnu";
	echo',';
	if( !($uneteam['IDJoueur3']==0)){
		$J=$uneteam['IDJoueur3'];
		$NomJoueur = $bdd->query('SELECT Nominv FROM compte WHERE ID = \'$J\'')  or die(print_r($bdd->errorInfo()) );
		echo($NomJoueur);
	}
	else echo"Joueur inconnu";
	echo',';
	if( !($uneteam['IDJoueur4']==0)){
		$J=$uneteam['IDJoueur4'];
		$NomJoueur = $bdd->query('SELECT Nominv FROM compte WHERE ID = \'$J\'')  or die(print_r($bdd->errorInfo()) );
		echo($NomJoueur);
	}
	else echo"Joueur inconnu";
	echo',';
	if( !($uneteam['IDJoueur5']==0)){
		$J=$uneteam['IDJoueur5'];
		$NomJoueur = $bdd->query('SELECT Nominv FROM compte WHERE ID = \'$J\'')  or die(print_r($bdd->errorInfo()) );
		echo($NomJoueur);
	}
	else echo"Joueur inconnu";
	echo'.';

Lors de l'affichage de la page, tout s'affiche normalement sauf le noms des joueurs.. je ne comprend pas où est mon erreur, pourtant après quelques tests je pense qu'elle est dans la ligne de requête SQL qui selectionne Nominv dans la table compte, mais impossible de voir ce qui cloche, je nage dans l'obscurité totale... :(
Si quelqu'un voit quelque chose je suis preneur ;)
Bon vent à tous!

17/01/2015, 12h40
laurentSc

Je crois que dans la clause WHERE SQL, il ne faut pas encadrer de cotes la variable. Par exemple, $NomJoueur = $bdd->query('SELECT Nominv FROM compte WHERE ID = \'$J\'') deviendrait $NomJoueur = $bdd->query('SELECT Nominv FROM compte WHERE ID = '.$J)
17/01/2015, 13h12
sabotage

$NomJoueur est le jeu de résultat entier, ce n'est pas une valeur.
tu as oublié l'étape "fetch".
17/01/2015, 13h28
laurentSc

Sabotage a raison mais je crois que moi aussi ; non ?
19/01/2015, 12h12
zaza576

Hello,

il y a un truc simple à faire avec PHP pour débugger une situation que tu ne comprends pas.
Tu fais une requête SQL. Pour s'assurer qu'elle te retournes déjà des résultats cohérents avec tes attentes AVANT de t'occuper de l'affichage dans ta page, tu peux faire :

$result = maRequeteSelect('SELECT * FROM blablabla ...);
var_dump($result);
// ou même print_r($result);

Ainsi, tu verras si déjà tu récupères les bonnes choses et si ta requête est valide.

Ensuite, pour l'affichage, fait étape par étape pour afficher déjà un premier attribut de ta requête, puis un second ... Cela te permet de suivre progressivement l'évolution de ton code et de comprendre à quel moment il peut y avoir une erreur dans ton code.

Après, fait attention aux types de données retournées. Les entiers doivent rester des entiers. Les chaînes des chaînes.

SELECT Nominv FROM compte WHERE ID = \'$J\'
Ton ID est-il un entier ? Si oui, pourquoi mets-tu $J entre côtes ? As tu vérifié aussi la valeur de ton $J ?
Le mieux serait que tu mettes la requête dans une variable comme ceci :
$maRequete = SELECT Nominv FROM compte WHERE ID = \'$J\';
Puis tu fais un echo pour visualiser textuellement la syntaxe de ta requête.
Enfin, tu copies colles cette requête directement dans ton PHPMyAdmin pour effectuer ta requête en dehors de ton script et visualiser directement dans ton SGBD la validité de ta requête.
Ainsi, tu comprendras mieux tes erreurs, puisque si elles ne sont pas applicatives (script PHP), alors elles sont au niveau du modèle et de l'API de gestion de tes données (SGBD et requêtes SQL).

Amuses toi bien et tiens nous informé de ton avancement !
19/01/2015, 13h45
laurentSc
Le conseil de ZAZA576 de tester tes requêtes dans phpmyAdmin est pertinent, mais je viens de penser aussi qu'il faudrait faire des requêtes préparées :
Code:

1 2 3 4 $maRequete = SELECT Nominv FROM compte WHERE ID = ?; $qid=$bdd->prepare($maRequete); $qid->execute(array($J));
19/01/2015, 19h35
ABCIWEB

@laurentSc

Oui c'est un bon conseil.

Cela dit dans ce cas particulier, si $J est un entier, et généré en interne par le programme (et pas une valeur saisie par le visiteur), on peut se dispenser de faire une requête préparée et même d'utiliser la fonction quote(). En effet il n'y a aucun caractère de la variable à échapper puisque c'est un entier, et cette variable est une valeur sûre puisqu'elle ne dépend pas de l'utilisateur.

Mais bon en utilisant systématiquement une requête préparée cela évite de se poser des questions pour chaque variable de la requête et d'utiliser un code générique ;)
19/01/2015, 22h48
laurentSc

@ABCIWEB : voici bien un discours de spécialiste ; cela dit, j'avais eu à faire à un spécialiste en PHP (un formateur, en 2012) qui pour pas s'embêter faisait des requêtes préparées dans tous les cas, même s'il n'y avait aucune variable...