Qu'est ce qu'elle a ma requête UPDATE ?

Version imprimable

13/05/2006, 15h34
psychoBob

Qu'est ce qu'elle a ma requête UPDATE ?
Bonjour,

Qui peut me dire pourquoi cette requête plante avec obstination ?
Code:

1 2 3 4 5 6 $connexionRatee= "UPDATE ipBloque SET tentative=tentative+1, dateLimite='".$dateUnix."' WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."' "; mysql_query($connexionRatee);
ça ne donne pas de message d'erreur, mais ça n'update pas non plus.
13/05/2006, 15h47
Julien.alkaza

Citation:

SET tentative=tentative+1

Je pense que ca vient de là...
Je ne crois pas que tu puisse utiliser la valeur d'un champs comme ca...
Il faut que tu récupère sa valeur via un "SELECT tentative FROM IPBloquee WHERE IP = '$IP';";

Puis après tu incrémente ta valeur tentative ( $tentative++;), et seulement ensuite tu fais un UPDATE avec tentative = '$tentative'...
13/05/2006, 15h48
billoum

Salut,

Pê qu'aucun enregistrement ne satisfait la condition.
Fais un select avec la même clause WHERE pour voir si ca te retourne des enregistrements.

En fait c'est tout le script qui foire...:?

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
//on récupère l'IP
$ip = $_SERVER["REMOTE_ADDR"];
// la date unix
$dateUnix=time();
// la date unix - 15 minutes
$dateMaxi=$dateUnix-900;
//on va chercher si des ip ont été enregistrée dans les 15 minutes.
$affiche="
SELECT ip  FROM ipBloque 
WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."'";
$resultat = mysql_query("$affiche");
$ipBloque=mysql_num_rows($resultat);
//si on a trouvé aucune ip, on insère celle ci (ça n'insère rien).
if($ipBloque=0)
{
$connexionRatee="INSERT INTO `ipBloque` (`ip`,`dateLimite`,`tentative`) 
VALUES
(
'$ip','$dateUnix','1'
)";
mysql_query($connexionRatee);
}
//si on a trouvé un résultat, on update le tuple en 
incrémentant de 1 le nombre de tentative, 
ainsi que la date, pour 
bloquer 15 minute si tentative est à 5 (ça c'est dans un autre script): 
ça n'update rien non plus.
if($ipBloque==1)
{
$connexionRatee="
UPDATE ipBloque 
SET tentative='tentative+1', dateLimite='".$dateUnix."'  
WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."'";
mysql_query($connexionRatee);
;}
mysql_free_result($resultat);

13/05/2006, 15h54
psychoBob

Citation:

Envoyé par billoum

Salut,

Pê qu'aucun enregistrement ne satisfait la condition.
Fais un select avec la même clause WHERE pour voir si ca te retourne des enregistrements.

Oui mais comme je viens de dire dans la réponse que j'écrivais pendant que tu faisais du même, si ça ne trouve rien, ça doit quand même insérer un tuple, justement. Et si ça en trouve un, celui qui vient d'être inséré, alors on l'update.

Je me demande si ça ne viendrait pas du fait que les variables tout en haut ($dateMaxi et $ip) ne sont pas transmise à l'intérieur du script, mais je ne comprend pas pourquoi alors.
13/05/2006, 15h55
Julien.alkaza

Tes requêtes marchent ou pas??
J'entends par là dans PHPMyAdmin (ou autre)??

Sinon, tu n'as pas besoin de faire tout un fouilli comme celui là :

Citation:

ip='".$ip."'

Si tu mets ta requête entre "", tu fais juste

Code:

"...WHERE ip = '$ip'..."

et c'est bon!!
13/05/2006, 16h06
psychoBob

La requête insert fonctionne bien dans phpMyAdmin, avec des chiffres comme 1 à la place des variables.
Mais dans le script ça ne fonctionne pas.

L'update j'ai pas testé dans phpMyAdmin, je veux déjà régler le INSERT.
13/05/2006, 16h12
billoum

T'as fait :

Code:

mysqlquery($requete) or die($requete.':<br>'.mysql_error());

après chaque requête?
Et aussi (même si jpense pas que ca changera gd chose), c'est une faute de frappe là (manque '=='):

Code:

if($ipBloque=0)

?
13/05/2006, 16h13
psychoBob
Bon alors, c'est réglé pour ce script, il manquait un = à if(ipBloque=0).

Maintenant c'est celle là qui foire :
Code:

1 2 3 4 5 6 " SELECT ip FROM ipBloque WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."' AND tentative <= 5 "
ça doit être au niveau du 5 je pense...

**edit**

On a posté ensemble, Billoum :D

Citation:

Et aussi (même si jpense pas que ca changera gd chose), c'est une faute de frappe là (manque '=='):

Code :

if($ipBloque=0)

Bah si c'était radical même comme erreur ;)
13/05/2006, 16h20
Julien.alkaza

Tiens...j'l'avais pas vu celui là :aie::aie:
13/05/2006, 16h21
billoum

Ben en fait j'avais testé chez moi un if pareil mais ca n'empêchait pas de rentrer dans la boucle ( ca t'aurait juste empêché de rentrer dans la seconde vu que 0 serait affecté à la variable, enfin bon...:roll:).

Sinon, pour tester tes requêtes,est-ce que tu fais:

Code:

mysqlquery($requete) or die($requete.':<br>'.mysql_error());

?
13/05/2006, 16h23
psychoBob

Je viens de faire le mysql_error mais ça ne retourne pas d'erreur. ça ne trouve pas le tuple, pourtant il y en a un qui correspond.

ça doit encore être une erreur bidon, mais bon...
13/05/2006, 16h26
psychoBob

Bon déjà l'opérateur était à l'envers, c'était >= et non <= .

Mais c'est pas ça quand même :roll:
13/05/2006, 16h27
Julien.alkaza

Essaye ce que je te dis...
remplace tes '".$ip."' par '$ip'

Ca simplifie grandement les choses....et ca vient peut être de là...

EDIT : par "datelimite" tu entends quoi dans ta table??
13/05/2006, 16h32
psychoBob

J'ai réussi, j'avais pas changé la logique du script, je fais toujours des if $ipBloque<=5 alors qu'il fallait désormais juste tester si il y a avait un tuple donc if$ipBloque==1.

Pour les '".$ip."' à remplacer par '$ip', on m'avait expliqué que les ".." renforçait la sécurité en gênant les injection sql.

Pour le dateLimite, t'étais sur la piste Julien.alkaza, mais sans le script en entier...

Merci à tous en tout cas.