script ne s'exécute que lorsque je mets une alert

Version imprimable

05/09/2010, 13h53
ratsmok

script ne s'exécute que lorsque je mets une alert

Salut mes amis

Mon scripte ne s'exécute que lorsque je mets une alert sinon le scripte ne s'exécute pas je ne sais pas pourquoi aide moi svp

Merci d'avance :oops:
05/09/2010, 13h57
gwyohm

bonjour,

sans le code ca va être dur de t'aider...

voila le code

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 myInputs = document.getElementsByTagName('input'), span = Array();
		var a=0;
		var inputs=Array();
		alert(myInputs.length);	//alert qui me pose le problème
		for(var i=0; i<myInputs.length; i++){
				if(myInputs[i].id=='ville_inter[]'){
					inputs[a]=myInputs[i];		
					a++;
				}
			}						
			for(a = 0; a < inputs.length; a++) {
				if(inputs[a].type == "checkbox"&& 
inputs[a].className == "styled") {
					span[a] = document.createElement("span");
					span[a].className = inputs[a].type;
					span[a].id = "span_"+inputs[a].id;
 
					if(inputs[a].checked == true) {
						if(inputs[a].type == "checkbox") {
							position = "0 -" + (checkboxHeight*2) + "px";
							span[a].style.backgroundPosition = position;
						} else {
							position = "0 -" + (radioHeight*2) + "px";
							span[a].style.backgroundPosition = position;
						}
					}
					inputs[a].parentNode.insertBefore(span[a], inputs[a]);
					inputs[a].onchange = Custom.clear;
					if(!inputs[a].getAttribute("disabled")) {
						span[a].onmousedown = Custom.pushed;
						span[a].onmouseup = Custom.check;
					} else {
						span[a].className = span[a].className += " disabled";
					}
				}
			}
	}

05/09/2010, 15h35
{F-I}

Une des premières choses qui saute aux yeux quand on regarde un code source, c'est son indentation.

Le principe de l'indentation, c'est d'utiliser intelligemment les tabulations pour "décaler" certaines parties de votre code, afin de montrer plus clairement la structure et donc gain du temps dans le repérage.
05/09/2010, 19h06
Auteur
bonjour,

trois choses que j'ai remarquées :
*
Code:

1 2 span = Array();
span est un objet HTML, je te conseille vivement de renommer cette variable.

*
Code:

1 2 3 4 5 6 if(myInputs[i].id=='ville_inter[]') { inputs[a]=myInputs[i]; a++; }
tes objets input ont tous le même id ? La valeur doit être unique.

*
Code:

1 2 3 span[a].onmousedown = Custom.pushed; span[a].onmouseup = Custom.check;
la syntaxe est étrange :koi:

Je serais curieux de voir le code HTML de la page (pas de PHP).
05/09/2010, 21h47
dukej
Citation:
Envoyé par Auteur

bonjour,
trois choses que j'ai remarquées :
*

Code:

1 2 span = Array();

span est un objet HTML, je te conseille vivement de renommer cette variable.
que la variable s'appelle div, span, form, input, c'est largement possible, span représente dans son cas un tableau de spans.
Citation:
Envoyé par Auteur

*

Code:

1 2 3 span[a].onmousedown = Custom.pushed; span[a].onmouseup = Custom.check;

la syntaxe est étrange :koi:

Je serais curieux de voir le code HTML de la page (pas de PHP).
ce code est tout à fait correct, puisque notre monsieur associe une méthode à un objet.

utiliser une fonction anonyme en programmation est plutôt source de bordel derrière.
06/09/2010, 10h17
Auteur

Citation:

Envoyé par dukej

que la variable s'appelle div, span, form, input, c'est largement possible, span représente dans son cas un tableau de spans.

certes, mais c'est un coup à se mélanger les pinceaux. Et ayant eu des surprises avec certains navigateurs lorsque des objets JS portent le même nom qu'un élément HTML, je préfère prévenir ;)