
org.sonar.java.checks.DanglingElseStatementsCheck Maven / Gradle / Ivy
/*
* SonarQube Java
* Copyright (C) 2012-2024 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 Sonar Source-Available License Version 1, as published by SonarSource SA.
*
* 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 Sonar Source-Available License for more details.
*
* You should have received a copy of the Sonar Source-Available License
* along with this program; if not, see https://sonarsource.com/license/ssal/
*/
package org.sonar.java.checks;
import java.util.Collections;
import java.util.List;
import org.sonar.check.Rule;
import org.sonar.plugins.java.api.IssuableSubscriptionVisitor;
import org.sonar.plugins.java.api.tree.IfStatementTree;
import org.sonar.plugins.java.api.tree.Tree;
import org.sonar.plugins.java.api.tree.Tree.Kind;
@Rule(key = "S5261")
public class DanglingElseStatementsCheck extends IssuableSubscriptionVisitor {
@Override
public List nodesToVisit() {
return Collections.singletonList(Kind.IF_STATEMENT);
}
@Override
public void visitNode(Tree tree) {
IfStatementTree ifstmt = (IfStatementTree) tree;
if (hasElse(ifstmt) && isNested(ifstmt)) {
reportIssue(ifstmt.elseKeyword(), "Add explicit curly braces to avoid dangling else.");
}
}
private static boolean hasElse(IfStatementTree ifstmt) {
return ifstmt.elseStatement() != null;
}
private static boolean isNested(IfStatementTree ifstmt) {
return ifstmt.parent().is(Kind.IF_STATEMENT) && ((IfStatementTree) ifstmt.parent()).thenStatement() == ifstmt;
}
}
© 2015 - 2025 Weber Informatics LLC | Privacy Policy