2016-09-09 2 views
0

У меня есть этот код, используемый для получения нескольких часов от объекта и умножения их на переменную.javascript- Как проверить, является ли объект конкретным свойством

Вот объект «работа»

var work = [ 
    {'day': 27, 'hours': 7.30}, 
    {'day': 29, 'hours': 7.30}, 
    {'day': 31, 'hours': 10}, 
    {'day': 1, 'hours': 8.30}, 
    {'day': 2, 'hours': 7}, 
    {'day': 3, 'hours': 7}, 
    {'day': 5, 'hours': 7.30}, 
    {'day': 6, 'hours': 7}, 
    {'day': 7, 'hours': 7.30}, 
    {'day': 8, 'hours': 8}, 
    {'day': 9, 'hours': 9.30} 
] 

var payPerHour = 7; 

и вот моя функция для расчета зарплаты

function calculatePay() 
{ 
    var result = 0, fResult = 0; 

    for(var i = 0; i < work.length; i++) { 
     Object.keys(work).forEach(function (val) { 
      if (work[i].hasOwnProperty('hours')) 
       result = work[i][val] * payPerHour; 

      fResult += result; 
     }); 
    } 

    return fResult; 
} 

Я использовал «hasOwnProperty», чтобы проверить, если свойство «часы» существует в работе , Результатом функции является NaN. Зачем?

+2

'Object.keys (работа)': что вы ожидаете здесь? –

+0

с ним для цикла i итерации по массиву, а затем с Object.keys (work) я просматриваю объект, который находится в массиве. –

+0

@DavidDume - вам не нужно перебирать объект tho. – tymeJV

ответ

4

Вы уже перебор своего массива с помощью цикла for - тогда вы

Object.keys(work).forEach(function(val) { 

Это не имеет никакого смысла. work - массив, а не объект. В принципе, если вы удалите эту строку, она будет работать:

for(var i = 0; i < work.length; i++) { 
    //Object.keys(work).forEach(function(val) { 
    if(work[i].hasOwnProperty('hours')) 
     result = work[i]["hours"] * payPerHour; //use the right property here 

    fResult += result; 
    //}); 
} 

Более простой способ может быть использование Array.reduce

var totalHourPay = work.reduce(function(total, workDay) { 
    if (workDay.hasOwnProperty("hours")) { 
     total += workDay.hours * payPerHour; 
    } 

    return total; 
}, 0); 
+0

Или просто 'work.reduce ((всего, работаDay) => всего + (работаDay.часов || 0) * оплатаPerHour, 0)'. –

+0

@ScottSauyet - Ya - если доступен синтаксис ES6: D – tymeJV

+0

Несомненно, ES6 - приятное изменение, но также удаление проверки 'hasOwnProperty' в пользу' || 0' делает его более кратким, и для моих глаз более читабельным. –

0

Вы можете использовать Javascript for..in петель, как показано ниже:

function calculatePay() { 
 
    var result = 0, fResult = 0; 
 
    for(var index in work) { 
 
     \t \t if(work[index].hasOwnProperty('hours')) 
 
\t \t \t result = work[index]['hours'] * payPerHour; 
 
\t \t \t fResult += result; 
 
    } 
 
    return fResult; 
 
}

0

Почему эти сложные бывшие pressions? Попробуйте это решение ES6:

var work = [{'day':27,'hours':7.30},{'day':29,'hours':7.30},{'day':31,'hours':10}, 
 
      {'day':1,'hours':8.30},{'day':2,'hours':7},{'day':3,'hours':7}, 
 
      {'day':5,'hours':7.30},{'day':6,'hours':7},{'day':7,'hours':7.30}, 
 
      {'day':8,'hours':8},{'day': 9, 'hours': 9.30}], 
 
      calculatePay=(b)=>work.reduce((c,d)=>c+d.hours*b,0) 
 

 
// Test 
 
console.log(calculatePay(7)) // Returns 600.6 and 7 is your rate per hour 
 

 
// Or redefine function calculatePay this way if you need ES5 compatibility 
 
// function calculatePay(a) { 
 
// return work.reduce(function(b,c) { 
 
//  return b + c.hours * a 
 
// }, 0) 
 
// } 
 

 
// And if you really needs use hasOwnProperty, define it this way 
 
// function calculatePay(a) { 
 
// return work.reduce(function(b,c) { 
 
//  return b + (c.hasOwnProperty('hours') && c.hours || 0) * a 
 
// }, 0) 
 
// } 
 

 
// But you dont need hasOwnProperty. You are processing simple array, 
 
// not object. And this array is full of ordinary object literals. None of 
 
// them does not inherits anything from another object, unless you override 
 
// Object itself.

0

Я считаю, что это то, что вы хотели:

var work = [{'day': 27, 'hours': 7.30}, 
     {'day': 29, 'hours': 7.30}, 
     {'day': 31, 'hours': 10}, 
     {'day': 1, 'hours': 8.30}, 
     {'day': 2, 'hours': 7}, 
     {'day': 3, 'hours': 7}, 
     {'day': 5, 'hours': 7.30}, 
     {'day': 6, 'hours': 7}, 
     {'day': 7, 'hours': 7.30}, 
     {'day': 8, 'hours': 8}, 
     {'day': 9, 'hours': 9.30} 
     ]; 

var payPerHour = 7; 

var result = 0, fResult = 0; 

function calculatePay(){ 

for(var i = 0; i < work.length; i++) { 
    Object.keys(work).forEach(function() { 

    result = work[i].hours * payPerHour; 
    fResult += result; 
}); 

} 

console.log("Final total: " +fResult); 
} 

calculatePay(); 
Смежные вопросы