Undefined qui est "defined"

Version imprimable

Bonjour à tous,

Je dois réaliser un projet en javascript or j'ai une erreur qui me bloque.
On me dit que mon tableau nommé global n'est pas défini quand je l'appelle dans une fonction.
Vous auriez une idée ??

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
var global = new Array();
var nbCase = window.prompt("Entrez le nombre de cases que vous voulez pour votre tableau");
var test=nbCase*nbCase;
 
	function TBL_virtuel() {
		for (var x=0; x<nbCase; x++) {
			global[x] = new Array();
			for (var y=0; y<nbCase; y++) {
 
					global[x][y]="vide"; 
			}	
		}
		return global;
	}
 
		function afficher()
	{
		var p=0;
		var coef=0.80;
		var nombreAs = nbCase*coef;
		var cellules = document.getElementsByTagName('td');
		for (var x=0; x<nbCase; x++) {
			for (var y=0; y<nbCase; y++) {
				calcul_place_A(global);
				calcul_place_B();
			}
		}
	}
 
	function calcul_place_A () {
		var p=0;
		var nombreAs = test/8.1;
		Math.round(nombreAs);
			do {
				var aleatoire = Math.random();
				var aleatoi = Math.random();
				var placeX = Math.round(aleatoire * nbCase);
				var placeY = Math.round(aleatoi * nbCase);
				if ((placeX>0) && (placeX<=nbCase) && (placeY>0) && (placeY<=nbCase)) {
				global[placeX][placeY]="As";
				alert (global[placeX][placeY]);
				}
 
				p++;
			} while (p<nombreAs);
		//document.write(global);
		return global;
	}
 
 
	function calcul_place_B () {
 
			if ((x>0)&&(y>0)&&(global[x-1][y-1]!="As"))
				{
					global[x-1][y-1]++;
				}
 
				if ((x>0)&&(y>0)&&(global[x][y-1]!="As"))
				{
					global[x][y-1]++;
				}
 
				if ((x<nbCase-1)&&(y>0)&&(global[x+1][y-1]!="As"))
				{
					global[x+1][y-1]++;
				}
 
				if ((x>0)&&(y>0)&&(global[x-1][y]!="As"))
				{
					global[x-1][y]++;
				}
 
				if ((x>0)&&(y>0)&&( global[x][y]!="As"))
				{
					global[x][y]++;
				}
 
				if ((x<nbCase-1)&&(y>0)&&(global[x+1][y]!="As"))
				{
					global[x+1][y]++;
				}
 
				if ((x>0)&&(y<nbCase-1)&&(global[x-1][y+1]!="As"))
				{
					global[x-1][y+1]++;vivant++;
				}
 
				if ((x>0)&&(y<nbCase-1)&&(global[x][y+1]!="As"))
				{
					global[x][y+1]++;
				}
 
				if ((x<nbCase-1)&&(y<nbCase-1)&&(global[x+1][y+1]!="As"))
				{
					global[x+1][y+1]++;
				}
 
 
	document.write(global);
	}

21/01/2009, 14h21
E.Bzz

Bonjour,
comme ton Array "global" est déclaré en variable .... globale ( :? ), il faut enlever les "return global;" à la fin de tes fonctions ...
De plus ces return ne sont visiblement pas "attendus" par la fonction appelante.

Code:

calcul_place_A(global);

Ne le passe pas en paramètre : c'est une variable globale (donc visible de toutes les fonctions).

De plus, bien que pas tout à fait compris le but de ton script, tu ne confondrais pas Array et <table> ?

Et que comptes-tu obtenir avec

Code:

global[x-1][y-1]++;

sachant que global[x-1][y-1] contient du caractère ?

A+
21/01/2009, 14h25
Auteur

bonjour,

- déjà tu peux supprimer les return global dans la mesure où ton tableau est une variable globale (en plus dans ton code tu ne récupères pas ce que retourne les fonctions).

- dans ta fonction afficher() :

Code:

calcul_place_A(global);

supprime le paramètre car d'après sa définition, calcul_place_A() n'a pas de paramètres

- pour faire du débogage, utilise plutôt alert() ou innerHTML sur un élément présent dans ta page :

Code:

document.getElementById("idElement").innerHTML = global

banni le document.write() !!

Ensuite :
1- A quoi sert ce code ?
2- comment sont appelées les différentes fonctions (événements qui les appellent, dans quel ordre sont-elles exécutées) ?
3- Perds l'habitude de déclarer tes variables dans des boucles, le programme perd de sa clarté et ça me donne des boutons ces :furieux: de déclarations variables qui se baladent à droite et à gauche. Déclare tes variables au début de tes fonctions, tu gagneras en lisibilité et sans doute en performance. ;)

[edit]

E.Bzz : nos messages se sont croisés ;)

[/edit]
21/01/2009, 14h29
rollergirl

Merci de ton aide mais j'ai toujours cette erreur dans firefox "global[placeX] is undefined"

Pour le tableau global[placeX] j'avais oublier que mon tableau avait vide par défaut...

en ce qui concerne le code, il doit me servir de base pour faire un démineur, sachant que pour le moment rien n'est affiché en html, l'appel au dom se fera en dernier.
21/01/2009, 14h31
E.Bzz

Citation:

Envoyé par Auteur

E.Bzz : nos messages se sont croisés ;)

J'ai failli crier au plagiat :mouarf:
21/01/2009, 14h34
E.Bzz

Citation:

Envoyé par rollergirl

Merci de ton aide mais j'ai toujours cette erreur dans firefox "global[placeX] is undefined"

Essaye

Code:

var placeX = Math.round(aleatoire * parseInt(nbCase, 10));

car nbCase est du string à l'origine (prompt() ).

En respectant bien sûr les conseils d'Auteur pour les déclarations ;)

A+

Je viens de tester en changeant avec parsint mais sa m'affiche toujour la même erreur... :/

La fonction parsint sert bien à lui dire que le pront saisie est un chiffre mais a quoi sert le 10 ?

Pour la déclaration des varaibles c'est mieux ainsi ?

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
	function calcul_place_A () {
 
	// Déclaration des variables utiliser pour cette fonction
		var aleatoire;
		var aleatoi;
		var placeX;
		var placeY;
		var p=0;
		var nombreAs = test/8.1;
		Math.round(nombreAs);
			do {
				aleatoire = Math.random();
				aleatoi = Math.random();
				placeX = Math.round(aleatoire * parseInt(nbCase, 10));
				placeY = Math.round(aleatoi * parseInt(nbCase, 10));
				if ((placeX>0) && (placeX<=nbCase) && (placeY>0) && (placeY<=nbCase)) {
				global[placeX][placeY]="A";
				//alert (global[placeX][placeY]);
				}
 
				p++;
			} while (p<nombreAs);
 
	}

21/01/2009, 14h47
E.Bzz

Citation:

Envoyé par rollergirl

Je viens de tester en changeant avec parsint mais sa m'affiche toujour la même erreur... :/

Tu as essayé de changé le nom de ton Array (sait-on jamais) ?

Citation:

Envoyé par rollergirl

La fonction parsint sert bien à lui dire que le pront saisie est un chiffre mais a quoi sert le 10 ?

C'est pour indiquer une conversion en base 10. Sans ça un bug de la fonction fait que certaines valeurs sont converties dans une autre base (hexa, octale ...) en fonction du 1° caractère de la valeur à convertir. Il y a un thread sur ce problème je crois ...

Citation:

Envoyé par rollergirl

Pour la déclaration des varaibles c'est mieux ainsi ?

:king:

A+
21/01/2009, 14h54
rollergirl

Citation:

Envoyé par E.Bzz

Tu as essayé de changé le nom de ton Array (sait-on jamais) ?

J'ai déja tester et malheureusement sa ne fonctionne pas... toujours le même message d'erreur

Tu ne sais pas d'ou sa peut venir ???
21/01/2009, 15h08
marcha
Salut,

à ce stade j'essaierai d'isoler l'erreur, commente presque la totalité de ton
code et essaie de laisser juste des choses dans ce genre:
Code:

1 2 3 4 5 6 7 8 var global = new Array(); global[0] = new Array(); global[0][0] = "A"; function calcul_place_A() { alert(global[0][0]); } calcul_place_A();
Si ça fonctionne, alors décommente petit à petit par section jusqu'à trouver
la section qui pose problème et reviens avec ça ici si tu n'as pas trouvé.

Bon je viens de tout tester et le problème semble vraiement venir de ma fonction calcul_place_Acar même avec ce code sa marche pas

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
27
28
29
30
31
32
33
34
var global = new Array();
var nbCase = window.prompt("Entrez le nombre de cases que vous voulez pour votre tableau");
var test=nbCase*nbCase;
 
	function init() {
 
		calcul_place_A();
	}
 
 
	function calcul_place_A () {
 
	// Déclaration des variables utiliser pour cette fonction
		var aleatoire;
		var aleatoi;
		var placeX;
		var placeY;
		var p=0;
		var nombreAs = test/8.1;
		Math.round(nombreAs);
			do {
				aleatoire = Math.random();
				aleatoi = Math.random();
				placeX = Math.round(aleatoire * parseInt(nbCase, 10));
				placeY = Math.round(aleatoi * parseInt(nbCase, 10));
				if ((placeX>0) && (placeX<=nbCase) && (placeY>0) && (placeY<=nbCase)) {
				global[placeX][placeY]="A";
				//alert (global[placeX][placeY]);
				}
 
				p++;
			} while (p<nombreAs);
 
	}

21/01/2009, 15h25
E.Bzz

Code:

Math.round(nombreAs);

ça serait pas plutôt

Code:

nombreAs=Math.round(nombreAs);

:?

Et pour pas être embêté partout

Code:

var nbCase = parseInt(window.prompt("Entrez le nombre de cases que vous voulez pour votre tableau"), 10);

Comme ça plus besoin de conversion par la suite ...
Parce que sinon

Code:

var test=nbCase*nbCase;

n'est pas bon non plus ...

A+
21/01/2009, 15h33
marcha
Salut,

Il te faut savoir que les tableaux multidimensionnels se gèrent
en javascript un peu différemment que dans d'autres langages.

Les tableaux à deux dimensions ne sont pas des grilles, se sont
des tableaux contenant des tableaux

En écrivant global[placeX][placeY] tu pensait écrire à la position X et Y
mais global[placeX] n'est pas initialisé et tu essaies d'accéder au tableau
de niveau 2 sans l'avoir initialisé.

tu peux essayer ceci:
Code:

1 2 3 if(typeof global[placeX] == undefined) global[placeX] = []; // ou new Array() global[placeX][placeY]="A";
21/01/2009, 15h41
rollergirl
E.Bzz >> J'ai bien changer le type de la variable attendu en prompt mais sans résultats..

Citation:

Envoyé par marcha

Salut,

En écrivant global[placeX][placeY] tu pensait écrire à la position X et Y
mais global[placeX] n'est pas initialisé et tu essaies d'accéder au tableau
de niveau 2 sans l'avoir initialisé.

En faites on m'avait vaguement parler de ce phènomène (qui est casse pied) mais en faite mon tableau est déclaré dans la fonction TBL_virtuel.

L'ajout de ton code n'a rien changer...
Code:

1 2 3 4 5 6 7 8 9 10 function TBL_virtuel() { for (var x=0; x<nbCase; x++) { global[x] = new Array(); for (var y=0; y<nbCase; y++) { global[x][y]="vide"; } } }
En tout cas merci de tenter de m'aider je suis en train de désespérer...

Citation:

Envoyé par rollergirl

L'ajout de ton code n'a rien changer...

Oui j'ai fait une petite erreur, undefined doit être entre guillemets.

voici un code qui marche chez moi, essaye de voir si c'est le cas chez toi aussi:

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
27
28
29
30
31
32
33
34
35
36
37
 
var global = new Array();
var nbCase = window.prompt("Entrez le nombre de cases que vous voulez pour votre tableau");
var test=nbCase*nbCase;
 
	function init() {
 
		calcul_place_A();
	}
 
 
	function calcul_place_A () {
 
	// Déclaration des variables utiliser pour cette fonction
		var aleatoire;
		var aleatoi;
		var placeX;
		var placeY;
		var p=0;
		var nombreAs = test/8.1;
		Math.round(nombreAs);
			do {
				aleatoire = Math.random();
				aleatoi = Math.random();
				placeX = Math.round(aleatoire * parseInt(nbCase, 10));
				placeY = Math.round(aleatoi * parseInt(nbCase, 10));
				if ((placeX>0) && (placeX<=nbCase) && (placeY>0) && (placeY<=nbCase)) {
				if(typeof global[placeX] == 'undefined') global[placeX] = [];
				global[placeX][placeY]="A";
				}
 
				p++;
			} while (p<nombreAs);
 
	}
 
	init();

21/01/2009, 16h08
rollergirl

Merci sa marche enfin...

Si jamais le problème se repose plus loin dans mon script je peux me débugger avec ce petit if ?
21/01/2009, 16h12
marcha
Salut,

Bon je crois avoir trouvé qqch, quand tu initialises le tableau
tu crée les indices 0, 1, 2, ... nbCase-1, soit pour nbCase = 4
ce serait les indices 0,1,2,3

Dans calcul_place_A() tu as ce test:
Code:

1 2 if ((placeX>0) && (placeX<=nbCase) && (placeY>0) && (placeY<=nbCase))
Et là tu accepte l'indice 4 avec un placeX<=4, il faudrait peut-être un "<" non ?
05/03/2009, 19h48
briegel

Eh, moi aussi j'ai un tableau à plusieurs dimensions, qui est créé et retourné à la fin d'une fonction, IE le retourne sans problèmes, et FF le considère "undefined".
08/03/2009, 10h42
Auteur

Citation:

Envoyé par briegel

Eh, moi aussi j'ai un tableau à plusieurs dimensions, qui est créé et retourné à la fin d'une fonction, IE le retourne sans problèmes, et FF le considère "undefined".

Tu as fait une erreur à la ligne 12 de ton script :aie:

Plus sérieusement : si tu ne postes pas ton code, on ne pourra pas t'aider...