[Java] Problema con gli iterator e la serializzazione

[kekkuz87]

Salve a tutti sto studiando java da un po di tempo e per "allenamento/studio" sto scrivendo un piccolo programma che gestisca un archivio personale
il mio problema è che per eliminare un elemento dal archivio ho creato il metodo removeVoceArchvio in questo modo

public class Archivio implements Serializable {

private static final long serialVersionUID= 1L;
protected ArrayList<VoceArchivio> archivio;

public Archivio(){
archivio= new ArrayList<VoceArchivio>();
}

public void addVoceArchivio(VoceArchivio v){
archivio.add(v);
}

public void removeVoceArchivio(VoceArchivio v){
//Iterator<VoceArchivio> it = archivio.iterator();
//while(it.hasNext()){
for(Iterator<VoceArchivio> it = archivio.listIterator();it.hasNext();){
VoceArchivio a = it.next();
if(a.equals(v))
archivio.remove(a);
//it.remove();
}
}

e il metodo funziona se creo ad esempio una classe main dove creo un archivio e applico il metodo removeVoceArchvio
ma la mia idea era ance di salvare in memoria l'archivio per poter accedere successivamente e poter aggiungere o togliere delle voci
quindi ho creato due classi x Serializzare e deserializzare l'archivio
e qui sorge il problema se creo un archivio lo serializzo e successivamente lo deserializzo
e all'archivio applivo il metodo removeVoceArchvio questo non fa nulla

Posto anke le due classi x la serializzazione

package Archivio;
import java.io.*;

public class SerializzaArchivio {
final String nomeFile="archivio.dat";
public SerializzaArchivio(Archivio archivio){

ObjectOutputStream output;
try{
output = new ObjectOutputStream(new FileOutputStream(nomeFile));
output.writeObject(archivio);
output.close();
}
catch (FileNotFoundException e)
{
System.out.println(e);
}

catch (IOException e) {
System.out.println(e);
}
}
}

package Archivio;

import java.io.*;

public class DeserializzaArchivio {
private Archivio archivio;
final String nomeFile="archivio.dat";

public DeserializzaArchivio(){

try{
ObjectInputStream input = new ObjectInputStream(new FileInputStream(nomeFile));
archivio =(Archivio) input.readObject();
input.close();
}
catch (FileNotFoundException e)
{
System.out.println("");
}

catch (IOException e) {
System.out.println(e);
}
catch (ClassNotFoundException e){
System.out.println(e);
}

}
public Archivio getArchivio(){
return archivio;
}
}

Credo che il problema sia un conflitto tra iterator e la serializzazione
Spero di essere stato abbastanza chiaro ma sono un paio di giorni che cerco di risolvere questo problema

[mone.java]

Ricorda che anche la classe VoceArchivio deve essere serializzabile....

inoltre il tuo metodo per la rimozione


public void removeVoceArchivio(VoceArchivio v){
//Iterator<VoceArchivio> it = archivio.iterator();
//while(it.hasNext()){
for(Iterator<VoceArchivio> it = archivio.listIterator();it.hasNext();){
VoceArchivio a = it.next();
if(a.equals(v))
archivio.remove(a);
//it.remove();
}


si può scrivere cosi:


public void removeVoceArchivio(VoceArchivio v){
//fa esattamente la stessa cosa che fai tu sopra (piu o meno)
archivio.remove(v);
}


e ti devi ricordare di implementare il metodo equals all'interno della classe VoceArchivio! altrimenti lui fa il confronto tra indirizzi in memoria! e quindi probabilmente tu stai cercando di rimuovere un oggetto diverso da quello che si trova nell'archivio...

mentre ricorda che per scorrere un archivio (non eliminare elementi però):


for (VoceArchivio va : archivio) {
faiQualcosaConVA(va);
}

[kekkuz87]

ciao grazie per l'aiuto ma per fortuna ho risolto da solo(cosa non fa una notte di sonno:D) dovevo riscrivere esattamente il metodo equals come da te suggerito..comunque mi hai dato la risposta corretta;) ;) ti ringrazio