forked from palantir/tslint
-
Notifications
You must be signed in to change notification settings - Fork 0
/
noSwitchCaseFallThroughRule.ts
118 lines (105 loc) · 4.27 KB
/
noSwitchCaseFallThroughRule.ts
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
/**
* @license
* Copyright 2013 Palantir Technologies, Inc.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
import * as utils from "tsutils";
import * as ts from "typescript";
import * as Lint from "../index";
export class Rule extends Lint.Rules.AbstractRule {
/* tslint:disable:object-literal-sort-keys */
public static metadata: Lint.IRuleMetadata = {
ruleName: "no-switch-case-fall-through",
description: "Disallows falling through case statements.",
descriptionDetails: Lint.Utils.dedent`
For example, the following is not allowed:
\`\`\`ts
switch(foo) {
case 1:
someFunc(foo);
case 2:
someOtherFunc(foo);
}
\`\`\`
However, fall through is allowed when case statements are consecutive or
a magic \`/* falls through */\` comment is present. The following is valid:
\`\`\`ts
switch(foo) {
case 1:
someFunc(foo);
/* falls through */
case 2:
case 3:
someOtherFunc(foo);
}
\`\`\``,
rationale: "Fall though in switch statements is often unintentional and a bug.",
optionsDescription: "Not configurable.",
options: null,
optionExamples: ["true"],
type: "functionality",
typescriptOnly: false,
};
/* tslint:enable:object-literal-sort-keys */
public static FAILURE_STRING_PART = "expected a 'break' before ";
public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
return this.applyWithWalker(new NoSwitchCaseFallThroughWalker(sourceFile, this.ruleName, undefined));
}
}
export class NoSwitchCaseFallThroughWalker extends Lint.AbstractWalker<void> {
public walk(sourceFile: ts.SourceFile) {
const cb = (node: ts.Node): void => {
if (node.kind === ts.SyntaxKind.SwitchStatement) {
this.visitSwitchStatement(node as ts.SwitchStatement);
}
return ts.forEachChild(node, cb);
};
return ts.forEachChild(sourceFile, cb);
}
private visitSwitchStatement(node: ts.SwitchStatement) {
const clauses = node.caseBlock.clauses;
const len = clauses.length - 1; // last clause doesn't need to be checked
for (let i = 0; i < len; ++i) {
if (clauses[i].statements.length !== 0 &&
// TODO type assertion can be removed with typescript 2.2
!utils.endsControlFlow(clauses[i] as ts.CaseClause) &&
!this.isFallThroughAllowed(clauses[i])) {
this.reportError(clauses[i + 1]);
}
}
}
private isFallThroughAllowed(clause: ts.CaseOrDefaultClause) {
const sourceFileText = this.sourceFile.text;
const comments = ts.getLeadingCommentRanges(sourceFileText, clause.end);
if (comments === undefined) {
return false;
}
for (const comment of comments) {
let commentText: string;
if (comment.kind === ts.SyntaxKind.MultiLineCommentTrivia) {
commentText = sourceFileText.substring(comment.pos + 2, comment.end - 2);
} else {
commentText = sourceFileText.substring(comment.pos + 2, comment.end);
}
if (commentText.trim() === "falls through") {
return true;
}
}
return false;
}
private reportError(clause: ts.CaseOrDefaultClause) {
const keyword = clause.kind === ts.SyntaxKind.CaseClause ? "case" : "default";
this.addFailureAt(clause.getStart(this.sourceFile), keyword.length, `${Rule.FAILURE_STRING_PART}'${keyword}'`);
}
}