Bekijk mijn programma

Status
Niet open voor verdere reacties.
Onnodige citaten (quotes) verwijderd. Tevens een aantal berichten verwijderd. Eentje was dubbel en de ander kon je prima met de "Aanpassen" functie in je laatste bericht plaatsen, wat ik nu gedaan heb.
 
N.a.v. van een misbruikmelding, er is voor zover ik kan zien niets mis met de link.
 
Drie belangrijke punten:

1. De Lay-out mag een stuk beter;
2. Het programma is onoverzichtelijk;
3. Als ik wat heb uitgerekend met je programma gaat-ie gelijk doorverwijzen naar een andere site.. ?

Vooral punt 3 is belangrijk.
 
Dat weet ik niet? Jij hebt de code geschreven :P

En ik heb nooit gejavascript. ;)
 
Okay, hier een lijst met wat er beter kan / anders moet:

  1. script-tags net voor de </body>, niet in de <head>
  2. variabelen aanmaken met var, niet zomaar gebruiken
  3. het is netter om getElementX functies te gebruiken, of een this in de functie aanroep
  4. ook wel handig om niet 20 variabelen te maken en deze te vullen, maar alleen nuttige te gebruiken (en geef ze een nuttige naam! A t/m Z is nogal vaag)
  5. return eens false aan het eind van de functie; op het moment zie je de resultaten eventjes en na de alert is alles weg. Dit komt door je form; je kan beter je eigen button scripten om de functie aan te roepen
  6. if mag best met brackets ({ en }), is wel zo netjes
  7. je moet GEEN HOOFDLETTER TAGS GEBRUIKEN want dan lijkt het net of je schreeuwt...


hier is een snel voorbeeldje hoe ik het zou hebben gedaan (tenminste, de code. Text etc. is gewoon gekopieerd):
HTML: 
<!doctype html>

<html lang='nl'>


   <head>

      <meta charset='utf-8' />

      <title>Waterprogramma</title>

   </head>

   <body>

      Water prijs / euro: <input id='waterprijs' />
      Water hoeveelheid / liter: <input id='hoeveelheid' />
      Max bedrag / euro: <input id='maxbedrag' />
      Liter gratis / persoon: <input id='gratis' />

      <button onclick='calculate();'>ga!</button>


      <pre id='output'></pre>


      <script>

         function output(what)
         {
            document.getElementById('output').innerHTML += what + '\n';
         }


function calculate()
{
   var waterprijs  = document.getElementById('waterprijs').value,
       hoeveelheid = document.getElementById('hoeveelheid').value,
       maxbedrag   = document.getElementById('maxbedrag').value,
       gratis      = document.getElementById('gratis').value;

   var totaal = waterprijs * hoeveelheid;
   var totaalMetGratis = waterprijs * (hoeveelheid - gratis);


   document.getElementById('output').innerHTML = '';   // wis vorige input



   output('Prijs WATER| ' + waterprijs + ' &euro;');
   output('Hoeveelheid LITER| ' + hoeveelheid + ' &euro;');
   output('Streefbedrag EURO| ' + maxbedrag + ' &euro; ');
   output(gratis + ' Liter gratis per persoon');
   output('Totaal liter| ' + hoeveelheid + ' Zonder gratis liter| ' + gratis);
   output('Totaal EURO| ' + totaal + ' Zonder gratis liter| ' + gratis);
   output('Totaal EURO| ' + totaalMetGratis + ' Met gratis liter');

   output('<br />');

   if(maxbedrag > totaalMetGratis)
   {
      // ?
      output('U zit onder uw streefbedrag');
   }else{
      output('U zit boven uw streefbedrag');
   }
}

      </script>


   </body>

</html>

De code waar het om gaat is niet geident zodat 'ie wat meer opvalt.
 
het is makkelijker om het in php te maken.. ik zal een voorbeeldje geven
index.html
HTML: 
<html>
<head>
<title></title>
</head>
<body>
<form action="joulink.php" method="post">
<input type="text" name="prijs">€
<input type="text" name="aantalliter">L
<input type="submit" name="submit" value="submit">
</form>
joulink.php
HTML: 
<?php

$waterprijs=$_POST['waterprijs'];
$aantalliter=$_POST['aantalliter'];


echo "De prijs is :€ $waterprijs";
echo "U wilt : $aantalliter L water";

?>
 
Status
Niet open voor verdere reacties.
                    Steun ons                    

Nieuwste berichten

Terug
Bovenaan Onderaan