@Override public void visitNode(Tree tree) { ExpressionTree condition = ((IfStatementTree) tree).condition(); if (condition.is(Tree.Kind.BOOLEAN_LITERAL)) { reportIssue(condition, "Remove this if statement."); } }
@Override public void visitNode(Tree tree) { ExpressionTree condition = ((IfStatementTree) tree).condition(); if (condition.is(Tree.Kind.BOOLEAN_LITERAL)) { reportIssue(condition, "Remove this if statement."); } }
@Override public void visitNode(Tree tree) { if (((IfStatementTree)tree).condition().is(Tree.Kind.BOOLEAN_LITERAL)) { addIssue(tree, "Remove this if statement."); } }
@Override public void visitIfStatement(IfStatementTree tree) { super.visitIfStatement(tree); ExpressionTree condition = tree.condition(); if (condition.is(Tree.Kind.ASSIGNMENT) && EQUALITY_RELATIONAL_OPERATORS.contains(getKind(((AssignmentExpressionTree) condition).expression()))) { raiseIssue(tree); } }
@Override public void visitIfStatement(IfStatementTree ifTree) { LevelTestVisitor levelTestVisitor = new LevelTestVisitor(); ifTree.condition().accept(levelTestVisitor); if (levelTestVisitor.match) { stackAndContinue(ifTree, super::visitIfStatement); } else { super.visitIfStatement(ifTree); } }
@Override public void visitIfStatement(IfStatementTree tree) { scan(tree.condition()); scan(tree.thenStatement()); scan(tree.elseStatement()); }
@Override public void visitIfStatement(IfStatementTree tree) { super.visitIfStatement(tree); ExpressionTree condition = tree.condition(); if (condition.is(Tree.Kind.ASSIGNMENT) && EQUALITY_RELATIONAL_OPERATORS.contains(((AssignmentExpressionTree) condition).expression().kind())) { raiseIssue(((AssignmentExpressionTree) condition).operatorToken().line(), tree); } }
@Override public void visitIfStatement(IfStatementTree tree) { scan(tree.condition()); scan(tree.thenStatement()); scan(tree.elseStatement()); }
@Override public void visitIfStatement(IfStatementTree tree) { scan(tree.condition()); scan(tree.thenStatement()); scan(tree.elseStatement()); }
@Override public void visitIfStatement(IfStatementTree tree) { super.visitIfStatement(tree); ExpressionTree condition = tree.condition(); if (condition.is(Tree.Kind.ASSIGNMENT) && EQUALITY_RELATIONAL_OPERATORS.contains(((AssignmentExpressionTree) condition).expression().kind())) { raiseIssue(((AssignmentExpressionTree) condition).operatorToken().line(), tree); } }
@Override public void visitIfStatement(IfStatementTree tree) { scan(tree.condition()); scan(tree.thenStatement()); scan(tree.elseStatement()); }
@Override public void visitIfStatement(IfStatementTree tree) { ExpressionVisitor expressionVisitor = new ExpressionVisitor(parameterSymbol); tree.condition().accept(expressionVisitor); if (expressionVisitor.typeChecked) { typeChecked = true; return; } scan(tree.thenStatement()); scan(tree.elseStatement()); }
private static boolean isInsideIfStatementWithNullCheckWithoutElse(MethodInvocationTree mit) { Tree parent = mit.parent(); while (parent != null && !parent.is(Tree.Kind.IF_STATEMENT)) { parent = parent.parent(); } if (parent == null) { return false; } IfStatementTree ifStatementTree = (IfStatementTree) parent; return ifStatementTree.elseStatement() == null && isNullCheck(ExpressionUtils.skipParentheses(ifStatementTree.condition())); }
private static boolean isInsideIfStatementWithNullCheckWithoutElse(MethodInvocationTree mit) { Tree parent = mit.parent(); while (parent != null && !parent.is(Tree.Kind.IF_STATEMENT)) { parent = parent.parent(); } if (parent == null) { return false; } IfStatementTree ifStatementTree = (IfStatementTree) parent; return ifStatementTree.elseStatement() == null && isNullCheck(ExpressionUtils.skipParentheses(ifStatementTree.condition())); }
private void visit(IfStatementTree tree) { scan(tree.condition()); scan(tree.thenStatement()); StatementTree elseStatementTree = tree.elseStatement(); if (elseStatementTree != null && elseStatementTree.is(Tree.Kind.IF_STATEMENT)) { visit((IfStatementTree) elseStatementTree); } else { scan(elseStatementTree); } }
private void visit(IfStatementTree tree) { scan(tree.condition()); scan(tree.thenStatement()); StatementTree elseStatementTree = tree.elseStatement(); if (elseStatementTree != null && elseStatementTree.is(Tree.Kind.IF_STATEMENT)) { visit((IfStatementTree) elseStatementTree); } else { scan(elseStatementTree); } }
@Override public void visitIfStatement(IfStatementTree tree) { ConditionalState conditionalState = visitCondition(tree.condition()); currentState = conditionalState.trueState; tree.thenStatement().accept(this); if (tree.elseStatement() != null) { currentState = conditionalState.falseState; tree.elseStatement().accept(this); } currentState = currentState.parentState.mergeValues(conditionalState.trueState, conditionalState.falseState); }
private void visit(IfStatementTree tree) { scan(tree.condition()); scan(tree.thenStatement()); StatementTree elseStatementTree = tree.elseStatement(); if (elseStatementTree != null && elseStatementTree.is(Tree.Kind.IF_STATEMENT)) { visit((IfStatementTree) elseStatementTree); } else { scan(elseStatementTree); } }