onChange en js non-intrusif

Version imprimable

bonjour à tous

je débute en javascript et j'ai un problème avec une fonction lié a une balise select :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 addLoadEvent(customerchangeAction()); function customerchangeAction() { if (document.getElementById("idsite")) { formulaire = document.getElementById('idsite'); formulaire.onChange = NewUrl(formulaire); } } function NewUrl(formulaire) { var url; url = '/site/index?type=customer&idsite='+formulaire.options[formulaire.selectedIndex].value; window.location.replace( url ); }
"idsite" est une balise select qui, lors qu'elle est changé, redirige vers une autre URL (fonction NewUrl), le problème étant que ma page boucle en se redirigeant en permanence vers le premier "<option>" de mon select !

Je précise que je n'avais pas ce problème en javascript intrusif, je mettais seulement un code du style : ' onChange="NewUrl(idsite)" ' dans ma balise select, et ça marchait très bien.

voila, je suppose qu'il s'agit d'une erreur de débutant mais je ne trouve pas de solutions nul part... :(

merci par avance

27/02/2008, 11h18
E.Bzz

Bonjour,
il faut déclencher

Code:

addLoadEvent(customerchangeAction());

sur un évènement (comme tu le faisais avant) et non au chargement de la page, comme actuellement ...
Qu'est-ce que tu entends par intrusif ?

A+
27/02/2008, 11h26
lpi-RiaS

merci pour ta réponse !

Euh j'apelle ça non-intrusif pasqu'il n'y a dans mon html que cette ligne : '<script type="text/javascript" src="/js/scripts.js"></script>'
, tout le reste se trouve dans le fichier script.js

C'est mon maitre de stage qui l'exige :p moi auparavant j'appelais ma méthode NewUrl dans mon select comme je l'ai dit plus haut et ça marchais bien...

Donc, sans rien ajouter dans mon html, je doit eviter cette maudite boucle au chargement de la page sur la première valeur du select... je sais pas si c'est clair ^^

Citation:

Envoyé par lpi-RiaS

je sais pas si c'est clair ^^

L'explication, oui, mais les raisons ... moins :mrgreen:
Je reviens sur ce que j'ai écrit : rien dans ton code n'indique un déclenchement du onchange au chargement de la page :?
As-tu donner tout le code concerné ?

Sinon :
Code:

1 2 3 4 5 6 function NewUrl() { var url; var le_select = document.getElementById('idsite'); url = '/site/index?type=customer&idsite='+le_select.options[le_select.selectedIndex].value; window.location.replace( url ); }
avec

Code:

formulaire.onchange = NewUrl;

Mais pourquoi appeler "formulaire" un objet <select> :?

A+

effectivement, ce nom de variable c'est douté, je l'ai renommé "select"
le code complet de script.js :

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
addLoadEvent(customerchangeAction());
 
function addLoadEvent(func)
{
	var oldonload = window.onload;
	if (typeof window.onload != 'function')
	{
    	window.onload = func;
    }
    else
    {
    	window.onload = function()
    	{
	    	if (oldonload)
	    	{
	        	oldonload();
	      	}
	      	func();
    	}
	}
}
 
function customerchangeAction()
{
	if (document.getElementById)
	{
		if (document.getElementById("idsite"))
		{
			le_select = document.getElementById('idsite');
			le_select.onchange = NewUrl();
		}
	}
}
 
function NewUrl()
{
	var url;
    var le_select = document.getElementById('idsite');
	url = '/site/index?type=customer&idsite='+le_select.options[le_select.selectedIndex].value;
	window.location.replace( url );
}

et donc ce code boucle toujours sans que je touche à la liste déroulante sur la page en question :(

27/02/2008, 11h54
E.Bzz

Code:

le_select.onchange = NewUrl;

(sans les parenthèses) ;)

A+
27/02/2008, 11h59
lpi-RiaS

merci :d quel boulet je suis ^^