123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105 |
- /**
- * @fileoverview Rule to flag use of unary increment and decrement operators.
- * @author Ian Christian Myers
- * @author Brody McKee (github.com/mrmckeb)
- */
- "use strict";
- //------------------------------------------------------------------------------
- // Helpers
- //------------------------------------------------------------------------------
- /**
- * Determines whether the given node is the update node of a `ForStatement`.
- * @param {ASTNode} node The node to check.
- * @returns {boolean} `true` if the node is `ForStatement` update.
- */
- function isForStatementUpdate(node) {
- const parent = node.parent;
- return parent.type === "ForStatement" && parent.update === node;
- }
- /**
- * Determines whether the given node is considered to be a for loop "afterthought" by the logic of this rule.
- * In particular, it returns `true` if the given node is either:
- * - The update node of a `ForStatement`: for (;; i++) {}
- * - An operand of a sequence expression that is the update node: for (;; foo(), i++) {}
- * - An operand of a sequence expression that is child of another sequence expression, etc.,
- * up to the sequence expression that is the update node: for (;; foo(), (bar(), (baz(), i++))) {}
- * @param {ASTNode} node The node to check.
- * @returns {boolean} `true` if the node is a for loop afterthought.
- */
- function isForLoopAfterthought(node) {
- const parent = node.parent;
- if (parent.type === "SequenceExpression") {
- return isForLoopAfterthought(parent);
- }
- return isForStatementUpdate(node);
- }
- //------------------------------------------------------------------------------
- // Rule Definition
- //------------------------------------------------------------------------------
- module.exports = {
- meta: {
- type: "suggestion",
- docs: {
- description: "disallow the unary operators `++` and `--`",
- category: "Stylistic Issues",
- recommended: false,
- url: "https://eslint.org/docs/rules/no-plusplus"
- },
- schema: [
- {
- type: "object",
- properties: {
- allowForLoopAfterthoughts: {
- type: "boolean",
- default: false
- }
- },
- additionalProperties: false
- }
- ],
- messages: {
- unexpectedUnaryOp: "Unary operator '{{operator}}' used."
- }
- },
- create(context) {
- const config = context.options[0];
- let allowForLoopAfterthoughts = false;
- if (typeof config === "object") {
- allowForLoopAfterthoughts = config.allowForLoopAfterthoughts === true;
- }
- return {
- UpdateExpression(node) {
- if (allowForLoopAfterthoughts && isForLoopAfterthought(node)) {
- return;
- }
- context.report({
- node,
- messageId: "unexpectedUnaryOp",
- data: {
- operator: node.operator
- }
- });
- }
- };
- }
- };
|