Event Handling Exception in thread "AWT-EventQueue-0" java.lang.ClassCastException

[Cresse]

Guten Tag, ich bräuchte mal Hilfe, ich bekomme eine Fehlermeldung, kann aber die Ursache nicht finden. Hier erst mal die komplette Fehlermeldung:

Exception in thread "AWT-EventQueue-0" java.lang.ClassCastException
at javax.swing.LayoutComparator.compare(Unknown Source)
at javax.swing.LayoutComparator.compare(Unknown Source)
at java.util.TimSort.countRunAndMakeAscending(Unknown Source)
at java.util.TimSort.sort(Unknown Source)
at java.util.TimSort.sort(Unknown Source)
at java.util.Arrays.sort(Unknown Source)
at java.util.Collections.sort(Unknown Source)
at javax.swing.SortingFocusTraversalPolicy.enumerateAndSortCycle(Unknown Source)
at javax.swing.SortingFocusTraversalPolicy.getFocusTraversalCycle(Unknown Source)
at javax.swing.SortingFocusTraversalPolicy.getFirstComponent(Unknown Source)
at javax.swing.LayoutFocusTraversalPolicy.getFirstComponent(Unknown Source)
at javax.swing.SortingFocusTraversalPolicy.getDefaultComponent(Unknown Source)
at java.awt.FocusTraversalPolicy.getInitialComponent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$000(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.SequencedEvent.dispatch(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$000(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Da verstehe ich nur, dass der Fehler etwas mit den ActionListener zu tun hat, aber ich habe dort meines Wissens keinen aktiviert, und auch meine ActionPerformed Methode wird nicht aufgerufen.
Da keine Zeilen dabei sind vermute ich, dass es zum Fehler in dieses Zeilen kommt:

	private void constructOrderviewPanel() 
	{
		BorderdLabel tempLabel = new BorderdLabel("      " + text.getText(15));
		orderPanel.add( tempLabel , "cell 0 0 , width 430!");
		for( int i = 0 ; i< orders.size() ; i++)
		{
			MyJButton temp = new MyJButton("Name: " + orders.get(i).getName() + "  Price: " + orders.get(i).getSum() , i);
			temp.addActionListener(this);
			orderPanel.add(temp, "cell 0 " + i+1 + " , width 430!");
		}
		
	}
	
	private void updateOrderviewPanel() 
	{
		Component[] allComponents = orderPanel.getComponents();
		for(int i=0; i<allComponents.length; i++)
		{
			if(allComponents[i].getClass().equals(MyJButton.class))
			{
				((MyJButton)allComponents[i]).removeActionListener(this);
			}
		}
		orderPanel.removeAll();
		constructOrderviewPanel();
		
	}

Im Programmablauf wird erst der Konstruktor dieser Klasse aufgerufen und die constructOrderviewPanel() Methode, dann ändere ich die Order und rufe die updateOrderviewPanel() Methode auf. Dann kommt es zur Fehlermeldung. Der meiner Meinung nach wichtige Teil des Konstructors:

orderPanel = new JPanel();
		orderPanel.setLayout(new MigLayout("width 450!"));
		orderScroll = new JScrollPane();
		constructOrderviewPanel();
		super.add(orderPanel , "width 450! , height 780! , cell 0 0");
		orderScroll.getViewport().add(orderPanel);
		orderScroll.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);
		orderScroll.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
		super.add(orderScroll, "cell 1 0 , height 780!");

Vielen dank schonmal für eure Hilfe

 

[Flown]

Passiert das auch wenn du diese Zeile auskommentierst?

((MyJButton)allComponents[i]).removeActionListener(this);

 

[Cresse]

Leider ja, dass einzige was ein bisschen hilft, ist wenn ich 2 systems outs einfüge:

private void constructOrderviewPanel() 
	{
		System.out.println("1");
		BorderdLabel tempLabel = new BorderdLabel("      " + text.getText(15));
		System.out.println("1");
		orderPanel.add( tempLabel , "cell 0 0 , width 430!");
		for( int i = 0 ; i< orders.size() ; i++)
		{
			MyJButton temp = new MyJButton("Name: " + orders.get(i).getName() + "  Price: " + orders.get(i).getSum() , i);
			temp.addActionListener(this);
			orderPanel.add(temp, "cell 0 " + i+1 + " , width 430!");
		}
		
	}

dann kommt der Fehler nur noch in ca 40% der Fälle, aber das macht für mich einfach keinen Sinn wie das helfen soll. War von mir eigentlich nur gedacht um rauszufinden wo der Fehler passiert.

 

[Flown]

Versuch mal im constructOrderviewPanel ein neues Panel anzulegen, das zu befüllen und dann dem aktuellen orderPanel zuzuweisen und danach

orderPanel.validate()

 

[SlaterB]

funktioniert die Anzeige mit dem Scrollen usw?

> orderScroll.getViewport().add(orderPanel);
stört mich, sowas sollte man nicht unüberlegt machen,

es gibt genaue Methoden im fraglichen (edit: fragil wollte ich schreiben) JScrollPane-Konstrukt, entweder den View im Konstruktor übergeben oder setViewPortView() benutzten,

> super.add(orderPanel , "width 450! , height 780! , cell 0 0");
ist auch sinnlos, wenn das orderPanel doch in das JScrollPane kommt


aber beides nicht unbedingt gute Gründe für die Exceptions, oder doch?

------

was du noch machen könntest:
deine Methoden zum Umbau der GUI in

SwingUtilities.invokeLater( new Runnable() {
  public void run() {
     // Code, Methodenaufrufe
  }
});

setzen, dann werden sie vom AWT-Thread ausgeführt
und laufen nicht gleichzeitig zu Hintergrund-GUI-Aktionen, denen du einfach die Basis abschneidest

dazu würde die Abmilderung durch Verzögerung, etwa durch Ausgabe, passen

edit: doch nicht, wird ja alles eh von einem Listener, also bereits vom AWT-Thread gemacht,
du benutzt keinen separaten Thread, oder?

 

[Ebenius]

Ich tippe darauf, dass Du Änderungen an den GUI-Komponenten außerhalb des Event Dispatch Threads machst.
[EDIT]Also in etwa das woran auch SlaterB denkt.[/EDIT]

Füg mal in [c]constructOrderviewPanel()[/c] ein:

System.out.println(Thread.currentThread().getName());

Was bekommst Du ausgegeben?

Ebenius

 

[Cresse]

Versuch mal im constructOrderviewPanel ein neues Panel anzulegen, das zu befüllen und dann dem aktuellen orderPanel zuzuweisen und danach

orderPanel.validate()

Dann bleibt es bei mir in der View nachher leer. Aber vielelicht mache ich auch was falsch

 

[Flown]

Ja es ist immer schwer zu raten, wenn man den ganzen Code nicht kennt, aber die anderen haben glaub ich schönere Lösungen parat.

 

[Cresse]

funktioniert die Anzeige mit dem Scrollen usw?

> orderScroll.getViewport().add(orderPanel);
stört mich, sowas sollte man nicht unüberlegt machen,

es gibt genaue Methoden im fraglichen JScrollPane-Konstrukt, entweder den View im Konstruktor übergeben oder setViewPortView() benutzten,

> super.add(orderPanel , "width 450! , height 780! , cell 0 0");
ist auch sinnlos, wenn das orderPanel doch in das JScrollPane kommt


aber beides nicht unbedingt gute Gründe für die Exceptions, oder doch?

Habe ich bis jetzt immer so gemacht, auch in anderen Projekten und das hat so geklappt, also man konnte scrollen und so, fügt man das nur zum JScrollPane hinzu und dann reicht es die ScrollPane hinzuzufügen, dann sieht man bei mir das orderviewPanel nicht mehr

funktioniert die Anzeige mit dem Scrollen usw?



was du noch machen könntest:
deine Methoden zum Umbau der GUI in

SwingUtilities.invokeLater( new Runnable() {
  public void run() {
     // Code, Methodenaufrufe
  }
});

setzen, dann werden sie vom AWT-Thread ausgeführt
und laufen nicht gleichzeitig zu Hintergrund-GUI-Aktionen, denen du einfach die Basis abschneidest

dazu würde die Abmilderung durch Verzögerung, etwa durch Ausgabe, passen

edit: doch nicht, wird ja alles eh von einem Listener, also bereits vom AWT-Thread gemacht,
du benutzt keinen separaten Thread, oder?

benutze noch keinen seperaten Thread und wollte das für dieses projekt auch nicht machen, aber den aufruf zur aktualisierung mache ich noch aus der main methode, zum testen habe ich da eine manulle änderung gemacht, und dann sage ich er soll aktualisierren

 

[Flown]

kannst du mal den ganzen Code posten oder hochladen, damit er ausführbar ist?

 

[Cresse]

Ich tippe darauf, dass Du Änderungen an den GUI-Komponenten außerhalb des Event Dispatch Threads machst.
[EDIT]Also in etwa das woran auch SlaterB denkt.[/EDIT]

Füg mal in [c]constructOrderviewPanel()[/c] ein:

System.out.println(Thread.currentThread().getName());

Was bekommst Du ausgegeben?

Ebenius

Die Ausgabe ist beidesmal main.

 

[Cresse]

kannst du mal den ganzen Code posten oder hochladen, damit er ausführbar ist?

kann ich sind aber ca 20 Klassen ,wenn auch teilweise nur kleine, weis nciht wie übersichtlich das dann noch ist, aber ich kann meine main methode und die klasse mit dem fehler posten:

package starting;

import java.util.ArrayList;

import product.*;
import language.TextNames;
import gui.PizzaFrame;
import gui.Restaurant;

public class Testing {

	private static TextNames text = new TextNames();

	public static void main(String[] args) 
	{
		// Restaurent Test mit Order
		Restaurant test;
		Pizza te = new Pizza(text);
		te.setName("SalamiPizza");
		te.setPrice(3.5f);
		te.setSize(15);
		Topping temp = new Topping();
		temp.setPrice(0.5f);
		temp.setName("Schinken");
		te.add(temp);
		Topping temp1 = new Topping();
		temp1.setPrice(0.5f);
		temp1.setName("Schinken3");
		te.add(temp1);
		Topping temp2 = new Topping();
		temp2.setPrice(0.5f);
		temp2.setName("Schinken2");
		te.add(temp2);
		Order t = new Order ();
		t.add(te);
		t.add(te);
		t.add(te);
		t.add(te);
		t.add(te);
		t.add(te);
		t.setName("asas");
		ArrayList<Order> q = new ArrayList<Order>();
		q.add(t);
		q.add(t);
		q.add(t);
		q.add(t);
		q.add(t);
		test = new Restaurant(text, q);
		

		Restaurant testq;
		Pizza teq = new Pizza(text);
		teq.setName("SalamiPizzqqqqqqqqqqqa");
		teq.setPrice(45.5f);
		teq.setSize(15);
		Topping tempq = new Topping();
		tempq.setPrice(4.5f);
		tempq.setName("Schinkenqqqqqqqqqqqqq");
		teq.add(tempq);
		Topping temp1q = new Topping();
		temp1q.setPrice(4.5f);
		temp1q.setName("Schinkenqqqqqqqqqqqqqq3");
		te.add(temp1q);
		Topping temp2q = new Topping();
		temp2q.setPrice(4.5f);
		temp2q.setName("Schinken2qqqqqqqqqqqqqq");
		teq.add(temp2q);
		Order tq = new Order ();
		tq.add(teq);
		tq.add(teq);
		tq.add(teq);
		tq.add(teq);
		tq.add(teq);
		tq.add(teq);
		tq.setName("asasqqqqqqqqqqqqqqqqqqqqq");

		
		q.add(tq);
		q.add(tq);
		q.add(tq);
		q.add(tq);
		q.add(tq);
		
		test.testUpdate();
	}

}

package gui;

import java.awt.Component;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.Observable;
import java.util.Observer;

import javax.swing.JMenu;
import javax.swing.JPanel;
import javax.swing.JScrollPane;

import net.miginfocom.swing.MigLayout;

import product.Order;

import language.TextNames;

public class Restaurant extends PizzaFrame implements Observer, ActionListener {

	private TextNames text;
	private JPanel orderPanel, pizzasPanel, toppingsPanel;
	private JScrollPane orderScroll, pizzasScroll, toppingsScroll;
	private ArrayList<Order> orders = new ArrayList<Order>();

	
	public Restaurant ( TextNames text , ArrayList<Order> orders)
	{
		super(text , 1);
		this.orders = orders;
		this.text = text;
		construct();
		 
	}
	
	
	public Restaurant(TextNames text) 
	{
		super(text, 1);
		this.text = text;
		construct();
	}
	
	private void construct()
	{

		
		orderPanel = new JPanel();
		orderPanel.setLayout(new MigLayout("width 450!"));
		orderScroll = new JScrollPane();
		constructOrderviewPanel();
		super.add(orderPanel , "width 450! , height 780! , cell 0 0");
		orderScroll.getViewport().add(orderPanel);
		orderScroll.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);
		orderScroll.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
		super.add(orderScroll, "cell 1 0 , height 780!");
		
		pizzasPanel = new JPanel();
		pizzasPanel.setLayout(new MigLayout("width 200!"));
		pizzasScroll = new JScrollPane();
		constructPizzasPanel();
		super.add(pizzasPanel, "width 200! , height 780! , cell 2 0");
		pizzasScroll.getViewport().add(pizzasPanel);
		pizzasScroll.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);
		pizzasScroll.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
		super.add(pizzasScroll, "cell 3 0 , height 780!");
		
		toppingsPanel = new JPanel();
		toppingsPanel.setLayout(new MigLayout("width 200!"));
		toppingsScroll = new JScrollPane();
		constructToppingsPanel();
		super.add(toppingsPanel, "width 200! , height 780! , cell 4 0");
		toppingsScroll.getViewport().add(toppingsPanel);
		toppingsScroll.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);
		toppingsScroll.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
		super.add(toppingsScroll, "cell 5 0 , height 780!");
		
		
		super.setSize(900, 900);
		super.setVisible(true);
		
		
		
	}
	
	private void constructOrderviewPanel() 
	{
		System.out.println(Thread.currentThread().getName());
		System.out.println("1");
		BorderdLabel tempLabel = new BorderdLabel("      " + text.getText(15));
		System.out.println("1");
		orderPanel.add( tempLabel , "cell 0 0 , width 430!");
		for( int i = 0 ; i< orders.size() ; i++)
		{
			MyJButton temp = new MyJButton("Name: " + orders.get(i).getName() + "  Price: " + orders.get(i).getSum() , i);
			temp.addActionListener(this);
			orderPanel.add(temp, "cell 0 " + i+1 + " , width 430!");
		}
		
	}
	
	private void updateOrderviewPanel() 
	{
		Component[] allComponents = orderPanel.getComponents();
		for(int i=0; i<allComponents.length; i++)
		{
			if(allComponents[i].getClass().equals(MyJButton.class))
			{
				((MyJButton)allComponents[i]).removeActionListener(this);
			}
		}
		orderPanel.removeAll();
		constructOrderviewPanel();
		
	}
	
	

	public void testUpdate()
	{
		updateOrderviewPanel();
	}


	private void constructPizzasPanel() 
	{
		
	}
	private void updatePizzasPanel() 
	{
		// TODO Auto-generated method stub
		
	}

	
	private void constructToppingsPanel() 
	{
		
	}
	private void updateToppingsPanel() 
	{
		// TODO Auto-generated method stub
		
	}

	@Override
	public void actionPerformed(ActionEvent arg0) 
	{
		if(arg0.getSource().getClass().equals(MyJButton.class))
		{
			new ViewOrder(orders.get(((MyJButton)arg0.getSource()).getNummber()));
		}
		else
		{
			super.actionPerformed(arg0);
		}
		
	}

	@Override
	public void update(Observable arg0, Object arg1) {
		// TODO Auto-generated method stub
		
	}

}

 

[Cresse]

SwingUtilities.invokeLater( new Runnable() {
  public void run() {
     // Code, Methodenaufrufe
  }
});

Versteh leider nicht ganz, wie ich das einbauen und nacher aufrufen soll

 

[SlaterB]

die main ist ein separater Thread, also da besteht ein gewisses Problem,

die gepostete main ist aber ziemlich unübersichtlich, mehrere Restaurant-Variablen..,
ein sleep() oder sonstigen zeitlichen Ablauf scheint es nicht zu geben, wozu das update()?

stelle doch erst alle Daten zusammen, übergebe sie an die GUI, und dann erst das setVisible(true)

> Versteh leider nicht ganz, wie ich das einbauen und nacher aufrufen soll

z.B. da wo aktuell
> updateOrderviewPanel();
steht meinen Code hinkopieren und den Aufruf updateOrderviewPanel(); hinein,
aber wie gesagt eher fraglich warum das überhaupt ausgeführt wird

 

[Cresse]

Habe den kompleten Code in eine Zip datei gesteckt, kann man hier runterladen :

File-Upload.net - pizzaprogramm.zip

 

[Cresse]

Habe das gemacht, um die update methode zu überprüfen, ob die klappt, deswegen erstelle ich erst eine order mir paar pizzen und rufe damit meine gui auf. Nachher im programm soll die sich dann verändert haben und ich wollte wissen, ob die update methode dies dann richtig verarbeiten kann, also ob die so klappt wie sie da steht

 

[Michael...]

Bei welcher (Anwender)aktion tritt denn der Fehler auf?
Ich würde mir mal grundsätzlich mal überlegen, ob das ganze Hinzufügen und wieder Entfernen von Komponenten wirklich notwendig ist. Sowas ist er unüblich und in den meisten Fällen überflüssig und deutet auf einen Fehler im Konzept hin.

Nur als Hinweis: Sowas wie

if(allComponents[i].getClass().equals(MyJButton.class))

macht man üblicherweise mit

if(allComponents[i] instanceof MyJButton)

 

[Cresse]

Das soll später eine Anwendung sein, die orders bekommt, von anderen PCs, also den usern (für pizzen) und wo man bereits zugestellte wieder löscht und jederzeit neue hinzukommen können

 

[Cresse]

danke für den hinweis, wusste nicht das es etwas weniger umständliches gibt

 

[SlaterB]

also es ist in jedem Fall ein Problem mit main, invokeLater() ist eine offiziele Lösung,
später von Buttons aus sollte es keine solchen Probleme geben, solange nicht wieder separate Threads, dann invokeLater()..

 

[Cresse]

Habe das jetzt ausprobiert, damit kommt die fehlermeldung aber trotzdem noch manchmal, der ausgegebene thread ist jetzt jedes mal AWT-...

private void constructOrderviewPanel() 
	{
		SwingUtilities.invokeLater( new Runnable() {
			  public void run() {
				  System.out.println(Thread.currentThread().getName());
					System.out.println("1");
					BorderdLabel tempLabel = new BorderdLabel("      " + text.getText(15));
					System.out.println("1");
					orderPanel.add( tempLabel , "cell 0 0 , width 430!");
				
			  }
			});
		
		for( int i = 0 ; i< orders.size() ; i++)
		{
			MyJButton temp = new MyJButton("Name: " + orders.get(i).getName() + "  Price: " + orders.get(i).getSum() , i);
			temp.addActionListener(this);
			orderPanel.add(temp, "cell 0 " + i+1 + " , width 430!");
		}

 

[Cresse]

also es ist in jedem Fall ein Problem mit main, invokeLater() ist eine offiziele Lösung,
später von Buttons aus sollte es keine solchen Probleme geben, solange nicht wieder separate Threads, dann invokeLater()..

Was genau heisst das invoke Later eingetlich? das gibt an welcher thread das bearbeiten soll? Wann wird das dann bearbeitet?

 

[SlaterB]

nun, wenn das Land überspült wird und jemand sagt bau einen Damm,
und du an der halbe Küste einen Damm baust und den Rest wie vorher beläßt,
dann kaufe dir lieber trotzdem noch Gummistiefel,

wie zuvor doch angesprochen einfach die ganze Methode bzw. die eine Zeile des Aufrufs
> updateOrderviewPanel();
hinein, was soll dieses Teil-Gewurschtel?

> Was genau heisst das invoke Later eingetlich? das gibt an welcher thread das bearbeiten soll? Wann wird das dann bearbeitet?

du hast den Suchbegriff und das Internet vor dir

 

[Cresse]

Bei welcher (Anwender)aktion tritt denn der Fehler auf?
Ich würde mir mal grundsätzlich mal überlegen, ob das ganze Hinzufügen und wieder Entfernen von Komponenten wirklich notwendig ist. Sowas ist er unüblich und in den meisten Fällen überflüssig und deutet auf einen Fehler im Konzept hin.

Daher sollte ich mir eine ArrayListe anlegen wo ich alle Componenten seperat speichere, damit ich sagen kann welche gelöscht werden soll und welche nicht?

Wie kann ich dann im MigLayout sagen, das alle elemten unter einem gewissen element eine zeile nach oben geschoben werden sollen, so dass dort keine lücke mehr ist?

 

[Cresse]

wie zuvor doch angesprochen einfach die ganze Methode bzw. die eine Zeile des Aufrufs
> updateOrderviewPanel();
hinein, was soll dieses Teil-Gewurschtel?

wenn die addActionlistener oder die removeActionlistener methode mit drinnen stehen verbietet eclipse mir das

 

[SlaterB]

ich kann speziell hierbei nicht ganz folgen, falls du nicht gerade temp vor dem invokeLater() deklarierst und
> temp.addActionListener(this);
dann darin,
aber allgemein ist die Welt kein Ponyhof, Probleme sind da und müssen gelöst werden,
deswegen auf invokeLater() zu verzichten habe ich kaum als Lösung zugelassen ( )

falls hier im Forum, dann bitte mehr Code und weitere Informationen, was immer du hast, etwa die Fehlermeldung

 

[Flown]

Funktioniert bei mir einwandfrei:

	private void updateOrderviewPanel() 
	{
		final Component[] allComponents = orderPanel.getComponents();
		for(int i=0; i<allComponents.length; i++)
		{
			if(allComponents[i] instanceof MyJButton)
			{
				final MyJButton temp = (MyJButton) allComponents[i];
				SwingUtilities.invokeLater(new Runnable() {
					
					@Override
					public void run() {
						// TODO Auto-generated method stub
						temp.removeActionListener(Restaurant.this);
						
					}
				});
			}
		}
		orderPanel.removeAll();
		constructOrderviewPanel();
		
	}

 

[SlaterB]

doch nicht solche Vorschläge machen
die ganze Aktion updateOrderviewPanel() in ein invokeLater() und gut ist

 

[Cresse]

Jetzt habe ich das invoke later glaube ich an der richtigen Stell, die Fehlermeldung ist weg:

	public void testUpdate()
	{
		 SwingUtilities.invokeLater( new Runnable() {
             public void run() {updateOrderviewPanel();}});
		//updateOrderviewPanel();
	}

Jetzt macht er aber das update nicht mehr, ich habe das Gefühl, das er das erste mal das construct macht, und dann zum invokeLater kommt und dann einfriert, weil danach klappen dann die actionlistener auf der Oberfläche nicht mehr und die neuen Elemente kommen nicht hinzu

 

[Cresse]

Funktioniert bei mir einwandfrei:

Das tut es bei mir auch gut, aber hatte jetzt einal den fall wo er einfach keine grapischen elemente angezeigt hat und einmal kamm nochmal die selbe fehlermeldung

 

[Cresse]

Vielen dank euch allen für eure Hilfe, vielleicht mache ich auch etwas grundsätzlich falsch, ich werde versuchen das Problem zu umgehen, in dem ich alles einmal erzeuge und nachher einfach nur noch die Änderungen durchführe, dass ist dann wahrscheinlich sehr viel leistungsschonender und kommt dann hoffentlich auch ohne diesen Fehler aus. Wenn ich es soweit habe, das es läuft werde ich den funktionierenden code hier nochmal posten, oder wenn es nicht klappt noch ein paar Fragen stellen.

Hatte mir überlegt, dass ich meine Klasse order so erweitere, das sie ein action event auslöst, wenn sie geändert wird. Die entsprechende action führe ich dann auch in der gui durch. Damit sollte dieses Problem dann behoben sein, oder?

 

[SlaterB]

> weil danach klappen dann die actionlistener auf der Oberfläche nicht mehr und die neuen Elemente kommen nicht hinzu

es wäre ein genereller Fehler, von Nichtfunktionalität B nach Korrektur A auf Verzicht von A zu schließen,
passe da auf,

je mehr Fehler man korrigiert, desto mehr andere bisher nicht drankommende oder in bestimmten Situationen nicht relevante kommen ans Tageslicht

-----

nach jeder nachträglichen GUI-Änderung, wobei sowas besser überhaupt nicht zu empfehlen ist, gehört z.B.
validate();
repaint();
am JFrame-Objekt aufgerufen,
schreibe das ans Ende z.B. von constructOrderviewPanel()

dann werden die neuen Komponenten angezeigt, im Moment noch die alten, die keine Listener mehr haben (spekuliert)

-----

> ich werde versuchen das Problem zu umgehen, in dem ich alles einmal erzeuge und nachher einfach nur noch die Änderungen durchführe, dass ist dann wahrscheinlich sehr viel leistungsschonender und kommt dann hoffentlich auch ohne diesen Fehler aus

absolut, aber neue andere Probleme sind wohl auch nicht ganz zu vermeiden

 

[Ebenius]

Eine Standard-Anwendung die AWT/Swing spielt baut sich am besten so auf:

/** Creates the GUI. Call on EDT, only! */
@SuppressWarnings("nls")
static void createAndShowGui() {
  // Hier die ganze GUI initialisieren
}

/** @param args ignored */
public static void main(String[] args) {
  SwingUtilities.invokeLater(new Runnable() {

    public void run() {
      createAndShowGui();
    }
  });
}

Ebenius

 

[Cresse]

nach jeder nachträglichen GUI-Änderung, wobei sowas besser überhaupt nicht zu empfehlen ist, gehört z.B.
validate();
repaint();
am JFrame-Objekt aufgerufen,
schreibe das ans Ende z.B. von constructOrderviewPanel()

dann werden die neuen Komponenten angezeigt, im Moment noch die alten, die keine Listener mehr haben (spekuliert)

Super, so klappt es ohne Probleme, vielen Dank für die Hilfe. Würde es trotzdem Sinn machen, das nochmal neu zu schreiben, ohne immer das ganze Panel neu aufzubauen?



Noch eine Frage zu validate und repaint: Wenn ich mehrere JPanel ineinander stecke, muss ich dann bei allen jeweils validate und repaint aufrufen oder generell nur beim JFrame in dem die Panel sind?

 

[Cresse]

Eine Standard-Anwendung die AWT/Swing spielt baut sich am besten so auf:

...

Also am besten jede Änderung die man an der GUI vornimmt immer vom AWT Thread ausführen lassen? und nur berechnungen, internetverbindung, ... über andere threads abarbeiten?

 

[Ebenius]

Also am besten jede Änderung die man an der GUI vornimmt immer vom AWT Thread ausführen lassen? und nur berechnungen, internetverbindung, ... über andere threads abarbeiten?

Genau so soll's sein. Siehe auch: The Java™ Tutorials / Lesson: Concurrency in Swing.

Ebenius

 

[Cresse]

nach jeder nachträglichen GUI-Änderung, wobei sowas besser überhaupt nicht zu empfehlen ist, gehört

Wieso sind nachträgliche Änderungen an der Gui nicht zu empfehlen, wenn neue elemente hinzukommen und alte gelöscht werden muss man doch den neuen stand wieder anzeigen, oder?

 

[SlaterB]

> Wenn ich mehrere JPanel ineinander stecke, muss ich dann bei allen jeweils validate und repaint aufrufen oder generell nur beim JFrame in dem die Panel sind?

ich bin notorisch schlecht im Merken offizieller Lösungen, falls es sie überhaupt gibt (eben vergessen oder nie gelesen)
und probiere eher alles aus, tendiere zu letzterem da bisher ausreichend

> Also am besten jede Änderung die man an der GUI vornimmt immer vom AWT Thread ausführen lassen?

genau, aber nicht gerade alle einzeln

> wenn neue elemente hinzukommen und alte gelöscht werden muss man doch den neuen stand wieder anzeigen, oder?

es geht ja wenn nötig, nur in der Regel unnötig,
und Dinge wie CardLayout gibts auch noch

Komponenten sind schwerfällig, während der Fahrt wechselt man auch nicht unbedingt ein Rad am Auto,
sondern sorgt von Anfang an dafür dass möglichst alle Fälle abgedeckt sind

 

[Cresse]

Vielen Dank allen die mir bei der Lösung dieses Problems geholfen haben, habe viel gelernt und muss jetzt erst mal noch ein paar Sachen nachlesen und verstehen (und natürlich probieren).

SlaterB, nochmal vielen Dank, besonders für die coolen vergleiche.

 

[Flown]

doch nicht solche Vorschläge machen
die ganze Aktion updateOrderviewPanel() in ein invokeLater() und gut ist

Stimmt

Also du benötigst das validate nur weil du removeAll verwendest:

Auszug aus der API

public void removeAll()

Removes all the components from this container. This method also notifies the layout manager to remove the components from this container's layout via the removeLayoutComponent method.

This method changes layout-related information, and therefore, invalidates the component hierarchy. If the container has already been displayed, the hierarchy must be validated thereafter in order to reflect the changes.

 

[SlaterB]

> Also du benötigst das validate nur weil du removeAll verwendest:

steht bei add() auch

Note: If a component has been added to a container that has been displayed, validate must be called on that container to display the new component. If multiple components are being added, you can improve efficiency by calling validate only once, after all the components have been added.

 

[bERt0r]

Stell dir mal vor, du wärst verantwortlich für eine analoge Anzeigetafel bei einem Fussballspiel (mit Kreide). Das heißt, wenn ein Tor fällt, bekommst du ein "Toooor" Event und jetzt musst du den Anzeigestand "updaten". Wenn du so vorgehst wie in deiner "updateOrderviewPanel" Methode, schnappst du dir jedes mal die Tafel, wirfst sie in dem Müll und holst dann eine neue Tafel raus die du dann mit dem aktuellen Spielstand beschreibst. Das ist doch äußerst umständlich. Noch schlimmer, wenn im Spiel ein Tor fällt während du deine Tafel holst kriegst du es nicht mit, weil es nicht auf dich wartet - es läuft "in einem anderen Thread".

Wenn du deinen Buttons einfach mit setText neue Bezeichnungen gegeben hättest wäre es zwar auch schön gewesen das im Event Dispatch Thread zu machen, deinen Fehler hättest du aber sicher nicht bekommen. Warum du den Buttons den Actionlistener wegnimmst, wenn du sie danach eh löscht, versteh ich auch nicht.

 

[Cresse]

Ich wollte vermeiden, das da jede Menge Actionlistener irgendwo noch dran sind, eigentlich um Fehler zu vermeiden, funktioniert das genausogut, wenn ich einfach ein remove all verwende?

 

[Michael...]

Ich wollte vermeiden, das da jede Menge Actionlistener irgendwo noch dran sind, eigentlich um Fehler zu vermeiden, funktioniert das genausogut, wenn ich einfach ein remove all verwende?

naja, wenn Die Buttons nicht mehr da sind, wird schon keiner mehr draufdrücken ;-)