予期しない以上のJavaScriptの可能な反復

Question 1

私は次のコードを持っています：

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

私のIDEは、上記のコードに関連してこのエラーを表示します。

予期しない（カスタム/継承された）メンバーに対する反復の可能性、おそらくhasOwnPropertyチェックが欠落している

JavaScriptでフィルタリングされていないfor-inループのインスタンスをチェックします。この構成を使用すると、継承されたプロパティまたは予期しないプロパティが処理されます。hasOwnProperty（）メソッドを使用して独自のプロパティをフィルタリングする必要があります。検証は、JavaScript、html、またはjspファイルで機能します。

この声明の意味を詳しく説明していただけますか？

Question 2

IDEは、テストを追加することを推奨しています。

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

内側のforループ。

個人的にはこれを行わず、可能であれば警告を無効にすることをお勧めします。ほとんどのコードでは必要がなく、ES5コードでは、を使用して列挙できないプロパティをオブジェクトに安全に追加できます。Object.defineProperty

このhasOwnPropertyチェックは、新しい（列挙可能な）プロパティを安全に追加していない場合にのみ必要なObject.prototypeので、最も簡単な修正はそれを行わないことです。

jQueryはこのテストを実行しません。安全でない方法で変更するとjQueryが機能しなくなることを明示的に文書化していObject.prototypeます。

Question 3

javascriptのすべてのオブジェクトには、独自のプロパティ（ネイティブ/継承されたメソッド/プロパティ）とオブジェクト自体に直接アタッチされたプロパティを持つプロトタイプがあります。

オブジェクトを反復処理すると、オブジェクト自体のプロパティとオブジェクトのプロトタイプのプロパティが反復処理されます。

したがって、プロトタイプの反復を回避するために、オブジェクトが指定されたプロパティを直接持っている場合にのみtrueを返すhasOwnPropertyメソッドを使用することをお勧めします。つまり、プロトタイプの内部ではありません

例

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

詳細については、こちらをご覧ください

Question 4

ループを次のようにリファクタリングすることもできます。

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

Question 5

また、forEachループを記述して警告を取り除き、より読みやすく機能的なアプローチをとることができます。

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

Question 6

このループの最初にもう1つの条件を追加する必要があります

if (awards.hasOwnProperty(i))