
org.sonar.java.checks.ConditionalOnNewLineCheck 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 org.sonar.check.Rule;
import org.sonar.plugins.java.api.IssuableSubscriptionVisitor;
import org.sonar.plugins.java.api.JavaFileScannerContext;
import org.sonar.plugins.java.api.location.Position;
import org.sonar.plugins.java.api.tree.IfStatementTree;
import org.sonar.plugins.java.api.tree.StatementTree;
import org.sonar.plugins.java.api.tree.SyntaxToken;
import org.sonar.plugins.java.api.tree.Tree;
import org.sonar.plugins.java.api.tree.Tree.Kind;
import java.util.Collections;
import java.util.List;
@Rule(key = "S3972")
public class ConditionalOnNewLineCheck extends IssuableSubscriptionVisitor {
private SyntaxToken previousToken;
@Override
public List nodesToVisit() {
return Collections.singletonList(Kind.IF_STATEMENT);
}
@Override
public void setContext(JavaFileScannerContext context) {
previousToken = null;
super.setContext(context);
}
@Override
public void visitNode(Tree tree) {
IfStatementTree ifStatementTree = (IfStatementTree) tree;
if(previousToken != null && isOnSameLineAsPreviousIf(ifStatementTree)) {
reportIssue(ifStatementTree.ifKeyword(), "Move this \"if\" to a new line or add the missing \"else\".",
Collections.singletonList(new JavaFileScannerContext.Location("", previousToken)), null);
}
StatementTree elsePart = ifStatementTree.elseStatement();
if (elsePart != null) {
previousToken = elsePart.lastToken();
} else {
previousToken = ifStatementTree.thenStatement().lastToken();
}
}
private boolean isOnSameLineAsPreviousIf(IfStatementTree ifStatementTree) {
// check column for nested if on one line case.
Position previousTokenStart = Position.startOf(previousToken);
Position ifStatementStart = Position.startOf(ifStatementTree.ifKeyword());
return previousTokenStart.line() == ifStatementStart.line() &&
previousTokenStart.column() < ifStatementStart.column();
}
}
© 2015 - 2025 Weber Informatics LLC | Privacy Policy