[Java] java.lang.RuntimeException - ciclare List

[ramboelmeio]

Ciao ragazzi...ho un problema. Ho una List di oggetti che devo ciclare, estrarre uno alla volta e controllare delle dei parametri, se questi parametri mi soddisfano lascio intatta la lista, in caso contario elimino l'oggetto e continuo a ciclare la collezione controllando l'oggetto successivo.

Il problema è che quando la condizione per la quale bisognerebbe eliminare l'oggetto dalla lista, l'oggetto viene rimosso ma al ciclo successivo mi viene "sputata" una bella eccezione.

il codice è una cosa del tipo:

Iterator it=bonusTokenList.iterator();
                while (it.hasNext())
                {
                    BonusToken bt=(BonusToken)it.next();
                    if (bt.getBonusGotDate()==null) {
                        bonusTokenList.remove(bt);
                        break;
                    }
                } 

e l'eccezione che mi viene lanciata è una "java.lang.RuntimeException: net.mioProgetto.package.ReadOnlyException: Read only persistent object"

che mi consigliate di fare?

[nether]

Il problema è che quando la condizione per la quale bisognerebbe eliminare l'oggetto dalla lista, l'oggetto viene rimosso ma al ciclo successivo mi viene "sputata" una bella eccezione.

non dovresti eliminare un oggetto presente nella collezione che stai iterando col metodo .remove() della collection, a meno che tu non esca immediatamente dopo dal ciclo (cosa che mi sembra tu faccia con "break"... invece non ho capito quando ricicli nuovamente la collezione). Usa il metodo .remove() di Iterator:

Iterator it=bonusTokenList.iterator();
                while (it.hasNext())
                {
                    BonusToken bt=(BonusToken)it.next();
                    if (bt.getBonusGotDate()==null) {
                        it.remove();
                        break;
                    }
                }

[ramboelmeio]

Grazie per la risposta,
purtroppo col metodo remove () di iterator fa la stessa cosa (lancia sempre l'eccezione)...Il break era solo un tentativo, io devo cicleare l'intera collezione prima di uscire dal ciclo e proseguire il flusso del programma.

[simo7784]

L'errore te lo da anche senza iteratore? Se devi solo ciclare la list per rimuovere elementi puoi fare anche

for(int i=0; i<bonusTokenList.size(); i++) {
    BonusToken bt=(BonusToken)bonusTokenList.get(i);
    if (bt.getBonusGotDate()==null) {
       bonusTokenList.remove(i);
       //Decremento la variabile perche' l'indice degli elementi successivi scala di uno e devo ricontrollare anche l'indice attuale
       i--;
   }
}

[ramboelmeio]

ho scoperto ora che mi dava questo errore perchè la lista era un oggetto persistente (ho preso in mano codice scritto da altri e ho poca esperienza). In questi casi cercando un po' qua e un po' la la soluzione è creare un'altra lista e aggiungerci gli oggetti che mi interessano.

Grazie a tutti...

[LeleFT]

Originariamente inviato da ramboelmeio
[...]
e l'eccezione che mi viene lanciata è una "java.lang.RuntimeException: net.mioProgetto.package.ReadOnlyException: Read only persistent object"

Non vorrei dire, ma te lo diceva chiaro e tondo la JVM qual era il problema...


Ciao.

[ramboelmeio]

hai ragione pure te :-( ma sono alle prima armi e soprattutto non ho mai usato un motore di persistenza!

ciao grazie

[LeleFT]

Originariamente inviato da ramboelmeio
hai ragione pure te :-( ma sono alle prima armi e soprattutto non ho mai usato un motore di persistenza!

ciao grazie

Nemmeo io. Ma quell'eccezione parlava da sola.


Ciao.

[ramboelmeio]

diciamo allora che anche se parlava da sola non sapevo come risolvere il problema!