Можлива ітерація JavaScript над несподіваною

У мене є такий код:

  for (i in awards) {
         if (awards[i] instanceof Array === false) {
               console.log(awards[i]);
                httpFactory.patch(awards[i], {"read": true}, false);
             }
       }

Моя IDE показує цю помилку, що стосується коду вище:

Можлива ітерація над несподіваними (спеціальними / успадкованими) членами, ймовірно відсутніми перевірка hasOwnProperty

Перевіряє наявність випадків нефільтрованих циклів входу в JavaScript. Використання цієї конструкції призводить до обробки успадкованих або несподіваних властивостей. Вам потрібно відфільтрувати власні властивості за допомогою методу hasOwnProperty (). Перевірка працює у файлах JavaScript, html або jsp.

Не могли б ви пояснити детальніше, що мається на увазі під цим твердженням?

IDE рекомендує додати тест:

if (awards.hasOwnProperty(i)) {
    ...
}

всередині forпетлі.

Я особисто рекомендую цього не робити і, якщо це можливо, вимкнути попередження. У більшості кодів просто немає потреби, а ще менше в коді ES5, де ви можете безпечно додавати неперелічені властивості до об’єкта за допомогоюObject.defineProperty

hasOwnPropertyПеревірка необхідна тільки якщо ви небезпечне додані нові (перераховуються) властивості Object.prototype, тому найпростіше виправлення не робити .

jQuery не виконує цей тест - вони явно документують, що jQuery зламається, якщо Object.prototypeвін небезпечно змінений.

Кожен об'єкт у javascript має прототип, який має власні властивості (власні / успадковані методи / властивості) та властивості, які безпосередньо приєднані до самого об'єкта.

Коли ви переглядаєте об’єкт, він повторюватиме властивості самого об’єкта та властивості прототипу об’єкта.

Отже, щоб уникнути ітерації над прототипом, рекомендується використовувати метод hasOwnProperty, який повертає true лише тоді, коли об’єкт має згадану властивість безпосередньо. тобто не всередині прототипу

Приклад

for (var k in object) {
  if (object.hasOwnProperty(k)) {
     // do your computation here.
  }
}

Детальніше тут

Ви також можете переформатувати ваш цикл у:

const keys = Object.keys(object);
for (const key of keys){
   // do something with object[key];
}

Також ви можете позбутися попередження про написання циклу forEach для більш читабельного та функціонального підходу:

Object.keys(object).forEach(key => {
    // Do something with object[key]
});

слід додати ще одну умову на початку цього циклу

if (awards.hasOwnProperty(i))