123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687 |
- "use strict";
- /*
- * eslint-plugin-sonarjs
- * Copyright (C) 2018-2021 SonarSource SA
- * mailto:info AT sonarsource DOT com
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 3 of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- */
- // https://sonarsource.github.io/rspec/#/rspec/S3626
- const docs_url_1 = require("../utils/docs-url");
- const loops = 'WhileStatement, ForStatement, DoWhileStatement, ForInStatement, ForOfStatement';
- const rule = {
- meta: {
- messages: {
- removeRedundantJump: 'Remove this redundant jump.',
- suggestJumpRemoval: 'Remove this redundant jump',
- },
- schema: [],
- type: 'suggestion',
- hasSuggestions: true,
- docs: {
- description: 'Jump statements should not be redundant',
- recommended: 'error',
- url: (0, docs_url_1.default)(__filename),
- },
- },
- create(context) {
- function reportIfLastStatement(node) {
- const withArgument = node.type === 'ContinueStatement' ? !!node.label : !!node.argument;
- if (!withArgument) {
- const block = node.parent;
- if (block.body[block.body.length - 1] === node && block.body.length > 1) {
- const previousComments = context.getSourceCode().getCommentsBefore(node);
- const previousToken = previousComments.length === 0
- ? context.getSourceCode().getTokenBefore(node)
- : previousComments[previousComments.length - 1];
- context.report({
- messageId: 'removeRedundantJump',
- node,
- suggest: [
- {
- messageId: 'suggestJumpRemoval',
- fix: fixer => fixer.removeRange([previousToken.range[1], node.range[1]]),
- },
- ],
- });
- }
- }
- }
- function reportIfLastStatementInsideIf(node) {
- const ancestors = context.getAncestors();
- const ifStatement = ancestors[ancestors.length - 2];
- const upperBlock = ancestors[ancestors.length - 3];
- if (upperBlock.body[upperBlock.body.length - 1] === ifStatement) {
- reportIfLastStatement(node);
- }
- }
- return {
- [`:matches(${loops}) > BlockStatement > ContinueStatement`]: (node) => {
- reportIfLastStatement(node);
- },
- [`:matches(${loops}) > BlockStatement > IfStatement > BlockStatement > ContinueStatement`]: (node) => {
- reportIfLastStatementInsideIf(node);
- },
- ':function > BlockStatement > ReturnStatement': (node) => {
- reportIfLastStatement(node);
- },
- ':function > BlockStatement > IfStatement > BlockStatement > ReturnStatement': (node) => {
- reportIfLastStatementInsideIf(node);
- },
- };
- },
- };
- module.exports = rule;
- //# sourceMappingURL=no-redundant-jump.js.map
|