[JAVA] Implementazione del metodo Clone()

[_Ilaria_]

Ciao!
Dunque io ho un problema. Sto facendo un progetto e c'è un pezzo che mi sta facendo impazzire.
Fra le tante classi ho una classe Frigo, la quale ha come attributo un ArrayList di oggetti di tipo Alimento (altra classe che ho implementato). E fin qui ok.
La classe Alimento implementa l'interfaccia Cloneable ed ha quindi un metodo clone(), in cui c'è un try catch. Nel try c'è return super.clone(), nel catch c'è return null.
Ok.
Ora, anche Frigo purtroppo deve avere un metodo clone(), quindi implementa anch'essa l'interfaccia Cloneable ed ha il metodo clone(). Problemaaaa: come diamine devo fare dato che il Frigo ha un ArrayList di Alimento tutti da clonare? Devo fare un ciclo for? Il problema è, non so perchè, mi dà errore quando faccio alimenti.size() (alimenti è un metodo di Frigo che semplicemente restituisce l'ArrayList di Alimento), non capisco perchè mi dà questo errore!
Aiutoooooo? Grazieee!

[valia]

ancora nessuno di noi è veggente e/o sa che succede nel tuo computer, per cui che errore ti dà??

ps scherzo ovviamente, ma se non ci dici l'errore, come ti aiutiamo??

[_Ilaria_]

ok, controllo...

dunque, mi dice (nel test) Exception in thread "main" java.lang.NullPointerException
e mi indica come errata la riga del ciclo for dove c'è for(int i=0;i<alimenti.size();i++)
aiutooooo cos'ha che non va il ciclo for???

[Alex'87]

Originariamente inviato da _Ilaria_
aiutooooo cos'ha che non va il ciclo for???

Considerato l'errore probabilmente non hai istanziato l'ArrayList ma senza vedere il codice non possiamo dirti cosa c'è che non va...

[isAlreadyInUse]

Originariamente inviato da _Ilaria_
ok, controllo...

dunque, mi dice (nel test) Exception in thread "main" java.lang.NullPointerException
e mi indica come errata la riga del ciclo for dove c'è for(int i=0;i<alimenti.size();i++)
aiutooooo cos'ha che non va il ciclo for???

alimenti è null

[_Ilaria_]

Allora, scusate. alimenti è l'ATTRIBUTO della classe frigo ed è un Arraylist di Alimento.
se provo a mettere, invece di alimenti, il metodo getAlimenti() mi dà lo stesso errore.

Volete tutto il codice?
Ve lo dò se volete...

[satifal]

Originariamente inviato da _Ilaria_
...
Volete tutto il codice?
Ve lo dò se volete...

Non sarebbe male per risolvere il problema.

[valia]

se tu hai un oggetto puoi dire

ArrayList alimenti ;

hai DICHIARATO che esiste una variabile di nome alimenti che è di tipo ArrayList

alimenti = new ArrayList();

hai ISTANZIATO alimenti.

Tu probabilmente hai solo dichiarato la variabile, non la istanzi da nessuna parte

Finchè non istanzi alimenti, questo assume un valore di default (null appunto) ecco perchè hai quella nullPointerException.

Istanzia correttamente le variabili di istanza e riprova

[_Ilaria_]

Ok, basta vi dò tutto il codice della classe Frigo. (La classe Alimento in realtà si chiama AlimentoFrigo):


package cucina;
import java.util.ArrayList;

import alimenti.*;

/**
* Classe che crea un frigorifero
* @author 3097536
*/

public class Frigo implements Cloneable{

private ArrayList<AlimentoFrigo> alimenti;

/**
* Crea un frigo con un insieme di alimenti
* @param a ArrayList di alimenti
*/

public Frigo(ArrayList<AlimentoFrigo> a)
{
alimenti = a;
}

/**
* Aggiunge un alimento alla lista di alimenti
* @param a alimento da agiungere
*/

public void aggiungiAlimento(AlimentoFrigo a){
alimenti.add(a);
}

/**
* Rimuove un alimento dalla lista degli alimenti
* @param a alimento da rimuovere
*/

public void rimuoviAlimento(AlimentoFrigo a)
{
alimenti.remove(a);
}

/**
* Restituisce gli alimenti presenti nel frigo
* @return un ArrayList di alimenti
*/

public ArrayList<AlimentoFrigo> getAlimenti()
{
return alimenti;
}

/**
* Ordina la lista di alimenti da quello che scade prima a quello che scade per ultimo
* usando l'algoritmo Inserction Sort
*/

public void ordinaAlimenti()
{
for(int j=2;j<alimenti.size();j++)
{
AlimentoFrigo key = alimenti.get(j);
int i = j-1;

while(i>0 && key.getScadenza().before(alimenti.get(i).getScaden za()))
{
alimenti.set(i+1, alimenti.get(i));
i = i-1;
}
alimenti.set(i+1, key);
}
}

public Object clone(){

try{
Frigo f = (Frigo) super.clone();
for(int i=0;i<alimenti.size();i++){
f.rimuoviAlimento(alimenti.get(i));
f.aggiungiAlimento((AlimentoFrigo) alimenti.get(i).clone());

}
return f;
}
catch (CloneNotSupportedException e){
return null;
}
}
}

[valia]

come pensavo, dove crei l'arraylist alimenti???
vale null, dichiararlo non equivale ad istanziarlo

al max fai

ArrayList alimenti = new ArrayList();

e stai a posto.