tslint se plaindre “des déclarations doit être filtré avec une instruction if” lors de l'utilisation de l'interrupteur

Permet de dire que j'ai la méthode suivante:

getErrorMessage(state: any, thingName?: string) {
    const thing: string = state.path || thingName;
    const messages: string[] = [];
    if (state.errors) {
        for (const errorName in state.errors) {
            switch (errorName) {
                case 'required':
                    messages.push(`You must enter a ${thing}`);
                    break;
                case 'minlength':
                    messages.push(`A ${thing} must be at least ${state.errors['minlength'].requiredLength}characters`);
                    break;
                case 'pattern':
                    messages.push(`The ${thing} contains illegal characters`);
                    break;
                case 'validateCardNumberWithAlgo':
                    messages.push(`Card doesnt pass algo`);
                    break;
            }
        }
    }
    return messages;
}

quand je lance

ng lint

J'obtiens l'erreur suivante :

pour ( ... ... ) les déclarations doivent être filtrées avec une instruction if

Avoir un look similaire à celui de question je ne pense pas que la réponse serait applicable à ma situation. Après tout, l'instruction switch se trouve dans la catégorie de si-sinon-si l'échelle.

tslint devraient envisager d'instruction switch comme une forme de déclaration, mais si il ne l'est pas?!

utiliser si d'autre à la place de l'interrupteur?
Voulez-vous suggérer la même chose si j'avais 100 conditions à vérifier?
En fait, je peux vous suggérer de refactoriser l'intérieur de la boucle en quelque chose d'autre. 🙂 Un appel de méthode, ou peut-être la carte sur les erreurs de la matrice de la création de la liste de messages d'erreur. 🙂
Double Possible de Ce qui ne l'JSLint erreur "corps de un pour dans doit être enveloppé dans une instruction if"?
Je pense que le point est que la règle dit qu'un if déclaration doit venir après le for...in, et une instruction switch peut être vu comme une sorte de if déclaration, pourtant, il ne permet pas de résoudre l'erreur.

OriginalL'auteur MHOOS | 2017-05-30

11

Cela m'a rendu curieux, j'ai donc vérifié les TSlint code source pour cette règle. Il a une fonction appelée isFiltered qui semble être la seule à vérifier ts.SyntaxKind.IfStatement, pas pour ts.SyntaxKind.SwitchStatement.
```
function isFiltered({statements}: ts.Block): boolean {
    switch (statements.length) {
        case 0: return true;
        case 1: return statements[0].kind === ts.SyntaxKind.IfStatement;
        default:
            return statements[0].kind === ts.SyntaxKind.IfStatement && nodeIsContinue((statements[0] as ts.IfStatement).thenStatement);
    }

}
```
De sorte que si vous voulez convertir votre objet dans un tableau, vous aurez besoin d'utiliser un correctif à partir du lien que vous avez fourni. Soit Object.keys, ou un if déclaration:
```
    for (const errorName in state.errors) {
      if (state.errors.hasOwnProperty(errorName)) {
        switch (errorName) {
```
La chose intéressante est que vous pouvez avoir tout type de if instruction et l'erreur s'en aller. Il n'existe pas de vérifier pour voir si vous appelez hasOwnProperty.

OriginalL'auteur Frank Modica

La règle est destinée à vous empêcher d'accéder à des propriétés définies sur l'objet prototype lors de l'utilisation de pour .. dans.

Vous pouvez cependant refactoriser le code de tout simplement ne pas l'utiliser, et de le rendre plus facile à maintenir et à développer ainsi.

Un exemple serait celui-ci:

interface ErrorMessageFactory {
  (thing: string, state?): string
}

type Errors = 'required' | 'minlength' | 'pattern' | 'validateCardNumberWithAlgo'

let errorFactory: {[e in Errors]: ErrorMessageFactory} = {
  required: (thing) => `You must enter a ${thing}`,
  minlength: (thing, state) => `A ${thing} must be at least ${state.errors['minlength'].requiredLength}characters`,
  pattern: (thing) => `The ${thing} contains illegal characters`,
  validateCardNumberWithAlgo: (thing) => `Card doesnt pass algo`
}



function getErrorMessage(state: any, thingName?: string) {
  if (state.errors) {
    return state.errors.map((error) => errorFactory[error](thingName, state));
  }
  return [];
}

Vous pouvez voir un extrait dans la cour de récréation ici.

J'aime vraiment la façon dont vous remaniée que le code,certainement un 'merci' et aussi un vote

OriginalL'auteur toskv

Vous devez vous connecter pour publier un commentaire.