JLabel Textausgabe funktioniert nicht

[Quao]

Servus,
ich habe ein Problem mit der Textausgabe in meinem JLabel.
Meine Methode ruft in einer for-Schleife immer wieder die Textausgabe des JLabels in der GUI Klasse auf. Mein JLabel zeigt aber nichts neues an. Brauche ich dafür vielleicht Threads? Sonst funktioniert das JLabel aber. Wenns auch ohne Threads gehn müsste, kann ich auch mal den Quellcode einstellen.
Ich hoffe mir kann einer sagen was das schlauste wär.
Danke

 

[Camill]

Ja, Code wäre hier sicherlich hilfreich.

 

[Marco13]

Vermutlich wird die Methode, die die for-Schleife ausführt, von einem ButtonKlick oder so ausgelöst? Wenn ja, muss diese Methode in einem eigenen Thread laufen. Das label.setText muss dann aber mit SwingUtilities.invokeLater wieder auf den Event-Dispatch-Thread gemacht werden.

 

[Quao]

Ich hab jetzt mal alles unnötige weggelassen.
Also hier ist der Code der GUI Klasse:

public JLabel konsole = new JLabel();
konsole.setBounds(32, 216, 379, 25);
konsole.setText("Choose two files and press 'Synchronise'!");
cp.add(konsole)
public void synchro_ActionPerformed(ActionEvent evt)
{
    if(in != null && out != null)
    {
        if(in.isDirectory())
        {
            if(out.isDirectory())
            {
                konsole.setText("Synchronising...");
                Syn s = new Syn(this);
                konsole.setText("Looking for files...");
                if(protocol.isSelected())
                {
                    try
                    {
                        Console console = new Console();
                        console.setVisible(false);
                        s.syn2(in, out, console);
                        console.setVisible(true);
                    }
                    catch(Exception e)
                    {
                        konsole.setText("Sorry, the protocol doesn't work");
                        s.syn(in, out);
                    }
                }
                else
                {
                    s.syn(in, out);
                }
                int n = s.getcopyed();
                int m = s.getshifted();
                konsole.setText("Finished synchronising; copied " + n + " files and shifted " + m + " files");
                in = null;
                eingangLabel.setText("no File choosed");
                out = null;
                zielLabel.setText("no File choosed");
            }
            else
            {
                konsole.setText("Targetfile is no directory");
            }
        }
        else
        {
            konsole.setText("Startfile is no directory");
        }
    }
    else
    {
        konsole.setText("No file choosed");
    }
};

und hier ist die Klasse mit der Methode, die an sich funktioniert, das JLabel aber nicht verändert wird während die Methode läuft (Ich glaube der Satz spricht für Threads:

    public void syn(File in, File out)
    {
        File[] files = in.listFiles();
        File[] filesout = out.listFiles();
        
        for(int i = 0; i < files.length; i++)
        {
            if(files[i].isDirectory())
            {
                if(Search.search(filesout, files[i]) >= 0)
                {
                    syn(files[i], filesout[Search.search(filesout, files[i])]);
                }
                else
                {
                    copyed++;
                    gui.konsole.setText("Copying directory " + copyed + " ...");
                    File newFile = new File(out.getAbsolutePath() + System.getProperty("file.separator") + files[i].getName());
                    newFile.mkdirs();
                    newDir(files[i], newFile, filesout);
                    gui.konsole.setText("Looking for files...");
                }
            }
            else
            {
                if(Search.search(filesout, files[i]) >= 0)
                {
                }
                else
                {
                    copyed++;
                    gui.konsole.setText("Copying file " + copyed + " ...");
                    File newFile = new File(out.getAbsolutePath() + System.getProperty("file.separator") + files[i].getName());
                    Copy.copyFile(files[i], newFile);
                    gui.konsole.setText("Looking for files...");
                }
            }
        }
    }

 

[Marco13]

Allgmein: Vorher

public void actionPerformed(ActionEvent evt)
{
    synchro_ActionPerformed(evt);
}

Nachher:

public void actionPerformed(final ActionEvent evt)
{
    Thread t = new Thread(new Runnable()
    {
        @Override 
        public void run()
        {
            synchro_ActionPerformed(evt);
        }
    };
    t.start();
}

UND jeden Aufruf von
konsole.setText(text);
oder
label.setText(text);
ersetzen durch
setTextOnEDT(konsole, text);
wobei

private static void setTextOnEDT(final JTextComponent textComponent, final String text)
{
    SwingUtilities.invokeLater(new Runnable()
    {
        @Override 
        public void run()
        {
            textComponent.setText(text);
        }
    });
}