SWING Anwendung mit 3 Buttons zum ändern der Farbe

[dasie]

Hallo an alle!

Wir sollen als Aufgabe eine Swing Anwendung mit 3 Buttons (zum ändern der Farbe) erstellen. Leider bringt mir der Compiler immer noch zwei Fehler, hab keinen blassen schimmer ob ich den ActionListener falsch implementiert habe oder ob da irgendwas anderes nicht stimmt. Wäre nett, wenn mir jemand einen Tip geben könnte. Danke schon mal. Gruß, dasie

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

public class ButtonBox implements ActionListener {
	public static void main (String args[]) {
		JFrame bb = new JFrame("ButtonBox");	
		JPanel pa = new JPanel( new GridLayout (3, 1));
		JButton g = new JButton("faerbe Buttons gelb");
		g.addActionListener(this);
		JButton r = new JButton("faerbe Buttons rot");
		r.addActionListener(this);
		JButton b = new JButton("faerbe Buttons blau");
		b.addActionListener(this);
		pa.add(g);
		pa.add(r);
		pa.add(b);
		
		bb.setForeground(Color.black);
		b.setForeground(Color.black);
		b.setBackground(Color.yellow);   
		g.setForeground(Color.black);
		g.setBackground(Color.yellow);
		r.setForeground(Color.black);
		r.setBackground(Color.yellow);
 
		bb.getContentPane().add(pa);
		bb.setVisible(true);
		bb.pack();
		
			public void actionPerformed(ActionEvent e){
  		
	   			if(e.getActionCommand()== "faerbe Buttons gelb"){
    	   		g.setBackground(Color.yellow);	
		   		r.setBackground(Color.yellow);	
		   		b.setBackground(Color.yellow);	
    	   		}
    	   	
    	   		if(e.getActionCommand() == "faerbe Buttons rot"){
    	   		g.setBackground(Color.red);	
			   	r.setBackground(Color.red);	
			   	b.setBackground(Color.red);	
	    	   	}
	    	   	
	    	   	if(e.getActionCommand()== "faerbe Buttons blau"){
	    	   	g.setBackground(Color.blue);	
			   	r.setBackground(Color.blue);	
			   	b.setBackground(Color.blue);	
	    	   	}
			}
	}	
}

 

[Wildcard]

Währe noch nett die Fehler zu posten. Der Compiler ist da schneller als ich :wink:

 

[Wildcard]

Ok, was mir auf Anhieb auffällt, ist das hier:

g.addActionListener(this);

die main Methode ist statisch, kennt also kein Objekt das man mit 'this' ansprechen könnte.
Du brauchst also ein ButtonBox Objekt.

EDIT:
und wenn ich deine Klammern richtig zähle, befindet sich die actionPerformed IN der main Methode :autsch:

 

[dasie]

Hi, also das sind die Fehler

---------- Capture Output ----------
> "C:\Programme\Java\jdk1.5\bin\javac.exe" ButtonBox.java
ButtonBox.java:38: illegal start of expression
public void actionPerformed(ActionEvent e){
^
ButtonBox.java:58: ';' expected
}
^
2 errors
> Terminated with exit code 1.

 

[dasie]

Ok, was mir auf Anhieb auffällt, ist das hier:

g.addActionListener(this);

die main Methode ist statisch, kennt also kein Objekt das man mit 'this' ansprechen könnte.
Du brauchst also ein ButtonBox Objekt.

EDIT:
und wenn ich deine Klammern richtig zähle, befindet sich die actionPerformed IN der main Methode :autsch:

bin anfänger :bahnhof:
Wo mu... ss die AktionPerfo.. den hin? Hinter die main Methode? da kommen jetzt aber viel mehr fehler :shock:

 

[Wildcard]

Du kannst keine Methode in einer Methode machen.
Zumindest die Syntax solltest du kennen.
Kommen noch mehr Fehler?

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

public class ButtonBox implements ActionListener {
   public static void main (String args[]) {
      ButtonBox box = new ButtonBox();
      JFrame bb = new JFrame("ButtonBox");   
      JPanel pa = new JPanel( new GridLayout (3, 1));
      JButton g = new JButton("faerbe Buttons gelb");
      g.addActionListener(box);
      JButton r = new JButton("faerbe Buttons rot");
      r.addActionListener(box);
      JButton b = new JButton("faerbe Buttons blau");
      b.addActionListener(box);
      pa.add(g);
      pa.add(r);
      pa.add(b);
      
      bb.setForeground(Color.black);
      b.setForeground(Color.black);
      b.setBackground(Color.yellow);   
      g.setForeground(Color.black);
      g.setBackground(Color.yellow);
      r.setForeground(Color.black);
      r.setBackground(Color.yellow);

      bb.getContentPane().add(pa);
      bb.setVisible(true);
      bb.pack();
      

   }   

         public void actionPerformed(ActionEvent e){
        
               if(e.getActionCommand()== "faerbe Buttons gelb"){
                g.setBackground(Color.yellow);   
               r.setBackground(Color.yellow);   
               b.setBackground(Color.yellow);   
                }
             
                if(e.getActionCommand() == "faerbe Buttons rot"){
                g.setBackground(Color.red);   
               r.setBackground(Color.red);   
               b.setBackground(Color.red);   
                }
                
                if(e.getActionCommand()== "faerbe Buttons blau"){
                g.setBackground(Color.blue);   
               r.setBackground(Color.blue);   
               b.setBackground(Color.blue);   
                }
         }
}

 

[Gast]

guck mal in der faq/buch/forensuche/google was static ist

 

[dasie]

gelöscht

 

[Wildcard]

Die ersten 3 hab ich dir doch schon korrigiert :autsch:

 

[dasie]

Danke! Oh je, ist es denn so schlimm

 

[Wildcard]

Naja, die Syntax solltest du schon können.

 

[dasie]

Ok, so schauts jetzt erst mal aus

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

public class ButtonBox implements ActionListener {
	public static void main (String args[]) {
		
		ButtonBox bbox = new ButtonBox();
		JFrame bb = new JFrame("ButtonBox");	
		JPanel pa = new JPanel( new GridLayout (3, 1));
		JButton g = new JButton("faerbe Buttons gelb");
		g.addActionListener(bbox);
		JButton r = new JButton("faerbe Buttons rot");
		r.addActionListener(bbox);
		JButton b = new JButton("faerbe Buttons blau");
		b.addActionListener(bbox);
		pa.add(g);
		pa.add(r);
		pa.add(b);
		
		bb.setForeground(Color.black);
		b.setForeground(Color.black);
		b.setBackground(Color.yellow);   
		g.setForeground(Color.black);
		g.setBackground(Color.yellow);
		r.setForeground(Color.black);
		r.setBackground(Color.yellow);
 
		bb.getContentPane().add(pa);
		bb.setVisible(true);
		bb.pack();
		
	}	
		
	public void actionPerformed(ActionEvent e){
  		
	 	if(e.getActionCommand()== "faerbe Buttons gelb"){
    	
		g.setBackground(Color.yellow);	
		r.setBackground(Color.yellow);	
		b.setBackground(Color.yellow);	
    	}
    	   	
    	if(e.getActionCommand() == "faerbe Buttons rot"){
    	g.setBackground(Color.red);	
		r.setBackground(Color.red);	
		b.setBackground(Color.red);	
	    }
	    	   	
	    if(e.getActionCommand()== "faerbe Buttons blau"){
	    g.setBackground(Color.blue);	
		r.setBackground(Color.blue);	
		b.setBackground(Color.blue);	
	    }
	}
}

 

[Wildcard]

Das kann immer noch nicht gehen, weil es in einem statischen Kontext kein 'this' gibt! :!:

 

[dasie]

wie kann ich mit der ActionEvent nun auf die g, b, r zugreifen? mit super?

 

[Wildcard]

Nein, dafür musst du sie als Instanzvariablen der Klasse deklarieren. (Also über der main-Methode)

 

[dasie]

Danke für deine Hilfe! Das kapier ich heute aber heute nicht mehr. Da mach ich morgen weiter! Aber du hast mir schon sehr geholfen. gruß, daniela

 

[Leroy42]

Bevor wir hier alles auseinanderfriemeln gebe ich dir
mal den Code der deiner Intention entspricht:

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

public class Test extends JFrame implements ActionListener { 
    JButton g = new JButton("faerbe Buttons gelb"); 
    JButton r = new JButton("faerbe Buttons rot"); 
    JButton b = new JButton("faerbe Buttons blau"); 

    public Test () { 
        JPanel pa = new JPanel( new GridLayout (3, 1)); 
        g.addActionListener(this); 
        r.addActionListener(this); 
        b.addActionListener(this); 
        pa.add(g); 
        pa.add(r); 
        pa.add(b); 
         
        b.setForeground(Color.black); 
        b.setBackground(Color.yellow);    
        g.setForeground(Color.black); 
        g.setBackground(Color.yellow); 
        r.setForeground(Color.black); 
        r.setBackground(Color.yellow); 

        getContentPane().add(pa); 
        pack(); 
        setVisible(true); 
         
     }    
       
    public void actionPerformed(ActionEvent e) { 
    	
    	if(e.getActionCommand().equals("faerbe Buttons gelb")){ 
    		
    		g.setBackground(Color.yellow);    
    		r.setBackground(Color.yellow);    
    		b.setBackground(Color.yellow);    
    	} 
    	
    	if(e.getActionCommand().equals("faerbe Buttons rot")){ 
    		g.setBackground(Color.red);    
    		r.setBackground(Color.red);    
    		b.setBackground(Color.red);    
    	} 
    	
    	if(e.getActionCommand().equals("faerbe Buttons blau")){ 
    		g.setBackground(Color.blue);    
    		r.setBackground(Color.blue);    
    		b.setBackground(Color.blue);    
    	} 
    } 
   public static void main (String args[]) {
   	new Test();
    }    
}

Versuche ihn zu verstehen und melde dich bei spezifischen Fragen.

 

[Guest]

Vielen Dank Leroy. Das muss ich mir jetzt mal genau anschauen. Aber dein Angebot werd ich sicher wahrnehmen

 

[Gast]

denk aber dran, vor der Abgabe noch einen JFrame dazuzumachen. Denn wenn du aufs Übungsblatt schaust, steht da in der Titelleiste der Name (ButtonBox)