123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229 |
- "use strict";
- const astUtils = require("./utils/ast-utils.js");
- const ARITHMETIC_OPERATORS = ["+", "-", "*", "/", "%", "**"];
- const BITWISE_OPERATORS = ["&", "|", "^", "~", "<<", ">>", ">>>"];
- const COMPARISON_OPERATORS = ["==", "!=", "===", "!==", ">", ">=", "<", "<="];
- const LOGICAL_OPERATORS = ["&&", "||"];
- const RELATIONAL_OPERATORS = ["in", "instanceof"];
- const TERNARY_OPERATOR = ["?:"];
- const COALESCE_OPERATOR = ["??"];
- const ALL_OPERATORS = [].concat(
- ARITHMETIC_OPERATORS,
- BITWISE_OPERATORS,
- COMPARISON_OPERATORS,
- LOGICAL_OPERATORS,
- RELATIONAL_OPERATORS,
- TERNARY_OPERATOR,
- COALESCE_OPERATOR
- );
- const DEFAULT_GROUPS = [
- ARITHMETIC_OPERATORS,
- BITWISE_OPERATORS,
- COMPARISON_OPERATORS,
- LOGICAL_OPERATORS,
- RELATIONAL_OPERATORS
- ];
- const TARGET_NODE_TYPE = /^(?:Binary|Logical|Conditional)Expression$/u;
- function normalizeOptions(options = {}) {
- const hasGroups = options.groups && options.groups.length > 0;
- const groups = hasGroups ? options.groups : DEFAULT_GROUPS;
- const allowSamePrecedence = options.allowSamePrecedence !== false;
- return {
- groups,
- allowSamePrecedence
- };
- }
- function includesBothInAGroup(groups, left, right) {
- return groups.some(group => group.includes(left) && group.includes(right));
- }
- function getChildNode(node) {
- return node.type === "ConditionalExpression" ? node.test : node.left;
- }
- module.exports = {
- meta: {
- deprecated: true,
- replacedBy: [],
- type: "suggestion",
- docs: {
- description: "Disallow mixed binary operators",
- recommended: false,
- url: "https://eslint.org/docs/latest/rules/no-mixed-operators"
- },
- schema: [
- {
- type: "object",
- properties: {
- groups: {
- type: "array",
- items: {
- type: "array",
- items: { enum: ALL_OPERATORS },
- minItems: 2,
- uniqueItems: true
- },
- uniqueItems: true
- },
- allowSamePrecedence: {
- type: "boolean",
- default: true
- }
- },
- additionalProperties: false
- }
- ],
- messages: {
- unexpectedMixedOperator: "Unexpected mix of '{{leftOperator}}' and '{{rightOperator}}'. Use parentheses to clarify the intended order of operations."
- }
- },
- create(context) {
- const sourceCode = context.sourceCode;
- const options = normalizeOptions(context.options[0]);
-
- function shouldIgnore(node) {
- const a = node;
- const b = node.parent;
- return (
- !includesBothInAGroup(options.groups, a.operator, b.type === "ConditionalExpression" ? "?:" : b.operator) ||
- (
- options.allowSamePrecedence &&
- astUtils.getPrecedence(a) === astUtils.getPrecedence(b)
- )
- );
- }
-
- function isMixedWithParent(node) {
- return (
- node.operator !== node.parent.operator &&
- !astUtils.isParenthesised(sourceCode, node)
- );
- }
-
- function getOperatorToken(node) {
- return sourceCode.getTokenAfter(getChildNode(node), astUtils.isNotClosingParenToken);
- }
-
- function reportBothOperators(node) {
- const parent = node.parent;
- const left = (getChildNode(parent) === node) ? node : parent;
- const right = (getChildNode(parent) !== node) ? node : parent;
- const data = {
- leftOperator: left.operator || "?:",
- rightOperator: right.operator || "?:"
- };
- context.report({
- node: left,
- loc: getOperatorToken(left).loc,
- messageId: "unexpectedMixedOperator",
- data
- });
- context.report({
- node: right,
- loc: getOperatorToken(right).loc,
- messageId: "unexpectedMixedOperator",
- data
- });
- }
-
- function check(node) {
- if (
- TARGET_NODE_TYPE.test(node.parent.type) &&
- isMixedWithParent(node) &&
- !shouldIgnore(node)
- ) {
- reportBothOperators(node);
- }
- }
- return {
- BinaryExpression: check,
- LogicalExpression: check
- };
- }
- };
|