使用は、

Question

Iは気づくことJava Enum Documentation状態序方法：

この列挙定数の序数（初期定数の序数が割り当てられ、その列挙宣言、その位置を返しゼロ）。ほとんどのプログラマーはこのメソッドを使用しません。 EnumSetやEnumMapなどの洗練されたenumベースのデータ構造で使用するように設計されています。

は、私は配列のインデックスへordinalを使用しないようお勧めオンラインすべての例を理解する代わりにEnumMap。特にItem 33 of Effective Java 私の質問です：それは私のEnumの定義内で使用するのは大丈夫ですか？

public enum Direction { 
    NORTH(0, 1), NORTH_EAST(1, 1), EAST(1, 0), SOUTH_EAST(1, -1), 
    SOUTH(0, -1), SOUTH_WEST(-1, 1), WEST(-1, 0), NORTH_WEST(-1, 1); 

    private final int xOffset; 
    private final int yOffset; 

    private final static int DEGREES = 360; 

    private Direction(int xOffset, int yOffset) { 
    this.xOffset = xOffset; 
    this.yOffset = yOffset; 
    } 

    public Position move(Position position) { 
    return new Position(position.getX() + xOffset, position.getY() + yOffset); 
    } 

    public Direction rotate(int degrees) { 
    int length = Direction.values().length; 
    int index = (ordinal() + (degrees/(DEGREES/length))) % length; 
    return Direction.values()[index]; 
    } 
} 

あなたは私が序（）を使用しています見ることができるように私ができるように、方向を循環（および次の関連する方向を返す）： は、たとえば私のコードは次のとおりです。例えば。 NORTHから90度回転すると、EASTが返されます。

しかし、序数を使用するのは良い方法ではないことがわかっているので、コードサイズを小さくして可読性を維持しながら、これを行うより良い方法があるかどうかは疑問でした。

アドバイスをいただきありがとうございます。

Answer 1

私はこれが完全に受け入れられるordinalだと言います。実際、それは私の意見では、非常にきれいでシンプルです。

あなたenumコードでordinalを使用しないことについてのコメントは、ほとんどの時間は、あなたの代わりにenum自体またはEnumMapを使用することができますので、そうすることからあなたを思いとどまらせるだけです。

EffectiveJavaへの参照は、配列に対してオフセットとして使用する正しいintを選択するのが難しく、一般的に不要なので、これを行うことを警告しています。あなたのケースでは、a）あなたはそれを正しくやっているので、b）それはあなたが探している機能を達成するための最も簡単な方法ですので、適用されません。

Answer 2

Enum定義で使用するのは問題ありませんか？

大丈夫です。しかし、理由はよくあなたが提供されたリンクに記載されてそれを使用するとEnumMapのために行くことはない

このプログラム作品（それはそれで説明した例に言及していたとして引用は変更部分だ）とさえエレガント表示される場合があります外見は欺くことができます。コンパイラのように、序数と配列インデックスの間の関係を知る方法はありません。変更を加えてそれに応じて更新することを忘れると、実行時にプログラムが失敗します。

EnumMapは、あなたのデザインにあなたのデザインに含めると非常に効率的です。

Answer 3

WEST_FAR_AWAY（-10、0）のように、今後誰かが新しいメンバーを導入すると、コードが破損することがあります。アプリケーションの要件に応じて、これが「マイナーで許容可能な欠陥」か「メンテナンスの悪夢」かどうかを評価する必要があります。