
org.sonar.java.checks.CatchUsesExceptionWithContextCheck Maven / Gradle / Ivy
/*
* SonarQube Java
* Copyright (C) 2012 SonarSource
* [email protected]
*
* 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 02
*/
package org.sonar.java.checks;
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.Sets;
import org.sonar.api.rule.RuleKey;
import org.sonar.check.BelongsToProfile;
import org.sonar.check.Priority;
import org.sonar.check.Rule;
import org.sonar.plugins.java.api.JavaFileScanner;
import org.sonar.plugins.java.api.JavaFileScannerContext;
import org.sonar.plugins.java.api.tree.AssignmentExpressionTree;
import org.sonar.plugins.java.api.tree.BaseTreeVisitor;
import org.sonar.plugins.java.api.tree.CatchTree;
import org.sonar.plugins.java.api.tree.ConditionalExpressionTree;
import org.sonar.plugins.java.api.tree.ExpressionTree;
import org.sonar.plugins.java.api.tree.IdentifierTree;
import org.sonar.plugins.java.api.tree.MemberSelectExpressionTree;
import org.sonar.plugins.java.api.tree.MethodInvocationTree;
import org.sonar.plugins.java.api.tree.NewClassTree;
import org.sonar.plugins.java.api.tree.ParenthesizedTree;
import org.sonar.plugins.java.api.tree.ThrowStatementTree;
import org.sonar.plugins.java.api.tree.Tree;
import org.sonar.plugins.java.api.tree.Tree.Kind;
import org.sonar.plugins.java.api.tree.TypeCastTree;
import org.sonar.plugins.java.api.tree.VariableTree;
import java.util.Iterator;
import java.util.List;
import java.util.Set;
@Rule(
key = CatchUsesExceptionWithContextCheck.RULE_KEY,
priority = Priority.MAJOR,
tags={"error-handling"})
@BelongsToProfile(title = "Sonar way", priority = Priority.MAJOR)
public class CatchUsesExceptionWithContextCheck extends BaseTreeVisitor implements JavaFileScanner {
public static final String RULE_KEY = "S1166";
private final RuleKey ruleKey = RuleKey.of(CheckList.REPOSITORY_KEY, RULE_KEY);
private static final Set EXCLUDED_EXCEPTION_TYPE = ImmutableSet.of(
"NumberFormatException",
"InterruptedException",
"ParseException",
"MalformedURLException");
private static final Set JAVA_SUB_PACKAGE = ImmutableSet.of(
"lang",
"text",
"net");
private JavaFileScannerContext context;
private final Set invalidCatchesCheck = Sets.newHashSet();
@Override
public void scanFile(JavaFileScannerContext context) {
this.context = context;
scan(context.getTree());
}
@Override
public void visitCatch(CatchTree tree) {
invalidCatchesCheck.add(tree);
super.visitCatch(tree);
if (invalidCatchesCheck.contains(tree)) {
invalidCatchesCheck.remove(tree);
if (!isExcludedType(tree.parameter().type())) {
context.addIssue(tree, ruleKey, "Either log or rethrow this exception along with some contextual information.");
}
}
}
private static boolean isExcludedType(Tree tree) {
return isUnqualifiedExcludedType(tree) ||
isQualifiedExcludedType(tree);
}
private static boolean isUnqualifiedExcludedType(Tree tree) {
return tree.is(Kind.IDENTIFIER) &&
EXCLUDED_EXCEPTION_TYPE.contains(((IdentifierTree) tree).name());
}
private static boolean isQualifiedExcludedType(Tree tree) {
if (!tree.is(Kind.MEMBER_SELECT)) {
return false;
}
MemberSelectExpressionTree memberSelectExpressionTree = (MemberSelectExpressionTree) tree;
if (!EXCLUDED_EXCEPTION_TYPE.contains(memberSelectExpressionTree.identifier().name()) || !memberSelectExpressionTree.expression().is(Kind.MEMBER_SELECT)) {
return false;
}
memberSelectExpressionTree = (MemberSelectExpressionTree) memberSelectExpressionTree.expression();
if (!JAVA_SUB_PACKAGE.contains(memberSelectExpressionTree.identifier().name()) || !memberSelectExpressionTree.expression().is(Kind.IDENTIFIER)) {
return false;
}
IdentifierTree identifierTree = (IdentifierTree) memberSelectExpressionTree.expression();
return "java".equals(identifierTree.name());
}
@Override
public void visitMethodInvocation(MethodInvocationTree tree) {
super.visitMethodInvocation(tree);
if (tree.arguments().size() >= 2) {
handleArguments(tree.arguments());
}
}
private void removeInvalidCatches(String exceptionName) {
Iterator it = invalidCatchesCheck.iterator();
while (it.hasNext()) {
CatchTree tree = it.next();
if (exceptionName.equals(tree.parameter().simpleName())) {
it.remove();
}
}
}
private void handleArguments(List trees) {
for (Tree tree : trees) {
if (tree.is(Kind.IDENTIFIER)) {
removeInvalidCatches(((IdentifierTree) tree).name());
}
}
}
@Override
public void visitThrowStatement(ThrowStatementTree tree) {
super.visitThrowStatement(tree);
allowIfExceptionInExpression(tree.expression(), true);
}
@Override
public void visitAssignmentExpression(AssignmentExpressionTree tree) {
super.visitAssignmentExpression(tree);
allowIfExceptionInExpression(tree.expression(), true);
}
@Override
public void visitVariable(VariableTree tree) {
super.visitVariable(tree);
if (tree.initializer() != null) {
allowIfExceptionInExpression(tree.initializer(), true);
}
}
private void allowIfExceptionInExpression(ExpressionTree tree, boolean allowIdentifier) {
if (tree.is(Kind.IDENTIFIER) && allowIdentifier) {
removeInvalidCatches(((IdentifierTree) tree).name());
} else if (tree.is(Kind.NEW_CLASS)) {
NewClassTree newClassTree = (NewClassTree) tree;
handleArguments(newClassTree.arguments());
} else if (tree.is(Kind.METHOD_INVOCATION)) {
MethodInvocationTree methodInvocationTree = (MethodInvocationTree) tree;
allowIfExceptionInExpression(methodInvocationTree.methodSelect(), false);
handleArguments(methodInvocationTree.arguments());
} else if (tree.is(Kind.TYPE_CAST)) {
allowIfExceptionInExpression(((TypeCastTree) tree).expression(), allowIdentifier);
} else if (tree.is(Kind.MEMBER_SELECT)) {
allowIfExceptionInExpression(((MemberSelectExpressionTree) tree).expression(), false);
} else if (tree.is(Kind.PARENTHESIZED_EXPRESSION)) {
allowIfExceptionInExpression(((ParenthesizedTree) tree).expression(), allowIdentifier);
} else if (tree.is(Kind.CONDITIONAL_EXPRESSION)) {
ConditionalExpressionTree conditionalExpressionTree = (ConditionalExpressionTree) tree;
allowIfExceptionInExpression(conditionalExpressionTree.trueExpression(), allowIdentifier);
allowIfExceptionInExpression(conditionalExpressionTree.falseExpression(), allowIdentifier);
}
}
}
© 2015 - 2025 Weber Informatics LLC | Privacy Policy