Discussion :

[bilel.1st]

Bonjour.

j'ai créé une fonction qui récupère le résultat (JSON) d'un traitement AJAX et parcoure ce dernier afin de l'afficher avec le code HTML approprié.

Au début le comportement de cette fonction était comme prévu mais après certains évolutions (Diverse contenu entré par les utilisateurs) la fonction se bloque au niveau de la boucle FOR qui parcoure le résultat, ce blocage fait bloquer le navigateur par la suite.

Pour remédier à ça j'ai changer la boucle FOR par une WHILE, le problème est régler jusqu'au présent.

Après tous ça je voudrai savoir si j'ai bien fais avec se changement et les risques/limites de tells boucles.

Merci.

[sebhm]

Bonjour,

il devait y avoir un probleme avec ta boucle For car on peut faire la meme chose avec les 2.

[bilel.1st]

En fait j'ai vérifié la boucle FOR et j'ai rien trouvé d'anormal, c'est ce qui m'a poussé de poster ce message dans le forum.

[sebhm]

oui bien sur j'avais compris que tu avais vérifié.
Mais nous on n'a pas vu le code, donc on a le droit d'être sceptique
parce que, normalement, ca fait la meme chose...

[bilel.1st]

Vous avez raison, 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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
Rsp = eval('(' + xhr.responseText + ')');
Length = Rsp.Objs.Obj.length;
 
for(i=1;i<Length;i++)
{
	var1 = Rsp.Objs.Obj[i].ID;
	var2 = Rsp.Objs.Obj[i].NOM;
 
	if(Condition1)
	{
		switch(TYPE)
		{
			case 1 : 
			{
				DIV_1 += var1 + " - " + var2;
			} break;
			case 2 : 
			{
				DIV_2 += var1 + " - " + var2;							
			} break;
			case 3 : 
			{
				DIV_3 += var1 + " - " + var2;
			} break;
			case 4 : 
			{
				DIV_4 += var1 + " - " + var2;
			} break;
		}
	}
	else if(Condition2)
	{
		//même type de traitement;
	}
	else
	{
		//même chose aussi;
	}
}

j'ai simplement remplacé le for par une while avec un i++ à la fin.

[Invité]

jour

for(i=1;i<Length;i++)

il est fort possible que se soit le nom de ta variable qui cause le probleme

[sebhm]

je suis pas tres calé en AJAX.
Tu évalues la réponse recu sous format texte : parce que c'est du code javascript ?
et c'est quoi les objets objs et obj ?

Code :

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

il est fort possible que se soit le nom de ta variable qui cause le probleme

oui possible. Quoiqu'il en soit, il est vivement conseillé de ne pas utiliser de mot clé pour les noms des variables

[bilel.1st]

Bonjour,

Peut être que c'est la source du problème
voici la réponse du serveur au format JSON c'est d'où viennent les Objs et Obj :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
{"Objs": 
	{"Obj":[ 
				{
                                 "ID":  "1",
				 "NOM": "nom"
				}
			]
	}
}

[Bovino]

Salut,
supprime les crochets

Code :

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

1
2
3
4
5
6
7
8
{"Objs": 
    {"Obj": 
        {
            "ID":  "1",
            "NOM": "nom"
        }
    }
}

[bilel.1st]

D'accord,
mais est ce que ça pose un problème ou est ce que c'est faut de mettre les crochets ?

[Gatsu35]

Salut, "
supprime les crochets

Code :

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

1
2
3
4
5
6
7
8
{"Objs": 
    {"Obj": 
        {
            "ID":  "1",
           NOM": "nom"
        }
    }
}

Nan mais je crois plutot que c'est la declaration json qui est maladroitement écrite.

au final je pense qu'on devrait plutot avoir un tableau d'objets Objs étant ce tableau:

Code :

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

1
2
3
4
5
6
7
8
{"Objs": 
    [
        {
            "ID":  "1",
            "NOM": "nom"
        }
    ]
}

Au final on se retrouve avec ça.
C'est ça que tu souhaites ? ou alors j'ai pas compris

[Bovino]

Bien sûr que non !
Si tu rajoutes des crochets, tu ajoutes surtout un sous-objet, les propriétés ne peuvent donc plus être récupérées avec la syntaxe

Code :

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

var1 = Rsp.Objs.Obj[i].ID;

.
Pour t'en persuader, teste cela :

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
var test1 = {"Objs": 
    {"Obj": 
        {
            "ID":  "1",
            "NOM": "nom"
        }
    }
};
var test2 = {"Objs": [
    {"Obj": 
        {
            "ID":  "1",
            "NOM": "nom"
        }
    }]
};
alert(test1.Objs.Obj.ID);
alert(test2.Objs.Obj.ID);

[bilel.1st]

Bonjour,

En fait la réponses devrait être sur plusieurs enregistrements donc un tableau d'objets c'est pour cela Rsp.Objs.Obj[i] et c'est pourquoi j'utilise les crochets, en fin c'est d'où vient la boucle.

[Bovino]

C'est sûr que si tu as plusieurs fois le même nom de propriété dans le même objet, c'est un peu moins bien
Effectivement, comme le suggère Gatsu35, ta propriété Obj semble un peu inutile

Pour revenir à la question initiale, il n'y a pas vraiment de raison pour qu'il y ait des différences entre une boucle for et une boucle while et si l'on en croit tes explications, il apparait surtout que ta structure est plus complexe que ce que tu montres, donc l'erreur se trouve certainement ailleurs.

Enfin, pour en revenir au nommage, même s'il n'est pas interdit d'utiliser une propriété "ID" et que JavaScript est sensible à la casse, je te conseillerais par prudence, au mieux de changer le nom, sinon de faire l'appel en notation crochets plutôt que pointée :

Code :

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

var1 = Rsp.Objs.Obj[i]["ID"];

[bilel.1st]

Bonsoir,

Merci pour vos conseilles JSON.

C'est vrais que la structure que j'utilise est plus complexe que le code montré mais le principe c'est pratiquement ça.

Ce que j'ai voulus savoir de cette discutions c'est ce que Javascript a t il des limites c'est à dire lorsqu'il s'agit d'un grand lot d'informations ou du code complexe (trop de boucles, tests, imbrications,...) ?