Calculation accepts space in input filds

Question

I have this script to calculate a form

    $(".mad, .mad2, .mad3, .mad4, .mad5, .mad6").each(function() {
        //add only if the value is number
        if (!isNaN(this.value) && this.value.length != 0) {
            antal += parseFloat(this.value);
             subtotal = antal*30;
             $(this).css("background-color", "#ffffff");
        }
        else if (this.value.length != 0){
             $(this).css("background-color", "#e89898");
       }
    });
        $("#antal").html(antal.toFixed(0));
        $("#subtotal").html(subtotal.toFixed(0));

it seems to work fine, if one is trying to ad a letter or its being removed and the input field turns red. however i dos accept space resulting in the antal and subtotal Show NaN

How do i get it to work for both letters AND space (not accepting space either ) ?

just replace spaces by empty string: `this.value.replace(/ /g,'')` — A. Wolff, Apr 02 '14 at 13:35
why do you need the `if` in the `else if`? shouldn't this just be `else`? — Liam, Apr 02 '14 at 13:37
possible duplicate of [Why does isNaN(" ") equal false](http://stackoverflow.com/questions/825402/why-does-isnan-equal-false) — Liam, Apr 02 '14 at 13:43

score 1 · Accepted Answer · answered Apr 02 '14 at 13:39

1

Try and use jquery's trim() method to remove all the whitespaces:

if (this.value.trim().length != 0 && !isNaN(this.value.trim()) ) {

}

answered Apr 02 '14 at 13:39

user1636130

1,615
5
29
47

Thanks buth where in my script do i ad that – Jesper Møller Apr 02 '14 at 14:33
I assume you figured it out? You want to replace the first if statement that you posted with what I posted. – user1636130 Apr 02 '14 at 16:54

score 0 · Answer 2 · answered Apr 02 '14 at 13:47

0

if (parseFloat(this.value) == this.value) {

answered Apr 02 '14 at 13:47

daghan

948
10
18

Calculation accepts space in input filds

2 Answers2