Discussion :

[lpi-RiaS]

bonjour à tous

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

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

[E.Bzz]

Bonjour,
il faut déclencher

Code :

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

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+

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

[E.Bzz]

je sais pas si c'est clair ^^

L'explication, oui, mais les raisons ... moins
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 :

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

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 :

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

formulaire.onchange = NewUrl;

Mais pourquoi appeler "formulaire" un objet <select>

A+

[lpi-RiaS]

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

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

[E.Bzz]

Code :

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

le_select.onchange = NewUrl;

(sans les parenthèses)

A+

[lpi-RiaS]

merci :d quel boulet je suis ^^