[HTML/JS]qu'est ce qui ne va pas avec mon programme?

qu'est ce qui ne va pas avec mon programme? [HTML/JS] - HTML/CSS - Programmation

Marsh Posté le 22-09-2003 à 22:45:40    

Bonjour,
Voilà je débute dans la programmation de site web.Je maitrise à peu près le HTML, et un peu plus vaguement le Javascript.  
Je souhait créer un site de démo pour mon patron (je travaille chez un administrateur de bien) pour lui montrer ce que pourrais donner un site internet avec un programme qui calcule le tarif des "mandats.". Plutôt que de calculer ça à la main, par téléphone etc. ce qui serait fastidieux, je voudrais diriger un maximum de clients vers le site et mon programme. (vu que c moi qui receptionne les appels).
J'ai tenté de faire un programme JS qui calcule un pourcentage du loyer selon le mandat choisi.
ça ne marche pas, j'ai une page blanche. je ne comprends pas pk. ça doit être qqc de ballot que je ne maitrise pas, mais je ne trouve pas. (je m'excuse c assez illisible et bordélique)
 
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
 
<html>
<head>
 <title>formulaireSOGECOPA</title>
 <script language="Javascript">
 <!-- function calcul(){
 var loyer=0
 var mandat=""
 var affichage="Votre loyer H.C. etant de "
 affichage=affichage+document.forms[0].loyer.value
 affichage=affichage+" euros, le coût de votre mandat"
 if(document.forms[0].mandat.options[0].selected)
    {affichage=affichage+" Primmo"}
 if(document.forms[0].mandat.options[1].selected)
    {affichage=affichage+" Sérénimmo"}
 if(document.forms[0].mandat.options[2].selected)
    {affichage=affichage+" Optimmo"}
 if(document.forms[0].mandat.options[3].selected)
    {affichage=affichage+" Optimmo plus"}
 
 affichage=affichage+if(document.forms[0].mandat[0].checked){var loyer*0.12)/2+"pour le locataire, et "+(var loyer*0.12)/2}
 if(document.forms[0].mandat[1].checked){loyer*0.08}
 if(document.forms[0].mandat[2].checked){loyer*0.09}
 if(document.forms[0].mandat[3].checked){loyer*0.1}
 affichage=affichage+" euros pour le propriétaire."
 }
 //-->
</script>    
</head>
 
 
<body>
 
<center><h2>Calculez le côut de votre mandat :</h2></center>
<h3>Indiquez le montant de votre loyer TTC  hors charges et cochez l'offre qui vous intéresse:</h3>
<form onClick="calcul()">
<table border="0" cellspacing="1" cellpadding="5">
 
<tr colspan="2">
       <td >Loyer hors charges en euros:</td>
       <td><input name="loyer" type="text" value="" size="10" maxlength="8"></td>
</tr>
<td> Selectionnez votre Mandat :</td>
 
<tr>
       
   <td><input type="radio" name="mandat" value="oui">Primmo</td>
       <td><input type="radio" name="mandat" value="oui">Sérénimmo</td>
       <td><input type="radio" name="mandat" value="oui">Optimmo</td>
       <td><input type="radio" name="mandat" value="oui">OptimmoPlus</td>
</tr>
<!--Ne sert à rien//-->
<tr>
       <td>Coût :</td>
       <td><input name="COUT" type="hidden" value=""></td>
       <td> euros par mois</td>
       <td></td>
 
 
</tr>
  <td><input type="SUBMIT" name="Envoyer" value="Envoyer" onClick="calcul()">
    </td>
       <td></td>
       <td><input type=reset value="Effacer"></td>
       <td></td>
</tr>
</table>
</form>
</body>
 
 
 
 
 
</html>

Reply

Marsh Posté le 22-09-2003 à 22:45:40   

Reply

Marsh Posté le 23-09-2003 à 08:56:31    

fais déjà une cellule en readonly pour afficher la réponse et prend un exemple javascript simple et adapte le

Reply

Marsh Posté le 23-09-2003 à 10:12:56    

bon c'est un peu n'importe quoi ton script.
D'abord vaut miex terminer toutes tes lignes par un ; en javascript c'est un peu plus propre.
Ensuite tu as deux trois endroits ou tu fais n'importe quoi:

Code :
  1. affichage=affichage+if(document.forms[0].mandat[0].checked){var loyer*0.12)/2+"pour le locataire, et "+(var loyer*0.12)/2}
  2. if(document.forms[0].mandat[1].checked){loyer*0.08}
  3. if(document.forms[0].mandat[2].checked){loyer*0.09}
  4. if(document.forms[0].mandat[3].checked){loyer*0.1}
  5. affichage=affichage+" euros pour le propriétaire."


A aucun endroit tu stockes le prix pour le proprio, comment tu veux t'en sortir? Bref moi je ferais plutot quelque chose comme ça:

Code :
  1. var mot_locataire;
  2.   var mt_proprio;
  4. if ( document.myForm.mandat[0].checked )
  5. {
  6.     mt_locataire = loyer*0.12/2;
  7.     mt_proprio   = loyer*0.12/2;
  8. }
  9. if(document.myForm.mandat[1].checked)
  10. {
  11.     mt_locataire = loyer*0.12/2;
  12.     mt_proprio   = loyer*0.12/2;
  13. }
  14. if(document.myForm.mandat[2].checked)
  15. {
  16.     mt_locataire = loyer*0.12/2;
  17.     mt_proprio   = loyer*0.12/2;
  18. }
  19. if(document.myForm.mandat[3].checked)
  20. {
  21.     mt_locataire = loyer*0.12/2;
  22.     mt_proprio   = loyer*0.12/2;
  23. }
  24. affichage+= mt_locataire+"pour le locataire, et "+mt_proprio+" euros pour le propriétaire.";


 
[edit un peu plus tard]
Ah oui et temps qu'a faire j'utiliserais des ids et la fonction getElementById plutot que de t'ennuyer avec tes tableaux de formulaire et tes tableaux d'options ( qui à vue de nez ne doivent pas marcher en plus)


Message édité par anapajari le 23-09-2003 à 10:14:34
Reply

Marsh Posté le 23-09-2003 à 18:21:02    

jagStang, oui oui j'y ai pensé, mais j'hésitais entre une zone de texte readonly ou une fenêtre d'alerte et je me suis mélangée. qu'est ce qui est le mieux?
 
Anapajari, merci de ta réponse.Kler que c n'importe quoi. Je sais que mon programme est foireux :D, je débute.
 
Pour le ; j'ai trop honte!![:angelfire]
 
En fait c juste pour l'offre primmo que c partagé entre le proprio et le locataire, sinon c tout pour le propriétaire. En fait, il paye un % du loyer hors charge.
 

Citation :

j'utiliserais des ids et la fonction getElementById


J'ai pas compris... :??:

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed