JFrame - Problem mit Bildaufbau

[MikeThomson]

Hallo,

ich habe ein Problem mit einem JFrame. Wenn ich diesen ab und zu öffne baut das Bild nicht komplett auf. Auf meinem Entwicklungsrechner (XP) habe ich keine Probleme. Wenn ich jedoch die Applikation auf Vista starte sehe ich die Buttons erst wenn ich mit der Maus drüberfahre. Der Hintergrund des Frames ist nicht zu sehen.

Nachdem ich alle Objekte auf dem Frame platziert habe mach ich ein pack(); und ein setSize(...);

Jemand eine Idee woran das liegen könnte?

 

[Fu3L]

Poste am besten mal Quelltext, kann man leichter überprüfen, als jezz rumzuraten...

 

[MikeThomson]

Da hast du natürlich recht. Anbei der Code des JFrames. Wie gesagt dieser und die einige Buttons der Toolbar werden (ab und zu) nicht gezeichnet.

[HIGHLIGHT="Java"]import java.awt.BorderLayout;
import java.awt.Image;
import java.awt.Insets;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.net.URL;

import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JSlider;
import javax.swing.SwingConstants;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;

import org.jdesktop.swingx.JXStatusBar;
import org.jdesktop.swingx.plaf.basic.BasicStatusBarUI;

import chrriis.dj.nativeswing.NativeComponentWrapper;

public class PlayerFrame extends JFrame implements WebBrowserListener, ServerListener{

private static final long serialVersionUID = 1L;

private Log log;
private Config conf;

private ToolBar ptb;

private JLabel curTitle;

private WebBrowser webBrowser;

private int minWidth;
private int minHeight;


public PlayerFrame(Log log, Config conf, Server server, Web web){
this.log = log;
this.conf = conf;

setLayout(new BorderLayout());

URL imageURL = getClass().getResource("/images/16x16/logo.png");
Image im = Toolkit.getDefaultToolkit().getImage(imageURL);
this.setIconImage(im);
ptb = new ToolBar(log, conf, web);
add(ptb,BorderLayout.NORTH);

String user = conf.get("user");
String pass = conf.get("pass");

webBrowser = new WebBrowser();
webBrowser.setVisible(true);
webBrowser.addWebBrowserListener(this);
server.addServerListener(this);

NativeComponentWrapper ncw = new NativeComponentWrapper(webBrowser);
add(ncw.createEmbeddableComponent(), BorderLayout.CENTER);
add(createStatusBar(), BorderLayout.SOUTH);
pack();

int width = 1024;
if (Toolkit.getDefaultToolkit().getScreenSize().width<width){
width = Toolkit.getDefaultToolkit().getScreenSize().width;
}
int height = 500;
if (Toolkit.getDefaultToolkit().getScreenSize().height<height){
height = Toolkit.getDefaultToolkit().getScreenSize().height;
}
setSize(width, height);

setLocationRelativeTo(getOwner());
}

private JXStatusBar createStatusBar(){
JXStatusBar statusBar = new JXStatusBar();
statusBar.putClientProperty(BasicStatusBarUI.AUTO_ADD_SEPARATOR, Boolean.FALSE);
JXStatusBar.Constraint ct = new JXStatusBar.Constraint(new Insets(0, 5, 0, 5));


curTitle = new JLabel("");
statusBar.add(curTitle, ct);
return statusBar;
}
}[/HIGHLIGHT]

 

[Ebenius]

Wie sieht denn der Code auf in dem Du die Frame-Instanz baust und sichtbar machst?

Anbei: pack(); kannst Du Dir sparen, wenn Du setSize() aufrufst.

Ebenius

 

[MikeThomson]

Hallo,

das sieht in etwa so aus. Ich habe den Code drumherum weggelassen:

public class MyTrayIcon extends TrayIcon{

private PlayerFrame plfa;

public MyTrayIcon(){
 plfa = new PlayerFrame(...);
 JMenuItem button = new JMenuItem("P");
 button.addActionListener(new ActionListener() {
    public void actionPerformed(ActionEvent ae){
       plfa.setVisible(true);
    }
 });
 popup.add(button);
}

}

Und das ganze läuft in einer EventQueue.invokeLager(new Runnable(){ ....

Jemand eine Idee. Ich habe jetzt in den Konstruktor auch mal einen Repaint reingepackt hilft leider nichts.

*Edit*
Ich habe das Problem mit dem Neuzeichnen jetzt auch auf dem XP-Rechner. Lustigerweise habe ich das ganze nur nachdem ich das Projekt deployed habe. Starte ich die Applikation von Eclipse im Debug-Modus trat das Problem (bis jetzt) nie auf.

 

[txd67689]

setVisible(); sollte aber im AWT-Thread stehen (invokeLater oder invokeAndWait)

Initial Threads (The Java™ Tutorials > Creating a GUI with JFC/Swing > Concurrency in Swing)

 

[MikeThomson]

Danke für den Link.

Leider sagt er mir die Webseite kann nicht angezeigt werden.

Ich möchte eigentlich nichts anderes als den JFrame über mein TrayIcon zu öffnen. Da mein TrayIcon selbst SwingBased ist wüsste ich jetzt nicht wie ich das anstellen soll?

*Edit*
Sorry ich glaub ich hab zu schnell gelesen. Ich führe doch .setVisible innerhalb des AWT Threads auf.

Und das ganze läuft in einer EventQueue.invokeLager(new Runnable(){ ....

Das sieht dann bei (etwa) mir so aus:

[HIGHLIGHT="Java"]
public class Start{

public static void main(String[] args){

EventQueue.invokeLater(new Runnable(){
public void run(){
UIManager.setLookAndFeel(new MyLook());
MyTrayIcon mti = new MyTrayIcon();
SystemTray.getSystemTray().add(mti);
}

}

}
[/HIGHLIGHT]

 

[txd67689]

leider hab ich jetz nicht die zeit alles genau durchzulesen. aber die beschriebenen probleme können evtl. daher kommen, dass du setVisible() nicht bzw. nicht für mich erkennbar im awt-thread aufrufst. die faustregel ist: das "realisieren" von guielementen (auf-und abbauen) immer im awt-thread machen!
ungef. so:

SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				deinZeugs.getInstance().getDeinFrameFrame().setVisible(true);
			}

hier nochmal der link als code zum kopieren:

http://java.sun.com/docs/books/tutorial/uiswing/concurrency/initial.html

und noch einer

http://java.sun.com/products/jfc/tsc/articles/threads/threads1.html

 

[MikeThomson]

Hi Buddelflink,

siehe letzt Änderung meines Posts.

Leider gehen bei mir beide Links nicht. Ggf. ein lokales Problem?

Meiner Meinung nach läuft das .setVisible() im AWT-Thread. Korrigiere mich wenn ich falsch liege.

 

[Ebenius]

setVisible(); sollte aber im AWT-Thread stehen (invokeLater oder invokeAndWait)

Die actionPerformed()-Methode wird ja bereits auf dem Event Dispatch Thread ausgeführt. InvokeAndWait wirft damit auf jeden Fall einen

throw new Error("Cannot call invokeAndWait from the event dispatcher thread");

InvokeLater kannst Du nehmen, um das Frame nachgelagert aufzubauen. Was passiert damit? [HIGHLIGHT="Java"]button.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent ae){
plfa.getRootPane().revalidate();
plfa.setVisible(true);
}
);[/HIGHLIGHT]
Nachtrag:

Leider gehen bei mir beide Links nicht. Ggf. ein lokales Problem?

Bei mir geht's.

Ebenius

 

[MikeThomson]

Hallo Ebenius,

InvokeLater habe ich ja auch gentutzt.

Wenn ich das statt dem einfachen plfa.setVisible() nutze.
[HIGHLIGHT="Java"]button.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent ae){
plfa.getRootPane().revalidate();
plfa.setVisible(true); });
[/HIGHLIGHT]

Passiert nichts besonderes. Der Frame wird wie auch nur bei setVisible nicht korrekt aufgebaut. Erst wenn ich mit der Maus drüber fahre klappts. Starte ich das ganze hingegen im Debug-Mode in Eclipse, klappt es wunderbar. Aber da tuts ein einfaches .setVisible() auch.

Die Links ziehe ich mir später zu gemüte. Wäre aber schön wenn noch jemand eine Idee hat?

 

[txd67689]

plfa wird in diesen codeausschnitt nicht im awt-thread realisiert oder ich muss zum optiker.

das ginge so:

//oder eben invokAndWait
SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				plfa.setVisible(true);			}
});

damit schon mal der frame ordentlich platziert

edit: "Die actionPerformed()-Methode wird ja bereits auf dem Event Dispatch Thread ausgeführt" <-- richtig. ich nehm den optiker.

 

[MikeThomson]

Hallo Buddelfink,

auch hiermit keine Besserung. Gleiches Problem wie zuvor.

 

[Ebenius]

plfa wird in diesen codeausschnitt nicht im awt-thread realisiert oder ich muss zum optiker.

Dann wird das wohl nötig werden...

Da steht doch das: [HIGHLIGHT="Java"]button.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent ae){
plfa.setVisible(true);
}
});[/HIGHLIGHT]

Alle Events werden vom Event Dispatch Thread verschickt. Also auch ActionEvents. Du kannst es gern probieren, Button irgendwohin, ActionListener dran und drinnen dann: [HIGHLIGHT="Java"]System.out.println("Am I on the EDT? " + (SwingUtilities.isEventDispatchThread() ? "yes" : "no");[/HIGHLIGHT]
Aber Du kannst Dir die Mühe auch sparen.

Ebenius

 

[txd67689]

ja, ich hatte die actionPerformed nicht gesehen.
das liegt auch ein bisschen daran, dass man alles rechenlastige da nicht drin hat, sonderen einen workerthread aufbauen sollte. deine revalidate(); könnte dafür ein kandidat sein. dann aber setVisible() auch wieder im awt-thread platzieren, wenn der worker durch ist.

 

[Ebenius]

@MikeThomson, das Thema beginnt mich zu nerven. Kannst Du das Problem mal so klein wie möglich zusammenstampfen und das ausführbare Ergebnis als Quelltext irgendwohin legen? Dann teste ich's mal aus.

Ebenius

 

[MikeThomson]

Jup ich werds versuchen.

Besten Dank für die Hilfe

 

[Ebenius]

ja, ich hatte die actionPerformed nicht gesehen.
das liegt auch ein bisschen daran, dass man alles rechenlastige da nicht drin hat, sonderen einen workerthread aufbauen sollte. deine revalidate(); könnte dafür ein kandidat sein. dann aber setVisible() auch wieder im awt-thread platzieren, wenn der worker durch ist.

Nein... "meine" revalidate()-Methode ist nicht "meine" sondern die des JComponent-Klasse und die legt sich selbst auf den EDT wenn sie da noch nicht ist: [HIGHLIGHT="Java"]public void revalidate() {
if (getParent() == null) {
// Note: We don't bother invalidating here as once added
// to a valid parent invalidate will be invoked (addImpl
// invokes addNotify which will invoke invalidate on the
// new Component). Also, if we do add a check to isValid
// here it can potentially be called before the constructor
// which was causing some people grief.
return;
}
if (SwingUtilities.isEventDispatchThread()) {
invalidate();
RepaintManager.currentManager(this).addInvalidComponent(this);
}
else {
Runnable callRevalidate = new Runnable() {
public void run() {
revalidate();
}
};
SwingUtilities.invokeLater(callRevalidate);
}
}[/HIGHLIGHT]



Ebenius