[JAVA] repaint(): perché non funziona?

[vincenzo83]

Ciao a tutti
Ho un problema con questo programmino di prova:

_______________________________________________________________________
import javax.swing.*;
import java.awt.*;

public class Main {
public static void main(String[] args) {
Pannello pannello = new Pannello();
Finestra finestra = new Finestra(pannello);
finestra.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
finestra.setVisible(true);

pannello.addBottoni();
finestra.repaint();
}
}


class Finestra extends JFrame {
Pannello pannello;
public Finestra(Pannello pannello) {
setSize(300, 200);
this.pannello = pannello;
add(pannello, BorderLayout.CENTER);
}

}

class Pannello extends JPanel {
public Pannello() {
setBackground(Color.BLUE);
}

public void addBottoni() {
for(int i = 0; i < 3; i++) {
JButton button = new JButton(i + " ");
add(button);
}
}
}
__________________________________________________________

Questo programma creare una finestra a cui viene aggiunto un pannello, dopo aver costruito la finestra con il pannello, viene richiamato il metodo addBottoni() che aggiunge dei bottoni al pannello.

Lanciando il programma si può notare che i bottoni non vengono visualizzati….appaiono solo se la finestra viene ridimensionata.


Ho provato a chiamare il metodo repaint() ma niente da fare…..cosa si deve fare per visualizzare i tre bottoni?

[PGI-Bis]

Ciò che riscontri è il risultato di una race condition tra il Thread "main" ed il Thread "event dispatcher" di AWT/Swing. Affida la proiezione della finestra e le modifiche del componente visibile allo AWT Event Dispatcher e il flusso di esecuzione tornerà ad essere prevedibile.

public class Main { 
	public static void main(String[] args) { 
		final Pannello pannello = new Pannello(); 
		final Finestra finestra = new Finestra(pannello); 
		finestra.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				finestra.setVisible(true); 
				pannello.addBottoni(); 
				finestra.repaint(); 
			}
		});
	} 
}

[vincenzo83]

Grazie. Ritornerò per ulteriori chiarimenti.

[vincenzo83]

Originariamente inviato da PGI-Bis

Ciò che riscontri è il risultato di una race condition tra il Thread "main" ed il Thread "event dispatcher" di AWT/Swing. Affida la proiezione della finestra e le modifiche del componente visibile allo AWT Event Dispatcher e il flusso di esecuzione tornerà ad essere prevedibile.

public class Main { 
	public static void main(String[] args) { 
		final Pannello pannello = new Pannello(); 
		final Finestra finestra = new Finestra(pannello); 
		finestra.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				finestra.setVisible(true); 
				pannello.addBottoni(); 
				finestra.repaint(); 
			}
		});
	} 
}

Analizzando il codice che hai scritto…un intuizione ….ho richiamato il metodo setVisible(true) dopo addBottoni() e il problema si è risolto…

Per capirci meglio, queste è il Main modificato:

public class Main {
public static void main(String[] args) {
Pannello pannello = new Pannello();
Finestra finestra = new Finestra(pannello);
finestra.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

finestra.setVisible(true);

pannello.addBottoni();
finestra.setVisible(true);

}
}

Quindi in pratica si deve richiamare setVisble(true ) ogni volta che si aggiungono componenti ?
Cosa è successo precisamente?

[PGI-Bis]

Permettimi di insistere .

L'invocazione di metodi che proiettano un componente o alterano un componente già visibile devono essere affidate ad un thread apposito, detto event dispatcher thread. Se questo non accade è possibile, oltre ad un dead lock del programma se le istruzioni coinvolte ne contengano i presupposti, che il comportamento del programma non segua la successione di istruzioni contenute nel codice.

Il Thread main, che è quello che esegue le istruzioni nel tuo codice, non è il thread event dispatcher. SwingUtilities.invokeLater(Runnable) è un modo per affidare un pacchetto di comandi alle cure dell'awt event dispatcher.

Secondo la documentazione Java non è necessario invocare due volte setVisible, una prima e una dopo, per ottenere l'esecuzione del comando "a finestra visible aggiungi dei pulsanti ad un contenitore nella finestra". E' sufficiente l'istruzione add sul contenitore, eseguita dall'awt event dispatcher.

SwingUtilies.invokeLater(new Runnable() {
    public void run() {
        finestra.setVisible(true);
        pannello.addBottoni();
   }
});

Nel caso in cui le dimensioni del contenitore cambino dopo l'inserimento del contenuto e se questo sia in grado di ridistribuire gli spazi assegnati dal contenitore del contenitore, allora dovrai anche invocare un revalidate() sul contenitore del contenitore.

A me è capitato un paio di volte di dover invocare un "frame.repaint()" dopo una modifica perchè l'interfaccia fosse aggiornata ma stando al framework Swing questo non dovrebbe essere mai necessario per cui in quelle occasioni devo aver commesso un qualche errore da qualche parte.

L'invocazione doppia di setVisible produce effetti nel caso di specie probabilmente per via della sua onerosità, ciò che da il tempo al thread main di mutare lo stato del contenitore "pannello" prima che l'awt event dispatcher aggiorni la superficie della finestra. Ma e' solo un'ipotesi. Comunque, senza lo SwingUtilities.invokeLater o alternative che garantiscano l'esecuzione nell'event dispatcher, il codice è tecnicamente errato e potenziale sorgente di un dead lock.

[vincenzo83]

Ok, messaggio ricevuto…mi hai convinto . Grazie ancora.