Discussion :

[gestgm]

Bonjour,

Je débute en js, et c'est mon premier script en fait (je ne parle pas des copiés collés !)

Voilà mon code, d'abord un div :

Code html :

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

1
2
3
<div class="carre" onClick="cours(this);">
	MUC2
</div>

Ensuite le JS :

Code :

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

1
2
3
4
5
6
7
8
9
10
function cours(tcours) {
	var txt;
    tcours.onclick=function(){
	txt=this.innerHTML;
	var datecours = document.getElementById('datecours');
	var datecours = datecours.options[datecours.selectedIndex].value;
	url = 'agenda.php?cours=' + txt + '&date=' + datecours;
	location.href = url;
    }
};

C'est pour un projet que je réalise dans le cadre d'un licence pro, mais je débute vraiment en js, je ne comprend pas pourquoi je dois cliquer deux fois pour que le click fonctionne.

Un exemple est visible en ligne, pour que le prof puisse voir le résultat.

http://reseau-web.fr/spxml/index.php

Merci pour votre aide

[Spartacusply]

Lors de ton premier click, tu redéfinis la fonction onclick sur ton bouton, mais tu ne l'appliques pas pour autant.

Il faut que tu supprimes cette ligne :

Code :

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

tcours.onclick=function(){

[gestgm]

Bon, j'essaye de l'enlever, mais le problème, c'est que c'est de cette façon que je récupère la valeur textuel de mon div et que je le passe pour construire mon url, franchement, je ne suis pas assez calé pour le moment pour faire autrement.

Un exemple serai le bienvenu !

En tout cas, merci pour la réponse

[Spartacusply]

Pourquoi utiliser this (qui ne référence pas le "this" que tu souhaites alors que tu le passes en paramètre (ce qui est la bonne méthode de faire) ?

Ensuite, indique une "value" sur tes options, c'est indispansable si tu ne veux pas qu'au moindre changement d'affichage, ton code ne soit plus valable.

Code html :

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

1
2
3
4
5
6
7
<select id="datecours" class="custom-dropdown__select custom-dropdown__select--white">
    <option value="2012-02-10">10 fév. 2012</option>
    <option value="2012-02-11">11 fév. 2012</option>
    <option value="2012-02-12">12 fév. 2012</option>
    <option value="2012-02-13">13 fév. 2012</option>
    <option value="2012-02-14">14 fév. 2012</option>
</select>

Voici qui pourrait faire l'affaire :

Code :

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

1
2
3
4
5
6
function cours(tcours) {
        var txt = tcours.innerHTML.trim();
        var datecours = document.getElementById('datecours');
        var date = datecours.options[datecours.selectedIndex].value;
        location.href = 'agenda.php?cours=' + txt + '&date=' + date;
}

[gestgm]

Bon effectivement, cela fonctionne comme ça et j'ai appris quelque chose

Merci beaucoup !

Pour le select, en fait, dans l'avancé du devoir il ne sert pas vraiment, c'est la raison pour laquelle je n'ai pas mis de value.