なかなか直せなかったバグが、Parcelable関連を見直すことで直ったので、その時に気づいた注意点を残しておく.
Parcelableを実装するときの注意
例えば、Stringの配列をreadする際は、readStringArrayではなく、createStringArrayを使う :
public class MyData implements Parcelable {
private String mStringArray[];
public String[] getStringArray() {
return mStringArray;
}
public void setStringArray(String stringArray[]) {
this.mStringArray = stringArray;
}
private MyData(Parcel in) {
mStringArray = in.createStringArray();
}
@Override
public void writeToParcel(Parcel dest, int flags) {
dest.writeStringArray(mStringArray);
}
@Override
public int describeContents() {
return 0;
}
public static final Creator<MyData> CREATOR = new Creator<MyData>() {
@Override
public MyData createFromParcel(Parcel in) {
return new MyData(in);
}
@Override
public MyData[] newArray(int size) {
return new MyData[size];
}
};
}
もし、readStringArray使った場合、画面回転程度の読み込みではエラーは出ないが、「アクティビティ破棄する(アクティビティを保持しない)」設定をオンにした状態で再起動する(ホームボタン→アプリ起動)と、私の環境では次のエラーが出た.
Caused by: java.lang.NullPointerException: Attempt to get length of null array
...
at MainActivity.onCreate(MainActivity.java:xxx)
Parcelableでデータを受渡しするときの注意
特に、
onRestoreInstanceState、
onSavedInstanceStateでアクティビティの復元を行う際、
putParcelableArrayや
getParcelableArrayなどで、Parcelableの配列を使わないほうが良い.
...
private MyData myData[];
...
@Override
protected void onRestoreInstanceState(Bundle savedInstanceState) {
...
myData = (MyData[])savedInstanceState.getParcelableArray(Key1);
...
}
上のようにキャストしないとコンパイルエラーが発生するが、アクティビティが破棄された後(「アクティビティを破棄する」設定をオンにしてアプリケーションを終了する)では、Bundle内で型の情報が保持されないようで、次のようなエラーが出た.
Caused by: java.lang.ClassCastException: android.os.Parcelable[] cannot be cast to com.foo.example.MyData[]
at com.foo.example.MainActivity.onViewStateRestored(MainActivity.java:xxx)
代わりに、
putParcelableArrayListや
getParcelableArrayListを使うことで、このエラーを回避できた.
...
private ArrayList<MyData> myData;
...
@Override
protected void onRestoreInstanceState(Bundle savedInstanceState) {
...
myData = savedInstanceState.getParcelableArrayList(Key1);
...
}
参考にしたサイト