Wo liegt der Fehler?

[DerVerwirrte]

Hallo Leute!
Ich muss für die Schule ein Programm schreiben, das Buttons enthält, bei deren Bedienung sich die Hintergrundfarbe des Programms ändert. Ich hab es auch schon geschrieben, aber irgendwie will der ActionListener nicht funktionieren.

So sieht es aus:

import java.awt.*;

public class Aufg2 extends Frame
{
 Panel pnl=new Panel(new GridLayout(4,2));
 Button rot=new Button("rot");
 Button rot(10);
 Button blau=new Button("blau");
 Button gruen=new Button("gruen");
 Button pink=new Button("pink");
 Button orange=new Button("orange");
 Button gelb=new Button("gelb");
 Button cyan=new Button("cyan");
 
 derWindowListener myWindowListener=new derWindowListener();
 derActionListener myActionListener=new derActionListener(this);
 
 public Aufg2()
 {
   super("");
   
   this.pnl.add(rot);
   this.pnl.add(blau);
   this.pnl.add(gruen);
   this.pnl.add(pink);
   this.pnl.add(orange);
   this.pnl.add(gelb);
   this.pnl.add(cyan);

   this.add(pnl,BorderLayout.CENTER);
   
   this.addWindowListener(new derWindowListener());
   
   rot.addActionListener(myActionListener);
   blau.addActionListener(myActionListener);
   //gruen.addActionListener(derActionListener());
   //pink.addActionListener(derActionListener());
   //orange.addActionListener(derActionListener());
   //gelb.addActionListener(derActionListener());
   //cyan.addActionListener(derActionListener());
   
   this.setSize(300,300);
   this.setVisible(true);
 }
 public static void main(String[]args)
 {
   Aufg2 f=new Aufg2();
 }
}

import java.awt.*;
import java.awt.event.*;

public class derActionListener implements ActionListener
{
  public Aufg2 f;
  public derActionListener(Aufg2 f)
  {
    this.f=f;
  }
  public void actionPerformed(ActionEvent e)
  {
    if(e.getSource()==f.rot)
    {
     f.setBackground(Color.red);
    }
    else if(e.getSource()==f.blau)
    {
     f.setBackground(Color.blue);
    }
  }
}

Mein Java-Editor zeigt keine Fehlermeldungen an und laufen tut das Programm auch.
Aber warum verändert sich die Hintergrundfarbe nicht?
Wie kann ich die Buttons kleiner machen bzw nur so groß wie die Beschriftung?

DerVerwirrte

 

[Guest]

Ups hab nen Fehler gemacht.
Zeile 8 des ersten Quellcodes müsst Ihr löschen. Dann ist es lauffähig, funktioniert aber trotzdem nicht.

 

[lin]

Das Panel verdeckt das Frame, von welchem du die Farbe änderst...

Panel pnl=new Panel(new GridLayout(4,2));

 

[DerVerwirrte]

Achso alles klar . Vielen Dank.

Eine Frage hätte ich aber immernoch: Wie bekomm ich die Größe der Buttons kleiner?

 

[lin]

Unter Swing kannst du mit

setPreferredSize(new Dimension(int w, int h));

die Werte verändern, unter AWT, hm? setSize()?
Kannst natürlich auch einfach die grösse des Frames verkleinern, dann werden die Buttons auch kleiner...

Ich hatte gerade etwas Freude am proggen ;-)
Ist wahrscheinlich nicht besonders gut, ging sehr rasch, aber vielleicht kannst du ja doch mal einen Blick drauf werfen... Ist leider Swing und nicht AWT.

import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.*;

public class Aufg2 extends JFrame implements ActionListener {
	
static JButton disp = new JButton(); //display Button, zeigt Farbe an
	
	public Aufg2() {
		setTitle("Change Colors");
		setSize(300, 200);
		int x = (int) (0.5 * (getToolkit().getScreenSize().width - getWidth()));
		int y = (int) (0.5 * (getToolkit().getScreenSize().height - getHeight()));
		setLocation(x, y);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		JPanel pane = new JPanel(new GridLayout(4, 2));
		
		//Erstelle Buttons
		String[] lbl = { "rot", "blau", "grün", "pink", "orange", "gelb", "cyan" };
		JButton[] bt = new JButton[lbl.length];
		for( int i = 0; i < bt.length; i++ ) {
			bt[i] = new JButton(lbl[i]);
			bt[i].addActionListener(this);
			bt[i].setActionCommand(lbl[i]);
			pane.add(bt[i]);
		}
		pane.add(disp);
		
		getContentPane().add("Center", pane);
		setVisible(true);
	}
	
	public void actionPerformed(ActionEvent ae) {
		String source = ae.getActionCommand();
		Dimmer.col = disp.getBackground();
		
		if(source == "rot") {
			Dimmer.dest = Color.RED;
		} else if (source == "blau") {
			Dimmer.dest = Color.BLUE;
		} else if (source == "grün") {
			Dimmer.dest = Color.GREEN;
		} else if (source == "pink") {
			Dimmer.dest = Color.PINK;
		} else if (source == "orange") {
			Dimmer.dest = Color.ORANGE;
		} else if (source == "gelb") {
			Dimmer.dest = Color.YELLOW;
		} else {
			Dimmer.dest = Color.CYAN;
		} 
		//Dimmer Thread
		Dimmer dim = new Dimmer();
		Thread thr = new Thread(dim);
		thr.start();
	}
	
	//Methode welche die Farbwerte setzt
	public static void updateDisp(Color col) {
		disp.setBackground(col);
		disp.repaint();
	}
	
	public static void main(String[] args) {
		new Aufg2();
	}	
}

class Dimmer implements Runnable {
	static Color col, dest;
	long duration = 1000; //Dauer des Farbwechsels in ms
	int changes = 20; //Anzahl Farbwechsel
	
	//run Methode des Dimmer-Threads
	public void run() {
		//Berechne um wie viel die einzelnen Farbwerte verändert werden müssen
		int r = dest.getRed() - col.getRed(); 
		int g = dest.getGreen() - col.getGreen();
		int b = dest.getBlue() - col.getBlue();
	
		try {
			//Berechne um wieviel die einzelnen Farben erhöht werden müssen
			double rInc = r/changes, gInc= g/changes, bInc = b/changes;
			r = Math.abs(col.getRed()); g = Math.abs(col.getGreen()); b = Math.abs(col.getBlue());
			for ( int k = 0; k < 20; k++) {
				r += rInc;
				g += gInc;
				b += bInc;
				//updateDisp Methdoe wird aufgerufen
				Aufg2.updateDisp(new Color(r, g, b));
				//Thread wird schlafen gelegt für gewisse Zeit
				Thread.sleep(duration/changes);
			}
		} catch (InterruptedException ie) {
			System.out.println(ie);
		}
	}
}

 

[DerVerwirrte]

Dankeschön.