non-exécution d'un fetch

Version imprimable

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:

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;
		}
...

02/01/2014, 01h17
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.
02/01/2014, 09h41
sabotage

Citation:

Pourquoi ?

Parce que ta requête ne retourne rien, soit elle produit une erreur, soit elle ne produit aucun résultat.
02/01/2014, 10h42
Invité

Bonjour Laurent,
tu n'as pas l'impression d'écrire le code en dépit du bon sens ? :roll:
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... :no:

Même ABCIWEB :calin: est en train de craquer...
02/01/2014, 11h58
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 :

Citation:

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...
02/01/2014, 12h19
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.
02/01/2014, 14h15
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...

OK, mais pas très explicite :

Citation:

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:

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 . ' </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é...

02/01/2014, 15h04
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 ?
02/01/2014, 15h08
laurentSc

Mais si je la donne, vu que c'est justement show tables...
02/01/2014, 15h46
ABCIWEB

Faut croire que t'as quand même réussi à faire une erreur dans une requête aussi simple...
02/01/2014, 16h12
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)
02/01/2014, 17h19
sabotage
Tu n'aurais pas fait un mélange entre le foreach sur query() et un fetch() ?
Code:

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 ?
02/01/2014, 17h58
laurentSc
Comme le prepare ne sert à rien et pour éviter toute confusion, j'ai fait ça :
Code:

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é :

Citation:

sql=
show tables

...et
Code:

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 ?

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:

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.

02/01/2014, 18h26
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:

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

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

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
$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 :

Citation:

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 ?

02/01/2014, 18h56
sabotage

Pourquoi utiliser tinymce pour faire saisir une requête ?
02/01/2014, 19h33
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 ;)
02/01/2014, 22h34
laurentSc

J'ai testé strip_tags et ça marche :ccool:
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).
03/01/2014, 04h08
ABCIWEB

Citation:

Envoyé par laurentSc

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).

Tu devrais faire des requêtes type (déjà écrites) à choisir dans un formulaire et en fonction du choix, ne laisser qu'un (ou plusieurs) paramètres comme variable à rentrer dans un autre formulaire ce qui permettrait au moins de contrôler les possibilités car rien n'est protégé actuellement. Ton "système de protection" est plus illusoire que réellement efficace.

Faudrait te faire un formulaire d'authentification avec login et mot de passe. C'est pas du temps perdu car tu pourras t'en servir pour tous les espaces administrateur.
03/01/2014, 13h48
Bovino

Citation:

y a que moi qui en connais les URLs

:cry:

No more comments...
04/01/2014, 18h20
laurentSc

Bon OK, il serait plus sûr de proposer une liste d'instructions dans une liste déroulante et de laisser libre le reste, si je mets pas le DROP dans la liste, sauf que je m'en sers aussi, donc comment faire ? Et d'autre part, qu'est-ce que je risque ?
04/01/2014, 19h11
Invité
Citation:

qu'est-ce que je risque ?

Que quelqu'un, malgré tout, trouve l'URL du fichier (non protégé !) !
Et sachant que tu utilises sûrement le même type d'URL sur chacun de tes sites, TOUS TES SITES seront vulnérables !
(du genre : www.site1.com/admin/meme-pas-peur.php - www.site2.com/admin/meme-pas-peur.php - ... !)

Citation:

Envoyé par laurentSc

..., donc comment faire ?

Lire/étudier les conseils déjà donnés.

Citation:

Envoyé par ABCIWEB

...faire un formulaire d'authentification avec login et mot de passe. C'est pas du temps perdu car tu pourras t'en servir pour tous les espaces administrateur.

+ voir aussi :
- Qu'est-ce qu'un fichier .htaccess ?
- Comment mettre en place une authentification simple des utilisateurs ?
04/01/2014, 20h40
ABCIWEB

Tu peux trouver ici un exemple de formulaire complet qui permet en plus un hash du mot de passe pour éviter qu'il transite en clair dans les tuyaux.

L'autre solution donnée en dernier exemple par jreaux62 (authentification via http et htaccess) est plus rapide à mettre en place, mais elle est moins souple d'utilisation, moins transposable d'un site à l'autre et elle n'est pas personnalisable. Mais pour aller vite c'est le minimum vital.

Enfin bon il faudrait que tu t'intéresse sans plus tarder à la protection d'un espace administrateur (appelé aussi "espace membres" si tu veux plus de réponses dans google). Normalement cela se fait naturellement avant la création d'un espace administrateur, pas après, c'est bizarre que tu n'aies pas compris cette priorité. Tu te serais aperçu qu'on utilise jamais la méthode que tu utilises qui est vraiment trop puérile (en fait elle n'est même jamais mentionnée).
05/01/2014, 18h18
Bovino

J'imagine que la notion d'utilisateur et de privilèges pour une base de données t'es inconnue... :no:
05/01/2014, 19h07
Invité

Si tu es LE SEUL à utiliser cet "admin", un .htaccess est le minimum vital pour protéger ton accès (avec login et mot de passe).
:alerte: C'est à mettre en place d'urgence !!!
(si quelqu'un trouve l'URL, il faudra aussi qu'il trouve le login et mot de passe pour accéder à la page).

Mais, dès que possible, installe un VRAI "panel d'administration" (appelé aussi "espace administrateur") !

Fuseau horaire GMT +2. Il est actuellement 23h41.