Discussion :

[laurentSc]

Bonsoir,

dans le code ci-dessous, on ne passe pas dans le foreach qui affiche la valeur retournée par la requête SQL (quelconque). Pourquoi ?

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
if (!empty($sql)) {
	try{
		$bdd = new MyPdo();
	}catch(PDOException $e){
		echo "argggggggggggggg".$e->getMessage();
	}
	echo "<table width='100%' border='1' style='text-align:center'><tr>";
	$first = true;
	var_dump($bdd); //on passe bien là
	foreach ($bdd->query($sql, PDO::FETCH_OBJ) as $ligne) {
		var_dump($ligne); //on ne passe pas là
		if ($first) {
			echo "<tr>";
			foreach ($ligne as $key => $value) {
				echo '<th>' . $key . '</th>';
			}
			echo "</tr>";
			$first = FALSE;
		}
...

[ABCIWEB]

Bonjour,

Et pourquoi tu ne mets pas la requête à l'intérieur du try{} ? C'est bien pour s'en servir qu'on a déclaré le mode exception dans l'initialisation de la connexion. Cela te donnerais les messages d'erreur appropriés.

Il me semble aussi qu'on a déjà déclaré également le mode PDO::FETCH_OBJ dans l'initialisation de la connexion, donc encore une fois (et je te l'ai déjà dit) IL N'Y A PAS BESOIN DE LE REDECLARER PAR LA SUITE.

Pour le reste je comprends pas trop l'histoire du deuxième foreach.

[sabotage]

Pourquoi ?

Parce que ta requête ne retourne rien, soit elle produit une erreur, soit elle ne produit aucun résultat.

[Invité]

Bonjour Laurent,
tu n'as pas l'impression d'écrire le code en dépit du bon sens ?
C'est quand même toujours plus ou moins écrit n'importe comment, non ?
Et tu refais toujours plus ou moins les mêmes erreurs de débutant...

On est encore quelques uns à avoir encore la bonne volonté de t'aider.
Tu vas finir par épuiser tout notre quota de patience...

Même ABCIWEB est en train de craquer...

[laurentSc]

Pas la peine de t'énerver ABCIWEB ; en fait, ce n'est pas la même application : ici, c'est un phpMyAdmin-like (bon, pas vraiment), en gros un script qui me permet d'exécuter des requêtes SQL et d'en afficher le retour s'il y en a. (J'avais fait ça à une époque car chez Free, je trouvais phpMyadmin hyper lent donc je préférais m'en passer et en gros, je n'y ai recours que pour faire des imports/exports, et en ayant pris l'habitude, je m'en sers chez tous les hébergeurs).
Mais du coup, j'ai aligné le connect.php sur l'autre, et du coup, j'ai enlevé le PDO::FETCH_OBJ ; d'autre part, j'ai aussi intégré la requête dans le try.

Sabotage, je peux tester cela avec n'importe quelle requête et même celles qui doivent retourner quelque chose ne retourne rien ici.

En fait, même avec echo "<br/>argggggggggggggg ".$e->getMessage(); dans le catch, ça affiche :

argggggggggggggg SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '
show tables
' at line 1

si je teste avec show tables.

Bon, Jérôme, ce code est assez ancien, même si je l'ai converti de _mySQL à PDO il y a peut-être 2 ans, mais comme jusque là, il marchait, je ne le modifiais pas...et les erreurs de débutant ici, je les vois pas...

[Invité]

La première "erreur de débutant" est que tu ne sais pas encore effectuer un débogage de base.

Ensuite, c'est de ne pas appliquer les connaissances que tu as apprises, ou les conseils qui t'ont été donnés.

Que le code soit ancien ou pas, tu es censé utiliser tes connaissances nouvelles.

[ABCIWEB]

En mettant ta requête dans le try tu as maintenant le code d'erreur approprié. Reste à l'interpréter. Tu ne sais pas encore lire les messages d'erreur de php ? C'est pourtant de l'anglais très rudimentaire...

[laurentSc]

OK, mais pas très explicite :

ERREUR PDO dans ... L.72 :SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '
show tables
' at line 1

(alors que la requête SQL est show tables)

et comme entre temps, j'ai mis des requêtes préparées et un vrai fetch (ce qui n'était le cas avant), voici le nouveau code :

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
if (!empty($sql)) {
	try{
		$bdd = new MyPdo();			
		echo "<table width='100%' border='1' style='text-align:center'><tr>";
		$first = true;
		$prep = $bdd->prepare($sql);
		$prep->execute();
		foreach ($prep->fetch() as $ligne) {	
			echo "ligne=";var_dump($ligne);echo "<br/>";
			if ($first) {
				echo "<tr>";
				foreach ($ligne as $key => $value) {
					echo '<th>' . $key . '</th>';
				}
				echo "</tr>";
				$first = FALSE;
			}
			echo "<tr>";
			foreach ($ligne as $value) {
				echo '<td>' . $value . '&nbsp;</td>';
			}
			echo "</tr>";
		}
		echo "</table >";
		//efface la variable (pour se déconnecter)
		$bdd = NULL;
	}
	catch(PDOException $e) {
    echo 'ERREUR PDO dans ' . $e->getFile() . ' L.' . $e->getLine() . ' : ' . $e->getMessage();
	}
}

et le fetch n'est pas exécuté vu que le var_dump juste après n'est pas affiché...

[ABCIWEB]

Le message d'erreur indique une erreur de syntaxe dans ta requête aux alentours de "show tables". Et comme tu nous montre tout sauf la requête, que veux tu qu'on te réponde ?

[laurentSc]

Mais si je la donne, vu que c'est justement show tables...

[ABCIWEB]

Faut croire que t'as quand même réussi à faire une erreur dans une requête aussi simple...

[laurentSc]

C'est ce que ça laisse supposer sauf que chez Free où j'ai laissé les fonctions _mySQL, ça fonctionne (alors que chez Evxonline, la connexion échoue ; je donc dois passer en pdo)

[sabotage]

Tu n'aurais pas fait un mélange entre le foreach sur query() et un fetch() ?

Code :

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

1
2
foreach ($bdd->query($sql) as $ligne) {	
			echo "ligne=";var_dump($ligne);echo "<br/>";

Ton prepare() ne sert à rien puisque tu n'as pas de paramètres.

Pour ton problème de SHOW TABLES, est-ce que tu as affiché ta requête complète pour voir ?

[laurentSc]

Comme le prepare ne sert à rien et pour éviter toute confusion, j'ai fait ça :

Code php :

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

1
2
3
$req = $bdd->query($sql);
while ($ligne=$req->fetch()) {
...

Par contre, juste avant, je fais un echo : echo "sql=".$sql."<br/>";et apparemment, un retour à la ligne est rajouté :

sql=
show tables

...et

Code :

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

1
2
3
4
5
6
 
<textarea name="requete" rows="4" cols="55"><?php echo $sql; ?></textarea>
...
<?php
if (isset($_POST['requete']))
	$sql = stripslashes($_POST['requete']);

Le fautif semble donc ce retour à la ligne, mais d'où vient-il et comment le supprimer ?

[ABCIWEB]

Quand tu auras résolu ton pb de connexion/requête il te resteras à corriger ton code car en l'état tu ne pourras lire qu'une ligne...

Faudrait que ça ressemble à un truc comme ç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
<?php
//...
$prep = $connect->prepare($sql);
$prep->execute();
 
$html = null;
$first = true;
 
while($ligne = $prep->fetch())
{ 
	if ($first == true)
	{
		// S'il y a au moins une ligne on fait une table
		$html .= '<table>';
		$html .= "\n".'<tr>';
		foreach ($ligne as $key => $value) $html .= '<th>'.$key.'</th>';
		$html .= '</tr>';
		$first = false;
	}
 
	$html .= "\n".'<tr>';
	foreach ($ligne as $value) $html .= '<td>'.$value.'</td>';
	$html .= '</tr>';
}
// S'il y a eu au moins une ligne on ferme la table
if ($first == false) $html .= "\n".'</table>';
 
?>
<!DOCTYPE html>
<html lang="fr">
<head>
<meta charset="UTF-8" />
<title>titre</title>
</head>
 
<body>
<?= $html?>
</body>
</html>

EDIT : J'ai utilisé prepare pour reprendre ton code. Même si utiliser des requêtes préparées ne sert à rien dans certains cas, je te conseille néanmoins de toujours utiliser des requêtes préparées afin de bien maîtriser au moins une façon de faire (la plus généraliste), le temps de ne plus être débutant.

[sabotage]

Je ne pense pas que ce soit un retour à la ligne seul car cela ne gênerait pas la requête.

Peut être des choses dans des balises :

Code :

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

echo "sql=".htmlentities($sql)."<br/>";

[laurentSc]

Bien vu Sabotage et merci à ABCIWEB dont j'ai pris en compte le code :

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
$bdd = new MyPdo();	
$prep = $bdd->prepare($sql);
$prep->execute();				
 
$html = null;
$first = true;
 
while($ligne = $prep->fetch())
{ 
      if ($first == true)
	{
               // S'il y a au moins une ligne on fait une table
               $html .= '<table>';
               $html .= "\n".'<tr>';
               foreach ($ligne as $key => $value) $html .= '<th>'.$key.'</th>';
               $html .= '</tr>';
               $first = false;
	}
 
$html .= "\n".'<tr>';
foreach ($ligne as $value) $html .= '<td>'.$value.'</td>';
$html .= '</tr>';
}
// S'il y a eu au moins une ligne on ferme la table
if ($first == false) $html .= "\n".'</table>';
 
//efface la variable (pour se déconnecter)
$bdd = NULL;
echo $html;

et si je fais ce que m'indique Sabotage, ça donne :

sql=<p>show tables</p>

donc c'est tinymce qui rajoute ces balises (avant, ça tournait sans cet éditeur) ; est-ce que pour s'en débarrasser, je ne peux faire qu'un traitement de chaîne de caractères ?

[sabotage]

Pourquoi utiliser tinymce pour faire saisir une requête ?

[ABCIWEB]

Eventuellement tu pourrais essayer strip_tags : $sql = strip_tags($sql);

Mais pas garanti que cela fonctionne à tous les coups. Le plus simple serait de suivre le conseil de sabotage et ne pas utiliser tinymce dans ce cas (évidemment puisque tinymce est prévu pour formater du texte...).

D'un autre coté, permettre d'exécuter des requêtes depuis un formulaire c'est vraiment dangereux. T'as intérêt que cette page soit blindée et accessible uniquement à des personnes de confiance ET compétentes sinon ta base peut être complétement détruite.

Enfin bon au final tu vois bien que le message d'erreur de php était bon

[laurentSc]

J'ai testé strip_tags et ça marche
Sinon c'est vrai que tinymce sert pas à grand chose, mais à moi, si car dans mes requêtes, y en a une que j'ai voulu utiliser et qui est un select et j'affiche des phrases générées avec tinymce et donc pour afficher correctement les smileys, j'avais besoin de tinymce...
Et sinon, c'est sûr que mon appli basée sur un formulaire est dangereuse (DROP DATABASE), mais y a que moi qui en connais les URLs (une par hébergeur).