SonarQube: Nel caso in numeri magici essere ammessi in enumerazioni costruttore java

Question

Relativamente alla Regola calamari: i numeri magici S109 non devono essere utilizzati

non dovrebbe essere permesso di avere i numeri nel costruttore di un enum in Java? Il seguente codice non dovrebbe violare la regola a mio parere.

public enum Color{ 
    RED(42), 
    GREEN(123456), 
    BLUE(666); 

    public final int code; 

    Color(int colorCode){ 
     this.code=colorCode; 
    }  
} 

Sto utilizzando la versione Sonar Java Plugin 3.3

Answer 1

Sarà fissato in versione 3.4

Vedere questo problema in SonarSource: http://jira.sonarsource.com/browse/SONARJAVA-1117

Answer 2

Tecnicamente, sì, non c'è niente di sbagliato con il codice. Il codice mostra che il codice colore per ogni tipo di enum si basa sul costruttore (come il rosso è 42). Ma, la regola del calamaro impone che questo possa essere "confuso" quando qualcuno sta cercando di eseguire il debug del codice (in particolare, una grande parte di codice).

Questo è preso dalla regola calamaro documentation.

L'uso di numeri magici può sembrare ovvio e diretto quando si scrive un pezzo di codice, ma sono molto meno chiari e chiari al momento del debug.

Ecco perché i numeri magici devono essere demistificati prima di essere assegnati a costanti chiaramente denominate prima di essere utilizzate.

-1, 0 e 1 non sono considerati numeri magici

Così, nel codice, si consiglia di fare qualcosa di simile.

public enum Color{ 
    public final int RED_CODE = 42; 
    RED(RED_CODE); 

    public final int code; 

    Color(int colorCode){ 
     this.code=colorCode; 
    }  
} 

o potrebbe anche disabilitare la regola: D