Menü addActionListener

[Sindbad1983]

hi...ich wollt jetzt mit Menüs arbeiten..hab aber irgendwie Probleme...

ich muss ja die MenuItems mit addActionListener(this) registrieren oder?
Das funktioniert aber irgendwie nicht!

Hab ich da was vergessen?

import java.awt.Color;
import java.awt.Container;
import java.awt.Menu;
import java.awt.MenuBar;
import java.awt.MenuItem;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowListener;
import java.awt.event.WindowAdapter;

import javax.swing.JFrame;

public class PaintFrame extends JFrame implements ActionListener{
	
	
	private Zeichenflaeche zeichenflaeche;
	Figure f;
	
	public PaintFrame(String titel){
		super(titel);
		
		
	
		
		zeichenflaeche=new Zeichenflaeche();
		getContentPane().add(zeichenflaeche); 
	    zeichenflaeche.setSize(getContentPane().getSize()); 
	    zeichenflaeche.setFocusable(true); 
	    zeichenflaeche.requestFocus(); 
	}
	
	
	public void makeMenu(){
		
	MenuBar bar=new MenuBar();
		
		
		 Menu m = new Menu("Look & Feel");
		 MenuItem red=new MenuItem("Red");
		 MenuItem blue=new MenuItem("Blue");
		 MenuItem green=new MenuItem("Green");
		
		 m.add(red);
		 red.addActionListener(this);
		 m.add(blue);
		 blue.addActionListener(this);
		 m.add(green);
		 green.addActionListener(this);
		 bar.add(m);
		 setMenuBar(bar);
		 System.out.println("ccc");
		 
		 
		
	
	}
	
	public void actionPerformed(ActionEvent e) {
		
		String menu=e.getActionCommand();
		
		if(menu.equals("Red")){
			f.setColor(Color.red);
			System.out.println("Rot wurde ausgewählt");
		}
		else if(menu.equals("Blue")){
			f.setColor(Color.blue);
			System.out.println("Blau wurde ausgewählt");
		}
		else if(menu.equals("Green")){
			f.setColor(Color.green);
			System.out.println("Grün wurde ausgewählt");
		}
		
		
	}
	
	public static void main(String [] args){
		PaintFrame frame=new PaintFrame("Zeichenprogramm");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
		
		frame.setSize(600,600);
		
		frame.setLocationRelativeTo(null); 
		frame.setVisible(true);
		frame.makeMenu();
		

		

			
	}

}

 

[Sky]

Das funktioniert aber irgendwie nicht!

Hab ich da was vergessen?

Ja. Wie man fragen richtig stellt...

 

[mic_checker]

Hast du schon das Tut von SUN dazu durchgelesen ?

How to use Menus

 

[Sindbad1983]

sorry..

es tut sich einfach nichts, wenn ich auf die Items klick..es wird nicht mal

System.out.println("Rot wurde ausgewählt");

ausgegeben..
Geschweige dem werden die Figuren auch nicht in der Farbe gezeichnet!
daraus resulierend kann da was nicht stimmen!

Wie bzw. was muss ich beim actionListener registieren?

Wo könnte hier der Fehler liegen?

 

[mic_checker]

habs gerad nochma getestet.

was funzt denn net?

Musste ein paar Sachen auskommentieren weil ich ja nicht alle Klassen von dir hatte, aber der rest hat geklappt.

edit:
hab meinen beitrag losgeschickt als deiner schon gepostet war....also bei mir klappts.

 

[Sky]

Bei mir erscheint die Ausgabe in der Java-Konsole!

 

[Roar]

desweiteren soltlest du nicht swing & awt mischen

 

[Sindbad1983]

aha...das ist aber seeehr interessant..wenn ich alles ausklammere, dann gehts bei mir auch..
find ich komisch
eigentlich müsste das doch gehn!
ich ruf das setColor von Figure auf und er spinnt

public abstract class Figure extends JComponent {
	
	int pressedX;
	int pressedY;
	int releasedX;
	int releasedY;
	Color c;
	
	public void setXY(int x, int y, int r, int h){
		pressedX=x;
		pressedY=y;
		releasedX=r;
		releasedY=h;
	}
	
	public void setColor(Color c){
		this.c=c;
	}
	
	

	public abstract void draw(Graphics g);

}

	public void actionPerformed(ActionEvent e) {
		
		String me=e.getActionCommand();
		
		if(me.equals("Red")){
			System.out.println("Rot wurde ausgewählt");
			f.setColor(Color.red); //oben hab ich Figure f deklariert
			
		}
		else if(me.equals("Blue")){
			System.out.println("Blau wurde ausgewählt");
			f.setColor(Color.blue);
			
		}
		else if(me.equals("Green")){
			System.out.println("Grün wurde ausgewählt");
			f.setColor(Color.green);
			
		}
		
		
	}

und in der Unterklasse Rectangle soll dann die ausgewählte Farbe weitergegeben werden!

public class Rectangle extends Figure{
	
	
	
	public void draw(Graphics g)   { 
		
		g.setColor(c);
		g.fillRect(pressedX,pressedY,releasedX-pressedX,releasedY-pressedY);
		

	}


}

 

[mic_checker]

gibt er denn die ausgaben nicht mehr aus wenn du das setColor drin lässt?

 

[Sindbad1983]

Rot wurde ausgewählt
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at figures.PaintFrame.actionPerformed(PaintFrame.java:67)
at java.awt.MenuItem.processActionEvent(Unknown Source)
at java.awt.MenuItem.processEvent(Unknown Source)
at java.awt.MenuComponent.dispatchEventImpl(Unknown Source)
at java.awt.MenuComponent.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

 

[Sky]

sagst Du denn auch irgendwo new Figure bzw. Rectangle??

 

[Sindbad1983]

ja auf das bin ich selber grad draufgekommen

muss ich mir mal überlegen, wie ich das machen könnte...
weil Figure f=new Figure geht ja nicht, weil diese Klasse abstrakt ist

und die Rectangles und Circle werden dynamisch gezeichnet...
ich werd die setColor Methoden wohl verlegen müssen