Swing JFrame nimmt keine anderen Elemente auf

[Captain83]

Hallo,

ich habe ein Problem mit der Darstellung einer Ampel. Vermutlich sind es Noob-Probleme, ich komme aber einfach nicht weiter. Nach stundenlangem Lesen von sowohl Javainsel und Teia-Lehrbuch als auch dem Stöbern in zig Foren und Tutorials, bin ich nun vollkommen verwirrt. ???:L
Ich versuche eine Ampel darzustellen... Datenbankzugriff, Auswertung, alles funktioniert(getestet mit println()s... Aber die Darstellung haut einfach nicht hin....
Ich habe eine Klasse Ampel... hier funktionert das mit dem Container nicht, kann mir jemand sagen, wieso?

public class Ampel extends JFrame {
	public enum Ampelzustand {ROT, GRÜN, STÖRUNG};
	Ampelzustand z;
	Container con;	
	
	Ampel() {
		super("Warenkorb - Füllstandsanzeige");
		this.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE);
		con = getContentPane();
		this.con = con;
		this.setSize(1680,1050);
		this.setVisible(true);
		this.setResizable(false);
		con.setBackground(Color.BLACK);
			
	}
	
	public void setZustand(String zustand) {
		if (Ampelzustand.GRÜN.toString().equals(zustand)) {
			System.out.println("GRÜN");
			
			Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.BLACK);
			getContentPane().add(lampeOben);
			//con.add(lampeOben);
			Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.GREEN);
			getContentPane().add(lampeUnten);
			//con.add(lampeUnten);
			
		}
		
		else if (Ampelzustand.ROT.toString().equals(zustand)) {
			System.out.println("ROT");
			
			Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.RED);
			con.add(lampeOben);
			Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.BLACK);
			con.add(lampeUnten);
		}
		
		else {
			System.out.println("Gelb blinkend");
			try {
				
				Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.BLACK);
				con.add(lampeOben);
				Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.YELLOW);
				con.add(lampeUnten);
				
				Thread.sleep(1000);
				
				/*
				Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.YELLOW);
				con.add(lampeOben);
				Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.BLACK);
				con.add(lampeUnten);
				*/
				
			}
			
			catch (Exception e) {
				e.printStackTrace();
			}
		
		}
		
	}
	
}

public class Ampellicht extends JPanel {
	int x;
	int y;
	int width;
	int height;
	Color color;
		
	Ampellicht(int x, int y, int width, int height, Color color) {
		this.x = x;
		this.y = y;
		this.width = width;
		this.height = height;
		this.color = color;
			
	}
	
	protected void paintComponent(Graphics g) {
		super.paintComponents(g);
		
		g.setColor(color);
		g.fillOval(x, y, width, height);
		
	}
	
}

Die Methode paintComponent wird gar nicht aufgerufen soweit ich das analysieren konnte. Wieso, wiederum ka....

Vielen Dank für eure Hilfe im Voraus

 

[Fab1]

Hi,

mach hier mal das "s" weg, vielleicht geht es dann.

 protected void paintComponent(Graphics g) {
        super.paintComponents(g); // hier super.paintComponent(g); glaub daran liegt es
        
        g.setColor(color);
        g.fillOval(x, y, width, height);
        
    }

 

[Captain83]

Danke Fab1, das war auf jeden Fall schon mal ein Fehler...

Bleibt immer noch das Problem mit dem Container...
Ich habe leider keine Ahnung, was daran nicht stimmt....

 

[Fab1]

Also ich habe noch nie mit so einem Container wie du hier gearbeitet.

Aber ich würde es so machen. So wie du die Ampel Objekte erzeugst brauchst du auf alle Fälle mal ein Null-Layout beim JFrame. Da du ja beim erzeugen die X und Y Koords. angegeben hast.

also einfach setLayout(null); in den Konstruktor vom JFrame rein.

überall wo du das con verwendest kannst ja mal auskommentieren.

anschließend

    Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.BLACK);
             add(lampeOben);
            
            Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.GREEN);
            add(lampeUnten);

Nach meinem glauben sollte es jetzt schon funktionieren, kann es nur gerade nicht testen. Bei den else-if und dem else natürlich auch entsprechend ändern.

Hoffe es funzt, ansonsten schau ich daheim nochmal drüber.

 

[xehpuk]

Ein JFrame hat standardmäßig das BorderLayout. Da fügt man nicht einfach so Komponenten hinzu.
Du solltest dich allgemein mit LayoutManagers beschäftigen. Dazu findet sich genug im Netz.
Passend wäre in deinem Fall wohl BoxLayout oder GridLayout.

g.fillOval(x, y, width, height);

wird relativ zur Komponente zeichnen. Du solltest da für x und y eher 0 haben.

 

[Captain83]

Ich wollte eigentlich nur zum Testen jetzt mal die Grafiken einfügen... Später soll das ganze natürlich in einen Layoutmanager, es kommen ja auch noch einige Dinge dazu....
Es sind 2 Lampen, eine oben, eine unten.... desshalb auch die Koords 0,0 bzw. 0,450...
Es sollte doch zum Testen eigentlich wenigstens die 2 Lampen anzeigen, oder?

Ich habe es nach deinem Vorschlag abgeändert Fab1. Beim Debuggen springt er aber nie in die paintComponent()-Methode hinein, das wird wohl das Problem sein. Aber ich weiß nicht wieso... Soweit ich weiß, muss ich die Methode doch nicht explizit aufrufen , oder?

public class Ampel extends JFrame {
	public enum Ampelzustand {ROT, GRÜN, STÖRUNG};
	Ampelzustand z;
		
	Ampel() {
		super("Warenkorb - Füllstandsanzeige");
		setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE);
		setSize(1680,1050);
		setVisible(true);
		setResizable(false);
		setLayout(null);
		getContentPane().setBackground(Color.BLACK);
			
	}
	
	public void setZustand(String zustand) {
		if (Ampelzustand.GRÜN.toString().equals(zustand)) {
			System.out.println("GRÜN");
			
			Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.BLACK);
			add(lampeOben);
			Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.GREEN);
			add(lampeUnten);
			
		}
		
		else if (Ampelzustand.ROT.toString().equals(zustand)) {
			System.out.println("ROT");
			
			Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.RED);
			add(lampeOben);
			Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.BLACK);
			add(lampeUnten);
		}
		
		else {
			System.out.println("Gelb blinkend");
			try {
				
				Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.BLACK);
				add(lampeOben);
				Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.YELLOW);
				add(lampeUnten);
				
				Thread.sleep(1000);
				
				/*
				Ampellicht lampeOben = new Ampellicht(0, 0, 400, 400, Color.YELLOW);
				con.add(lampeOben);
				Ampellicht lampeUnten = new Ampellicht(0, 450, 400, 400, Color.BLACK);
				con.add(lampeUnten);
				*/
				
			}
			
			catch (Exception e) {
				e.printStackTrace();
			}
		
		}
		
	}
	
}

 

[xehpuk]

setLayout(null)

bitte schnell wieder vergessen.

Es sind 2 Lampen, eine oben, eine unten.... desshalb auch die Koords 0,0 bzw. 0,450...
Es sollte doch zum Testen eigentlich wenigstens die 2 Lampen anzeigen, oder?

Wieso sollte er das machen? Die "Koordinaten" sind lediglich Instanzvariablen in deiner Ampellicht-Klasse. Aufs Layout haben sie also keinen Einfluss.

Lektüre: Lesson: Laying Out Components Within a Container (The Java™ Tutorials > Creating a GUI With JFC/Swing)

 

[Captain83]

Hm was das Layout angeht, hast du wohl Recht... Ich guck mir trotzdem mal deinen Link an.

Aber es müsste doch wenigstens irgendwas, vielleicht sich was überlagerndes oder sowas, erzeugt werden. Bei mir tut sich leider gar nix..... Es wird ja nicht mal die paintComponent()-Methode aufgerufen...

 

[bERt0r]

Am besten machst du das so: Du hast 2 (oder besser wohl 3) Ampellichter. Die fügst du einmal im Konstruktor des Frames ein, merkst dir die Instanzen aber in einer Klassenvariable.
Verpass deiner Ampellicht Klasse eine setColor Methode.
Ändert sich jetzt dein Ampelzustand, erzeugst du keine neuen Ampellichter, sonder änderst einfach die Farbe deiner bestehenden.
Zum Testen würde ich dir erstmal einfach ein FlowLayout empfehlen.

edit: Du solltest ausserdem noch die PreferredSize deiner Ampellichter verändern. Standardmäßig sind die JPanels 2x2 Pixel groß.

 

[Captain83]

Danke bERt0r, ich werd das gleich mal ausprobieren...
Trotzdem werd ich immer noch ned ganz schlau daraus, wieso wird die paintComponent()-Methode nie aufgerufen?

 

[L-ectron-X]

Die wird schon aufgerufen. Nur macht das nicht der Programmierer direkt. Sondern durch anfordern eines Neuzeichnens. Aber das ist situationsabhängig.
Java selbst ruft ansonsten die Methode auf, wenn es nötig wird.

 

[Captain83]

Guten Morgen allerseits,

zuerst noch einmal Dankeschön für eure Antworten. Ich rieche förmlich, dass die Lösung des Problems naht , naja hoffen wir es zumindest mal :lol:
Hm, was ist denn dann an meiner Ampellicht-Klasse so falsch, dass die paintComponent()-Methode nicht aufgerufen wird`?

Ist ein JPanel für ein Ampellicht überhaupt der richtige Container oder sollte ich lieber Canvas nehmen?

 

[bERt0r]

int x;
    int y;
    int width;
    int height;

Du solltest dir andere Variablennamen suchen, diese gibts schon in JComponent bzw. JPanel. Ausserdem solltest du die Größe deines Panels mit setSize und/oder setPreferredSize auf deine Größe anpassen.

 

[Captain83]

Hey, wie geil, es funtioniert. :toll:
Ich danke euch allen für eure Antworten und ein extra großes Danke nochmal an bERt0r :applaus: