Java Preisrechner

puur · 18.12.2008

Hello Leute hab ein Problem und zwar mit dem Preisrechner und den Checkboxen. Im moment rechnet er mir beide checkboxen auf den Preis drauf aber ich möchte natürlich nur den Aufpreis berechnet haben die auch gewählt wurde.

HTML:

HTML:

<form name="form" id="form">
    <div style="float: left;">
        <div style="float: left;" class="text">Slots</div>
        <div style="float: left; padding-left: 30px;">
            <select style="width: 43px;" name="wert1" id="wert1" onChange="rechnen()">
                <option value="1">1</option>
                <option value="2">2</option>
                <option value="3">3</option>
                <option value="4">4</option>
            </select>
        </div>
        <!--<input name="wert2" id="wert2" type="text" >-->
        
        <br style="clear:left;" />
        
        <div style="float: left;" class="text">Monate</div>
        <div style="float: left; padding-left: 15px;">
            <select name="wert2" id="wert2" onChange="rechnen()">
                <option value="3">3</option>
                <option value="6">6</option>
                <option value="9">9</option>
                <option value="12">12</option>
            </select>
        </div>
    </div>
    <div style="float: left;">
        <label>
            <div style="float: left; padding-right: 5px; padding-left: 10px;" class="text">Teamspeak2</div>
            <div style="float: left; padding-top: 3px;">
                <input type="radio" name="TSorVent" id="wert3" value="10" onClick="rechnen()">
            </div>
        </label>
        <br style="clear: left;" />
        <label>
            <div style="float: left; padding-top: 5px; padding-right: 36px; padding-left: 10px;" class="text">Ventrilo</div>
            <div style="float: left; padding-top: 7px;">
                <input type="radio" name="TSorVent" id="wert4" value="20" onClick="rechnen()">
            </div>
        </label>
    </div>
    
    <!--<input name="berechnen" type="button" id="berechnen" value="Berechnen" onClick="rechnen()">-->
    <br style="clear: left;" /> <br />
    <input name="ergebnis" type="text" id="ergebnis">
</form>

JAVA:

Code:

var mwSt = 1.19;

function rechnen()
{
var Slots = document.form.wert1.value;
var Monate = document.form.wert2.value;
var Teamspeak2 = document.form.wert3.value;
var Ventrilo = document.form.wert4.value;

/*var wert2 = document.form.wert2.value;*/


var erg = Slots * Monate * Teamspeak2 * Ventrilo;

erg = erg;
erg = erg.toFixed(2);
document.form.ergebnis.value=erg;

}

sokie · 18.12.2008

AW: Java Preisrechner

Das ist kein JAVA

Duddle · 18.12.2008

AW: Java Preisrechner

Drei Dinge:

least. useful. line. ever.
Code:
```
erg = erg;
```
das sind keine Checkboxen, sondern Radiobuttons
du prüfst nirgendwo, ob der Radiobutton jeweils gesetzt ist, das kann also garnicht funktionieren. Also mach eine Fallabfrage und berechne darin deinen Endpreis.

Duddle

puur · 18.12.2008

AW: Java Preisrechner

ok leute

so funktionierts dann sind noch dinge zu beachten sicherheits technisch oder kann man das so lassen ?

Code:

var erg;
var mwSt = 1.19;
function rechnen()
{
    erg = 0;
        if (document.form.wert3.checked) { erg = erg + 10 * document.form.wert1.value * document.form.wert2.value * mwSt; }
        if (document.form.wert4.checked) { erg = erg + 20 * document.form.wert1.value * document.form.wert2.value * mwSt; }


    erg = erg.toFixed(2);
    document.form.ergebnis.value=erg;
}

saila · 22.12.2008

AW: Java Preisrechner

Hi,

bin zwar nicht der JavaScript-Profi, aber ich denke folgende Version dürfte korrekt sein:

JS innerhalb head:

Code:

<script type="text/javascript">
    var mwSt = 1.19;
    var vMwst = 0;
    var T2 = 0;
    var erg = 0;
    var ergo = 0;

    function rechnen() {

        var Slots         = document.form.wert1.value;
        var Monate         = document.form.wert2.value;
        
        if(document.form.wert3.checked) { T2 = document.form.wert3.value; }
        if(document.form.wert4.checked) { T2 = document.form.wert4.value; }
        
        ergo = Slots * Monate * T2;
        erg = ergo * mwSt;
        document.form.ergebnis.value=erg.toFixed(2);
        vMwst = ergo * 0.19;
        document.form.viewMwst.value=vMwst.toFixed(2);
    }
</script>

HTML innerhalb body:

Code:

<form name="form" id="form">
    <div style="float: left;">
        <div style="float: left;" class="text">Slots</div>
        <div style="float: left; padding-left: 30px;">
            <select style="width: 43px;" name="wert1" id="wert1" onChange="rechnen()">
                <option value="1">1</option>
                <option value="2">2</option>
                <option value="3">3</option>
                <option value="4">4</option>
            </select>
        </div>
        <!--<input name="wert2" id="wert2" type="text" >-->
        
        <br style="clear:left;" />
        
        <div style="float: left;" class="text">Monate</div>
        <div style="float: left; padding-left: 15px;">
            <select name="wert2" id="wert2" onChange="rechnen()">
                <option value="3">3</option>
                <option value="6">6</option>
                <option value="9">9</option>
                <option value="12">12</option>
            </select>
        </div>
    </div>
    <div style="float: left;">
        <label>
            <div style="float: left; padding-right: 5px; padding-left: 10px;" class="text">Teamspeak2</div>
            <div style="float: left; padding-top: 3px;">
                <input type="radio" name="TSorVent" id="wert3" value="10" onClick="rechnen()">
            </div>
        </label>
        <br style="clear: left;" />
        <label>
            <div style="float: left; padding-top: 5px; padding-right: 36px; padding-left: 10px;" class="text">Ventrilo</div>
            <div style="float: left; padding-top: 7px;">
                <input type="radio" name="TSorVent" id="wert4" value="20" onClick="rechnen()">
            </div>
        </label>
    </div>
    <br style="clear: left;" /> <br />
    EUR <input name="ergebnis" type="text" readonly="readonly"  id="ergebnis" />
    Enthaltene Mwst EUR <input style="border: none;" readonly="readonly" name="viewMwst" type="text" id="viewMwst" />
</form>

Die Berechnungen laufen bei mir korrekt.

Duddle · 22.12.2008

AW: Java Preisrechner

Ein Radiobutton ist standardmässig nicht gesetzt, T2 wird also nicht gesetzt und das Ergebnis ist 0.
Und der Threadstarter hat ja schon eine quasi richtige Lösung gefunden.

Duddle

saila · 22.12.2008

AW: Java Preisrechner

Ok - ich nehm das mal für mich zum üben

Wenn dem so ist, was ist dann daran nicht korrekt? T2 ist erst mal 0 und wenn gesetzt wird es auch erst berücksichtigt? Korrekt?