Discussion :

[spawns]

Re à tous,
je sais ca a déja été demandé des milllllllions de fois, mais mon code ne fonctionne pas.
comment recuperer la valeurs selectionnée par l'internaute dans mon select

function locacalculpro()
{
var pop=(document.getElementById('locduree').options[document.getElementById('locduree').selectedIndex].value)*12;
}

mon select

<select id="locduree" onchange="locacalculpro();" onchange="locaaffic('locnewmen');">
<option id="1">1</option>
<option id="2">2</option>
<option id="3">3</option>
<select>

cela ne fonctionne pas. j'ai impression que cela ne recupere rien...

[Bovino]

Code :

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

1
2
3
4
5
function locacalculpro(elt)
{
var pop=parseInt(elt.options[elt.selectedIndex].value)*12;
alert(pop);
}

et

Code :

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

1
2
3
4
5
<select id="locduree" onchange="locacalculpro(this);locaaffic('locnewmen');">
<option id="1">1</option>
<option id="2">2</option>
<option id="3">3</option>
<select>

[spawns]

re,
je ne peux pas utiliser

function locacalculpro(elt)

car cette fontion integre plein d'autre parametes qui ne fonctionne plus si je met (elt)

fonction entiere

function locacalculpro()
{
document.getElementById('formu').locmont.value=Number(document.getElementById('formu').loccumu.value);
document.getElementById('formu').loctotal.value=Number(document.getElementById('formu').locmons.value)+Number(document.getElementById('formu').locmde.value)+Number(document.getElementById('formu').locautre.value);
var pop=Number(document.getElementById('formu').locduree.value)*12;
var pap=(1+(0.08/12));
var pup=Math.round((Number(document.getElementById('formu').locmont.value)*(0.08/12))/(1 - Math.pow(pap,-pop))*100)/100;
document.getElementById('formu').locnewmensu.value=pup;
var pip=Math.round((Number(document.getElementById('formu').locnewmensu.value)*100)/Number(document.getElementById('formu').loctotal.value));
document.getElementById('formu').locendet.value=pip;
}

Comme je suis novice, en javascript j'ai un peu de mal à tous integrer.

je me sers ensuite de cette fonction pour remplir une partie de mon formulaire automatiquement comme :

...<br />Cumul du capital restant en consommation :<br />
<input type="text" name="loccumu" size="10" onkeyup="locacalculpro();" />€
<br />Mensualités en conso :<br />
<input type="text" name="locmensu" size="10" />€
<br />Loyer :<br />
<input type="text" name="locloyer" size="10" />€
<br />Montant du prêt :<br />
<input type="text" name="locmont" size="10" />€
<br />Durée (en année, de 1 à 12 ans max):<br />
<select id="locduree" onchange="locacalculpro();locaaffic('locnewmen');">
<option id="1">1</option>
<option id="2">2</option>
<option id="3">3</option>
<select>...

[Bovino]

Je ne vois pas pourquoi ta fonction ne fonctionnerais plus en ajoutant un paramètre...
En revanche :

Code :

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

var pop=Number(document.getElementById('formu').locduree.value)*12;

Code :

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

var pop=parseInt(document.getElementById('formu').locduree.value)*12;

[bigboomshakala]

oh my god

tu ne suis absolument pas les conseils qu'on te donne...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
function locacalculpro()
{
document.getElementById('formu').locmont.value=Number(document.getElementById('formu').loccumu.value);
document.getElementById('formu').loctotal.value=Number(document.getElementById('formu').locmons.value)+Number(document.getElementById('formu').locmde.value)+Number(document.getElementById('formu').locautre.value);
var pop=Number(document.getElementById('formu').locduree.value)*12;
var pap=(1+(0.08/12));
var pup=Math.round((Number(document.getElementById('formu').locmont.value)*(0.08/12))/(1 - Math.pow(pap,-pop))*100)/100;
document.getElementById('formu').locnewmensu.value=pup;
var pip=Math.round((Number(document.getElementById('formu').locnewmensu.value)*100)/Number(document.getElementById('formu').loctotal.value));
document.getElementById('formu').locendet.value=pip;
}

ce code est très lourd, très lent (pour peu que la page soit grosse) et très difficilement maintenable parce qu'on n'y voit rien e qu'on fait facilement des erreurs (parenthèses et autres)

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
function locacalculpro()
{
  var formu = document.getElementById("formu");
  var locmons = parseInt(formu.locmons.value);
  var loccumu = parseInt(formu.loccumu.value);
  var locmde = parseInt(formu.locmde.value);
  var locautre = parseInt(formu.locautre.value);
  var locduree = parseInt(formu.locduree.value);
  var locnewmensu = parseInt(formu.locnewmensu.value);
 
  formu.locmont.value = loccumu;
  formu.loctotal.value = locmons+locmde+locautre;
  var pop = locduree*12;
  var pap = (1+(0.08/12));
  var pup = Math.round((parseInt(formu.locmont.value)*(0.08/12))/(1 - Math.pow(pap,-pop))*100)/100;
  formu.locnewmensu.value=pup;
  var pip = Math.round((locnewmensu*100)/parseInt(formu.loctotal.value));
  formu.locendet.value=pip;
}

on y voit déjà plus clair ou pas ?
ensuite pour récupérer la valeur d'un select on demande tout simplement la value (comme pour un input text)

Code :

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

1
2
var toto = monSelect.value;
// équivaut à monSelect.options[monSelect.selectedIndex].value

[spawns]

Bonjour et merci pour votre aide,
desolé pour le code
mais effectivement ta facon c'est beaucoup + clair et probablement plus legere. Mais je ne connais pas encore assez javascript pour en juger.

Donc j'ai mis en place ton code à savoir :

function locacalculpro()
{
var formu = document.getElementById("formu");
var locmons = parseInt(formu.locmons.value);
var loccumu = parseInt(formu.loccumu.value);
var locmde = parseInt(formu.locmde.value);
var locautre = parseInt(formu.locautre.value);
var locduree = parseInt(formu.locduree.value);
var locnewmensu = parseInt(formu.locnewmensu.value);

formu.locmont.value = loccumu;
formu.loctotal.value = locmons+locmde+locautre;
var pop = locduree*12;
var pap = (1+(0.08/12));
var pup = Math.round((parseInt(formu.locmont.value)*(0.08/12))/(1 - Math.pow(pap,-pop))*100)/100;
formu.locnewmensu.value=pup;
var pip = Math.round((locnewmensu*100)/parseInt(formu.loctotal.value));
formu.locendet.value=pip;
}

et un morceau de formulaire :

...<div id="loc" style="display: none">
<br />Cumul du capital restant en consommation :<br />
<input type="text" name="loccumu" size="10" onkeyup="locacalculpro();" />€
<br />Mensualités en conso :<br />
<input type="text" name="locmensu" size="10" />€
<br />Loyer :<br />
<input type="text" name="locloyer" size="10" />€
<br />Montant du prêt :<br />
<input type="text" name="locmont" size="10" />€
<br />Durée (en année, de 1 à 12 ans max):<br />
<select id="locduree" onchange="locacalculpro();locaaffic('locnewmen');">
<option id="1">1</option>
<option id="2">2</option>
<option id="3">3</option>
<select>
<div id="locnewmen" style="display: none">
<br />Nouvelle mensualité :<br />
<input type="text" name="locnewmensu" size="10" />€...

Mais cela ne fonctionne pas, lors de la saisie des données par l'internaute cela doit calculer automatiquement le champ "<input type="text" name="locnewmensu" size="10" />" mais la non.

Cela affiche NaN dans le champ.

[bigboomshakala]

dans un 1er temps vérifie que ce que tu manipules dans localcalculpro ce sont bien des entiers (parseInt) ou des nombres à virgule (parseFloat) en les affichant avec alert.

si l'un des champs est vide, le parseInt va te renvoyer NaN ou une exception (je ne sais plus)

donc il faudrait penser à empêcher l'utilisateur d'entrer autre chose que des nombres, ou alors à filtrer dans localcalculpro, genre :

Code :

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

1
2
3
4
5
6
var nombre = 0;
try{
 nombre = parseInt(champ.value);
}
catch(ex){
}

ou

Code :

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

1
2
3
4
var nombre = 0;
if (!isNaN(champ.value))
  nombre = parseInt(champ.value);
}

[spawns]

j'ai testé, et je pense que le code ne trouve pas la valeur du select justement.

le truc que j'essaie de faire depuis 3jours en gros
ahhhhhhhhhhhhhhhhhhhhhhhhhhh.............

[bigboomshakala]

je pense

genre ?

[spawns]

la var pop n'est pas bonne.
elle ne recupere pas la valeur du menu deroulant.

c'est ca que je n'arrive pas faire

[bigboomshakala]

c'est normal

Code :

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

<option id="1">1</option>

tu n'as pas de value dans tes options

[spawns]

ok c'est bon j'ai trouvé

c'est une erreur dans les variable

var locduree= document.getElementById('locduree');
var bbb= locduree.options[locduree.selectedIndex].text;

Merci pour votre aide vous me sauvez la vie
A bientôt