Probleme mit GUI

[kulturfenster]

Hallo zusammen,

ich schriebe gerade ein Progrämmchen, womit man per Knopfdruck die Hintergrundfarbe verändern können soll.

ich bin folgendermassen vorgegangen:

1) bigPanel mit BorderLayout kreiert

2) northPanel mit FlowLayout kreiert

3) colorPanel kreiiert

4) Add Buttons ins northPanel

5) bigPanel(northPanel, BorderLayout.NORTH)

6) bigPanel(colorPanel, BorderLayout.CENTRAL)

7) frame.add(bigPanel)

Probleme:

- es werden nur 2 anstatt 3 Buttons angezeigt. Der letzte fehlt.
- die Knöpfe haben keine Wirkung, obwohl ich ihnen allen einen ACtionListener angehängt habe, der mittels

colorPanel.setBackground(Color.RED)

die Hintergrundfarbe verändern sollte.

Sieht jemand ein Problem im Aufbau? Oder muss ich mal ein bisschen Code posten?

 

[DellCapone]

kannst du bitte den ganzen code posten

 

[OneAndZero]

die add Methode auf ein JFrame sollte nicht aufgerufen werden...

besser:

public class Test extends JFrame{
public Test(){

this.getContentPane.addComponent(bla);
}
}

so ausm Kopf eben

 

[Quaxli]

Nach colorPanel.setBackground(...) fehlt noch ein repaint();

 

[L-ectron-X]

die add Methode auf ein JFrame sollte nicht aufgerufen werden...

Warum?

 

[SlaterB]

und wenn schon getContentPane(), dann dort zumindest auch die add()-Methode,
addComponent() gibts net

 

[OneAndZero]

zitat:

Neben add darf auch die Methode setLayout nicht mehr direkt auf dem Hauptfenster aufgerufen werden. Analog zu add ist statt dessen getContentPane().setLayout() aufzurufen. Ein Verstoß gegen diese Regel löst in beiden Fällen eine Exception aus.

ja, stimmt ist add() ich sag ja war ausm Kopf wusste nicht ob das da jetzt add() oder addComponent() war... aber danke

 

[SlaterB]

das stammt wohl aus
http://gd.tuwien.ac.at/languages/java/GoToJava2/html/k100225.html
aus dem Jahre 2000,


2008 ist das aber nicht mehr so, genauer: ab Java 1.5, wenn ich mich recht erinnere

 

[L-ectron-X]

@OneAndZero: Zeig mir mal die Quelle von dem Unsinn.

Dein Zitat bezieht sich dann doch wohl eher auf Java-Versionen von 1.2 bis 1.4.2.
Ab Java 1.5 ist das dann wieder völlig legitim.

Edit: Stimmt genau, SlaterB.

 

[OneAndZero]

stammt aus

Handbuch der Java-Programmierung, 5. Auflage, Addison Wesley, Version 5.0.2

wobei es der selbe Text ist... schon komisch!

Habe eben auch nochmal nachgelesen! Stimmt, ab 1.5 ist das nicht mehr notwendig das der Befehl add intern an die ContentPane weitergeleitet wird...
Scheiß Gewohnheit!

danke, man lernt nie aus

 

[kulturfenster]

ich stör nur ungern, aber hier ist mein Code:

repaint hat auch nix gebracht.

import javax.swing.JFrame;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import javax.swing.JButton;
import javax.swing.JPanel;
import java.awt.Color;
import java.awt.BorderLayout;
import java.awt.FlowLayout;

public class GUI {

    public static void main(String[] args)
    {
        JFrame frame = new JFrame("Farbenfroh");
        JButton blue_button = new JButton("Blue");
        JButton red_button = new JButton("Red");
        JButton yellow_button = new JButton("Yellow");
       
        JPanel bigPanel = new JPanel(new BorderLayout());
        JPanel northPanel = new JPanel(new FlowLayout());
        final JPanel colorPanel = new JPanel();
       
       
        class MakeItRed implements ActionListener
        {
            public void actionPerformed(ActionEvent e)
            {
                colorPanel.setBackground(Color.RED);
                colorPanel.repaint();
            }
        }
       
        class MakeItBlue implements ActionListener
        {
            public void actionPerformed(ActionEvent e)
            {
                colorPanel.setBackground(Color.BLUE);
                colorPanel.repaint();
            }
        }
       
        class MakeItYellow implements ActionListener
        {
            public void actionPerformed(ActionEvent e)
            {
                colorPanel.setBackground(Color.YELLOW);
                colorPanel.repaint();
            }
        }
       
        ActionListener blue_listener = new MakeItBlue();
        ActionListener red_listener = new MakeItRed();
        ActionListener yellow_listener = new MakeItYellow();
       
        blue_button.addActionListener(blue_listener);
        red_button.addActionListener(red_listener);
        yellow_button.addActionListener(yellow_listener);
       
        northPanel.add(blue_button);
        northPanel.add(red_button);
        northPanel.add(yellow_button);
       
        bigPanel.add(colorPanel, BorderLayout.CENTER);
        bigPanel.add(northPanel, BorderLayout.NORTH);
       
        frame.add(bigPanel);
       
        frame.setSize(200,200);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);
    }
}

 

[Verjigorm]

Zieh mal das Fenster breiter

 

[SlaterB]

bei mir ändert sich die Farbe,

alternativ zum Breiter-Ziehen kannst du das Nord-Panel auch vergrößern:
northPanel.setPreferredSize(new Dimension(50,80));

ob das auch automatisch geht, kann ich nicht sagen

 

[Quaxli]

Das ist ja alles in die main-Methode reingefrickelt. ???:L

Gegenvorschlag:

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

public class GUI extends JFrame implements ActionListener{

	private static final long	serialVersionUID	= 1L;
	
	JButton blue_button;
	JButton red_button;
	JButton yellow_button;
	
	JPanel bigPanel;
	JPanel northPanel;
	JPanel colorPanel;
	
	public static void main(String[] args) {
		new GUI();
	}
	
	public GUI(){
		setSize(400, 200);
		setLocation(100,100);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		
		blue_button = new JButton("Blue");
		red_button = new JButton("Red");
		yellow_button = new JButton("Yellow");

		bigPanel = new JPanel(new BorderLayout());
		northPanel = new JPanel(new FlowLayout());
		colorPanel = new JPanel();

		blue_button.addActionListener(this);
		blue_button.setActionCommand("blue");
		red_button.addActionListener(this);
		red_button.setActionCommand("red");
		yellow_button.addActionListener(this);
		yellow_button.setActionCommand("yellow");

		northPanel.add(blue_button);
		northPanel.add(red_button);
		northPanel.add(yellow_button);

		bigPanel.add(colorPanel, BorderLayout.CENTER);
		bigPanel.add(northPanel, BorderLayout.NORTH);

		add(bigPanel);
    
		setVisible(true);
	}

	public void actionPerformed(ActionEvent e) {
		
		if(e.getActionCommand().equals("red")){
			colorPanel.setBackground(Color.RED);
			this.repaint();
		}

		if(e.getActionCommand().equals("blue")){
			colorPanel.setBackground(Color.BLUE);
			this.repaint();
		}
		
		if(e.getActionCommand().equals("yellow")){
			colorPanel.setBackground(Color.YELLOW);
			repaint();
		}		
		
	}
}

 

[SlaterB]

statt ActionCommand könnte man übrigens auch direkt die Objecte vergleichen

Object source = e.getSource();
if (source == blue_button) {
...
} else {
...
}

 

[kulturfenster]

himmel, was läuft heute noch alles schief? hab eben gepostet und nix passiert.

also, bei mir scheint eclipse nicht richtig zu funktionieren.
GUI Applikationen stürzen regelmässig ab (not responding) und auch sonst läuft Eclipse extrem langsam.

Ich verwende eine mittelalten Pentium mit Ubuntu 8.10, Compiz deaktiviert und Eclipse anhand dieser Anleitung installiert.

Jemand eine Ahnung, was da nicht stimmen könnte?