par
Ryle » 13 févr. 2011, 12:19
En complément tu as également quelques petites erreurs de conception, pas méchants, mais puisque tu demandes les erreurs

:
- Si jamais la variable $_GET['modifier_news'] n'est pas définie, tu ne vas pas rentrer dans le if(), et tu n'exécuteras donc pas la requête. Pourtant juste après, requête ou pas tu vas faire un while pour parcourir les résultats de la requête, ce qui peut engendrer une erreur puisque $retour ne sera peut être pas défini.
- Second point, d'après ta requête sql, on peut supposer que celle-ci ne doit retourner qu'un seul enregistrement (celui dont l'id est passé en paramètre). Du coup il n'est pas nécessaire de "boucler" sur l'ensemble des résultats retournés avec un while. Un simple if() pour tester qu'il y a bien un résultat ferait tout aussi bien l'affaire, voire même, si tu es certains qu'il y aura toujours un résultat trouvé, un simple appel au fetch.
C'est vraiment du détail, mais ça permet de comprendre plus facilement ton code (surtout pour celui qui reviendra dessus dans 6 mois, et qui pourrait très bien être toi

) parce que là, mon premier réflexe, c'est de me dire que tu as un formulaire html dans une boucle while, et donc que tu veux afficher plusieurs formulaires sur ta page, alors que ça ne sera à priori jamais le cas

En complément tu as également quelques petites erreurs de conception, pas méchants, mais puisque tu demandes les erreurs :) :
- Si jamais la variable $_GET['modifier_news'] n'est pas définie, tu ne vas pas rentrer dans le if(), et tu n'exécuteras donc pas la requête. Pourtant juste après, requête ou pas tu vas faire un while pour parcourir les résultats de la requête, ce qui peut engendrer une erreur puisque $retour ne sera peut être pas défini.
- Second point, d'après ta requête sql, on peut supposer que celle-ci ne doit retourner qu'un seul enregistrement (celui dont l'id est passé en paramètre). Du coup il n'est pas nécessaire de "boucler" sur l'ensemble des résultats retournés avec un while. Un simple if() pour tester qu'il y a bien un résultat ferait tout aussi bien l'affaire, voire même, si tu es certains qu'il y aura toujours un résultat trouvé, un simple appel au fetch.
C'est vraiment du détail, mais ça permet de comprendre plus facilement ton code (surtout pour celui qui reviendra dessus dans 6 mois, et qui pourrait très bien être toi ;)) parce que là, mon premier réflexe, c'est de me dire que tu as un formulaire html dans une boucle while, et donc que tu veux afficher plusieurs formulaires sur ta page, alors que ça ne sera à priori jamais le cas :)