Cách tốt nhất để triển khai `next` và` previous` trên một kiểu enum là gì?

Question

Giả sử tôi có một enum:

enum E { 
    A, B, C; 
} 

Như đã trình bày trong this answer bởi lucasmo, các giá trị enum được lưu trữ trong một mảng tĩnh theo thứ tự mà chúng được khởi tạo, và sau đó bạn có thể lấy (một bản sao của) mảng này với E.values().

Bây giờ giả sử tôi muốn thực hiện E#getNext và E#getPrevious như rằng tất cả các từ ngữ dưới đây đánh giá để true:

E.A.getNext() == E.B 
E.B.getNext() == E.C 
E.C.getNext() == E.A 

E.A.getPrevious() == E.C 
E.B.getPrevious() == E.A 
E.C.getPrevious() == E.B 

thực hiện hiện tại của tôi cho getNext như sau:

public E getNext() { 
    E[] e = E.values(); 
    int i = 0; 
    for (; e[i] != this; i++) 
     ; 
    i++; 
    i %= e.length; 
    return e[i]; 
} 

và một tương tự phương thức cho getPrevious.

Tuy nhiên, mã này có vẻ rườm rà lúc tốt nhất (ví dụ, "trống rỗng" for vòng lặp, lạm dụng gây tranh cãi của một biến đếm, và có khả năng sai lầm lúc tồi tệ nhất (nghĩ suy, có thể).

Điều gì sẽ là cách tốt nhất để thực hiện getNext và getPrevious phương pháp với nhiều loại enum trong Java 7

---

LƯU ý: tôi làm không có ý định câu hỏi này là chủ quan theo yêu cầu của tôi cho implementa "tốt nhất". tion là viết tắt của yêu cầu cho việc thực hiện đó là nhanh nhất, sạch nhất, và duy trì nhất.

Answer 1

Hãy thử điều này:

public static enum A { 
    X, Y, Z; 
    private static A[] vals = values(); 
    public A next() 
    { 
     return vals[(this.ordinal()+1) % vals.length]; 
    } 

Thực hiện previous() là trái như một bài tập, nhưng nhớ lại rằng in Java, the modulo a % b can return a negative number.

EDIT: Như được đề xuất, tạo bản sao tĩnh riêng tư của mảng values() để tránh sao chép mảng mỗi lần next() hoặc previous() được gọi.

Answer 2

Ngoài ra, người ta có thể đi bằng cách nào đó dọc theo dòng của ý tưởng sau đây:

public enum SomeEnum { 
    A, B, C; 

    public Optional<SomeEnum> next() { 
    switch (this) { 
     case A: return Optional.of(B); 
     case B: return Optional.of(C); 
     // any other case can NOT be mapped! 
     default: return Optional.empty(); 
    } 
} 

Ghi chú:

1. Ngược lại với câu trả lời khác, cách này hiện một số ngầm mapping; thay vì dựa vào ordinal(). Tất nhiên điều đó có nghĩa là nhiều mã hơn; nhưng nó cũng buộc tác giả phải xem xét ý nghĩa của việc thêm các hằng số mới hoặc xóa các hằng số hiện tại. Khi dựa vào thứ tự, giả định ngụ ý tiềm ẩn của bạn là đơn đặt hàng được dựa trên thứ tự được sử dụng cho khai báo hằng số enum. Vì vậy, khi ai đó trở lại 6 tháng sau đó và phải thêm một hằng số mới, anh ta phải hiểu rằng hằng số Y mới cần X, Y, Z ... thay vì chỉ phụ thêm X, Z, Y!
2. Có thể có những trường hợp không có ý nghĩa gì đối với hằng số enum "cuối cùng" để có chữ "đầu tiên" làm người kế thừa. Hãy suy nghĩ về kích cỡ áo phông cho các ví dụ. XXL.next() chắc chắn không phải XS. Đối với các tình huống như vậy, sử dụng Tùy chọn là câu trả lời phù hợp hơn.