diff options
author | Florian Dold <florian.dold@gmail.com> | 2017-12-10 21:51:33 +0100 |
---|---|---|
committer | Florian Dold <florian.dold@gmail.com> | 2017-12-10 21:51:33 +0100 |
commit | 0469abd4a9c9270a1fdc962969e36e63699af8b4 (patch) | |
tree | f9864d4a4148621378958794cbbfdc2393733283 /node_modules/tslint/lib/rules/noConditionalAssignmentRule.js | |
parent | 6947e79bbc258f7bc96af424ddb71a511f0c15a3 (diff) |
upgrade dependencies
Diffstat (limited to 'node_modules/tslint/lib/rules/noConditionalAssignmentRule.js')
-rw-r--r-- | node_modules/tslint/lib/rules/noConditionalAssignmentRule.js | 119 |
1 files changed, 69 insertions, 50 deletions
diff --git a/node_modules/tslint/lib/rules/noConditionalAssignmentRule.js b/node_modules/tslint/lib/rules/noConditionalAssignmentRule.js index f437d1a63..30e77aa84 100644 --- a/node_modules/tslint/lib/rules/noConditionalAssignmentRule.js +++ b/node_modules/tslint/lib/rules/noConditionalAssignmentRule.js @@ -17,6 +17,7 @@ */ Object.defineProperty(exports, "__esModule", { value: true }); var tslib_1 = require("tslib"); +var tsutils_1 = require("tsutils"); var ts = require("typescript"); var Lint = require("../index"); var Rule = /** @class */ (function (_super) { @@ -25,14 +26,13 @@ var Rule = /** @class */ (function (_super) { return _super !== null && _super.apply(this, arguments) || this; } Rule.prototype.apply = function (sourceFile) { - var walker = new NoConditionalAssignmentWalker(sourceFile, this.getOptions()); - return this.applyWithWalker(walker); + return this.applyWithFunction(sourceFile, walk); }; /* tslint:disable:object-literal-sort-keys */ Rule.metadata = { ruleName: "no-conditional-assignment", description: "Disallows any type of assignment in conditionals.", - descriptionDetails: "This applies to `do-while`, `for`, `if`, and `while` statements.", + descriptionDetails: "This applies to `do-while`, `for`, `if`, and `while` statements and conditional (ternary) expressions.", rationale: (_a = ["\n Assignments in conditionals are often typos:\n for example `if (var1 = var2)` instead of `if (var1 == var2)`.\n They also can be an indicator of overly clever code which decreases maintainability."], _a.raw = ["\n Assignments in conditionals are often typos:\n for example \\`if (var1 = var2)\\` instead of \\`if (var1 == var2)\\`.\n They also can be an indicator of overly clever code which decreases maintainability."], Lint.Utils.dedent(_a)), optionsDescription: "Not configurable.", options: null, @@ -45,55 +45,74 @@ var Rule = /** @class */ (function (_super) { return Rule; }(Lint.Rules.AbstractRule)); exports.Rule = Rule; -var NoConditionalAssignmentWalker = /** @class */ (function (_super) { - tslib_1.__extends(NoConditionalAssignmentWalker, _super); - function NoConditionalAssignmentWalker() { - var _this = _super !== null && _super.apply(this, arguments) || this; - _this.isInConditional = false; - return _this; - } - NoConditionalAssignmentWalker.prototype.visitIfStatement = function (node) { - this.validateConditionalExpression(node.expression); - _super.prototype.visitIfStatement.call(this, node); - }; - NoConditionalAssignmentWalker.prototype.visitWhileStatement = function (node) { - this.validateConditionalExpression(node.expression); - _super.prototype.visitWhileStatement.call(this, node); - }; - NoConditionalAssignmentWalker.prototype.visitDoStatement = function (node) { - this.validateConditionalExpression(node.expression); - _super.prototype.visitDoStatement.call(this, node); - }; - NoConditionalAssignmentWalker.prototype.visitForStatement = function (node) { - if (node.condition != null) { - this.validateConditionalExpression(node.condition); +function walk(ctx) { + var checking = 0; + return ts.forEachChild(ctx.sourceFile, cb); + function cb(node) { + var kind = node.kind; + if (!tsutils_1.isNodeKind(kind)) { + return; // return early for tokens } - _super.prototype.visitForStatement.call(this, node); - }; - NoConditionalAssignmentWalker.prototype.visitBinaryExpression = function (expression) { - if (this.isInConditional) { - this.checkForAssignment(expression); - } - _super.prototype.visitBinaryExpression.call(this, expression); - }; - NoConditionalAssignmentWalker.prototype.validateConditionalExpression = function (expression) { - this.isInConditional = true; - if (expression.kind === ts.SyntaxKind.BinaryExpression) { - // check for simple assignment in a conditional, like `if (a = 1) {` - this.checkForAssignment(expression); + switch (kind) { + case ts.SyntaxKind.ConditionalExpression: + check(node.condition); + cb(node.whenTrue); + cb(node.whenFalse); + return; + case ts.SyntaxKind.IfStatement: + check(node.expression); + cb(node.thenStatement); + maybeCallback(cb, node.elseStatement); + return; + case ts.SyntaxKind.DoStatement: + case ts.SyntaxKind.WhileStatement: + check(node.expression); + cb(node.statement); + return; + case ts.SyntaxKind.ForStatement: + maybeCallback(cb, node.initializer); + maybeCallback(check, node.condition); + maybeCallback(cb, node.incrementor); + cb(node.statement); + return; } - // walk the children of the conditional expression for nested assignments, like `if ((a = 1) && (b == 1)) {` - this.walkChildren(expression); - this.isInConditional = false; - }; - NoConditionalAssignmentWalker.prototype.checkForAssignment = function (expression) { - if (isAssignmentToken(expression.operatorToken)) { - this.addFailureAtNode(expression, Rule.FAILURE_STRING); + if (checking !== 0) { + switch (kind) { + case ts.SyntaxKind.BinaryExpression: + if (tsutils_1.isAssignmentKind(node.operatorToken.kind)) { + ctx.addFailureAtNode(node, Rule.FAILURE_STRING); + } + cb(node.left); + cb(node.right); + return; + case ts.SyntaxKind.ParenthesizedExpression: + case ts.SyntaxKind.NonNullExpression: + case ts.SyntaxKind.AsExpression: + case ts.SyntaxKind.TypeAssertionExpression: + return cb(node.expression); + case ts.SyntaxKind.PrefixUnaryExpression: + return cb(node.operand); + default: + return noCheck(node); + } } - }; - return NoConditionalAssignmentWalker; -}(Lint.RuleWalker)); -function isAssignmentToken(token) { - return token.kind >= ts.SyntaxKind.FirstAssignment && token.kind <= ts.SyntaxKind.LastAssignment; + return ts.forEachChild(node, cb); + } + function check(node) { + ++checking; + cb(node); + --checking; + } + function noCheck(node) { + var old = checking; + checking = 0; + ts.forEachChild(node, cb); + checking = old; + } +} +function maybeCallback(cb, node) { + if (node !== undefined) { + cb(node); + } } var _a; |