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.

[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