tslint / codelyzer / ng lint hatası: “for (… in…) ifadeleri bir if ifadesiyle filtrelenmelidir”

Tiftik hata mesajı:

src / app / detail / edit / edit.component.ts [111, 5]: for (... in ...) ifadeleri bir if ifadesiyle filtrelenmelidir

Kod snippet'i (Bu bir çalışma kodudur. Angular.io form doğrulama bölümünde de mevcuttur ):

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] + ' ';
        }
      }
    }

Bu tiftik hatasını nasıl düzeltebilirim?

To gerçek sorunu açıklamak tslint işaret olduğunu, JavaScript belgelerine bir alıntı için ... açıklamada :

Döngü, nesnenin kendisinin tüm numaralandırılabilir özellikleri üzerinde yineleyecektir ve nesne, yapıcısının prototipinden (prototip zincirindeki nesneye yakın olan özellikler prototiplerin özelliklerini geçersiz kılar) devralır.

Yani, temelde bu, elde etmeyi beklemediğiniz özellikleri (nesnenin prototip zincirinden) alacağınız anlamına gelir.

Bunu çözmek için sadece nesnelerin kendi özellikleri üzerinde yineleme yapmamız gerekir. Bunu iki farklı şekilde yapabiliriz (@Maxxx ve @Qwertiy tarafından önerildiği gibi).

İlk çözüm

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

Burada, belirli bir nesnenin kendi numaralandırılabilir özelliklerinin bir dizisini, for ... in döngüsü tarafından sağlanan sırayla döndüren Object.Keys () yöntemini kullanırız (fark, bir for-in döngüsünün özellikleri numaralandırmasıdır) prototip zinciri de).

İkinci çözüm

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

Bu çözümde, prototip zincirindeki olanlar da dahil olmak üzere nesnenin tüm özelliklerini yineliyoruz , ancak nesnenin belirtilen özelliğe sahip (miras alınmamış) özelliği olup olmadığını gösteren bir boole döndüren Object.prototype.hasOwnProperty () yöntemini kullanıyoruz. miras alınan özellikler.

@ Helzgate'in cevabını uygulamanın daha temiz bir yolu muhtemelen '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 kullanın:

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 ...) için davranışınız amaçlarınız için kabul edilebilir / gerekli ise, tslint'e izin vermesini söyleyebilirsiniz.

tslint.json içinde, bunu "kurallar" bölümüne ekleyin.

"forin": false

Aksi takdirde, @Maxxx ile doğru fikre sahip

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

Bu mesajın kullanmaktan kaçınmakla ilgili olmadığını düşünüyorum switch. Bunun yerine kontrol etmenizi istiyor hasOwnProperty. Arka plan buradan okunabilir: https://stackoverflow.com/a/16735184/1374488