0

给定以下代码：

public interface DTMFKeys {

    public String getKeyStr();

    public static enum Sign implements TahoeDTMFKeys {

        SIGN_1("*1"), SIGN_2("*2"), SING_3("*3"), SIGN_4("*4"), SIGN_5("*5"), SIGN_6(
                "*6"), SIGN_7("*7"), SIGN_8("*8"), SIGN_9("*9");

        private String keyStr;

        private Sign(String keyStr) {
            this.keyStr = keyStr;
        }

        @Override
        public String getKeyStr() {
            return keyStr;
        }
    }

    public static enum Pound implements TahoeDTMFKeys {

        POUND_1("1#"), POUND_2("2#"), POUND_3("3#"), POUND_4("4#"), POUND_5(
                "5#"), POUND_6("6#"), POUND_7("7#"), POUND_8("8#"), POUND_9(
                "9#");

        private String keyStr;

        private Pound(String keyStr) {
            this.keyStr = keyStr;
        }

        @Override
        public String getKeyStr() {
            return keyStr;
        }
    }
}

如何删除重复的代码：

private Sign(String keyStr){
    this.keyStr=keyStr;
}

@Override
public String getKeyStr() {
    return keyStr;
}
4

2 回答 2

1

它的代码不多，所以我真的不会担心重构它。Java 枚举不能扩展其他类。避免代码中的重复实际上是关于重复大量复杂的逻辑，而不是 getter/setter 字段。

如果您真的担心“重复”，您可以完全取消枚举吗？

public static String getKeyString(int n, char sign) {
  if (n < 0 || n > 9) throw new IllegalArgumentException();
  if (sign == '#') {
    return ""+n+sign;
  else if (sign == '*') {
    return ""+sign+n;
  else {
    throw new IllegalAgumentException()
  }
}

此外，SING_3 拼写错误。

于 2013-02-01T02:23:34.237 回答
1

删除这些既不可能也不可取。因为枚举不能从抽象类继承，所以这是不可能的。keyStr对 Pound 和 Sign 类来说都是一个私有概念，它们在概念上是相互独立的。

于 2013-02-01T02:24:41.670 回答