Analizzatore Java SonarQube, regola S2183, perché dovrei rimuovere questo spostamento inutile?

Question

mi hanno scritto:

public static final int MY_GREAT_COLOR = (91 << 16) + (155 << 8) + 213 + (255 << 32); 

e Sonar dice:

Nuovo: Calamaro: S2183 Gravità: Critica, Messaggio: Rimuovere questo spostamento inutile

Qualcuno potrebbe dirmi perché? È solo il fatto che non dovrebbero esserci calcoli sui letterali, anche se si aggiunge alla leggibilità?

Answer 1

Immagino che 255 sia il tuo canale alfa, in tal caso suppongo che tu voglia 255 << 24.

SonarQube non conosce le tue intenzioni, ma ha un punto valido, come dichiarato da Toby.

Per quanto riguarda la leggibilità, è anche possibile provare il codice hexa (ad esempio 0xFF5B9BD5), molti sviluppatori sono già abituati. Naturalmente, un numero decimale sarebbe totalmente illeggibile e brutto.

Answer 2

255 << 32 overflow Java (32 bit) int, che diventa zero.

Vuoi un long e 255L << 32?