Effective Java 31 Use instance fields instead of ordinals
2014-03-25 19:40 小郝(Kaibo Hao) 阅读(447) 评论(0) 编辑 收藏 举报Principle
Never derive a value associated with an enum from its ordinal; store it in an instance field instead.
Bad practice Demo
// 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;}
}
// The right way
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; }
}
Disadvantages
- If the constants are reordered, the numberOfMusicians method will break.
- If you want to add a second enum constant associated with an int value that you've already used, you're out of luck.
- You can't add a constant for an int value without adding constants for all intervening int values.
Summary
The Enum specification has this to say about ordinal: "Most programmers will have no use for this method. It is designed for use by general-purpose enum based data structures such as EnumSet and EnumMap." Unless you are writing such a data structure, you are best off avoiding the ordinal method entirely.
出处:http://www.cnblogs.com/haokaibo/
本文版权归作者和博客园共有,欢迎转载,但未经作者同意必须保留此段声明,且在文章页面明显位置给出原文连接,否则保留追究法律责任的权利。