聲納違規：安全 - 陣列直接存儲

Question

有一個聲納違規：

聲納違規：安全 - 陣列直接存儲

public void setMyArray(String[] myArray) { 
    this.myArray = myArray; 
} 

解決方案：

public void setMyArray(String[] newMyArray) { 
    if(newMyArray == null) { 
    this.myArray = new String[0]; 
    } else { 
    this.myArray = Arrays.copyOf(newMyArray, newMyArray.length); 
    } 
} 

但我想知道爲什麼？

Answer 1

這是抱怨你正在存儲的數組是由調用者持有的相同數組。也就是說，如果調用者隨後修改這個數組，那麼存儲在該對象中的數組（也就是對象本身）將會改變。

解決方案是在對象通過時創建副本。這叫做防禦性複製。集合的後續修改不會影響存儲在對象中的數組。

通常在返回集合時（例如，在相應的getMyArray()調用中）這樣做也是很好的做法。否則，接收方可以執行修改並影響存儲的實例。

請注意，這顯然適用於所有可變集合（實際上所有可變對象） - 不只是數組。還要注意，這具有性能影響，需要與其他問題一起評估。

Answer 2

它被稱爲防禦性複製。關於這個主題的一篇很好的文章是Brian Goetz的"Whose object is it, anyway?"，它討論了getter和setter的值和引用語義之間的區別。

基本上，引用語義（沒有副本）的風險是您錯誤地認爲自己擁有數組，並且在修改它時，還要修改其他具有數組別名的結構。您可以在線找到許多有關防止複製和與對象別名有關的問題的信息。

Answer 3

走防守實施路可以爲您節省很多時間。 番石榴你得到另一個很好的解決方案，以達到目標：ImmutableCollections

http://code.google.com/p/guava-libraries/wiki/ImmutableCollectionsExplained

Answer 4

我有同樣的問題：

安全 - 陣列直接存儲用戶提供的陣列 'palomitas'直接存儲。

我原來的方法：

public void setCheck(boolean[] palomitas) { 
     this.check=palomitas; 
    } 

固定轉向：

public void setCheck(boolean[] palomitas) { 
     if(palomitas == null) { 
     this.check = new boolean[0]; 
     } else { 
     this.check = Arrays.copyOf(palomitas, palomitas.length); 
     } 
} 

其它實施例：

安全 - 陣列直接存儲用戶提供的陣列

private String[] arrString; 

    public ListaJorgeAdapter(String[] stringArg) {  
     arrString = stringArg; 
    } 

修正：

public ListaJorgeAdapter(String[] stringArg) { 
    if(stringArg == null) { 
     this.arrString = new String[0]; 
    } else { 
     this.arrString = Arrays.copyOf(stringArg, stringArg.length); 
    } 
} 

Answer 5

消除它們，你必須克隆存儲/恢復它在下面的類實現顯示前陣，因此沒有人可以修改或獲得的原始數據你的班級，但只有他們的副本。

public byte[] getarrString() { 
    return arrString.clone(); 
} 
/** 
* @param arrStringthe arrString to set 
*/ 
public void arrString(byte[] arrString) { 
    this.arrString= arrString.clone(); 
} 

我用它這樣的，現在我沒有得到任何違反SONAR ...

Answer 6

有某些情況下，這是一個設計決策，而不是錯過了。在這些情況下，您需要修改Sonar規則以排除它，以便它不會在報告中顯示此類問題。

Answer 7

這比所有這些更容易。您只需要將方法參數重命名爲其他任何內容即可避免聲納違規。

http://osdir.com/ml/java-sonar-general/2012-01/msg00223.html

public void setInventoryClassId(String[] newInventoryClassId) 
    {     
      if(newInventoryClassId == null) 
      { 
        this.inventoryClassId = new String[0]; 
      } 
      else 
      { 
        this.inventoryClassId = Arrays.copyOf(newInventoryClassId, newInventoryClassId.length); 
      } 

    }