Valeur dans un tableau qui ne s'affiche pas - Cannot read property '1' of undefined

Version imprimable

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:

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 :

Citation:

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

13/06/2013, 15h28
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:

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

est incorrect.
13/06/2013, 15h30
Barsy

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

mon tableau :

Code:

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

Désolé pour le double post

Code:

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:

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];
		}
	}
}

13/06/2013, 16h55
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.
13/06/2013, 17h27
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.
13/06/2013, 17h31
Barsy

Citation:

Envoyé par kirua99

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

Voilà d'où vient l'erreur.
13/06/2013, 17h36
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 .
13/06/2013, 19h10
sekaijin
var monTableau = new Array manque (), monTableau[nb manque un crochet ]=new Array()
Code:

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
13/06/2013, 19h58
SylvainPV

Code:

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

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

Code:

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

:ccool:

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

Code:

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:

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:

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

14/06/2013, 18h05
SylvainPV
Citation:
Envoyé par sekaijin

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

Code:

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:

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
14/06/2013, 20h57
NoSmoking
Bonjour,

Citation:

Envoyé par SylvainPV

...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:

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); } }

Une version un peu plus "moderne" :

pour un tableau (Array)

Code:

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

pour un objet (Object)

Code:

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

15/06/2013, 01h11
SylvainPV

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