Question

I have this serializable class, wich i use to store on a binary file an ArrayList of Strings.

public class SaveState implements Serializable{
   public static ArrayList <String> favoriteBusStopNumbers = new ArrayList<String>();
   public static SaveState instance=new SaveState();
}

I’m using this method to store the instance with the arrayList of strings once this array is full with the data i must store:

public static void saveData(){
    ObjectOutput out;
    try {
        //primero comprobamos si existe el directorio, y si no, lo creamos.
        File folder = new File(Environment.getExternalStorageDirectory() + DIRECTORY_NAME);
        if(!folder.exists())
            folder.mkdirs();

        File outFile = new File(Environment.getExternalStorageDirectory(), DIRECTORY_NAME+"appSaveState.data");
        out = new ObjectOutputStream(new FileOutputStream(outFile)); 
        out.writeObject(SaveState.instance);
        out.close();
    } catch (Exception e) {e.printStackTrace();}
}

And finally, i use this method on the init of my app to load the file and fill my SaveState.instance variable with the previously stored data:

public static void loadData(){
    ObjectInput in;
    try {
        File inFile = new File(Environment.getExternalStorageDirectory(), DIRECTORY_NAME+"appSaveState.data");
        in = new ObjectInputStream(new FileInputStream(inFile));        
        SaveState.instance=(SaveState) in.readObject();
        in.close();
    } catch (Exception e) {e.printStackTrace();}
}

When i save the data, the file is being created correctly with the object fill of data, i know it because the file has more than 0KB of disc space. But something is going wrong here because when i start my app and load the data, my SaveState.instance variable gets an empty ArrayList of strings……. then ¿what is wrong in the code?

Thanks

Answer 1

Your arraylist is static. Static variables are associated with the class and not with the object. So they dont get serialized.

If you do want to serialize static variables, you have to override readObject and writeObject.

Here is some additional info — http://java.sun.com/developer/technicalArticles/Programming/serialization/

Best solution here is to reconsider your data structure and make the arraylist non-static if its saving the state of an object.

EDIT: Alternatively, you could serialize just the arraylist ( as @Ted suggests below )