blob: 4a74f05f3a8403f8c0d11a1281c5000781dedb29 [file] [log] [blame]
/*
* Copyright 2011-2013 Bas Leijdekkers
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.siyeh.ig.style;
import com.intellij.codeInspection.ProblemDescriptor;
import com.intellij.openapi.progress.ProcessCanceledException;
import com.intellij.openapi.project.Project;
import com.intellij.psi.*;
import com.intellij.psi.tree.IElementType;
import com.intellij.psi.util.PsiUtilCore;
import com.intellij.util.IncorrectOperationException;
import com.siyeh.InspectionGadgetsBundle;
import com.siyeh.ig.BaseInspection;
import com.siyeh.ig.BaseInspectionVisitor;
import com.siyeh.ig.InspectionGadgetsFix;
import com.siyeh.ig.PsiReplacementUtil;
import org.jetbrains.annotations.Nls;
import org.jetbrains.annotations.NonNls;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
public class UnclearBinaryExpressionInspection extends BaseInspection {
@Nls
@NotNull
@Override
public String getDisplayName() {
return InspectionGadgetsBundle.message("unclear.binary.expression.display.name");
}
@NotNull
@Override
protected String buildErrorString(Object... infos) {
return InspectionGadgetsBundle.message("unclear.binary.expression.problem.descriptor");
}
@Override
protected InspectionGadgetsFix buildFix(Object... infos) {
return new UnclearBinaryExpressionFix();
}
private static class UnclearBinaryExpressionFix extends InspectionGadgetsFix {
@Override
@NotNull
public String getFamilyName() {
return getName();
}
@NotNull
@Override
public String getName() {
return InspectionGadgetsBundle.message("unclear.binary.expression.quickfix");
}
@Override
protected void doFix(Project project, ProblemDescriptor descriptor) throws IncorrectOperationException {
final PsiElement element = descriptor.getPsiElement();
if (!(element instanceof PsiExpression)) {
return;
}
final PsiExpression expression = (PsiExpression)element;
final StringBuilder newExpressionText = createReplacementText(expression, new StringBuilder());
PsiReplacementUtil.replaceExpression(expression, newExpressionText.toString());
}
private static StringBuilder createReplacementText(@Nullable PsiExpression expression, StringBuilder out) {
if (expression instanceof PsiPolyadicExpression) {
final PsiPolyadicExpression polyadicExpression = (PsiPolyadicExpression)expression;
final IElementType tokenType = polyadicExpression.getOperationTokenType();
final PsiElement parent = expression.getParent();
if (parent instanceof PsiPolyadicExpression) {
final PsiPolyadicExpression parentPolyadicExpression = (PsiPolyadicExpression)parent;
final IElementType parentOperationSign = parentPolyadicExpression.getOperationTokenType();
final boolean parentheses = !tokenType.equals(parentOperationSign);
appendText(polyadicExpression, parentheses, out);
} else {
final boolean parentheses = parent instanceof PsiConditionalExpression || parent instanceof PsiInstanceOfExpression;
appendText(polyadicExpression, parentheses, out);
}
}
else if (expression instanceof PsiParenthesizedExpression) {
final PsiParenthesizedExpression parenthesizedExpression = (PsiParenthesizedExpression)expression;
final PsiExpression unwrappedExpression = parenthesizedExpression.getExpression();
final PsiElement parent = expression.getParent();
if (!(parent instanceof PsiParenthesizedExpression)) {
out.append('(');
createReplacementText(unwrappedExpression, out);
out.append(')');
}
else {
createReplacementText(unwrappedExpression, out);
}
}
else if (expression instanceof PsiInstanceOfExpression) {
final PsiInstanceOfExpression instanceofExpression = (PsiInstanceOfExpression)expression;
final PsiElement parent = expression.getParent();
final boolean parentheses = mightBeConfusingExpression(parent);
appendText(instanceofExpression, parentheses, out);
}
else if (expression instanceof PsiConditionalExpression) {
final PsiConditionalExpression conditionalExpression = (PsiConditionalExpression)expression;
final PsiElement parent = expression.getParent();
final boolean parentheses = mightBeConfusingExpression(parent);
appendText(conditionalExpression, parentheses, out);
}
else if (expression instanceof PsiAssignmentExpression) {
final PsiAssignmentExpression assignmentExpression = (PsiAssignmentExpression)expression;
final PsiElement parent = expression.getParent();
final boolean parentheses = mightBeConfusingExpression(parent) && !isSimpleAssignment(assignmentExpression, parent);
appendText(assignmentExpression, parentheses, out);
}
else if (expression != null) {
out.append(expression.getText());
}
return out;
}
private static boolean isSimpleAssignment(PsiAssignmentExpression assignmentExpression, PsiElement parent) {
if (!(parent instanceof PsiAssignmentExpression)) {
return false;
}
final PsiAssignmentExpression parentAssignmentExpression = (PsiAssignmentExpression)parent;
final IElementType parentTokenType = parentAssignmentExpression.getOperationTokenType();
final IElementType tokenType = assignmentExpression.getOperationTokenType();
return parentTokenType.equals(tokenType);
}
private static void appendText(PsiAssignmentExpression assignmentExpression, boolean parentheses, StringBuilder out) {
if (parentheses) {
out.append('(');
}
final PsiExpression lhs = assignmentExpression.getLExpression();
out.append(lhs.getText());
final PsiJavaToken sign = assignmentExpression.getOperationSign();
out.append(sign.getText());
final PsiExpression rhs = assignmentExpression.getRExpression();
createReplacementText(rhs, out);
if (parentheses) {
out.append(')');
}
}
private static void appendText(PsiInstanceOfExpression instanceofExpression, boolean parentheses, @NonNls StringBuilder out) {
if (parentheses) {
out.append('(');
}
final PsiExpression operand = instanceofExpression.getOperand();
createReplacementText(operand, out);
out.append(" instanceof ");
final PsiTypeElement checkType = instanceofExpression.getCheckType();
if (checkType != null) {
out.append(checkType.getText());
}
if (parentheses) {
out.append(')');
}
}
private static void appendText(PsiConditionalExpression conditionalExpression, boolean parentheses, StringBuilder out) {
if (parentheses) {
out.append('(');
}
final PsiExpression condition = conditionalExpression.getCondition();
createReplacementText(condition, out);
out.append('?');
final PsiExpression thenExpression = conditionalExpression.getThenExpression();
createReplacementText(thenExpression, out);
out.append(':');
final PsiExpression elseExpression = conditionalExpression.getElseExpression();
createReplacementText(elseExpression, out);
if (parentheses) {
out.append(')');
}
}
private static void appendText(PsiPolyadicExpression polyadicExpression, boolean parentheses, StringBuilder out) {
if (parentheses) {
out.append('(');
}
final PsiExpression[] operands = polyadicExpression.getOperands();
for (PsiExpression operand : operands) {
if (operand == null) {
continue;
}
if (operand.getType() == PsiType.VOID) {
throw new ProcessCanceledException();
}
if (operands.length == 1) {
createReplacementText(operand, out);
}
final PsiJavaToken token = polyadicExpression.getTokenBeforeOperand(operand);
if (token != null) {
final PsiElement beforeToken = token.getPrevSibling();
if (beforeToken instanceof PsiWhiteSpace) {
out.append(beforeToken.getText());
}
out.append(token.getText());
final PsiElement afterToken = token.getNextSibling();
if (afterToken instanceof PsiWhiteSpace) {
out.append(afterToken.getText());
}
}
if (operands.length != 1) {
createReplacementText(operand, out);
}
}
if (parentheses) {
out.append(')');
}
}
}
@Override
public BaseInspectionVisitor buildVisitor() {
return new UnclearBinaryExpressionVisitor();
}
private static class UnclearBinaryExpressionVisitor extends BaseInspectionVisitor {
@Override
public void visitPolyadicExpression(PsiPolyadicExpression expression) {
super.visitPolyadicExpression(expression);
final PsiElement parent = expression.getParent();
if (mightBeConfusingExpression(parent)) {
return;
}
final IElementType tokenType = expression.getOperationTokenType();
final PsiExpression[] operands = expression.getOperands();
for (PsiExpression operand : operands) {
if (operand instanceof PsiInstanceOfExpression) {
registerError(expression);
return;
}
if (!(operand instanceof PsiPolyadicExpression)) {
continue;
}
final PsiPolyadicExpression polyadicExpression = (PsiPolyadicExpression)operand;
final IElementType childTokenType = polyadicExpression.getOperationTokenType();
if (!tokenType.equals(childTokenType)) {
registerError(expression);
return;
}
}
}
@Override
public void visitConditionalExpression(PsiConditionalExpression expression) {
super.visitConditionalExpression(expression);
final PsiElement parent = expression.getParent();
if (mightBeConfusingExpression(parent) || PsiUtilCore.hasErrorElementChild(expression)) {
return;
}
final PsiExpression condition = expression.getCondition();
final PsiExpression thenExpression = expression.getThenExpression();
final PsiExpression elseExpression = expression.getElseExpression();
if (!mightBeConfusingExpression(condition) && !mightBeConfusingExpression(thenExpression) &&
!mightBeConfusingExpression(elseExpression)) {
return;
}
registerError(expression);
}
@Override
public void visitInstanceOfExpression(PsiInstanceOfExpression expression) {
super.visitInstanceOfExpression(expression);
final PsiElement parent = expression.getParent();
if (mightBeConfusingExpression(parent)) {
return;
}
final PsiExpression operand = expression.getOperand();
if (!mightBeConfusingExpression(operand)) {
return;
}
registerError(expression);
}
@Override
public void visitAssignmentExpression(PsiAssignmentExpression expression) {
super.visitAssignmentExpression(expression);
final PsiElement parent = expression.getParent();
if (mightBeConfusingExpression(parent)) {
return;
}
final PsiExpression rhs = expression.getRExpression();
if (!mightBeConfusingExpression(rhs)) {
return;
}
if (!(rhs instanceof PsiAssignmentExpression)) {
return;
}
final PsiAssignmentExpression nestedAssignment = (PsiAssignmentExpression)rhs;
final IElementType nestedTokenType = nestedAssignment.getOperationTokenType();
final IElementType tokenType = expression.getOperationTokenType();
if (nestedTokenType.equals(tokenType)) {
return;
}
registerError(expression);
}
}
static boolean mightBeConfusingExpression(@Nullable PsiElement element) {
return element instanceof PsiPolyadicExpression || element instanceof PsiConditionalExpression ||
element instanceof PsiInstanceOfExpression || element instanceof PsiAssignmentExpression;
}
}