Discussion :

[jowelle]

Salut,
En utilisant le code suivant je voulais afficher la page index.jsp si showdata=Welcome User!
sinon afficher la page Authentification.jsp
mais quand j’exécute le code il m'affiche toujours la même page dans tout les cas.
quel est le problème?

Code :

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

1
2
3
4
5
6
7
8
9
10
var showdata = xmlHttp.responseText; 
    alert(showdata);
    var txt = new String("Welcome User!");
 
    if(txt=showdata)
    {window.open("index.jsp");}  
    else 
    {
        window.open("Authentification.jsp");
    }

Merci.

[vermine]

Bonjour,

Ici vous faites une affectation :

Code :

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

txt=showdata

C'est-à-dire que txt prend la valeur de showdata. Vous devez doubler l'égal pour faire une comparaison :

Code :

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

txt==showdata

[jowelle]

le problème persiste encore.

[jowelle]

notons que c'est un code qui est inclut dans un javascript ,les instructions "if else" ne sont pas toujours comprises, le programme affiche les deux pages ensembles.

[Watilin]

Salut,

une affectation dans un if est syntaxiquement correcte. Ça n'envoie pas de message d'erreur, et peut donc entraîner des problèmes difficiles à déboguer.

Je te conseille une astuce assez répandue :

Code :

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

1
2
3
if ("Welcome User!" == showdata) {
   ...
}

En mettant la partie constante (ici, une chaîne) en partie gauche de la comparaison, tu te protèges contre une faute de frappe ou d'inattention. Car si tu oublies un signe égal, ça donne "Welcome User!" = showdata et ça, ça génère une erreur que tu peux détecter tout de suite

Sinon, pour ton problème, il doit venir d'ailleurs. Si showdata était incorrect, tu le verrais grâce au alert, et le seul bout de code que tu nous montres ne permet pas d'ouvrir les deux fenêtres à la fois.

[NoSmoking]

Bonsoir,
compares les longueurs de tes 2 variables, il existe peut être un \r\n ou autre de caché dans showdata.

[Willpower]

en version bien indentée :

Code :

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

1
2
3
4
5
6
7
8
var showdata = xmlHttp.responseText;
alert(showdata);
if ("Welcome User!" == showdata) {
    window.open("index.jsp");
} 
else {
    window.open("Authentification.jsp");
}

avec toutes les lettres :

l'instruction if else n'est Pas comprisE toujours le Programme affiche les deux pages ensembles.

en français correct :

les instructions "if else" ne sont pas toujours comprises, le programme affiche les deux pages ensembles.

Je pense que si tu viens demander de l'aide, le minimum est de soigner au minimum tes phrases (on ne demande pas une orthographe irréprochable) et le code que tu nous présentes. Aussi, si tu codes de la même façon que tu écris en oubliant une lettre par mot, tu vas avoir beaucoup de mal.


Enfin, n'ayant pas compris les phrases de ton dernier poste, je ne peux pas vraiment t'aider.

C'est quelle page qui s'ouvre maintenant ? "index.jsp" ou bien "Authentification.jsp" ?

Le conseil que je peux néanmoins te donner est de vérifier ton string, s'il ne contient pas d'espaces supplémentaires ou des caractères spéciaux comme le saut de ligne. ( \n )

edit: pris de vitesse par noSmoking ^^

[NoSmoking]

edit: pris de vitesse par noSmoking ^^

eh oui c'est le résultat du NoSmoking

[jowelle]

Merci infinement NoSmoking pour votre aide ,oui c'était un problème d'espace supplémentaire, j'ai résolu maintenant le problème.

pour WillPower pardon si je t'es dérangé avec ma manière d'écriture, en fait j'ai pas relu qu'est ce que j'ai écrit en plus la langue française n'est pas ma langue maternelle.mais l'essentiel je pense que vous avez compris qu'est ce que je veux dire par ma question et la preuve est que vous avez répondu , en tout cas merci pour vous.