PMD UselessParentheses vi phạm

Question

tôi có phương pháp Java sau:

private int calculate() { 
    return (bytes[0] & 0xff) + ((bytes[1] & 0xff) << 8); 
} 

PMD phàn nàn về mã này với "UselessParentheses" vi phạm.

Tôi đã xem xét operator precentence rules và tôi vẫn không thấy dấu ngoặc đơn thừa trong mã đó. Tui bỏ lỡ điều gì vậy?

Answer 1

Không có ngoặc không cần thiết trong mã này, như bạn có thể thấy nếu bạn chạy này:

 byte [] bytes = new byte[] {1,2}; 

     System.out.println((bytes[0] & 0xff) + ((bytes[1] & 0xff) << 8)); 
     System.out.println(bytes[0] & 0xff + ((bytes[1] & 0xff) << 8)); 
     System.out.println((bytes[0] & 0xff) + (bytes[1] & 0xff) << 8); 
     System.out.println((bytes[0] & 0xff) + (bytes[1] & 0xff << 8)); 

Hơn nữa, đôi khi nó thực sự tốt để thêm dấu ngoặc thêm cho dễ đọc. Ví dụ:

int i = x << y + z; // this will shift x by y+z bits 
int j = x << (y + z); // equivalent, but more readable 

Answer 2

Sau khi đọc các ưu đãi nhà điều hành, các dòng mã, và cảnh báo PMD, đây có lẽ là một trong những trường hợp hiếm hoi nơi ưu tiên có nghĩa là để được áp dụng như

PMD complains on this code with a useless (parenthesis warning) 

hơn

PMD complains on this code with a (useless parenthesis) warning. 

Mã của bạn là đúng và dấu ngoặc đơn không thừa. Loại bỏ chúng sẽ làm cho mã ít đọc được, và mỗi một trong số chúng là cần thiết. Trong thực tế, toàn bộ vấn đề này là xứng đáng với một xkcd comic