@Override public void visitIfStatement(IfStatementTree tree) { statements++; super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { statements++; super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { statements++; super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { statements++; super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { blame.add(tree.firstToken()); super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { blame.add(tree.firstToken()); super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { if (!outerIf.isEmpty() && !hasElseClause(tree)) { context.reportIssue(this, tree.ifKeyword(), "Merge this if statement with the enclosing one.", Lists.newArrayList(new JavaFileScannerContext.Location("", outerIf.peek().ifKeyword())), null); } if (!hasElseClause(tree) && hasBodySingleIfStatement(tree.thenStatement())) { // children of this if statement are eligible for issues outerIf.push(tree); // recurse into sub-tree super.visitIfStatement(tree); if (!outerIf.isEmpty()) { outerIf.pop(); } } else { // direct children of this if statement not eligible for issues. Reset nesting count outerIf.clear(); super.visitIfStatement(tree); } }
@Override public void visitIfStatement(IfStatementTree tree) { if (!outerIf.isEmpty() && !hasElseClause(tree)) { context.reportIssue(this, tree.ifKeyword(), "Merge this if statement with the enclosing one.", Lists.newArrayList(new JavaFileScannerContext.Location("", outerIf.peek().ifKeyword())), null); } if (!hasElseClause(tree) && hasBodySingleIfStatement(tree.thenStatement())) { // children of this if statement are eligible for issues outerIf.push(tree); // recurse into sub-tree super.visitIfStatement(tree); if (!outerIf.isEmpty()) { outerIf.pop(); } } else { // direct children of this if statement not eligible for issues. Reset nesting count outerIf.clear(); super.visitIfStatement(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) { 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) { 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) { 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) { 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) { addKind(tree.condition(), UastNode.Kind.CONDITION); addKind(tree.thenStatement(), UastNode.Kind.THEN); StatementTree elseStatement = tree.elseStatement(); if (elseStatement != null) { addKind(elseStatement, UastNode.Kind.ELSE); } super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { checkBlock(tree.closeParenToken(), tree.thenStatement()); if (tree.elseKeyword() != null) { checkBlock(tree.elseKeyword(), tree.elseStatement()); } super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { checkBlock(tree.closeParenToken(), tree.thenStatement()); if (tree.elseKeyword() != null) { checkBlock(tree.elseKeyword(), tree.elseStatement()); } super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { insideIfStatement = true; if (isResourceNullCheckOfType(tree, NOT_EQUAL_TO.name())) { updateNullCheckedResources(tree, true); } else if (isResourceNullCheckOfType(tree, EQUAL_TO.name())) { insideEqualNullCheck = true; } super.visitIfStatement(tree); if (returnOccurredInsideEqualNullCheck) { updateNullCheckedResources(tree, true); } cleanFlagsAfterIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { Tree parentTree = tree.parent(); if (parentTree.is(Tree.Kind.IF_STATEMENT) && tree.equals(((IfStatementTree) parentTree).elseStatement())) { checkForReport(tree.thenStatement(), ((IfStatementTree) parentTree).elseKeyword(), tree.closeParenToken(), tree.ifKeyword().firstToken().text()); } else { checkForReport(tree.thenStatement(), tree.ifKeyword(), tree.closeParenToken(), tree.ifKeyword().text()); } SyntaxToken elseKeyword = tree.elseKeyword(); if (elseKeyword != null && !tree.elseStatement().is(Tree.Kind.IF_STATEMENT)) { checkForReport(tree.elseStatement(), elseKeyword, elseKeyword, elseKeyword.firstToken().text()); } super.visitIfStatement(tree); }
@Override public void visitIfStatement(IfStatementTree tree) { Tree parentTree = tree.parent(); if (parentTree.is(Tree.Kind.IF_STATEMENT) && tree.equals(((IfStatementTree) parentTree).elseStatement())) { checkForReport(tree.thenStatement(), ((IfStatementTree) parentTree).elseKeyword(), tree.closeParenToken(), tree.ifKeyword().firstToken().text()); } else { checkForReport(tree.thenStatement(), tree.ifKeyword(), tree.closeParenToken(), tree.ifKeyword().text()); } SyntaxToken elseKeyword = tree.elseKeyword(); if (elseKeyword != null && !tree.elseStatement().is(Tree.Kind.IF_STATEMENT)) { checkForReport(tree.elseStatement(), elseKeyword, elseKeyword, elseKeyword.firstToken().text()); } super.visitIfStatement(tree); }