Custom Validator с .forEach - это можно сделать по-другому (лучше)?

Question

Я создал свой собственный класс валидатора для практики.Это хороший подход или лучше?

import { AbstractControl } from '@angular/forms';

export class ProjectNameValidator {
   private static blackList = ['Test1'];
    static correctName(control: AbstractControl): {[key: string]: boolean} | null {
       const name: string = control.value;
       let isValid = true;
       ProjectNameValidator.blackList.forEach(forbiddenName => {
           if (forbiddenName === name) {
              isValid = !isValid;
           }
       });
       return isValid ? null : {'Forbidden name': true};
    }
}

Answer 1

Я бы использовал includes следующим образом:

export class ProjectNameValidator {
  private static blackList = ['Test'];
  static correctName(control: AbstractControl): { [key: string]: boolean } | null {
    return ProjectNameValidator.blackList.includes(control.value) ?  { 'Forbidden name': true } : null;
  }
}

Answer 2

Вы перебираете весь список, чтобы проверить, является ли один из них недействительным (запрещенным), но как только вы обнаружите один недействительный, вы все равно продолжите проверку, и, кроме того, существует ли четное количество запрещенных имен, которым вы соответствуетев конечном итоге вы позволите это.(isValid =! isValid).

Я рекомендую использовать indexOf следующим образом:

ProjectNameValidator.blackList.indexOf(name) >= 0;

, чтобы быстро проверить, содержится ли введенное значение в черном списке.

Answer 3

Как теперь выглядит мой пользовательский валидатор:

import { AbstractControl } from '@angular/forms';

export class ProjectNameValidator {
   private static blackList = ['Test1', 'Test2', 'Test3'];
    static correctName(control: AbstractControl): {[key: string]: boolean} | null {
       const name: string = control.value;
       const isInvalid =  ProjectNameValidator.blackList.indexOf(name) !== -1;
       return isInvalid ? {'Forbidden name': true} : null;
    }
}