Toán tử điều kiện trong câu lệnh if?

Question

Tôi đã viết những if -statement sau trong Java:

if(methodName.equals("set" + this.name) || 
    isBoolean() ? methodName.equals("is" + this.name) : 
        methodName.equals("get" + this.name)) { 
    ... 
} 

Đây có phải là một thực hành tốt để viết các biểu thức như vậy trong if, tách nhà nước từ điều kiện? Và biểu thức này có thể được đơn giản hóa không?

Answer 1

tôi sẽ thay đổi nó để

if (methodName.equals("set" + this.name) 
|| methodName.equals((isBoolean() ? "is" : "get") + this.name)) { 
    ... 
} 

Answer 2

nó là thực hành tốt? Sẽ tốt nếu nó dễ đọc hơn. Nó làm cho nó dễ dàng hơn để đọc nếu (1) nó và (2) loại người sẽ bị nhầm lẫn bởi nó sẽ không được đọc nó. Ai sẽ đọc nó?

Answer 3

Chẳng phải công việc như sau?

if (methodName.equals("set" + this.name) 
    || methodName.equals("get" + this.name) 
    || (isBoolean() && methodName.equals("is" + this.name))) { 
    ... 
} 

Dễ đọc hơn cách bạn sử dụng toán tử bậc ba và dễ hiểu hơn. Nó cũng có lợi thế là nó có thể tránh một cuộc gọi phương thức không cần thiết đến phương thức isBoolean (nó có các cuộc gọi phương thức 1, 2 hoặc 4 trong khi bạn luôn có 1 hoặc 3; hiệu suất/lỗ có thể quá ít phút để chú ý).

Cũng có một câu hỏi tương tự ở đây có tựa đề "Is this a reasonable use of the ternary operator?" Một người dùng có the following to say:

Nhà điều hành ternary có nghĩa là để trở lại một giá trị.

IMO, không được biến đổi trạng thái và giá trị trả lại sẽ được sử dụng.

Trong trường hợp khác, hãy sử dụng nếu câu lệnh. Nếu báo cáo có nghĩa là để thực hiện khối mã.

Lưu ý rằng tôi bao gồm dấu ngoặc đơn xung quanh biểu thức có chứa '& &' để dễ đọc. Chúng không cần thiết vì x && y được đánh giá trước m || n.

Cho dù bạn chọn sử dụng tùy thuộc vào bạn, nhưng tôi có xu hướng tránh nó có lợi cho khả năng đọc.

Answer 4

Tôi xin được nghiêng để thay đổi nó để

if (methodName.equals(setterForThis()) 
    || methodName.equals(getterForThis())) { 
    ... 
} 

với một số chức năng trích xuất:

private String setterForThis() { 
    return "set" + this.name; 
} 

private String getterForThis() { 
    return (isBoolean() ? "is" : "get") + this.name; 
} 

Đó là còn tất nhiên, nhưng tôi không thực sự vào sân anyway.