2 Frames Farbwahl durch Buttons

[Chucky]

Hallo,
habe mich jetzt lange Zeit mit AWT beschäftigt und wollt jetzt mal bissl SWING dazu lernen.
Als Aufgabe hatte ich mir gesetzt 2 Frames zu erzeugen. Der eine Hauptframe sollte Buttons
enthalten mit denen man die Hintergrundfarbe des Nebenframes steuern kann.

Nun stehe ich aber dem Problem, dass er meinen Nebenframe in der Ereignisbehandlungsmethode meiner Buttons
nicht erkennt. Komme an dieser Stelle nicht weiter, weiss aber das dieses Problem nichts mit Swing zu tun hat :lol:

Wäre nett wenn ihr mir weiterhelfen könntet.

Hier der Code:

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

public class Swingprobe extends JFrame{
	
	JButton rot;
	JButton grün;
	JButton blau;
	JButton gelb;
	JButton grau;
	static Nebenfenster secondwind;
	
	AktionenLauscher hoerzu=new AktionenLauscher();
	
	public Swingprobe(){
		
		setTitle("Farbwahl");
		getContentPane().setBackground(Color.BLUE);
		
		rot=new JButton("Rot");
		grün=new JButton("Grün");
		blau=new JButton("Blau");
		gelb=new JButton("Gelb");
		grau=new JButton("Grau");
		
		rot.addActionListener(hoerzu);
		grün.addActionListener(hoerzu);
		blau.addActionListener(hoerzu);
		gelb.addActionListener(hoerzu);
		grau.addActionListener(hoerzu);
		
		getContentPane().add(rot);
		getContentPane().add(grün);
		getContentPane().add(blau);
		getContentPane().add(gelb);
		getContentPane().add(grau);
		
		setLayout(new FlowLayout());
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
	}
	
	public static void main(String args[]){
		
		Swingprobe hauptfenster=new Swingprobe();
		hauptfenster.setSize(200,200);
		hauptfenster.setLocation(0,0);
		hauptfenster.setVisible(true);
		
		
		secondwind=new Nebenfenster();
		secondwind.setSize(200,200);
		secondwind.setLocation(200,200);
		secondwind.setVisible(true);
		
	}
}

class Nebenfenster extends JFrame{	
	public Nebenfenster(){
		setTitle("Farbscreen");
		getContentPane().setBackground(Color.WHITE);
	}
}

class AktionenLauscher implements ActionListener{
	public void actionPerformed(ActionEvent e){
		String s=e.getActionCommand();
		if(s=="Rot")
			secondwind.getContentPane().setBackground(Color.RED);
			//weitere Farben folgen
	}
}

Und hier die Fehlerbeschreibung laut Eclipse:

Exception in thread "AWT-EventQueue-0" java.lang.Error: Unresolved compilation problem: 
	secondwind cannot be resolved

	at AktionenLauscher.actionPerformed(Swingprobe.java:71)

Bin gerne für ALLE Verbesserungsvorschläge offen.

Vielen Dank schonmal Bob

 

[Reality]

Es muss so heissen:

public void actionPerformed(ActionEvent e){
      Object o = e.getSource();
      if(o == rot)
         secondwind.getContentPane().setBackground(Color.RED);
         //weitere Farben folgen
   }

Liebe Grüße
Reality

 

[Chucky]

Hm funktioniert leider auch nicht aber ist doch auch egal ob ich
jetzt per e.getSource (Objekt) oder e.getActionCommand (Aufschrift des Buttons)vergleiche oder? :/

Mfg Bob

 

[Roar]

Hm funktioniert leider auch nicht aber ist doch auch egal ob ich
jetzt per e.getSource (Objekt) oder e.getActionCommand (Aufschrift des Buttons)vergleiche oder? :/

Mfg Bob

ja, eigentlich schon, nur wird dein vergleich auf == "rot" immer false ergeben (s. FAQ) und warum ist secondwind statisch? initialisiere es im konstruktor von Swingprobe

 

[Chucky]

Wie gewünscht geändert:

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

public class Swingprobe extends JFrame{
	
	JButton rot;
	JButton grün;
	JButton blau;
	JButton gelb;
	JButton grau;
	
	AktionenLauscher hoerzu=new AktionenLauscher();
	
	public Swingprobe(){
		Nebenfenster secondwind;
		secondwind=new Nebenfenster();
		secondwind.setSize(200,200);
		secondwind.setLocation(200,200);
		secondwind.setVisible(true);
		
		setTitle("Farbwahl");
		getContentPane().setBackground(Color.BLUE);
		
		rot=new JButton("Rot");
		grün=new JButton("Grün");
		blau=new JButton("Blau");
		gelb=new JButton("Gelb");
		grau=new JButton("Grau");
		
		rot.addActionListener(hoerzu);
		grün.addActionListener(hoerzu);
		blau.addActionListener(hoerzu);
		gelb.addActionListener(hoerzu);
		grau.addActionListener(hoerzu);
		
		getContentPane().add(rot);
		getContentPane().add(grün);
		getContentPane().add(blau);
		getContentPane().add(gelb);
		getContentPane().add(grau);
		
		setLayout(new FlowLayout());
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
	}
	
	public static void main(String args[]){
		
		Swingprobe hauptfenster=new Swingprobe();
		hauptfenster.setSize(200,200);
		hauptfenster.setLocation(0,0);
		hauptfenster.setVisible(true);

		
	}
}

class Nebenfenster extends JFrame{	
	public Nebenfenster(){
		setTitle("Farbscreen");
		getContentPane().setBackground(Color.WHITE);
	}
}

class AktionenLauscher implements ActionListener{
	public void actionPerformed(ActionEvent e){
		Object s=e.getSource();
		if(s==rot)
			secondwind.getContentPane().setBackground(Color.RED);
			//weitere Farben folgen
	}
}

Fehlermeldung immernoch "secondwind cannot be resolved"
und dazu auch noch "rot cannot be resolved" (auch in der Klasse AktionenLauscher)

Exception in thread "AWT-EventQueue-0" java.lang.Error: Unresolved compilation problems: 
	rot cannot be resolved
	secondwind cannot be resolved

	at AktionenLauscher.actionPerformed(Swingprobe.java:69)

Mfg Bob

 

[Reality]

Ahh! Du musst auch nicht die Fehler kopieren, die ich mache.

Object s=e.getSource();
      if(o==rot)
         secondwind.getContentPane().setBackground(Color.RED);
         //weitere Farben folgen
   }

Also die Variable o und nicht s vergleichen!

Liebe Grüße
Reality

 

[Chucky]

Jetzt machst du ein Object s
und vergleichst mit o???????

:bahnhof:

Wenn du das meinst:

public void actionPerformed(ActionEvent e){
		Object s=e.getSource();
		if(s==rot)
			secondwind.getContentPane().setBackground(Color.RED);
			//weitere Farben folgen
	}

funktioniert nicht

Bob

 

[Reality]

Ups. hab das mit meinem Quellcode und deinem verwechselt.
Ich kompiliert mal und schau nach.
Bis gleich

 

[Reality]

So:
Entweder du machst eine innere Klasse oder du machst die Variablen static und greifst dann darauf zu.

Ich habe die innere Klasse genommen:

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

public class Swingprobe extends JFrame{

   JButton rot;
   JButton grün;
   JButton blau;
   JButton gelb;
   JButton grau;

   AktionenLauscher hoerzu=new AktionenLauscher();
   Nebenfenster secondwind;
   
   public Swingprobe(){
      
      secondwind=new Nebenfenster();
      secondwind.setSize(200,200);
      secondwind.setLocation(200,200);
      secondwind.setVisible(true);

      setTitle("Farbwahl");
      getContentPane().setBackground(Color.BLUE);

      rot=new JButton("Rot");
      grün=new JButton("Grün");
      blau=new JButton("Blau");
      gelb=new JButton("Gelb");
      grau=new JButton("Grau");

      rot.addActionListener(hoerzu);
      grün.addActionListener(hoerzu);
      blau.addActionListener(hoerzu);
      gelb.addActionListener(hoerzu);
      grau.addActionListener(hoerzu);

      getContentPane().add(rot);
      getContentPane().add(grün);
      getContentPane().add(blau);
      getContentPane().add(gelb);
      getContentPane().add(grau);

      setLayout(new FlowLayout());
      setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

   }

   public static void main(String args[]){

      Swingprobe hauptfenster=new Swingprobe();
      hauptfenster.setSize(200,200);
      hauptfenster.setLocation(0,0);
      hauptfenster.setVisible(true);


   }
   class Nebenfenster extends JFrame{   
   public Nebenfenster(){
      setTitle("Farbscreen");
      getContentPane().setBackground(Color.WHITE);
   }
}

class AktionenLauscher implements ActionListener{
   public void actionPerformed(ActionEvent e){
      Object o = e.getSource();
      if(o == rot)
         secondwind.getContentPane().setBackground(Color.RED);
         //weitere Farben folgen
   }
}

}

Liebe Grüße
Reality

 

[Guest]

Und ich hatte es doch oben mit der static deklaration versucht.
Was war denn daran falsch?

Mfg Bob

 

[Reality]

static ist nur erwünscht, wenn du willst, dass für alle Instanzen derselbe Variablen-Wert vorliegt; ansonsten kommt es zu Problemen.

Liebe Grüße
Reality