Discussion :

[maxfive7]

Bonjour,

j'ai une fonction qui marche très bien, jusqu'à un certain nombre d'exécution,
tout d'abord voici le 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
	$pseudo="admin";
	$sql = "SELECT C.Case FROM COMPTE C WHERE C.Pseudo = '".addslashes($pseudo)."'";
	$req = mysql_query($sql) or die('Erreur SQL : <br />'.$sql);
 
	if (mysql_num_rows($req) > 0) {
		$data = mysql_fetch_assoc($req);
		$caseJoueur =  $data['Case'];
	}
 
	// Si c'est un changement de case 
	if(isset ($_GET['NvelleCase'])){  
 
		$NewCase=$_GET['NvelleCase'];
		$caseJoueur = $NewCase;
		$sql = "UPDATE COMPTE C SET C.Case=$NewCase WHERE Pseudo = '".$pseudo."'";
		$req = mysql_query($sql) or die('Erreur SQL : <br />'.$sql);
		echo "vérification execution";
 
	}

Donc en fait, ce code est utilisé lors d'un changement de case sur une map,
j'effectue plusieurs déplacement, tout marche bien, sauf qu'au bout d'un moment la requete UPDATE ne marche plus, il n'y a pas d'erreur d'execution mais dans la base de donnée la valeur de la case n'est plus modifiée!!! pourtant ça a marché plein de fois! donc je disais au bout d'un moment sa ne marche plus, ou alors ça marche de temps en temps après de nombreux déplacement...

j'ai ajouté "echo "vérification execution";" pour vérifier que le code à bien été exécuté, c'est le cas à chaque déplacement, mais c'est comme si la requete avait échouée...

Je ne comprend pas du tout là!!!!

Une idée??? Merci d'avance

[Séb.]

Ce que je ferais à ta place :

Code :

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

1
2
3
4
5
6
7
if(isset ($_GET['NvelleCase'])){  
    $NewCase=$_GET['NvelleCase'];
    $caseJoueur = $NewCase;
    $sql = "UPDATE COMPTE C SET C.Case=$NewCase WHERE Pseudo = '".$pseudo."'";
    $req = mysql_query($sql) or die('Erreur SQL : <br />'.$sql);
    echo 'sql[', $sql, ']erreur[', mysql_error( ), ']n°[', mysql_errno( ), ']' ;
}

Simplifie également ta requête en :

Code :

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

$sql = "UPDATE COMPTE SET Case=$NewCase WHERE Pseudo = '$pseudo'";

Sinon tu utilises quelle version de MySQL ?

[maxfive7]

Ok j'ai simplifié sauf pour le "C." il ne marche pas sans...,
par contre ton code m'affiche ça :

sql[UPDATE COMPTE C SET C.Case=9 WHERE Pseudo = 'admin']erreur[]n�[0]


et sinon j'ai testé le retour de la requête par :

Code :

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

1
2
$req = mysql_query($sql) or die('Erreur SQL : <br />'.$sql);
if ($req) { echo "update fait"; }

et il me dit bien que l'update a réussi, et pourtant quand je vérifie dans la base la valeur n'a pas changé. Pourtant quand je me déplace sur la carte au début ça marche bien, dans la base sql la valeur est bien changé, et puis à un moment ça commence à déconner, et la valeur de la change ne change que de temps en temps... vraiment bizarre! (j'ai aussi vérifié qu'il avait bien la bonne valeur de la case dans la requête)

[Shugo78]

Tu utilise :

Code :

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

1
2
 
or die(mysql_error());

L'erreur devrait déjà être plus clair

[maxfive7]

Ok j'ai modifié, mais sinon y'a pas d'erreur
et il me dit bien que l'update a reussi à chaque changement de case, mais desfois la base de données n'est pas modifié... ça dépend quand

[Shugo78]

Hum Hum, embêtant
Repasse ton code tout modifié STP

[Dudule-le-poisson]

Apparemment, la requête SQL est toujours syntaxiquement bonne puisque mysql_query n'échoue jamais. Peut-être que sa sémantique dévie avec le temps ?
Je pense que tu peux regarder si ta requête agit en utilisant la fonction mysql_affected_rows, qui renvoie le nombre d'enregistrements affectés par ta requête. À priori, il doit toujours valoir 1. S'il vaut 0, c'est le début de ton bug.
Enfin, je constate que tu utilises addslashes($pseudo) pour le SELECT et $pseudo pour l'UPDATE. Est-ce que tu utilises toujours "admin" comme pseudo ? Est-ce que le pseudo que tu utilises comporte des quotes ou backslashs ?

[maxfive7]

Je vais tester avec mysql_affected_rows.

Pour le pseudo, comme je débute la création du site, temporairement je n'utilise que ce pseudo, qui s'ecrit tout simplement "admin"

le "addslashes" j'ai oublié de l'enlever, ça vient d'une morceau de code d'un autre site que j'avais fait.

Ma version de MySql : 4.1.9

[maxfive7]

Merci beaucoup!!!
En effet ça marche mieux maintenant
Je vais étudier ça! Merci encore pour le temps consacré à mon problème


EDIT :

Sur internet j'ai vu une "solution" qui permet d'interdire la mise en cache de la page, avec ces balises à mettre dans le header :

Code :

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

1
2
<META HTTP-EQUIV="Pragma" CONTENT="no-cache"> 
<META HTTP-EQUIV="Expires" CONTENT="-1">

Mais ça ne marche pas, je l'ai ai mise dans index.php,
je me demandais si ce'est parce que l'adresse concerné est : map.php?case=numCase et non index.php ?
mais map.php n'étant qu'un morceau de index.php, il n'a pas de header ni rien...
Je cherche

[Dudule-le-poisson]

C'est bien ta page map.php qui doit contenir ce genre d'instructions, car c'est elle qui est rechargée (par XHR) et mise en cache par le navigateur. Mais comme elle n'a pas d'en-tête HTML et que les navigateurs ne sont pas tenus de respecter les diverses <meta>, essaye :

Code :

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

header("Expires : 0");

dans map.php.

[maxfive7]

Warning: Cannot modify header information - headers already sent by (output started at c:\progs\easyphp1-8\www\index.php:7) in c:\progs\easyphp1-8\www\map.php on line 1

Pas encore ça

[Dudule-le-poisson]

Ah oui, c'est vrai qu'au début, tu l'includes ; tu te retrouves alors avec une instruction header() après avoir écrit du texte, ce qui ne va pas.
Je te propose la petite bidouille suivante : dans index.php, avant l'include, mets une petite variable $mettre_headers = true;.
Dans map.php, rajoute if (isset($mettre_headers)) devant ton instruction header().
Ça permettra de n'exécuter l'instruction header() que lorsque la page est appelée par XHR.

[maxfive7]

Toujours le même message d'erreur