verification de disponibilité de pseudo dans une table mysql [Résolu]

[DeD b]

Bonjour
je débute en javascript
je viens d'essayer un bout de code que j'ai déniché sur un site, qui vérifie si un pseudo est disponible
alors, il comporte deux fonction, une pour lire le fichier qui contient la requete vers ma base de donner, et une autre qui teste les valeurs retourné par la 1ere fonction et exécute le traitement nécéssaires.
le bout de code est très lisible, le passage de paramètres entre mon formulaire, le fichier .php et la fonction qui lit le fichier s'effectue sans problème.

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
<?
 
 
// CONNECION SQL
mysql_connect("localhost", "XXXXX", "XXXXX");
mysql_select_db("XXXXX");
 
// VERIFICATION
$result = mysql_query("SELECT PSEUDO FROM membre WHERE PSEUDO='".$_GET["pseudo"]."'");
if(mysql_num_rows($result)>=1)
        echo "1";
else
        echo "2";
?>

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
40
 
function writediv(texte)
     {
     document.getElementById('pseudobox').innerHTML = texte;
     }
function file(fichier)
     {
     if(window.XMLHttpRequest) // FIREFOX
          xhr_object = new XMLHttpRequest();
     else if(window.ActiveXObject) // IE
          xhr_object = new ActiveXObject("Microsoft.XMLHTTP");
     else
          return(false);
     xhr_object.open("GET", fichier, false);
     xhr_object.send(null);
     if(xhr_object.readyState == 4) return(xhr_object.responseText);
     else return(false);
     }
 
function verifPseudo(pseudo)
     {
     if(pseudo != '')
               {
               if(pseudo.length<2)
                         writediv('<span style="color:#cc0000"><b>'+pseudo+' :</b> ce pseudo est trop court</span>');
               else if(pseudo.length>30)
                         writediv('<span style="color:#cc0000"><b>'+pseudo+' :</b> ce pseudo est trop long</span>');
               else if(texte = file('verifpseudo.php?pseudo='+escape(pseudo)))
                         {
 
          if(texte == 1)
               writediv('<span style="color:#cc0000"><b>'+pseudo+' :</b> ce pseudo est deja pris</span>');
          else if(texte == 2)
               writediv('<span style="color:#1A7917"><b>'+pseudo+' :</b> ce pseudo est libre</span>');
          else
               writediv(texte);
                         }
               }
 
     }

dans la dernière fonction VerifPseudo, lorsque la longueur du pseudo est appropriée, mais au lieu que la variable texte se contente de prendre la valeur 1 ou 2 (comme il est indiqué dans le fonction .php ) elle prend tout le reste , voila ce que j'ai eu dans mon navigateur :
=1) echo "1"; else echo "2"; ?>
c'est le 2eme else qui s'est exécuté...

[NoSmoking]

Bonjour,
pas un spécialiste du PHP, mais il semblerait que tu ais un problème de syntaxe

Code php :

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
// CONNECION SQL
mysql_connect("localhost", "XXXXX", "XXXXX");
mysql_select_db("XXXXX");
// VERIFICATION
$result = mysql_query("SELECT PSEUDO FROM membre WHERE PSEUDO='".$_GET["pseudo"]."'");
if( mysql_num_rows($result)){
  echo "1";
}
else {
  echo "2";
}
?>

le fait d'injecter directement $_GET dans ta requête reste une opération suicidaire...

[DeD b]

Envoyé par NoSmoking

Bonjour,
pas un spécialiste du PHP, mais il semblerait que tu ais un problème de syntaxe

Code php :

le fait d'injecter directement $_GET dans ta requête reste une opération suicidaire...

bon, disons que l'ensemble de teste sur le pseudo saisi est effectué ailleurs...
la variable pseudo est récupéré sans problème, mais uniquement dans le cas ou sa taille est <1 ou dépasse 30, je peux même l'afficher dans le navigateur

[SpaceFrog]

Le problème ne se situe pas coté javascript vu que c'est le responseText qui n'est pas correct.

[emmanuel.remy]

Envoyé par DeD b

bon, disons que l'ensemble de teste sur le pseudo saisi est effectué ailleurs...
la variable pseudo est récupéré sans problème, mais uniquement dans le cas ou sa taille est <1 ou dépasse 30, je peux même l'afficher dans le navigateur

Ton code Javascript dit: si la longueur du pseudo est strictement inférieur à 2 je fais ça, si la longueur du pseudo est strictement supérieur à 30 je fais ça, sinon je fais mon appel ajax synchrone

Cherche pas plus loin.

ERE

[andry.aime]

Bonsoir,
1- Tu n'as pas besoin de tester le readyState parce que tu travailles en mode synchrone.

Code :

1
2
xhr_object.send(null);
     return(xhr_object.responseText);

2- Le responseText est un String mais pas un int.

Code :

if(texte == "1")

A+.

[Watilin]

Envoyé par DeD b

Code :

[…] voila ce que j'ai eu dans mon navigateur :
=1) echo "1"; else echo "2"; ?>

Le problème vient de la façon dont tu écris tes balises PHP : <? ... ?>. C'est la syntaxe « raccourcie ». À mon avis, ton serveur ne la supporte pas, ce qui expliquerait pourquoi il repasse en mode texte quand il rencontre un chevron fermant (>) dans la comparaison if (mysql_num_rows($result)>=1). Regarde le code HTML généré pour être sûr.

Je te conseille, à titre personnel, d'écrire systématiquement les balises PHP « longues » : <?php ... ?>.