[Java]Impossibile visualizzare componenti Swing in un JDialog o_O

[UnknownSoldier]

Salve a tutti. Mi capita uno stranissimo problema riguardo i componenti grafici Swing... ma ora vi spiego prima a cosa serve il programma che sto sviluppando: sto creando un programma che mi consente di collegarmi ad un sito internet e scaricare il suo catalogo di articoli. Questo sito ha una struttura specifica di directory che vado a creare, e con appositi script PHP che fanno da ponte tra la mia applicazione Java e il database MySQL che si trova on-line.
Siccome è un programma che lavora quasi completamente facendo richieste POST agli script PHP, io ho bisogno di una JProgressBar con progressione indeterminata o determinata. Quindi ho creato una classe apposita, che mi consente facilmente di creare un piccolo JDialog con una JLabel e una JProgressBar, da mostrare quando si sta effettuando una richiesta POST, che indica all'utente che c'è una connessione in corso e che bisogna attendere.
Ecco la classe:

package classes;

import javax.swing.JDialog;
import javax.swing.JLabel;
import javax.swing.JProgressBar;
import java.awt.FlowLayout;

public class ProgressBar
{
    private JProgressBar bar;
    private JDialog d;

    public void createIndeterminateBar()
    {
        bar = new JProgressBar();
        bar.setIndeterminate(true);
    }

    public void createDeterminateBar (int min, int max)
    {
        bar = new JProgressBar(min, max);
        bar.setValue (min);
    }
    
    public void showBar (final String title, final String message)
    {
        JLabel label = new JLabel (message);

        d = new JDialog();
        d.setTitle (title);
        d.setSize (200, 90);
        d.setLocationRelativeTo (null);
        d.setDefaultCloseOperation(JDialog.DO_NOTHING_ON_CLOSE);
        d.setLayout(new FlowLayout());

        d.add (label);
        d.add (bar);

        d.setVisible (true);
    }

    public void closeBar()
    {
        d.dispose();
        d = null;
        bar = null;
    }
    
    public void setValue (int value)
    {
        bar.setValue(value);
    }

    public JProgressBar getBar()
    {
        return bar;
    }
}

Il mio problema è che, se creo un oggetto di tale classe e mostro il JDialog utilizzando il metodo showBar(), il JDialog viene visualizzato, ma il contenuto della finestra è grigio! Non c'è nè la JLabel nè la JProgressBar! Ho notato che solo in determinati punti del programma succede questa cosa (proprio in quei punti in cui ne ho bisogno!). Ho pensato subito alla gestione dei Thread, ho provato con SwingUtilities.invokeLater oppure creare un Thread apposito... e per certi versi funzionava! Ma capii che non faceva al caso mio poichè se utilizzassi un altro Thread, non potrei più sapere quand'è che la richiesta POST finirà!
Vi posto la classe principale e in grassetto evidenzio le parti in cui uso tale classe:

import javax.swing.JDialog;
import javax.swing.JPasswordField;
import javax.swing.JButton;
import javax.swing.JOptionPane;
import javax.swing.JFileChooser;
import javax.imageio.ImageIO;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.FlowLayout;
import java.awt.image.BufferedImage;

import java.io.File;

import java.net.URL;
import java.net.MalformedURLException;

import java.util.ArrayList;

import classes.*;

public class Catalog implements ActionListener
{
    private String website;
    private URL loginURL;
    private URL collectionsURL;
    private URL typesURL;
    private URL articlesURL;
    private JPasswordField passwordField;
    private JButton loginButton;
    private JDialog dialog;

    private ArrayList < Collection > collections;

    public Catalog (String website) throws MalformedURLException
    {
        this.website = website;
        this.loginURL = new URL (website + "/admin/CatalogDownload/login.php");
        this.collectionsURL = new URL (website + "/admin/CatalogDownload/collections.php");
        this.typesURL = new URL (website + "/admin/CatalogDownload/types.php");
        this.articlesURL = new URL (website + "/admin/CatalogDownload/articles.php");
        showLogin();
    }

    public void showLogin()
    {
        dialog = new JDialog();
        dialog.setTitle ("Accedi");
        dialog.setSize (200, 96);
        dialog.setResizable (false);
        dialog.setLocationRelativeTo (null);
        dialog.setDefaultCloseOperation(JDialog.DISPOSE_ON_CLOSE);
        dialog.setLayout(new FlowLayout());

        passwordField = new JPasswordField (10);
        passwordField.addActionListener (this);
        loginButton = new JButton ("Accedi");
        loginButton.addActionListener (this);

        dialog.add (passwordField);
        dialog.add (loginButton);

        dialog.setVisible (true);
    }

    private String login (String password)
    {
        dialog.dispose();
        ProgressBar bar = new ProgressBar();
        bar.createIndeterminateBar();
        bar.showBar("Attendere", "Verifica della password...");
        String response = null;
        try
        {
            RequestSender request = new RequestSender (loginURL);
            request.setMethod("POST");
            request.addParam("password", password);
            response = request.send();
        }
        catch (Exception exception)
        {
            exception.printStackTrace();
            JOptionPane.showMessageDialog (null, "Si è verificato un errore durante la connessione", "Errore", JOptionPane.ERROR_MESSAGE);
        }
        bar.closeBar();

        return response;
    }

    private void process (String password)
    {
        String res = login (password);
        if (!res.trim().equals ("ok"))
        {
            JOptionPane.showMessageDialog (null, res, "Errore", JOptionPane.ERROR_MESSAGE);
            showLogin();
        }
        else
        {
            collections = new ArrayList < Collection >();
            ProgressBar bar = new ProgressBar();
            bar.createIndeterminateBar();
            bar.showBar("Attendere", "Connessione in corso...");

            try
            {
                getCollections();
                getTypes();
                getArticles();
                bar.closeBar();
                download();
            }
            catch (Exception exception)
            {
                exception.printStackTrace();
                JOptionPane.showMessageDialog(null, "Si è verificato un errore durante il download del catalogo", "Errore", JOptionPane.ERROR_MESSAGE);
            }
        }
    }

    private void download() throws Exception
    {
        JFileChooser fileChooser = new JFileChooser();
        fileChooser.setMultiSelectionEnabled(false);
        fileChooser.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
        int ris = fileChooser.showSaveDialog(null);
        if (ris == JFileChooser.APPROVE_OPTION)
        {
            int dim = 0;
            for (Collection collection : collections)
                dim += collection.articles.size();

            ProgressBar bar = new ProgressBar();
            bar.createDeterminateBar(0, dim);
            bar.showBar("Attendere", "Download in corso...");

            File location = fileChooser.getSelectedFile();
            File directory = new File (location, "Catalogo");
            directory.mkdir();
            File articles = new File (directory, "articles");
            articles.mkdir();
            File images = new File (articles, "images");
            images.mkdir();
            File thumbs = new File (images, "thumbs");
            thumbs.mkdir();

            File[] images_files = images.listFiles();
            File[] thumbs_files = thumbs.listFiles();
            if (images_files.length > 0)
                for (File f : images_files)
                    if (f.isFile())
                        f.delete();

            if (thumbs_files.length > 0)
                for (File f : thumbs_files)
                    f.delete();


            for (Collection collection : collections)
            {
                for (Article article : collection.articles)
                {
                    if (!article.getImage().equals (""))
                    {
                        BufferedImage img = ImageIO.read(new URL(website + "/images/jewels/" + article.getImage()));
                        ImageIO.write(img, getExtension(article.getImage()), new File(images, article.getImage()));
                        bar.setValue(bar.getBar().getValue() + 1);
                    }
                }
            }

            bar.closeBar();
        }
    }

    private void getCollections() throws Exception
    {
        RequestSender requester = new RequestSender(collectionsURL);
        requester.setMethod("POST");
        String response = requester.send();
        String[] collectionsResponse = response.split("-newline-");
        for (String collectionResponse : collectionsResponse)
        {
            if (!collectionResponse.trim().equals(""))
            {
                String[] fields = collectionResponse.split("-separator-");
                Collection collection = new Collection();
                collection.setId(Integer.parseInt(fields[0]));
                collection.setName(fields[1]);
                collection.setDesc(fields[2]);

                this.collections.add (collection);
            }
        }
    }

    private void getTypes() throws Exception
    {
        for (Collection collection : collections)
        {
            RequestSender requester = new RequestSender(typesURL);
            requester.setMethod("POST");
            requester.addParam("id_collection", "" + collection.getId());
            String response = requester.send();
            String[] types = response.split("-newline-");

            for (String type : types)
                collection.types.add(type);
        }
    }

    private void getArticles() throws Exception
    {
        for (Collection collection : collections)
        {
            RequestSender requester = new RequestSender(articlesURL);
            requester.setMethod("POST");
            requester.addParam("id_collection", "" + collection.getId());
            String response = requester.send();
            String[] articles = response.split("-newline-");
            for (String articleResponse : articles)
            {
                String[] fields = articleResponse.split("-separator-");
                if (!articleResponse.trim().equals(""))
                {
                    Article article = new Article();
                    article.setId(Integer.parseInt(fields[0]));
                    article.setCode(fields[1]);
                    article.setCollections_id(Integer.parseInt(fields[2]));
                    article.setType(fields[3]);
                    article.setImage(fields[4]);
                    article.setCt_gold(fields[5]);
                    article.setCt_precious_stone(fields[6]);
                    article.setCt_diamond(fields[7]);
                    article.setGr_pearl(fields[8]);
                    article.setGr_coral(fields[9]);
                    article.setGr_semiprecious_stone(fields[10]);
                    article.setDescription(fields[11]);
                    article.setPrice(Float.parseFloat(fields[12]));
                    article.setDiscount(Integer.parseInt(fields[13]));

                    collection.articles.add(article);
                }
            }
        }
    }

    public String getPassword (JPasswordField field)
    {
        char[] characters = field.getPassword();
        String password = "";
        for (char c : characters)
            password += c;

        return password;
    }

	private String getExtension (String name)
	{
        String extension = "";
		int i = name.lastIndexOf ('.');

		if (i > 0 && i < name.length() - 1)
			extension = name.substring (i + 1).toLowerCase();

		return extension;
	}

    public void actionPerformed (ActionEvent event)
    {
        process (getPassword (passwordField));
    }
}

Grazie mille per l'attenzione, spero che qualcuno mi aiuti.

[PGI-Bis]

Il problema è proprio quello dei thread. Te ne serve un altro da affiancare all'EDT per l'esecuzione dell'input-output. Puoi usare SwingWorker per coordinarli. Per ciò che riguarda il "quando termina la richiesta" non cambia nulla rispetto al contesto single-thread: la richiesta termina quando il metodo che gestitsce l'IO della richiesta http restituisce il controllo all'invocante.

[UnknownSoldier]

Originariamente inviato da PGI-Bis

Il problema è proprio quello dei thread. Te ne serve un altro da affiancare all'EDT per l'esecuzione dell'input-output. Puoi usare SwingWorker per coordinarli. Per ciò che riguarda il "quando termina la richiesta" non cambia nulla rispetto al contesto single-thread: la richiesta termina quando il metodo che gestitsce l'IO della richiesta http restituisce il controllo all'invocante.

Intanto grazie per la risposta! Per problemi di Thread e interfacce grafiche ho sempre utilizzato invokeLater di SwingUtilities, mentre SwingWorker non l'ho mai utilizzata. Non è che potresti farmi qualche esempio? Cos'è che fa, e quali sono le differenze rispetto a invokeLater?

[UnknownSoldier]

Sono riuscito a far visualizzare correttamente il contenuto dei JDialog creando un nuovo Thread in actionPerformed, che si attiva quando si clicca sul pulsante Accedi o si preme Invio nel JPasswordField!

Ora però ho un problema con la JProgressBar: procede correttamente ma una volta arrivata a metà percorso il programma termina (correttamente). Forse ho sbagliato ad impostare i valori min e max anche se non capisco come. Il minimo l'ho impostato a zero, e il massimo l'ho impostato al numero di articoli. Ad ogni salvataggio dell'immagine di un articolo viene incrementata di 1 unità o_O

[UnknownSoldier]

Credo di aver capito, il problema era che non tutti gli articoli hanno un'immagine

[PGI-Bis]

invokeLater di EventQueue (e SwingUtilities) ti consente, in soldoni, di passare da un Thread diverso dall'EDT all'EDT. Ad esempio nel metodo main sappiamo di non essere nell'EDT, se vogliamo aprire una finestra dobbiamo passare a quel thread. Messa così sembra un po' magia, in realtà l'EDT è un comunissimo consumatore, ma è per capirsi.

SwingWorker si occupa, in un certo senso, del caso inverso: siamo nell'EDT e vogliamo far qualcosa in un thread diverso - per evitare di sovraccaricare l'EDT.

Meglio ancora, si occupa di una particolare classe di casi inversi: siamo nell'edt, vogliamo far fare qualcosa ad un altro thread E ciò che quel thread fa ha effetto su parti del sistema che devono essere gestire dall'EDT (solitamente qualche cambiamento in un componente AWT-Swing).

Di norma, infatti, se ci troviamo nell'EDT e vogliamo passare ad un altro thread basta Thread. In ipotesi:

public void actionPerformed(ActionEvent e) { //EDT
   new Thread() {
       public void run() { //NON EDT
           
       }
   }.start();
}

In questo schema di solito l'esecuzione in background si coordina con l'EDT limitatamente ad un qualche effetto iniziale e finale, ad esempio premo il pulsante, apro una finestra di attesa, eseguo la computazione parallela, chiudo la finestra:

public void actionPerformed(ActionEvent e) {//EDT
    apriFinestraAttesa(); //qui siamo sempre nell'EDT
    new Thread() {
        public void run() { //NON EDT
            eseguiComputazioneInBackground();

            EventQueue.invokeLater(new Runnable() {
                public void run() { //EDT
                    chiudiFinestraAttesa(); //qui siamo nell'EDT per via di invokeLater
                }
            });
        }
    }.start();
}

Tra indentazioni e graffe non è particolarmente bello a vedersi ma è relativamente "facile".

La trama si complica quando il thread in background vuole passare dei valori all'edt durante la computazione (per valori preformati che l'EDT voglia passare al thread in background esiste già una sincronizzazione tra l'azione che avvia un thread e la prima istruzione eseguita dal thread avviato).

Quando un Thread in esecuzione vuole passare un valore ad un altro Thread in esecuzione il primo Thread deve scrivere ciò che vuole comunicare sotto la vigilanza di un monitor e passare quel monitor insieme al valore al secondo Thread il quale, prima di tentare la lettura, si approprierà del monitor.

Sono operazioni assolutamente elementari ma piuttosto noiose. E qui arriva swing worker. SwingWorker somiglia ad un Thread solo che ha due metodi "run", uno principale che viene eseguito in concorso con l'EDT e uno secondario eseguito dall'EDT e attivabile a richiesta.

Il metodo principale è doInBackground, deputato ad eseguire la computazione parallela. Il metodo secondario è "process(List<T> valori)".

Il contenuto di process è eseguito dall'EDT. Process viene attivato dopo un'esecuzione di "publish(T)" - di solito in doInBackground. Il parametro T è il secondo nella lista dei parametri di SwingWorker:

new SwingWorker(..., T)

Ad esempio uno SwingWorker che, in background, genera dei punti, sarà:

new SwingWorker<Void, Point>() {

}

Uno che produce interi sarà:

new SwingWorker<Void, Integer>() {

}

eccetera.

Quando doInBackground genera un valore che vuole passare all'EDT lo fa tramite publish:

new SwingWorker<Void, Point>() {

    public Void doInBackground() { //fuori dall'EDT
        Point p = new Point(10, 20);
        publish(p);
        return null;
    }
}

L'invocazione di publish "va a finire" in process.

new SwingWorker<Void, Point>() {

    public Void doInBackground() { //fuori dall'EDT
        Point p = new Point(10, 20);
        publish(p);
        return null;
    }

    public void process(java.util.List<Point> valori) {
        Point p = valori.get(0);
    }
}

La ragione per cui publish riceve un valore e process ne piglia una lista sta in ciò che invocazioni multiple di publish sono passibili di accorpamento per questioni di efficienza. Vale a dire che più invocazioni di publish nel doInBackground possono causare una sola invocazione di process con tutti gli argomenti nella stessa lista.

new SwingWorker<Void, Point>() {

    public Void doInBackground() { //fuori dall'EDT
        publish(new Point(10, 20));
        publish(new Point(30, 40));
        publish(new Point(50, 60));
        return null;
    }

    public void process(java.util.List<Point> valori) {//nell'EDT
        //le tre invocazioni di publish in doInBackground possono
        //causare tre invocazioni di process ma anche una sola invocazione
        //con una lista di tre punti o due invocazioni. in ogni caso la faccenda
        //si gestisce con un ciclo sulla lista
        for(Point p : valori) {
                fai qualcosa con p
        }
    }
}

Alla fine della fiera il punto centrale di SwingWorker sta nella garanzia di sincronizzazione tra publish e process: i valori che process riceve sono quelli che doInBackground ha creato prima di pubblicarli. Si tratta di un comportamento che le API non garantiscono invece per invokeLater:

new Thread() {
    public void run() {
        final Point p = new Point();
        p.setLocation(10, 20);
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                p.x ? //può valere o zero o dieci
            }
        });
}.start();

new SwingWorker<Void, Point>() {

    public Void doInBackground() {
        Point p = new Point();
        p.setLocation(10, 20);
        publish(p);
        return null;
    }

    public void process(List<Point> valori) {
        Point p = valori.get(0);
        p.x = ? //sicuramente 10
    }
}.execute();

A margine, l'omologo di SwingWorker con Thread e invokeLater sarebbe:

new Thread() {
    public void run() {

        final Point p = new Point();
        synchronized(p) {
            p.setLocation(10, 20);
        }

        EventQueue.invokeLater(new Runnable() {
            public void run() {
                synchronized(p) {
                    p.x = ?; //deve valere 10
                }
            }
        });
    }
}.start();

Tutto qua, nulla di trascendentale.

[UnknownSoldier]

Grazie mille sei stato molto esaustivo! Anche se alla fine non è che ci abbia capito molto
L'importante è che sono riuscito a risolvere il problema, poi se un giorno avrò un bisogno indispensabile di SwingWorker, approfondirò