tslint / codelyzer / ng lintエラー：「for（…in…）ステートメントはifステートメントでフィルタリングする必要があります」

リントエラーメッセージ：

src / app / detail / edit / edit.component.ts [111、5]：for（... in ...）ステートメントは、ifステートメントでフィルタリングする必要があります

コードスニペット（これは動作するコードです。angular.ioフォーム検証セクションでも入手できます）：

for (const field in this.formErrors) {
      // clear previous error message (if any)
      this.formErrors[field] = '';
      const control = form.get(field);

      if (control && control.dirty && !control.valid) {
        const messages = this.validationMessages[field];
        for (const key in control.errors) {
          this.formErrors[field] += messages[key] + ' ';
        }
      }
    }

この糸くずのエラーを修正する方法はありますか？

tslintが指摘している実際の問題を説明するために、for ... inステートメントの JavaScriptドキュメントからの引用：

ループは、オブジェクト自体の列挙可能なすべてのプロパティと、オブジェクトがコンストラクターのプロトタイプから継承するプロパティ（プロトタイプチェーンのオブジェクトに近いプロパティがプロトタイプのプロパティをオーバーライドする）を反復します。

したがって、これは基本的に、（オブジェクトのプロトタイプチェーンから）予期しないプロパティを取得することを意味します。

これを解決するには、オブジェクトのプロパティのみを反復処理する必要があります。これは2つの異なる方法で実行できます（@Maxxxと@Qwertiyで提案されています）。

最初の解決策

for (const field of Object.keys(this.formErrors)) {
    ...
}

ここではObject.Keys（）を利用しています、for ... inループによって提供されるのと同じ順序で、特定のオブジェクト自体の列挙可能なプロパティの配列を返すメソッドします（違いは、for-inループがプロパティを列挙することです）プロトタイプチェーンも）。

第二の解決策

for (var field in this.formErrors) {
    if (this.formErrors.hasOwnProperty(field)) {
        ...
    }
}

このソリューションでは、プロトタイプチェーン内のプロパティを含むすべてのオブジェクトのプロパティを反復処理しますが、Object.prototype.hasOwnProperty（）メソッドを使用して、オブジェクトが指定されたプロパティを（継承されていない）プロパティとして持つかどうかを示すブール値を返し、継承されたプロパティ。

@Helzgateの応答を適用するより適切な方法は、「for .. in」を次のように置き換えることです。

for (const field of Object.keys(this.formErrors)) {

for (const field in this.formErrors) {
  if (this.formErrors.hasOwnProperty(field)) {

for (const key in control.errors) {
  if (control.errors.hasOwnProperty(key)) {

Object.keysを使用します。

Object.keys(this.formErrors).map(key => {
  this.formErrors[key] = '';
  const control = form.get(key);

  if(control && control.dirty && !control.valid) {
    const messages = this.validationMessages[key];
    Object.keys(control.errors).map(key2 => {
      this.formErrors[key] += messages[key2] + ' ';
    });
  }
});

for（... in ...）の動作が目的に応じて許容可能/必要な場合は、tslintに許可するように指示できます。

tslint.jsonで、これを「rules」セクションに追加します。

"forin": false

そうでなければ、@ Maxxxは正しい考えを持っています

for (const field of Object.keys(this.formErrors)) {

このメッセージは、使用を避けることに関するものではないと思いますswitch。代わりに、それをチェックするように求めますhasOwnProperty。背景はここで読むことができます：https：//stackoverflow.com/a/16735184/1374488