第31条:用实例域代替序数

许多枚举天生就与一个单独的int值相关联。所有的枚举都有一个ordinal方法,它返回每个枚举常量在类型中的数字位置。你可以试着从序数中得到关联的int值:

// Abuse of ordinal to derive an associated value - DON'T DO THIS
public enum Ensemble {
    SOLO,   DUET,   TRIO,  QUARTET, QUINTET,
    SEXTET, SEPTET, OCTET, NONET,   DECTET;

    public int numberOfMusicians{ return ordinal() + 1; }
}

虽然这个枚举不错,但是维护起来就想一场恶梦。如果常量进行重新排序,numberOfMusicians方法就会遭到破坏。如果要再添加一个与已经用过的int值关联的枚举常量,就没那么走运了。例如,给双四重奏(double quartet)添加一个常量,它就像个八重奏一样,是由8位演奏家组成,但是没有办法做到。

要是没有给所有这些int值添加常量,也无法给某个int值添加常量。例如,假设想要添加一个常量表示三四重奏(triple quartet),它由12位演奏家组成。对于由11位演奏家组成的合奏曲并没有标准的术语,因此只好给没有用过的int值(11)添加一个虚拟(dummy)常量。这么做顶多就是不太好看。如果有许多int值都是从未用过的,可就不切实际了。

幸运的是,有一种很简单的方法可以解决这些问题。永远不要根据枚举的序数导出与它关联的值,而是要将它保存在一个实例域中

public enum Ensemble {
    SOLO(1), DUET(2), TRIO(3), QUARTET(4), QUINTET(5),
    SEXTET(6), SEPTET(7), OCTET(8), DOUBLE_QUARTET(8),
    NONET(9), DECTET(10), TRIPLE_QUARTET(12);

    private final int numberOfMusicians;
    Ensemble(int size) { this.numberOfMusicians = size; }
    public int numberOfMusicians{ return numberOfMusicians; }
}

Enum规范中谈到ordinal时这么写到:

“大多数程序员都不需要这个方法。它是设计成用于像EnumSetEnumMap这种基于枚举的通用数据结构的。”

除非你在编写的是这种数据结构,否则最好完全避免使用ordinal方法。

results matching ""

    No results matching ""