Exception in thread "Thread-6" java.lang.NullPointerException

[playbasfa]

Salve ragazzi ho un problema a runtime che genera l'errore presente nel titolo.
Il codice che genere l'errore è:

public class CountDown extends Thread{
    
    private javax.swing.JLabel jLabel1;
    
    public CountDown(javax.swing.JLabel a)
    {
        jLabel1=a;
    }
    
    public CountDown(){}
    
    int b=0;
    @Override
    public void run()
    {
        for(int i=0;i<=30;i++)
        {
            b=30-i;
            jLabel1.setText(Integer.toString(b)); //L'ERRORE STA QUI!!!!
            try
            {           	
                Thread.sleep(1000);
            }
            catch (Exception e)
            {
                System.out.println(e.getMessage());
            }
        }
        
        jLabel1.setText("Tempo Scaduto");
    }
    
}

Le chiamate avvengono in questo metodo:

Classe 1

Classe2 obj1= new Classe2();
CountDown obj2=new CountDown(jLabel1);

Classe 2

CountDown obj=new Countdown();
obj.start();

Classe1, Classe2, CountDown risiedono in file diversi all'interno dello stesso progetto.
Grazie in anticipo a chiunque voglia aiutarmi!

[valia]

l'errore parla chiaro: c'è qualcosa null.

separa le istruzioni

jLabel1.setText(Integer.toString(b)); //L'ERRORE STA QUI!!!!

fai

String tmp = Integer.toString(b); 
jLabel1.setText(tmp);

mi aspetto che ad essere null sia jLabel1 e mi aspetto problemi di accesso concorrente

[playbasfa]

Grazie per la risposta.
Fatto, rimane lo stesso errore, relativo alla seconda riga

JLabel1.setText(tmp)

nessun'altro adopera quella jLabel1..

[valia]

sapevo sarebbe rimasto lo stesso, quando uso metodi composti come hai fatto tu per capire chi effettivamente è null separo le istruzioni, ti ripeto mi aspettavo quel valore null.

Devi vedere come istanzi e usi quel thread (cioè come fai ad avere lì null)

[playbasfa]

Grazie per il consiglio
Non riesco a trovare conflitti..
Ecco il codice:

MainClass

private void jMenuItem1ActionPerformed(java.awt.event.ActionEvent evt) {                                           
    SendFile obj=new SendFile("127.0.0.1");
    CountDown ob=new CountDown(jLabel1);
    obj.takeFile();
    obj.start();
}

SendFile

package tesi;


public class SendFile extends Thread
{
    
String ip;
public SendFile(String ip)
    {
     this.ip=ip;
    }
     
    String text;
    DataInputStream in1;
    BufferedReader in;
    FileInputStream fis;
    File f;
    Socket s1;
    Socket s2;
    
    
    //Mette in fis il file pronto per essere inviato
    public void takeFile()
    {
        
        
        JFileChooser chooser = new JFileChooser();
        int n = chooser.showOpenDialog(chooser);
        if(n==JFileChooser.APPROVE_OPTION)
        {
            f = chooser.getSelectedFile();
            try 
            {
                fis = new FileInputStream(f);
            } 
            catch (FileNotFoundException ex) 
            {
                Logger.getLogger(SendFile.class.getName()).log(Level.SEVERE, null, ex);
            }
       }
    }
    
    // Si connette al Server e invia il File
    @Override
    public void run()
    {
        try 
        {
           s2 = new Socket (ip,5556);
            
           //Invio al server il nome del File 
           OutputStream out2 = s2.getOutputStream();
           PrintWriter alServer= new PrintWriter(out2,true);
           
           String nome=f.getName();
           alServer.println(nome);
          // out2.close();
          // s2.close();
           System.out.println(nome);
            
            
            
            byte[] buf = new byte[fis.available()]; 

            
                        s1 = new Socket (ip,5555);

            // Invio l'intero flusso di bytes al server e chiudo file in ingresso flusso di comunicazione e socket
            int read;
            OutputStream out1 = s1.getOutputStream();
            CountDown obj=new CountDown();
            obj.start();      -------- ECCO QUI ----------- 
            while ((read = fis.read(buf)) != -1) 
            {
                out1.write(buf, 0, read);
            }
            fis.close();
            out1.close();
            s1.close();
            
            
        } 
        catch (UnknownHostException ex) 
        {
            Logger.getLogger(SendFile.class.getName()).log(Level.SEVERE, null, ex);
        } 
        catch (IOException ex) 
        {
            Logger.getLogger(SendFile.class.getName()).log(Level.SEVERE, null, ex);
        }
        
    }
    
    
}

La classe CountDown è quella già postata.

[valia]

ooookkkkkk ma visto che abbiamo capito che jLabel1 è in quel punto null, perché non metti qualche breakpoint e fai un po' di debug?
punti chiave quelli in cui fai qualsiasi modifica a jLabel1

[playbasfa]

Non ho mai usato i breakpoint, sto provando ma non ci capisco un granchè.
Ho provato a mettere qualche print e ho visto che la JLabel1 viene correttamente passata, è come però se, quando la classe SendFile invoca il costruttore vuoto della classe CountDown, la jLabel1 si venisse e perdere.
Prima di adoperare la classe CountDown in questo progetto mi sono accertato che funzionasse "a solo" ed effettivamente era così, l'unione ha generato questo problema.
Non ci sono proprio altri punti in cui qualcuno adopera la jLabel1, questa viene creata e direttamente passata al CountDown, nessuno la adopera.

[valia]

ho capito, ma se tu chiami un costruttore vuoto e non ti preoccupi di inizializzare ad un valore di default una variabile interna che usi, come speri che tutto vada bene?
Elimina il costruttore senza parametri, per come hai fatto il tutto non deve esistere

[playbasfa]

Originariamente inviato da valia
ho capito, ma se tu chiami un costruttore vuoto e non ti preoccupi di inizializzare ad un valore di default una variabile interna che usi, come speri che tutto vada bene?

Mi sono preoccupato di rimepire la jLabel1 prima di tutto ovvero quando dico:

CountDown ob=new CountDown(jLabel1);

Elimina il costruttore senza parametri, per come hai fatto il tutto non deve esistere

Il costruttore vuoto mi serve per la chiamata a run che faccio all'interno della classe send, puoi trovarlo nelle ultime righe di codice della classe send, l'ho evidenziato in grassetto due risposte fa.


La sequenzialità delle azioni fa si che quando run viene invocato, jLabel1 è già riempito.. o almeno nella mia testa

[LeleFT]

Originariamente inviato da playbasfa
Il costruttore vuoto mi serve per la chiamata a run che faccio all'interno della classe send, puoi trovarlo nelle ultime righe di codice della classe send, l'ho evidenziato in grassetto due risposte fa.

La tua classe ha due costruttori: il primo prende una JLabel e inizializza la variabile locale al thread "jLabel1"; il secondo non prende nulla e non inizializza nulla.

Di conseguenza, se richiami il secondo, non puoi aspettarti di poter usare jLabel1 perchè non è inizializzata.

Hai due scelte da fare:

1) Elimini il costruttore vuoto: per inizializzare un oggetto CountDown devi passargli una label
2) Verifichi nel metodo run se la variabile jLabel1 è nulla: in quel caso, non dovrai usarla.

Altre possibilità non ce ne sono.


Ciao.