com.xenoamess.p3c.pmd.lang.java.rule.constant.UndefineMagicConstantRule Maven / Gradle / Ivy

Go to download
Show more of this group Show more artifacts with this name
Show all versions of p3c-pmd Show documentation
Alibaba Java Coding Guidelines PMD implementations(XenoAmess TPM)
There is a newer version: 2.2.1.0x
/*
 * Copyright 1999-2017 Alibaba Group.
 *
 * 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.xenoamess.p3c.pmd.lang.java.rule.constant;

import com.xenoamess.p3c.pmd.lang.java.rule.AbstractAliRule;
import com.xenoamess.p3c.pmd.lang.java.util.namelist.NameListConfig;
import net.sourceforge.pmd.lang.ast.Node;
import net.sourceforge.pmd.lang.java.ast.ASTCompilationUnit;
import net.sourceforge.pmd.lang.java.ast.ASTForStatement;
import net.sourceforge.pmd.lang.java.ast.ASTIfStatement;
import net.sourceforge.pmd.lang.java.ast.ASTLiteral;
import net.sourceforge.pmd.lang.java.ast.ASTWhileStatement;
import org.apache.commons.lang3.StringUtils;
import org.jaxen.JaxenException;

import java.util.ArrayList;
import java.util.List;

/**
 * [Mandatory] Magic values, except for predefined, are forbidden in coding.
 *
 * @author shengfang.gsf
 * @date 2016/12/13
 */
public class UndefineMagicConstantRule extends AbstractAliRule {

    /**
     * white list for undefined variable, may be added
     * @return UndefineMagicConstantRule LITERAL_WHITE_LIST
     */
    private static List getLiteralWhiteList() {
        return NameListConfig.getNameListService().getNameList(
                UndefineMagicConstantRule.class.getSimpleName(),
                "LITERAL_WHITE_LIST"
        );
    }

    private static final String XPATH = "//Literal/../../../../..[not(VariableInitializer)]";

    /**
     * An undefined that belongs to non-looped if statements
     *
     * @param node compilation unit
     * @param data rule context
     */
    @Override
    public Object visit(ASTCompilationUnit node, Object data) {
        // removed repeat magic value , to prevent the parent class to find sub-variable nodes when there is a repeat
        List currentLiterals = new ArrayList<>();
        try {
            // Find the parent node of the undefined variable
            List parentNodes = node.findChildNodesWithXPath(XPATH);

            for (Node parentItem : parentNodes) {
                List literals = parentItem.findDescendantsOfType(ASTLiteral.class);
                for (ASTLiteral literal : literals) {
                    if (inBlackList(literal) && !currentLiterals.contains(literal)) {
                        currentLiterals.add(literal);
                        String imageReplace = StringUtils.replace(literal.getImage(), "{", "'{");
                        addViolationWithMessage(data, literal,
                                "java.constant.UndefineMagicConstantRule.violation.msg", new Object[]{imageReplace});
                    }
                }
            }
        } catch (JaxenException e) {
            e.printStackTrace();
        }
        return super.visit(node, data);
    }

    /**
     * Undefined variables are in the blacklist
     *
     * @param literal literal
     * @return ifInBlackList
     */
    private boolean inBlackList(ASTLiteral literal) {
        String name = literal.getImage();
        int lineNum = literal.getBeginLine();
        // name is null,bool literal，belongs to white list
        if (name == null) {
            return false;
        }
        // filter white list
        for (String whiteItem : getLiteralWhiteList()) {
            if (whiteItem.equals(name)) {
                return false;
            }
        }
        ASTIfStatement ifStatement = literal.getFirstParentOfType(ASTIfStatement.class);
        if (ifStatement != null && lineNum == ifStatement.getBeginLine()) {
            ASTForStatement forStatement = ifStatement.getFirstParentOfType(ASTForStatement.class);
            ASTWhileStatement whileStatement = ifStatement.getFirstParentOfType(ASTWhileStatement.class);
            return forStatement == null && whileStatement == null;
        }

        // judge magic value belongs to  for statement
        ASTForStatement blackForStatement = literal.getFirstParentOfType(ASTForStatement.class);
        if (blackForStatement != null && lineNum == blackForStatement.getBeginLine()) {
            return true;
        }

        // judge magic value belongs to while statement
        ASTWhileStatement blackWhileStatement = literal.getFirstParentOfType(ASTWhileStatement.class);
        return blackWhileStatement != null && lineNum == blackWhileStatement.getBeginLine();
    }
}