Discussion :

[kirua99]

Bonjour à tous ,

j'ai un soucis avec un script.
Des valeurs sont dans un tableau à deux dimensions.

je voulais simplement réaliser un affichage :

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
function affiche()
{ 
...
 
for(var i=1; i<= monTableau.length; i++){
			for(var j=0; j<= 29; j++){
	if ( i == 1 ) 
{
			res = 
		"<center><h3 style='color:#3d85c6';> Information </h3></center>"+
		"<p style='color:#3d85c6';> "+other+" <br>"+
		country+" / "+City +"<br>" +
		" valeur " + monTableau[i][2];
 
}
if ( i == 2 )
{
 res = res + 
		"<center><h3 style='color:#3d85c6';> Information 2 </h3></center>"+
		"<p style='color:#3d85c6';> "+other+" <br>"+
		country+" / "+City +"<br>" +
		" valeur " + monTableau[i][2];
}
document.getElementById("info").innerHTML = res;
 
}

mon erreur :

Uncaught TypeError: Cannot read property '2' of undefined

ca doit être un truc simple , mais je comprends pas pourquoi , je peux pas afficher la même choses plus tard dans ma boucle.

Merci d'avance.
kirua99

[Bovino]

On ne sait pas à quoi ressemble ton tableau, donc difficile de trouver la cause...

Peut-être as-tu oublié que les indices d'un tableau commencent à 0...
Donc

Code :

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

for(var i=1; i<= monTableau.length; i++)

est incorrect.

[Barsy]

Comment génères-tu la variable "monTableau". Il doit lui manquer une valeur sur la deuxième colonne à la position 'i'.

[kirua99]

mon tableau :

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
var monTableau = new Array;
for(var i=0; i<data.getNumberOfRows(); i++){
	//alert(data.getValue(i, 1) +" --- "+ other); 
 
		if (  data.getValue(i, 0) == Zone
		&& data.getValue(i, 1) == other 
		&& data.getValue(i, 2) == country 
		&& data.getValue(i, 3) == City 
		&&  data.getValue(i, 4) == LabName  )		
		{
 
nb = nb + 1;
 
		monTableau[nb=new Array()
 
			for(var b=0; b<=29; b++){
 
	monTableau[nb][b] = data.getValue(i, b);
 
	}
}

Pour mettre ces valeurs mon tableau , en faisant des contrôles d'alerte , tout semble correct.

Kirua99

[kirua99]

Désolé pour le double post

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for(var i=1; i<= monTableau.length; i++){
	for(var j=0; j<= 29; j++){
	if ( i == 1 ) 
	{		
		res = 
		"<center><h3 style='color:#3d85c6';> Information </h3></center>"+
		"<p style='color:#3d85c6';> "+other+" <br>"+
		country+" / "+City +"<br>" +
		" valeur " + monTableau[i][2] +
		" valeur2 " + monTableau[i][5];
	}

ce code si marche correctement aucun problème


Par contre celui la non

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
for(var i=1; i<= monTableau.length; i++){
	for(var j=0; j<= 29; j++){
		if ( i == 1 ) 
		{		
			res = 
			"<center><h3 style='color:#3d85c6';> Information </h3></center>"+
			"<p style='color:#3d85c6';> "+other+" <br>"+
			country+" / "+City +"<br>" +
			" valeur " + monTableau[i][2] +
			" valeur2 " + monTableau[i][5];
		}
		if ( i == 2 ) 
		{		
			res = res +
			"<br><center><h3 style='color:#3d85c6';> Information </h3></center>"+
			"<p style='color:#3d85c6';> "+other+" <br>"+
			country+" / "+City +"<br>" +
			" valeur " + monTableau[i][2] +
			" valeur2 " + monTableau[i][5];
		}
	}
}

[Barsy]

Alors c'est que la variable "monTableau" ne contient pas 3 valeurs.
Ou que la variable "monTableau[2]" ne contient pas 3 valeurs.

Fait un alert sur monTableau[2] et sur monTableau[2][2] pour voir.

[kirua99]

ba normalement j'affiche si dans mon tableau si le i est a 2 ou il est a 1 .

et oui mais y a des moments ou le tableau à pour i seulement 1.

[Barsy]

mais y a des moments ou le tableau à pour i seulement 1.

Voilà d'où vient l'erreur.

[kirua99]

pourtant il génere uniquement si je met un tableau a 1 dimension ou a 2 dimension

je vais devoir regarder plus prés sa alors .
si j ai des probléme , je vous en fera part alors .

[sekaijin]

var monTableau = new Array manque (), monTableau[nb manque un crochet ]=new Array()

Code :

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

1
2
for(var i=1; i<= monTableau.length; i++){
	for(var j=0; j<= 29; j++){

pourquoi une boucle sur j alors que j n'est jamais utilisé.

enfin je te renvois vers tableau-double-entree-plus-infos/

A+JYT

[SylvainPV]

Code :

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

for(var i=1; i<= monTableau.length;i++)

Les index commencent à zéro et se terminent à length -1 !

Code :

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

for(var i=0; i< monTableau.length;i++)

[sekaijin]

dans 90% des cas il est plus simple d'utiliser

Code :

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

1
2
3
4
5
for (data in monTableau) {
  for (value in data) {
    alert(value);
  }
}

d'ailleurs je ne vois pas dans ton cas l'intérêt de passer par un tableau de tableau

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
var monTableau = new Array();
for(var i=0; i<data.getNumberOfRows(); i++){
  if (  data.getValue(i, 0) == Zone
		&& data.getValue(i, 1) == other 
		&& data.getValue(i, 2) == country 
		&& data.getValue(i, 3) == City 
		&& data.getValue(i, 4) == LabName  )		
  {
  var row = { //définir un objet permet de nommer les attributs et d'être sur de sa structure
    other :    data.getValue(i, 1),
    country :  data.getValue(i, 2),
    City :     data.getValue(i, 3),
    LabName  : data.getValue(i, 4),
  }
  monTableau.push(row); //un simple push évite de trainer un indice n pour rien
}

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
res = "";//en initialisant res avant la boucle on n'a plus besoin du if
for (row in monTableau) {
  res = [
    res, 
    "<br><center><h3 style='color:#3d85c6';> Information </h3></center>",
    "<p style='color:#3d85c6';> ",
    other, // <= ICI je ne sais pas d'où sort la variable other
    " <br>",
    country,// <= ICI je ne sais pas d'où sort la variable country 
    " / ",
    City ,// <= ICI je ne sais pas d'où sort la variable City 
    "<br> valeur ",
    row.country, // <= on utilise les attributs de l'objet à la place d'indice dans un tableau
    " valeur2 ",
    row.City
  ].join() ;
}

le code est bien plus lisible
A+JYT

[SylvainPV]

dans 90% des cas il est plus simple d'utiliser

Code :

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

1
2
3
4
5
for (data in monTableau) {
  for (value in data) {
    alert(value);
  }
}

Au contraire il est déconseillé d'utiliser for...in pour boucler sur les éléments d'une Array, car la boucle passe par toutes les propriétés énumérables de l'objet Array et pas seulement les éléments du tableaux. Il peut arriver de nombreux cas où cela entraîne des bugs très difficiles à déceler, par exemple lorsque le prototype de Array a été étendu par certaines librairies ou lorsque la longueur de l'array a été modifiée dynamiquement.

Exemple:

Code :

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

1
2
3
4
 
var arr = new Array(5); 
for(var e in arr){ alert("in:" + e); }
for(var e=0; e<arr.length; e++){ alert("length:" + e); }

En revanche, for..in est tout à fait approprié pour parcourir les propriétés d'un objet.
Plus d'infos ici : http://stackoverflow.com/questions/5...uch-a-bad-idea

[NoSmoking]

Bonjour,

...car la boucle passe par toutes les propriétés énumérables de l'objet Array et pas seulement les éléments du tableaux...

dans ce cas un bon réflexe hasOwnProperty

Code :

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

1
2
3
4
5
6
7
Object.prototype.test = function(){};
var i, tab = new Array('1', '2');
for( var i in tab) {
  if (tab.hasOwnProperty(i)) {
    console.log(i);
  }
}

[Bovino]

Une version un peu plus "moderne" :

• pour un tableau (Array)

Code :

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

1
2
3
monTableau.forEach(function(valeur, indice, monTableau){
    console.log(valeur, indice, monTableau);
}, objetThis);

• pour un objet (Object)

Code :

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

1
2
3
4
var liste = Object.keys(monObjet);
liste.forEach(function(valeur){
    console.log(this[valeur]);
}, monObjet)

[SylvainPV]

Si on veut être ultra-moderne, on peut aussi parler de for...of