Почему игнорируется !isNaN?

Question

[viva cristo64]

Учу js. Нужно было сделать программку, которая суммирует все числа в объекте, и если это не число просто пропускает его.
Возможно задача и простая, но не для меня, видимо.
Код:

const salaries = {"Vasiliy": 100,"Piotr": 300,"Daria": 250,"isPaid": false};

function sum(obj) {
    var sum = 0;
    for(var el in obj) {
        if( obj.hasOwnProperty(el) ) {
            if (!isNaN(obj[el])) {
                sum += parseFloat( obj[el] );
            }
        }
    }
    return sum;
}

alert(sum(salaries))

На первый взгляд все суммируется и норм, но почему-то игнорируется !isNaN и вместе с числами пытается суммироваться "isPaid", равная false. В итоге, что логично, пользователю выводится NaN.
Скажите пожалуйста - в чём проблема?

Answer 1

[Сергей Соколов]

isNaN() проверяет, что аргумент — это NaN – специальное значение НеЧисло ("NaN" = "Not a Number")

Вам же, наоборот, нужно убедиться что аргумент — именно число:

Object.values(
  { "Vasiliy": 100, "Piotr": "300", "Daria": 250, isPaid: false }
)
  .map(parseFloat) // что можно – преобразовать в число, или NaN
  .filter(value => !isNaN(value)) // отсеить NaN, оставить только числа
  .reduce((acc, c) => acc + c) // суммировать
// 650

Comments

[WbICHA]

Только вот если isPaid: true, то будет 651 в ответе.)

[WbICHA]

viva cristo64, .map(parseFloat) используй, а то ты, судя по всему, не тестил это решение.)

[Сергей Соколов]

WbICHA, точно. Поправил.

Answer 2

[Александр]

isNaN(false) --> false
!isNaN(false) --> true
parseFloat(false) --> NaN
100 + NaN --> NaN
https://developer.mozilla.org/en-US/docs/Web/JavaS...
Также if( obj.hasOwnProperty(el) ) { лишнее.