Sonar Vi phạm: An ninh - Mảng được lưu trữ trực tiếp

Question

Có một Vi phạm Sonar:

Sonar Vi phạm: An ninh - Mảng được lưu trữ trực tiếp

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

Giải pháp:

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

Nhưng tôi tự hỏi tại sao?

Answer 1

Việc khiếu nại mảng mà bạn đang lưu trữ là cùng một mảng do người gọi giữ. Nghĩa là, nếu người gọi sau đó sửa đổi mảng này, mảng được lưu trữ trong đối tượng (và do đó chính đối tượng đó) sẽ thay đổi.

Giải pháp là tạo một bản sao trong đối tượng khi nó được thông qua. Điều này được gọi là sao chép phòng thủ. Một sửa đổi tiếp theo của bộ sưu tập sẽ không ảnh hưởng đến mảng được lưu trữ trong đối tượng.

Thực tiễn tốt là thường làm điều này khi trả lại bộ sưu tập (ví dụ: trong một cuộc gọi getMyArray() tương ứng). Nếu không, người nhận có thể thực hiện một sửa đổi và ảnh hưởng đến thể hiện được lưu trữ.

Lưu ý rằng điều này rõ ràng áp dụng cho tất cả các bộ sưu tập có thể thay đổi (và trên thực tế tất cả các đối tượng có thể thay đổi) - không chỉ là mảng. Cũng lưu ý rằng điều này có tác động hiệu suất cần được đánh giá cùng với các mối quan tâm khác.

Answer 2

Nó được gọi là sao chép phòng thủ. Một bài viết hay về chủ đề là "Whose object is it, anyway?" bởi Brian Goetz, thảo luận về sự khác biệt giữa giá trị và ngữ nghĩa tham chiếu cho getters và setters. Về cơ bản, rủi ro với ngữ nghĩa tham chiếu (không có bản sao) là bạn giả tưởng rằng bạn sở hữu mảng và khi bạn sửa đổi nó, bạn cũng sửa đổi các cấu trúc khác có bí danh cho mảng đó. Bạn có thể tìm thấy nhiều thông tin về sao chép phòng thủ và các vấn đề liên quan đến bí danh đối tượng trực tuyến.

Answer 3

Để thực hiện biện pháp phòng thủ, bạn có thể tiết kiệm rất nhiều thời gian. Trong ổi bạn có được một giải pháp tốt đẹp để đạt được mục tiêu: ImmutableCollections

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

Answer 4

tôi đã cùng một vấn đề:

An ninh - Mảng được lưu trữ trực tiếp Các người dùng cung cấp mảng 'palomitas' được lưu trữ trực tiếp.

phương pháp ban đầu của tôi:

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

cố định quay sang:

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

Ví dụ khác:

An ninh - Mảng được lưu trữ trực tiếp Mảng người dùng cung cấp

private String[] arrString; 

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

Cố định:

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

Answer 5

Để loại bỏ chúng, bạn phải clone Array trước khi lưu trữ/trả lại nó như thể hiện trong việc thực hiện lớp sau, vì vậy không ai có thể sửa đổi hoặc lấy dữ liệu ban đầu của lớp học của bạn nhưng chỉ có một bản sao của chúng.

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

tôi đã sử dụng nó như thế này và Bây giờ tôi không nhận được bất kỳ sự vi phạm SONAR ...

Answer 6

Có một số trường hợp nó là một quyết định thiết kế và không bỏ lỡ. Trong những trường hợp này, bạn cần phải sửa đổi các quy tắc Sonar để loại trừ nó để nó không hiển thị các vấn đề như vậy trong báo cáo.

Answer 7

Dễ dàng hơn tất cả điều này. Bạn chỉ cần đổi tên tham số phương thức thành bất kỳ thứ gì khác để tránh vi phạm Sonar.

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

    }