Der Niederrheiner: Neuling: Komma und zweistellig mit kfm. Runden

Beitrag lesen

Ich würde es wie folgt machen :

1. wieso packst Du alle functions in einen eigenen script-Tag ? Pack die doch alle in einen einzigen. das spart schon mal viel Zeit und Platz.

2. Den toFixed() kannte ich nicht; macht aber definitiv Sinn :o)  !

3. Arbeite mit Funktionen, die Dir die Arbeit abnehmen

BISHER :

<script type="text/javascript">function perc1() {  document.getElementById("euro1").value = parseFloat(document.getElementById("rabatt1").value)*parseFloat(document.getElementById("summe1").value)/100;}</script>

usw.

NEU :

<script type="text/javascript">

function kommazupunkt(zahl) {
    zahl=zahl.toString();
    zahl=zahl.replace(/,/,".");
    zahl=parseFloat(zahl);
    return zahl;
  }

function perc1() {
    r=document.getElementById("rabatt1").value;
    s=document.getElementById("summe1").value;
    r=kommazupunkt(r);
    s=kommazupunkt(s);
    erg=s*(r/100);
    document.getElementById("euro1").value = erg.toFixed(2);
  }

function perc2 ......
  usw. usw.
</script>

Man kann den Code natürlich noch schöner verschachteln, aber so ist er für einen newbie besser lesbar
(hoffe ich)

Stephan