PMD UselessParentheses naruszenie

Question

Mam następujący sposób Java:

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

PMD narzeka na ten kod z naruszeniem "UselessParentheses".

Sprawdziłem operator precentence rules i nadal nie widzę nadmiarowych nawiasów w tym kodzie. Czy czegoś brakuje?

Answer 1

Nie ma niepotrzebnych nawias w tym kodzie, jak widać, jeśli uruchomić ten:

 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)); 

Ponadto, czasami jest to rzeczywiście dobry, aby dodać dodatkowe nawiasy dla czytelność. Np

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

Answer 2

Po przeczytaniu preferencji operatora, wiersz kodu oraz ostrzeżenie PMD, jest to prawdopodobnie jeden z tych rzadkich przypadkach, gdy pierwszeństwo ma być stosowana jak

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

zamiast

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

Masz kodeks, a nawias nie jest zbyteczny. Usunięcie ich spowodowałoby, że kod byłby mniej czytelny, a każdy z nich jest potrzebny. W rzeczywistości, ta cała sprawa jest godna xkcd comic