erreur "sans erreur" dans getElementById

Version imprimable

Bonjour,

je voudrais bien savoir si l'écriture suivante peut poser un problème dans un des IE - PC. Car ailleurs tout s'affiche sans broncher, seulement un des IE s'amuse à balancer "erreur de script". Effectivement, debugeur de Firefox trouve que "document.getElementById(puc) has no properties"

C'est un problème de mentionner id par une variable, donc sans écriture en dure entouré des paranthèses ?
Code:

1 2 3 4 5 6 7 8 9 10 11 function change(d) { if ((d==1) || (d==2)){ num = d; var puc = 'sit_'+d; document.getElementById(puc).style.visibility = "visible"; } else { var pac = 'sit_'+num; document.getElementById(pac).style.visibility = "visible"; } }
Big problème est que je dévéloppe sur Mac et je n'arrive pas pour l'instant à vérifier toutes les versions de IE sur PC. IE PC que j'ai vérifié ne donnait aucun message d'erreur, c'est le client qui me signale le problème sans pouvoir me dire quelle est sa version de IE...

C'est ici, il faut cliquer sur "Start" et vous devrez pouvoir profiter des messages d'erreur, si vous avez "la bonne" version de IE :
http://valodrome.tek-studio.eu/jeux/map.html

La vraie question est : la variable dans GetElementById(var) est un problème ?
Si non, vous voyez la raison qui provoque l'affichage d'alert ? (mais ça marche quand même malgré l'affichage d'alert...)

Merci d'avance pour vos idées !!!
A+
Tooomas

23/03/2007, 16h45
SpaceFrog
Code:

var puc = 'sit_'+d.toString();

ou
Code:

1 2 3 var puc='sit_'+d d=d.toString()
??
23/03/2007, 17h03
Bisûnûrs

Ce que je vois surtout c'est que ta variable num est affectée dans le if mais tu t'en sers dans le else.
23/03/2007, 17h06
SpaceFrog

heu oui en effet fillot !

et puis pourquoi ne pas utiliser d directement ???
23/03/2007, 17h23
Tooomas

Citation:

Envoyé par SpaceFrog

heu oui en effet fillot !

et puis pourquoi ne pas utiliser d directement ???

Il est utilisé directement dans le if. Seulement si d ne correspond pas à la liste dans if (d correspond à quelques situations définies entre 40, il faut transférer sa valeur dans une autre variable "num" qui sera valable aussi longtemps que n'arriverait pas un "d" prédéfini. Le "num" est stocké en attendant dans l'adresse et assure que le visiteur voit toujours la même image tant qu'il ne choisi pas une autre situation qui en a une bien à elle.

Citation:

Envoyé par Bisûnûrs

Ce que je vois surtout c'est que ta variable num est affectée dans le if mais tu t'en sers dans le else.

Ca pose un problème ? Je ne sais pas, c'est une question. Une chose est sur, à la première utilisation de cette fonction le num est défini et il est envoyé dans l'adresse. En ce moment là il n'est pas utilisé dans else. Il est utilisé dans else au prochain tour, mais dans ce cas là il provient donc de l'adresse...

En tous cas, cet histoire de .toString() me parait très intéressante, je vais l'essayer ce soir (là, je dois me casser...). Merci beaucoup pour l'instant !
23/03/2007, 17h34
Tooomas
Citation:
Envoyé par SpaceFrog

Code:

1 2 3 var puc='sit_'+d d=d.toString()

??
en effet, le résultat, s'il était en dure, serait par exemple comme ça :
document.getElementById("sit_4").style.visibility = "visible";

Ton histoire de .tostring() s'apliquerai dans ce cas là ?

Merci !! T

Ehm... finalement, je ne me casse pas...

Code:

1
2
3
4
function change(d) {
var puc = d<3 ? 'sit_'+d : 'sit_'+num;
document.getElementById(puc).style.visibility = "visible";
}

???

23/03/2007, 18h18
Tooomas

enfet, il y a une vingtaine de "d" sur la liste, genre 1,2,4,8,10,11,17...

Dans la même idée qua javatwister en repensant le code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
function chang(d) {
if ((d==1) || (d==2) || (d==3) || (d==4) || (d==6) || (d==10) || (d==11) || (d==13) || (d==14) || (d==15) || (d==16) || (d==17) || (d==18) || (d==20) || (d==22) || (d==26) || (d==32) || (d==38)){ 
	num = d;
	document.write(num);
	var puc = 'sit_'+d;
	document.getElementById(puc).style.visibility = "visible";
	}
else {
	var pac = 'sit_'+num;
	document.getElementById(pac).style.visibility = "visible";
	}
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
var myarray ={1,2 /*toutes tes valeurs*/};
 
function isInArray(d) {
var ret = false;
//une boucle pour savoir si oui ou non  c est dans le tableau
for(var i=0 ;i<myarray.length && ret ===false;i+=1){
if(myarray[i] === d){ret = true;}
}
return ret;
}
 
function chang(d) {
var id = 'sit_'+d ;
if(isInArray(d)){ 
	document.write(d);
	id = 'sit_'+d;
}
document.getElementById(id).style.visibility = "visible";
}

26/03/2007, 12h23
Tooomas

Merci les gars pour vos suggestions, mais finalement, le problème est "ailleurs". Car le .toString n'arrange rien et malgré les apparences, la fonction marche comme il faut. Je rappelle que c'est juste une alerte d'une version de IE PC qui apparaît, et le script s'exécute ensuite, après avoir validé "OK". Tous les autres navigateurs ne posent aucun problème.

En effet, ça affiche le même problème aussi si je remplace la variable par le nom écrit en dure, comme ("sit_1") dans le if et ("sit_2) dans else.

J'ai regardé le problèmes similaires chez des autres, quand la console de FF affiche également "getElementById("nomid") has no properties" et il se trouve, que c'était toujours des problèmes de nom - quand c'était un peu plus compliqué avec des formulaires et il fallait mettre un autre nom.

Sauf ici, je n'ai pas l'impression qu'il pourrait y avoir de confusion et éléments appelés existent...

Vous avez une idée please, le pire c'est quand ça l'air d'être bon et ça ne l'est pas !

Merci infiniment d'avance !
Tomas