Question

我想改进我对JDK 1.5的使用并停止使用私有静态final String而不是enum。这似乎是建议的。

但现在我的常数课程看起来像这样：

public class CEnum{
    /**
     * @author JJA
     * date : 20/10/2010
     */
    public enum ListTypeAffichage {
        DEP("DEPOT_TVA"), PAD("PAS_DEPOT_TVA"), NORM("CAT_NORMALE"), CAP("CAT_PARTICULIERE"), CAA("CAT_AUTRE_CAS");

        private final String sName;

        /**
         * @param name String
         */    
        ListTypeAffichage(String name) {
            this.sName = name;
        }

        /**
         * @return String
         */
        public String getType() {
            return sName;
        }        
    }

    /**
     * @author JJA
     * date : 20/10/2010
     */
    public enum ListTypeCategorie {
        DEDUIRE("SOMME_A_DEDUIRE"), AJOUTER("SOMME_A_AJOUTER");

        private final String sName;

        /**
         * @param name String
         */
        ListTypeCategorie(String name) {
            this.sName = name;
        }

        /**
         * @return String
         */
        public String getType() {
            return sName;
        }    
    }

    /**
     * @author JJA
     * date : 26/10/2010
     */
    public enum ListInterval {
        POS("POSITIF"), NS("NON_SIGNE");

        private final String sName;

        /**
         * @param name String
         */
        ListInterval(String name) {
            this.sName = name;
        }

        /**
         * @return String
         */
        public String getInterval() {
            return sName;
        }    
    }
}

而不是

public class ConstantesADMD {
    public static final List<String> typeAffich = new ArrayList<String>();
...
    ConstantesADMD(){
        typeAffich.add("DEPOT_TVA");
        typeAffich.add("PAS_DEPOT_TVA");
        typeAffich.add("CAT_NORMALE"); 

...       
    }
}

我的代码似乎非常糟糕，但至少有效。对于每个枚举，我必须添加冗余代码：

private final String sName;

/**
 * @param name String
 */    
ListTypeAffichage(String name) {
    this.sName = name;
}

/**
 * @return String
 */
public String getType() {
    return sName;
}

你建议我改进什么？注意：忘记我的第一个问题的最后一句话，我需要索引。告诉我如果我必须发布另一个问题，编辑我的fisrt似乎更容易。

Answer 1

我会命名我的枚举常量，因为你已经命名了你的字符串。然后，您可以使用Enum.toString()方法访问该名称。例如：

public enum ListTypeAffichage {
    DEPOT_TVA, PAS_DEPOT_TVA, CAT_NORMALE, CAT_PARTICULIERE, CAT_AUTRE_CAS;

    /**
     * @return String
     */
    public String getType() {
        return toString();
    }        
}

您当然也可以同时跳过getType()所有内容，然后使用toString()访问“类型”：

ListTypeAffichage myEnum = ListTypeAffichage.CAT_PARTICULIRE;
System.out.println("Type: " + myEnum.toString());             // like this...
System.out.println("Type: " + myEnum);                        // ...or like this

根据API，这比直接使用Enum.name()更好：

public final String name()
[...] 大多数程序员应优先使用toString（）方法 [...]

Answer 2

每个枚举都有一个name（）方法，它返回常量的精确字符串表示形式。所以你可以这样做：

 public enum ListTypeAffichage {
        DEPOT_TVA, PAS_DEPOT_TVA, CAT_NORMALE, CAT_PARTICULIERE, CAT_AUTRE_CAS                  
    }

然后

ListTypeAffichage.DEPOT_TVA.name();

Answer 3

通过使用缩写（DEP，PAD，NORM）等，您创建了别名（DEPOT_TVA，PAS_DEPOT_TVA，CAT_NORMALE）等。

如果你想保留缩写，那么你必须保持枚举。

如果您愿意取消缩写，那么您可以更改枚举如下所示，我在枚举中包含了一个主要方法来演示它的用法。

public enum ListTypeAffichageNames {
    DEPOT_TVA,
    PAS_DEPOT_TVA,
    CAT_NORMALE,
    CAT_PARTICULIERE,
    CAT_AUTRE_CAS;


    public static void main(String[] args) {
        System.out.println(DEPOT_TVA.toString());
        ListTypeAffichageNames affichage = ListTypeAffichageNames.valueOf("DEPOT_TVA");
        System.out.println(affichage.toString());
    }
}

除此之外，您当前的常量结构不会为您提供编译时类型检查，并且不会阻止在运行时发生以下情况：

    ConstantesADMD.typeAffich.clear();
    // or
    ConstantesADMD.typeAffich.remove("DEPOT_TVA");
    ConstantesADMD.typeAffich.add("dEpOt-tVa");

Java枚举在实践中，糟糕的代码需要改进

3 个答案: