Discussion :

[Greatmangaka]

bonjour
Je suis un étudiant en BTS IG et je débute en java script et je fais un site de vente et je voudrai faire un caddie avec une texbox et deux bouton + et - comme dans beaucoup de site de vente.
J'ai réussi à faire le script mais le problème est que la value de la textbox change une fraction de seconde et reprend sa valeur initial.
Si quelqu'un peut m'aider à comprendre.
Voici le code:

<form id="form1" name="form1" method="post" action="">
<label>
<input type="submit" name="-" value="-" onclick="clickbutton(this,this.form);"/>
<input name="qte" type="text" value=1 size="2" maxlength="2" />
</label>
<label>
<input type="submit" name="+" value="+" onclick="clickbutton(this,this.form);" />
</label>
</form>
<script type="text/javascript">



function clickbutton(bouton,form)
{
var qte= parseInt(form.qte.value) ;
if (bouton.value=="-")
{
form.qte.value = qte -1 ;
}
else if (bouton.value=="+")
{
form.qte.value = qte+1 ;
}
}


</script>

[nabab]

Tout betement parce que tes boutons sont des "submit" au lieu d'etre des "button", donc meme si ton formulaire n'a pas d'action, le fait d'appuyer sur ton bouton recharge la page (car cela poste ton formulaire).
Change juste submit pour button...

[Auteur]

Autre remarque qui porte sur l'attribut name : donne à cet attribut un terme composé de lettres et de chiffres.

Le fait d'écrire name="+" ou name="-" va de te poser des problèmes.
Donc écris en toutes lettres : name="plus" et name="moins"

Autre chose qui me chagrine, l'appel de ta fonction :

Code htm :

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

1
2
 
onclick="clickbutton(this,this.form);"

dans ce cas this désigne l'objet qui génère l'événement onclick c'est à dire le bouton (this est donc un objet de type input) et non le document. Donc this.form me semble incohérent car le bouton (this) ne possède pas l'objet form.

[Greatmangaka]

Tout betement parce que tes boutons sont des "submit" au lieu d'etre des "button", donc meme si ton formulaire n'a pas d'action, le fait d'appuyer sur ton bouton recharge la page (car cela poste ton formulaire).
Change juste submit pour button...

Merci nabab sa marche nikel

Autre remarque qui porte sur l'attribut name : donne à cet attribut un terme composé de lettres et de chiffres.

Le fait d'écrire name="+" ou name="-" va de te poser des problèmes.
Donc écris en toutes lettres : name="plus" et name="moins"

Merci Auteur, j 'ai suivi ton conseil pour les attributs name.

Autre chose qui me chagrine, l'appel de ta fonction :

Code htm :

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

1
2
 
onclick="clickbutton(this,this.form);"

dans ce cas this désigne l'objet qui génère l'événement onclick c'est à dire le bouton (this est donc un objet de type input) et non le document. Donc this.form me semble incohérent car le bouton (this) ne possède pas l'objet form.

Par contre sa marche très bien this.form renvoi l'objet form dans laquelle elle se trouve.
Quelques améliorations et sa roule. merci @+