[JAVA] Problema test con i Thread

[Villain]

Dovrei testare con i thread un'applicazione che gestisce una tavola calda.
1 thread per crea 1 per deposita e 1 per eroga.

il problema è che mi torna nullpointerException su questa istruzione

if (mappaProdotti.get(k.toString())==null)

quando faccio questo controlloreCorrente.deposita(id,10);

vi prego, help me!
import java.io.File;

import javax.swing.JOptionPane;


public class ThreadInserisci implements Runnable {

private Acqua acqua;
private ControlloreGestione controlloreGestione;
private IDEsterno id;
private Orologio oggi;
private Controllore controlloreCorrente;

public ThreadInserisci(Controllore controllore, IDEsterno id){

new File("Ser").mkdir();
new File("Log").mkdir();
this.id = id;
this.controlloreCorrente=controllore;
this.acqua = acqua;
new Thread(this, "inserisci").start();
}

public void run(){
// codice del thread
try{
ControlloreGestione controlloreGestione = new ControlloreGestione();
//IDEsterno id = new IDEsterno("Ferrarelle");
//Acqua acqua = new Acqua(id);
controlloreGestione.nuovoProdotto("Acqua", id);
//IDEsterno articolo = new IDEsterno(iDEsternoProdotto.getText());
int giac= 0;
oggi=new Orologio();
SalvataggioVeloce salva = new SalvataggioVeloce();
// salva.salvaLog(articolo, oggi/*, ruolo, pass*/);
salva.salvaMenu(id, "Acqua");
salva.salvaLogErogaDeposita(id, oggi, "APPENA INSERITO", giac, "0");

}
catch(DirittiException e)
{
JOptionPane.showMessageDialog(null, " NuovoProdottoAA " + e.getMessage(), " non autorizzati", JOptionPane.ERROR_MESSAGE);
}
catch(ClasseAlimentareException e)
{
JOptionPane.showMessageDialog(null, " NuovoProdottoAA " + e.getMessage(), " Errore di tipo", JOptionPane.ERROR_MESSAGE);
}
catch(ProdottoException e)
{
JOptionPane.showMessageDialog(null, " NuiovoProdottoAA " + e.getMessage(), "Errore", JOptionPane.ERROR_MESSAGE);
}
catch (IDEsternoException e)
{
JOptionPane.showMessageDialog(null, e.getMessage(), "Errore", JOptionPane.ERROR_MESSAGE);
}
}

}



import javax.swing.JOptionPane;


public class ThreadDeposita implements Runnable {

private Acqua acqua;
//private ControlloreDeposito controlloreDeposito;
private ControlloreGestione controlloreGestione;
private IDEsterno id;
private Orologio oggi;
private Controllore controlloreCorrente;


public ThreadDeposita(Controllore controllore, IDEsterno id){
//this.acqua = acqua;
this.id = id;
this.controlloreCorrente=controllore;
new Thread(this, "deposita").start();
}

public void run(){
// codice del thread
try
{

controlloreCorrente.deposita(id,10);
//ControlloreDeposito deposito = new ControlloreDeposito();
acqua.deposita(10);
int giac=acqua.giacenza();

// String ruolo=controlloreCorrente.utenteCorrente.getRuolo();
// String pass=controlloreCorrente.utenteCorrente.getPassword();
oggi=new Orologio();
SalvataggioVeloce salva = new SalvataggioVeloce();
salva.salvaLogErogaDeposita(id, oggi, "DEPOSITA", giac, Integer.toString(10));

}
catch (NumberFormatException e)
{
JOptionPane.showMessageDialog(null, " DepositaAA Numero non valido. ");
}
catch(ProdottoException e)
{
JOptionPane.showMessageDialog(null, " DepositaAA " + e.getMessage(), "Errore", JOptionPane.ERROR_MESSAGE);
}
catch (QuantitaException e)
{
JOptionPane.showMessageDialog(null, " DepositaAA Quantità non valida. ");
}
catch (IDEsternoException e)
{
JOptionPane.showMessageDialog(null, " DepositaAA Nome prodotto non non valido. ");
}
catch(DirittiException e)
{
JOptionPane.showMessageDialog(null, " DepositaAA " + e.getMessage(), "non autorizzato", JOptionPane.ERROR_MESSAGE);
}


}


}



public void deposita(IDEsterno k, int quantita) throws DirittiException, ProdottoException, IDEsternoException, QuantitaException
{
if (k==null)
{
throw new IDEsternoException("L'identificatore del prodotto deve essere non nullo");
}
if (mappaProdotti.get(k.toString())==null)
{
throw new ProdottoException("Prodotti " + k.toString() + " non sono presenti");
}
else
{
if (quantita <=0) {
throw new QuantitaException ("La quantita da depositare deve essere positiva\n");
}
else
mappaProdotti.get(k.toString()).deposita(quantita);
}
}

[banryu79]

Ciao, dal codice che hai postato direi che se viene lanciata una NullPointerException in questa riga:

if (mappaProdotti.get(k.toString())==null)

le uniche due cause possibili per cui questo accade sono:
1) mappaProdotti è null
2) il tipo di mappa usato non permette elementi null e l'argomento passato k (IDEsterno) è null

[tylerdurden83]

Anche se il tipo di mappa usato permettesse valori null, chiamare k.toString con k null darebbe comunque una nullPointerException. Tuttavia questo non può essere il caso, dato che c'è un if(k==null) prima dell'istruzione incriminata, quindi k non è null, quindi l'unica possibilità è che non hai inizializzato mappaProdotti a qualcosa che non sia null.

[banryu79]

Anche se il tipo di mappa usato permettesse valori null, chiamare k.toString con k null darebbe comunque una nullPointerException.
Yep, mi era sfuggito.