Message d'erreur MySQL

**jlb59** · 14/04/2008, 15h09

Envoyé par darkstar123456

non mais cherche pas et lis ce que j'écris. Tu dois forcément avoir une condition ou une boucle quelque part (ça fait juste la 3ieme fois que je le dis hein !).

Montres-nous le code en entier, car je suis certain que ton erreur mysql_close vient du fait qu'il n'a rien à fermer car tu ne fais pas de requête (condition qui ne passe pas/jamais).

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
40
41
42
43
44
45
46
<?php
if ((isset($_GET['motscles']) && !empty($_GET['motscles'])) && (isset($_GET['id']) && !empty($_GET['id'])))
{
$id = $_GET['id'];
$motscles = $_GET['motscles'];
?>
 
<!-- /////////////////////////////////////////////////// -->
<!-- //////////// Mise à jour des mots-clés //////////// -->
<!-- /////////////////////////////////////////////////// -->
 
<?php
$host = "localhost";	
$user = "root";	
$password = "";	
$nombase = "recherche";
$nomtable = "recherche";
 
$lien = mysql_connect($host,$user,$password) or die('Erreur de connexion : ' . mysql_error() . ""); 
mysql_select_db($nombase,$lien) or die ('Erreur de connexion à la base. ' . mysql_error() . ""); 
$sql = mysql_query("SELECT * FROM $nomtable LIMIT 0, 30") or die('Erreur de la requête : ' . mysql_error() . ""); 
 
echo '<fieldset>';
echo '<legend>Mise à jour de mots-clés de la table <strong>', $nomtable, '</strong></legend>';
 
$requete = mysql_query("SELECT * FROM $nomtable WHERE id=$id");
      $reponse = mysql_fetch_assoc($requete);
      $contenu_actuel = $reponse['motscles'];
      $nouveau_contenu = $contenu_actuel.$motscles;
 
echo 'Base : <strong>', $nombase , '</strong> <br /> Table : <strong>',  $nomtable , '</strong>';
echo 'ID modifié : <strong>' ,$id ,'</strong> <br />';
echo 'Titre : <strong>',$reponse[titre], '</strong> <br /><br />';
echo 'Nouveaux mots clés enregistrés : <br /><strong>', $nouveau_contenu ,'</strong><br /><br />';
 
mysql_query("UPDATE $nomtable SET motscles = '$nouveau_contenu' WHERE id=$id"); 
}
mysql_close();
?>
 
<br /><br />
[ <a href="javascript:history.go(-1)">Retour page précédente</a> ]
<br /><br />
<form method="post" action="../choix.html">
<input type="submit" name="choix" value="Retour au choix">
</form>

J'ai encore cherché "{" mais je vois pas, sauf peut-être au tout début du code.
C'est peut-être de là que vient l'erreur !
Je regarde ça de plus près.

**darkstar123456** · 14/04/2008, 15h13

et bien voilà c'est bien le if() du début...

La condition n'est pas remplie DONC pas de requête DONC mysql_close() renvoie une erreur car pas de requête.

CQFD.

Vérifie ta condition pour qu'elle renvoie true

PS : Petit conseil, indente ton code... tu aurais vu tout de suite la faute, même sans l'habitude de coder

PS 2 : tu as des doubles () dans ta condition :

Code php :

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

1
2
3
 
if( (tes conditions) ) {
}

les 2iemes parenthèses ne sont pas obligatoires car tu n'as que des ET (&&)

**Jumano** · 14/04/2008, 15h27

Positionne ton "}" juste après mysql_close();

**darkstar123456** · 14/04/2008, 15h31

Envoyé par jumano

Positionne ton "}" juste après mysql_close();

ouais mais bon, ca résoudra pas son problème parce que ça je l'avais déjà dit.

Faut vérifier les conditions pour que le script passe de temps en temps

**jlb59** · 14/04/2008, 15h32

Envoyé par darkstar123456

La condition n'est pas remplie DONC pas de requête DONC mysql_close() renvoie une erreur car pas de requête.

CQFD, CQFD, pour toi peut-être !
Pour moi, et j'en ai encore beaucoup à apprendre, tant que la base n'est pas fermée, je fais la ou les requête !
Je suis certainement à côté de mes pompes...

Envoyé par darkstar123456

les 2iemes parenthèses ne sont pas obligatoires car tu n'as que des ET (&&)

Là, j'ai très bien saisi...
Les () ne sont pas nécessaires avec ce genre d'opérateurs ?

Alors bonjour l'angoisse...

**darkstar123456** · 14/04/2008, 15h46

CQFD, CQFD, pour toi peut-être !
Pour moi, et j'en ai encore beaucoup à apprendre, tant que la base n'est pas fermée, je fais la ou les requête !
Je suis certainement à côté de mes pompes...

En fait oui, tant que tu n'es pas connecté à la BDD, tu ne peux pas faire de mysql_close(). Or dans ton code, la connection se faire dans le if() [qui à première vue ne passe pas]. Donc ton mysql_close() te dit qu'il se la coule douce car il a rien fait du tout :p

les 2iemes parenthèses ne sont pas obligatoires car tu n'as que des ET (&&)

En fait, en logique pour qu'un ET (&&) soit vrai, il faut que toutes les conditions soient vraies, et donc pas besoin de parenthèses supplémentaires

**jlb59** · 14/04/2008, 15h48

C'est noté darkstar123456, en espérant m'en rappeler.

**darkstar123456** · 14/04/2008, 16h01

Voilà pour t'aider les "tables de vérité" (c'est vraiment leur nom :p)

LE "ET" (&&)

LE "OU" (||)

**Jumano** · 14/04/2008, 16h08

darkstar123456, selon toi quelle serait la bonne syntaxe pour cette ligne :

Code :

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

1
2
 
if ((isset($_GET['motscles']) && !empty($_GET['motscles'])) && (isset($_GET['id']) && !empty($_GET['id'])))

**jlb59** · 14/04/2008, 16h08

Merci pour ces tables darkstar123456.

Là je m'adresse à toi et à Jumano.

Mon problème est résolu, c'était juste une question de "$_GET[...]"

En vérité, il fallait mettre POST à la place du GET, puisque dans mon formulaire le type était en "post".

Je comprends plus parce qu'à cause de ça uniquement cela engendre toutes ces erreurs !

Ah ! l'informatique...

**darkstar123456** · 14/04/2008, 16h11

Envoyé par jumano

darkstar123456, selon toi quelle serait la bonne syntaxe pour cette ligne :

Code :

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

1
2
 
if ((isset($_GET['motscles']) && !empty($_GET['motscles'])) && (isset($_GET['id']) && !empty($_GET['id'])))

haa mais la synthaxe n'est pas fausse... c'est juste qu'en tant que programmeur, moins on en fait mieux c'est

Donc j'aurais juste mis :

Code php :

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

1
2
 
if( isset($_GET['motscles']) && !empty($_GET['motscles']) && isset($_GET['id']) && !empty($_GET['id']) )

PS : lolilol j'ai l'impression de passer une interro

**Jumano** · 14/04/2008, 16h15

Ok, merci car depuis tout à l'heure j'essayais en vain d'enlever certaine "(" mais je voulais absolument laisser une "(' devant isset,

donc message d'erreur à chaque fois ...

**darkstar123456** · 14/04/2008, 16h19

j'avoue que c'est aussi pour la bonne santé mentale de tout le monde qu'il vaut parfois mieux limiter les parenthèse

**jlb59** · 14/04/2008, 16h37

Jumano et darkstar123456,

Je tiens beaucoup à vous remercier de votre aide.

J'en ai appris pas mal, maintenant, le plus dur, c'est de retenir ! Bonjour Alzheimer... Non mais je plaisante !

Quand je pense qu'il faut faire vachement gaffe avec les "()" et les "{}", brrrr ! Rien que d'y penser...

A bientôt les gars.

Message d'erreur MySQL [MySQL]

PHP & Base de données

Discussions similaires

Partager

Partager