Androidの未読フィールドの警告をバイトで返す

Question

Androidスタジオでjava.lang.SuppressWarningsパッケージを使用しています。

私はこの1つを取り除くことができません：それはsetterメソッドで起こっている

EI_EXPOSE_REP2: May expose internal representation by incorporating reference to mutable object (findbugs task) 

。

この警告を取り除く方法に関するアイデアはありますか？ ありがとう！

public class PropertyDetailDocumentStorageModel implements Parcelable { 
@SerializedName("picture") 
private byte[] mPicture;  
public void setmPicture(byte[] mPicture) { this.mPicture = mPicture; } 

警告：

setmPicture(byte[]) may expose internal representation by storing an externally mutable object into PropertyDetailDocumentStorageModel.mPicture 

注これは、型がbyte[]ある専用フィールドに起こっています。ゲッターを持つ同じクラスの他のフィールドは、この警告を投げていません。

Answer 1

@Thomasのように、配列は常に変更可能です。 修正ではなく、プロパティ自体のプロパティのコピーを返していました：

public byte[] getmPicture() { return Arrays.copyOf(mPicture, mPicture.length); } 

public void setmPicture(final byte[] picture) { this.mPicture = Arrays.copyOf(picture, picture.length); } 

代わりの

public byte[] getmPicture() { return mPicture; } 

public void setmPicture(byte[] picture) { this.mPicture = picture; } 

私は知りませんでしたどのような簡単な、例えば文字列のような他のタイプのためにということですgetterは常にオブジェクトのコピーを返します。配列の場合はそうではありません。

Answer 2

コメントの一部を明確にした後、私は答えがこれだと思います。

1. URF_UNREAD_FIELD - 読んようゲッターが存在するならば、FindBugsのは、フィールドはクラスの外部から読み込まれることを前提としているのでフィールドは、カウントされます。それが起こらなければ、さらなる分析や抑制を必要とする何らかの偽陽性があります。
2. EI_EXPOSE_REP2 - 配列は常に変更可能です。したがって、ゲッターから配列を返すことで、配列を変更することができます。配列のコピーをArrays.copyOf()経由で返すか、警告を抑制します。

FindBugsの警告は@SuppressFBWarningsアノテーション（doc）を介して抑制されます。 FindBugsアノテーションが機能するためには、解析プロセスのクラスパス上のFindBugs libフォルダからannotations.jarとjsr305.jarが必要です。例：

@SuppressFBWarnings("URF_UNREAD_FIELD")