[英]tslint / codelyzer / ng lint error: “for (… in …) statements must be filtered with an if statement”
[英]tslint complaining “statements must be filtered with an if statement” when using switch
可以说我有以下方法:
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;
}
我跑的时候
ng lint
我收到以下错误:
for(... in ...)语句必须使用if语句进行过滤
看一下类似的问题,我认为答案不适用于我的情况。 所有switch语句都在if-else-if阶段的类别中。
tslint应该将switch语句视为if语句的形式,但它不是?!
这让我很好奇,所以我查看了这条规则的TSlint源代码 。 它有一个名为isFiltered
的函数,它似乎只检查ts.SyntaxKind.IfStatement
,而不是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);
}
}
因此,除非您想将对象转换为数组,否则您需要使用您提供的链接中的修复。 Object.keys
或if
语句:
for (const errorName in state.errors) {
if (state.errors.hasOwnProperty(errorName)) {
switch (errorName) {
有趣的是,你可以使用任何类型的if
语句,错误就会消失。 没有检查你是否正在调用hasOwnProperty
。
该规则旨在防止您在使用for .. in时访问在对象原型上定义的属性。
但是,您可以重构代码,以便不使用它,并使其更易于维护和开发。
一个例子是这样的:
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 [];
}
你可以在这里的操场上看到一个工作片段。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.