Udreng procent af decimaltal

Den skriver bare NaN istedet for et resultat!

Tjah, rabat er jo en tekststreng, når du aflæser den som værdi i et felt, og hvis der er komma i, kan javascript ikke gætte sig til at det er et tal ...

Du kan finde en funktion, som kan noget forskelligt med tekstformattering af tal her: http://exp.dk/spm/640316 !-)

-- råt, uden afrunding osv., ville jeg prøve sådan:

function opdaterpris(foerpris){
  rabat = +document.form['rabat_vare'].value.replace(/,/,".");

  if (rabat > 100){
    alert("For højt tal, maks. 100!");
    document.form['rabat_vare'].value='0';
    document.getElementById('pris_efter_rabat').innerHTML = foerpris+' Kr.';
  }else{
    nyrabat = foerpris*100/rabat;
    document.getElementById('pris_efter_rabat').innerHTML = nyrabat+' Kr.';
  }
}

-- men den vil jo returnere rabatten, ikke prisen efter rabat, så nok:

    document.getElementById('pris_efter_rabat').innerHTML = foerpris-nyrabat+' Kr.';

Ret lige:

nyrabat = foerpris*100/rabat;

- til:

nyrabat = foerpris*rabat/100;

Godmorgen

Tja, jeg må nok indrømme at mit javascript er ret håbløst i denne situation. Jeg er kommet frem til koden (inspireret af det link du gav mig, med validering), men det virker ikke korrekt endnu.

function opdaterpris(vare_id,foerpris){

rabat = +document.form['rabat_vare_'].value.replace(/,/,".");

if(!rabat.match(/^-?\d+([,\.]\d+)*$/)){
    alert("Fejl, ikke gyldige tegn blev indtastet. Kun tal og komma er tilladt!");
    return "";
  }else{
if (rabat > 100){
alert("For højt tal, maks. 100!");
document.form['rabat_vare_'+vare_id+''].value='0';
document.all['pris_efter_rabat'].innerHTML = ''+foerpris+' Kr.';
}else{
nyrabat = foerpris*rabat/100;
document.all['pris_efter_rabat'].innerHTML = foerpris-nyrabat+' Kr.';
}
}
 

}

bare lige se bort fra "vare_id"... Det er noget som jeg har planer om at bygge på, men ikke noget jeg skal bruge endnu...

Jeg kan se fra dit pattern at du tillader at man kan skrive et minus-tegn foran din rabbat-sats. Det er ikke nogen god ide. Du skal gøre op ned dig selv om rabatten altid skal være et positivt eller et negativt tal.

Okay, hvordan ændre jeg det? :)

if(!rabat.match(/^\d+([,\.]\d+)*$/)){

Hej igen... Jeg er kommet frem til følgende indtil videre:

function opdaterpris(foerpris){
var rabat;
foerpris = parseInt(foerpris);
string_rabat = document.form['rabat_vare'].value;
if(!string_rabat.match(/^\d+([,\.]\d+)*$/)){
document.form['rabat_vare'].value='0';
document.all['pris_efter_rabat'].innerHTML = ''+foerpris+' Kr.';
    alert("Fejl, ikke gyldige tegn blev indtastet. Kun tal og komma er tilladt!");
    return "";
  }else{
rabat = parseInt(document.form['rabat_vare'].value);
if (rabat > 100){
alert("For højt tal, maks. 100!");
document.form['rabat_vare'].value='0';
document.all['pris_efter_rabat'].innerHTML = ''+foerpris+' Kr.';
}else{
nyrabat = foerpris*rabat/100;
document.all['pris_efter_rabat'].innerHTML = foerpris-nyrabat+' Kr.';
}
}

}

Men, jeg kan stadig ikke få den til at bruge kommatal, hvis jeg indtaster dem i formen, altså i document.form['rabat_vare'], hvordan gør jeg det?!? Samt, vil jeg gerne at den i pris_efter_rabat, skriver tallet med tusindseperator samt 2 decimaler efter komma...

Jeg har selv fået lavet det, med lidt diverse koder jeg har fundet... Det kan godt være at det ikke lige har gjort det nemt for mig, men det virker... Den eneste justering jeg har lavet, er at foerpris nu er med "punktum-komma" istedet for traditionelt dansk komma, så nu virker det. Hvis I begge lige vil lave et svar, så kan I få lidt point for at hjælpe på rette kurs. Min kode blev som følger:

function updateprisefterrabat(vare_id,foerpris){
var rabat;
string_rabat = document.tilfoej_kunde['rabat_vare_'+vare_id+''].value;
if(!string_rabat.match(/^\d+([,\.]\d+)*$/)){
        document.tilfoej_kunde['rabat_vare_'+vare_id+''].value='0';
        document.all['pris_efter_rabat_'+vare_id+''].innerHTML = ''+foerpris+' Kr.';
    alert("Fejl, ikke gyldige tegn blev indtastet. Kun tal og komma er tilladt!");
    return "";
}else{
        rabat = parseInt(document.tilfoej_kunde['rabat_vare_'+vare_id+''].value);
        if (rabat > 100){
                alert("For højt tal, maks. 100!");
                document.tilfoej_kunde['rabat_vare_'+vare_id+''].value='0';
                document.all['pris_efter_rabat_'+vare_id+''].innerHTML = ''+foerpris+' Kr.';
        }else{
                nyrabat = (foerpris)-(foerpris*rabat/100);
                var val = nyrabat;
                val+="";
              komma = val.indexOf('.');
              if(komma>-1){
                    dec = val.substring(komma)+'00';
                            if(dec.length>3)dec = Math.round(parseFloat('0'+dec)*100);
                    else dec = dec.substring(1,3);
                            val = val.substring(0,komma);
                        }
                else dec = "00";
                      val = val.split('').reverse().join('');
                      val = val.replace(/(\d{3})/g, "$1.");
              if(val.length%4==0)val = val.substring(0,val.length-1);
              val = val.split('').reverse().join('');
                document.all['pris_efter_rabat_'+vare_id+''].innerHTML = ""+val+","+dec+" Kr.";
                }
        }

}

Ingen point til mig på denne her. :^)

Så velbekomme '-)

Tak for point ;~}