Discussion :

[Tooomas]

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 :

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

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

[SpaceFrog]

Code :

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

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

ou

Code :

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

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

??

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

[SpaceFrog]

heu oui en effet fillot !

et puis pourquoi ne pas utiliser d directement ???

[Tooomas]

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.

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 !

[Tooomas]

Code :

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

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

[javatwister]

Code :

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

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

???

[Tooomas]

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

[magnus2005]

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

Code :

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

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 :

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