[AJAX] Condition non prise en compte!

**axl2008** · 14/11/2008, 20h59

Bonjour,
Je suis nouveau sur ce forum et nouveau aussi dans le domaine de la programmation Ajax.
J'ai fait un script dont voici un bout en piece jointe.
Ce code fonctionne correctement si je supprime le if :

Code :

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

if(nombre==3){document.getElementById('flash_block').style.display="none"; }

Par contre si je le remets, tout ce qui se trouve dans le if est ignoré même si la condition est vérifiée. La variable nombre existe bien car en faisant
document.write(nombre) j'obtiens bien un résultat.
Si quelqu'un pouvait m'aider, ce serait vraiment sympa car j'ai vraiment du mal à comprendre.
Merci d'avance

**axl2008** · 14/11/2008, 21h02

désolé je reposte la pièce jointe car je me suis trompé d'extension.
Merci d'avance pour vos conseils

**Bovino** · 15/11/2008, 09h56

Comme l'indique son nom, responseText te renvoies du texte, donc une chaîne que tu compares avec un nombre, or 3 et '3' ne sont pas égaux et ta condition est fausse, essaye avec :

Code :

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

if(nombre=='3')

**axl2008** · 15/11/2008, 11h16

Merci pour ta réponse Bovino.
J'ai essayé mais ça ne marche toujours pas !!!
J'ai vraiment du mal à comprendre

**Bovino** · 15/11/2008, 11h36

Montre-nous le code html correspondant.

**axl2008** · 15/11/2008, 11h57

la page appelée est en pièce jointe.
Elle est toute simple. Je récupère le nombre d'images et de l'autre côté je fait une condition suivant le nombre.

**supersnail** · 15/11/2008, 12h14

Tu peux poster ton code directement sur le forum? (en cliquant avant sur le bouton # de ta barre d'outils)
J'ai un principe : je n'ouvre aucune pièce jointe (on sait jamais ce qu'il y a dedans...)

**axl2008** · 15/11/2008, 12h31

Voici la page appelante : (Merci pour le # c'est plus simple

)

(xhr est définie plus haut dans la page)

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
<script language="javascript">
 
function nombreimages(){
				var xhr = getXhr()
				// On défini ce qu'on va faire quand on aura la réponse
				xhr.onreadystatechange = function(){
					// On ne fait quelque chose que si on a tout reçu et que le serveur est ok
					if(xhr.readyState == 4 && xhr.status == 200){
 
					var nombre=xhr.responseText;
					//document.write(xhr.responseText)
					if(nombre==3){
					document.getElementById('flash_block').style.display="none";
								 }				
 
 
																}	
 
													}
 
				//xhr.open("GET","ajax.php",true);
				xhr.open("GET", "nombre_images.php?nocache=" + Math.random(), true); 
				xhr.send(null);
 
 
					  }	
</script>

voici le code HTML :
Cette page est la page nombre_images.php qui est appelée par le code qui est ci-dessus:

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
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Document sans titre</title>
</head>
 
<body>
<?
 
/* déclaration des variables de connexion*/ 
$host = ;
$user = '';
$pass = '';
$db = '';
 
// connection à la DB
$link = mysql_connect ($host,$user,$pass) or die ('Erreur : '.mysql_error() );
mysql_select_db($db) or die ('Erreur :'.mysql_error());
 
 ///////////////////////////////////////////        nombre d'images        ////////////////////////////
$select = "select idimage FROM recupimage";
$result = mysql_query($select,$link)  or die ('Erreur : '.mysql_error() );	
 
 
 
			// si résultat on l'affiche.
if($result)  {
    				while($row1 = mysql_fetch_array($result)) {
						$nbre_images_table+=1;	
 
															 }
 
			}
 
echo $nbre_images_table;
 
</body>
</html>

**jeffray03** · 17/11/2008, 12h17

as-tu essayé :

Code :

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

1
2
 
if(eval(nombre) == 3 )

**supersnail** · 17/11/2008, 19h01

effectivement c'est la conditon qui pose problème (déjà dit plus haut)
On peut pas comparer un entier un nombre avec un texte,çe renverra toujours false...

**jeffray03** · 18/11/2008, 09h50

si c´est pour comparer un texte a un entier tu fais donc ceci :

1. si t´es sur que cela ne comporte pas de balise html

Code :

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

1
2
3
4
5
6
7
8
 
 
var nombre=  eval(xhr.responseText);
 
//document.write(xhr.responseText)
if( parseInt(nombre) == 3){
					document.getElementById('flash_block').style.display="none";
								 }

sinon essaie ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
   var existe = 0;
   var nombre=  eval(xhr.responseText);
   for (var i in nombre) {
   if (  parseInt(i) == 3)
   {
     existe = 1;
     break;  
   } 
  }

et la tu traitera la variable existe.

**DoubleU** · 18/11/2008, 10h35

Comme l'indique son nom, responseText te renvoies du texte, donc une chaîne que tu compares avec un nombre, or 3 et '3' ne sont pas égaux

On peut pas comparer un entier un nombre avec un texte,çe renverra toujours false...

Justement, non.
"3" == 3 >> true, car l'opérateur == compare uniquement la valeur
"3" === 3 >> false, car l'opérateur === compare la valeur ET le type

**marcha** · 18/11/2008, 11h25

Salut,

à mon avis le problème vient de nombre_images.php qui ne renvoie pas
juste un nombre, mais une page html, donc dans ton responseText tu
as qqch du genre "...<html>...<body>3</body></html>"

et tu ne t'en es pas aperçu car tu as utilisé document.write pour ton débugage
et tu as juste vu le "3", les balise html ayant été interprétées par ton navigateur.

avec alert(xhr.responseText) tu devrais y voir plus clair. (Un petit conseil au passage, utilise FireBug ou similaire pour ton développement)

Donc, modifie ta page nombre_images.php pour qu'elle ne retourne que le "3"
tu commences ta page directement par

<?php

et tu fini par

?>

en faisant attention qu'il n'y aie aucun espace avant ni après les balises php.

**DoubleU** · 18/11/2008, 11h31

marcha +1

[AJAX] Condition non prise en compte!

JavaScript

Discussions similaires

Partager

Partager