Buenas, creo que he entendido lo que estás haciendo después de darle vueltas un rato (y haber respondido erróneamente antes).
Tal como veo que tienes el programa escrito, sin ver cómo haces las llamadas a los métodos, llamas a la función que escribe EscriureFitxer
con los parámetros que tiene un Alumne, la ruta de un File donde escribe y un ArrayList de Alumne que escribes en el fichero.
Visto lo que haces en el método, podrías pasarle directamente un objeto Alumne y luego añadírselo a la lista, o pasar directamente la lista con el Alumno añadido, pero eso no viene al caso.
Tal como lo tienes ahora mismo lo que hace es sobreescribir los datos que había porque, como te habían dicho, para hacer que cuando escriba no escriba encima de lo que había, en el constructor de FileOutputStream hay que meterle un segundo parámetro que es true, para que haga append a lo que había.
Si le añado el parámetro true a mi me funciona correctamente el añadir, ojo, sólo el añadir.
Ahora bien, antes en tu programa tenías esto: ArrayList<Alumne> list = (ArrayList<Alumne>) objectInput.readObject();
Eso lee un objeto, que en este caso es una lista, pero solo lee uno. Aunque el programa funcionase, solo leería la primera lista que añades, no las demás.
Lo que no sé es si tu idea es tener una sola lista en tu fichero o tener más de una, pero tal como lo tienes planteado ahora mismo tendrías muchas, pero sólo estás leyendo una.
Si quisieras tener solo una lista, no tienes que hacer que el fichero vaya añadiendo cosas nuevas, si no sobre escribir la lista que ya había con la nueva, pero para eso en el método de escritura primero tendrías que recuperar la lista y añadir el nuevo elemento.
Voy a interpretar que quieres ir añadiendo nuevas listas para comentar un fallo que veo que quizás es importante puntualizar.
Si en el constructor del FileOutputStream
ponemos el segundo parámetro para que sobreescriba new FileOutputStream(FILE_NAME, true)
vamos a tener que hacer algo para poder leer todas las listas que se hayan almacenado. Algo como esto:
Object actual = null;
ArrayList<Alumne> lista = null;
while((actual = objectInput.readObject()) != null) {
lista = (ArrayList<Alumne>) actual;
for (int u = 0; u < lista.size(); ++u) {
System.out.println(u + " --> " + lista.get(u).getNom());
}
}
Esto iteraría sobre todas las listas que haya hasta que no haya más. Por cierto, habría que controlar EOFException
para cuando llegue al final del fichero.
Pero hay un problema con esto. Cuando se escribe un fichero con objetos se escribe una cabecera al inicio. Si escribes una vez e intentas leer no habrá fallo. El programa ve la cabecera y leerá todo perfectamente. Si intentas añadir algo nuevo al fichero escribirá después de lo ya existente una nueva cabecera. Cuando intentes leer usará la primera cabecera y leerá el primer valor, pero cuando intentes leer el segundo, escrito con otra cabecera, lanzará un error por cabecera corrupta.
la solución a este problema es, que cada vez que quieras escribir algo nuevo crees un nuevo lo primero que hay que hacer es crear un canal de lectura y de escritura e ir leyendo y escribiendo todo (sin cerrar el canal de escritura) y cuando llegues al final escribir lo nuevo y entonces cerrar los canales.
Como sospecho que tu intención es que haya una sola lista en tu fichero te propongo este método para escribir:
public static void EscribirFichero(String nom, String cognom1,
String cognom2, String dni, File FILE_NAME) {
ObjectOutputStream objectOutput = null;
ObjectInputStream objectInput = null;
ArrayList<Alumne> list = new ArrayList<Alumne>();
try {
if(FILE_NAME.exists()) {
objectInput = new ObjectInputStream(new FileInputStream(FILE_NAME));
list = (ArrayList<Alumne>) objectInput.readObject();
}
Alumne a = new Alumne(nom, cognom1, cognom2, dni);
list.add(a);
objectOutput = new ObjectOutputStream(new FileOutputStream(FILE_NAME));
objectOutput.writeObject(list);
System.out.println("Se ha generado el fichero " + FILE_NAME);
} catch (FileNotFoundException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
} catch (ClassNotFoundException e) {
e.printStackTrace();
} finally {
if (objectOutput != null) {
try {
objectOutput.close();
} catch (IOException ignored) {
}
}
}
}
Leerá la lista que hubiera, le añadirá el elemento nuevo y escribirá en el fichero la nueva lista.
Para leer he usado este método, ligeramente modificado:
public static void LeerFichero(File FILE_NAME) {
ObjectInputStream objectInput = null;
try {
objectInput = new ObjectInputStream(new FileInputStream(FILE_NAME));
Object actual = null;
ArrayList<Alumne> lista = null;
while((actual = objectInput.readObject()) != null) {
lista = (ArrayList<Alumne>) actual;
for (int u = 0; u < lista.size(); ++u) {
System.out.println(u + " --> " + lista.get(u).getNom());
}
}
objectInput.close();
} catch (FileNotFoundException e) {
e.printStackTrace();
} catch(EOFException e) {
} catch (IOException e) {
e.printStackTrace();
} catch (ClassNotFoundException e) {
e.printStackTrace();
} finally {
if (objectInput != null) {
try {
objectInput.close();
} catch (IOException ignored) {
}
}
}
}
Espero haber sido de ayuda esta vez :)
ObjectOutputStream
.