JPanel/Menu

[hansmann]

Hallo,
ich bin gerade dabei ein wenig Java zu lernen. Ich möchte ein Programm in dem man über eine JMenuBar ein JPanel verändern kann. Die JMenuBar habe ich in die Klasse MainMenu gepackt. Somit bekomme ich ein Fenster mit verschiedenen Menüeinträgen.
Um bei Click auf einen Menüpunkt zu reagieren habe ich eine Klasse MyActionListener implementiert. Wenn ich jetzt z.B auf Test klicke, möchte ich, dass in dem Fenster ein Kreis gezeichnet wird. Kann mir jemand sagen, wie ich jetzt weiter vorgehe. Irgendwie muss ich ja ein neues JPanel erzeugen, auf dem der Kreis gezeichnet wird und das auf meinen JFrame drauflegen. Nur weiß ich halt nicht wie???:L

public class MainMenu extends JFrame
{
	
	public MainMenu(ActionListener listener)
	{
	}
	

	
	
	private JMenu createFileMenu(ActionListener listener) 
	  { 
	
	  } 
	 
	  private JMenu createEditMenu(ActionListener listener)
	  {
	  }
	  
	  private void setCtrlAccelerator(JMenuItem MenuItem, char cAcceleration) 
	  { 
	  } 
	private JMenuItem addNewMenuItem(String CMenuItemName, char cShortcut, char cAccelerationKey, ActionListener Listener)
	   {
		
	   }
	
}

public class MyActionListener implements ActionListener
 
{
       public void actionPerformed(ActionEvent event)
	   {
		 String cmd = event.getActionCommand();
		 if (cmd.equals("Beenden")) 
		 {
			 System.exit( 0 ); 
		 }     
		 else if (cmd.equals("Test"))
		 { 
                     //Kreis zeichnen
		 }
		 	
	   }
}

public class CMain {
	
	
	public static void main(String[] args) 
	  { 
	   MyActionListener cmd =new MyActionListener();
	   MainMenu GUI			=new MainMenu(cmd);
	  
	  }
	
	
}

 

[Marco13]

Du solltest präziser sagen (oder, falls nötig, präziser überlegen ) was du willst. Die "pragmatischste" Antwort auf deine Frage wäre sowas wie

public class MyActionListener implements ActionListener
{
    private MainMenu mainMenu;

    public void setMainMenu(MainMenu mainMenu) 
    {
        this.mainMenu = mainMenu;
    }

...
// In der methode:
         else if (cmd.equals("Test"))
         { 
              mainMenu.machwas();
         }

und dann in der main;

public class CMain {

       MyActionListener cmd =new MyActionListener();
       MainMenu GUI         =new MainMenu(cmd);
       cmd.setMainMenu(GUI);

allerdings ist das krampfig-murksig und strukturell sehr unschön....

 

[hansmann]

Hallo,
wieso strukturell unschön? Habe es jetzt so versucht umzusetzen, wie du es meintest.CMain und setMainMenu habe so wie du geschrieben hast. Leider passiert noch nichts, wenn ich auf Punkte klicke. Liegt der Fehler jetzt in der Zeichenfläche?

//in der Methode:
                       else if (cmd.equals("Punkte"))
		 {
			 Zeichenfläche ll= new Zeichenfläche();
			    ll.setBackground(Color.white);
			    ll.setSize(400,400);
			    JPanel p = new JPanel(new BorderLayout());
			    p.add(ll,BorderLayout.CENTER);
			    mainMenu.add(p);
			 
		 }

Die Klasse Zeichenfläche sieht so aus:

public class Zeichenfläche extends JPanel 
{
	
	public void paintComponent(Graphics g){
		super.paintComponent(g);
		g.setColor(new Color(0,155,155));
		g.drawOval(50,50,50,50);
	}	
	
}

 

[Michael...]

Ich würde empfehlen eine Komponente zu verwenden auf die gezeichnet wird, anstelle immer neue Komponenten zu erzeugen und auszutauschen. Diese verwaltet auch die zu zeichnenden Objekte.
Bsp.:

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.*;

import javax.swing.*;

public class PaintingDemo extends JFrame {
	private PaintingPanel paintPanel;
	
	public PaintingDemo() {
		JMenuBar bar = new JMenuBar();
		JMenu menu = new JMenu("Datei");
		JMenuItem item;
		menu.add(item = new JMenuItem("Beenden"));
		item.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				System.exit(0);
			}
		});
		bar.add(menu);
		
		menu = new JMenu("Einfügen");
		menu.add(item = new JMenuItem("Kreis"));
		item.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				paintPanel.add(new Circle());
				paintPanel.repaint();
			}
		});
		menu.add(item = new JMenuItem("Rechteck"));
		item.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				paintPanel.add(new Rectangle());
				paintPanel.repaint();
			}
		});
		menu.add(item = new JMenuItem("Radieren"));
		item.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				paintPanel.clear();
				paintPanel.repaint();
			}
		});
		bar.add(menu);
		
		this.setJMenuBar(bar);
		paintPanel = new PaintingPanel();
		paintPanel.setBackground(Color.WHITE);
		this.getContentPane().add(paintPanel);
	}

	public static void main(String[] args) {
		JFrame frame = new PaintingDemo();
		frame.setBounds(0, 0, 500, 300);
		frame.setLocationRelativeTo(null);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
	}
	
	class PaintingPanel extends JPanel {
		List<DrawingObject> objList = new ArrayList<DrawingObject>();
		
		public void add(DrawingObject obj) {
			this.objList.add(obj);
		}
		
		public void clear() {
			this.objList.clear();
		}
		
		public void paintComponent(Graphics g) {
			super.paintComponent(g);
			for (int i=0; i<objList.size(); i++)
				objList.get(i).paintObject(g);
		}
	}
	
	interface DrawingObject {
		public void paintObject(Graphics g);
	}
	
	class Circle implements DrawingObject {
		public void paintObject(Graphics g){
			g.drawOval(50, 50, 50, 50);
		}
	}
	
	class Rectangle implements DrawingObject {
		public void paintObject(Graphics g){
			g.drawRect(80, 60, 50, 50);
		}
	}
}

 

[hansmann]

Hallo,
ja es wäre wohl besser auf einer Komponente zu zeichnen, statt immer wieder neue zu erzeugen. Ich möchte aber die GUI und Anwendungslogik trennen. Ich kann also nicht alles in eine Klasse packen-dieser Punkt bereitet mir Schwierigkeiten. Also wenn mir noch jemand Tipps geben kann, wäre es sehr hilfreich.

 

[Michael...]

Ich habe hier ja auch mehrere (innere) Klassen verwendet, die kann man selbstverständlich auch in eigene Dateien auslagern. Ging mir nur darum, dass man den Code per Copy&Paste&Compile ausführen kann.
Wenn Du mit Logik die ActionListener meinst kann man auch diese auslagern. Hier verweise ich immer gerne auf das MVC

 

[Marco13]

Das "strukturell unschön" bezog sich auf das, was du selbst gesagt hast: Die Trennung von GUI und Anwendung. Mit dem, was ich... (ich will nich sagen "vorgeschlagen", bestenfalls) erwähnt habe, ist es eben so, dass das Panel den Listener kennt, und umgekehrt :autsch: Deswegen "strukturell unschön".

Was sind denn die Kreise? Willst du "irgendwas malen, was (hier, rein zufällig, weil es halt Swing ist) mit einem g.drawOval auf einem JPanel dargestellt wird", oder willst du ein JPanel mit einem Kreis drauf anzeigen? Angenommen, das sollen Fußbälle sein, die auf einem Spielfeld liegen, dann würde man natürlich (GANZ grob, nur zur Verdeutlichung) sowas machen wie

class Spielfeld
{
    int getAnzahlBälle() {...}
    Point getPosition(i) {...}

    void fügeBallHinzu(int x, int y) {...}
}


class SpielfeldGUI extends JPanel
{
    private Spielfeld spielfeld;


    public void paintComponent(...)
    {
        ....
        for (int i=0; i<spielfeld.getAnzahlBälle(); i++) maleBallBei(spielfeld.getPosition(i));
    }


        ....
        // In irgendeinem ActionListener, der an einem Button hängt:
        void actionPerformed(...) 
        {
            spielfeld.fügeBallHinzu(10,10);
            repaint();
        }

}

Das GUI kennt zwar das Spielfeld, aber das Spielfeld weiß nicht mal, dass es ein GUI gibt.

Deswegen meinte ich, dass du genauer sagen (oder überlegen) solltest, was du erreichen willst. Wenn du vom GUI aus ein JPanel ins GUI legen willst, dann GIBT es nämlich nur ein GUI, und keine "Anwendung", die davon getrennt sein könnte

 

[hansmann]

Das Stichwort MVC ist in der Vorlesung (noch??) nicht gefallen. Aber das kommt dem was ich versuche wohl nahe. In meinem ursprünglichen Code hatte ich in der Klasse MyActionListener ja den Action Listener implementiert.
Wieso sehe ich denn jetzt immer noch keine Rechtecke?
Hier mein Versuch:
[Java]public class CMain {


public static void main(String[] args)
{
MyActionListener cmd =new MyActionListener();
MainMenu GUI =new MainMenu(cmd);
cmd.setMainMenu(GUI);
}


}[/code]

import java.awt.Graphics;
class Rectangle implements DrawingObject {
        public void paintObject(Graphics g){
            g.drawRect(80, 60, 50, 50);
        }
    }

public class MyActionListener implements ActionListener
 
{

	private MainMenu mainMenu;
	public void setMainMenu(MainMenu mainMenu) 
    {
        this.mainMenu = mainMenu;
    }

	
	PaintingPanel Zeichnung = new PaintingPanel();

	 public void actionPerformed(ActionEvent event)
	   {
		 String cmd = event.getActionCommand();
		 if (cmd.equals("Beenden")) 
		 {
			 System.exit( 0 ); 
		 }     
		 else if (cmd.equals("Rechteck"))
		 {
			 
			 Zeichnung.add(new Rectangle());
			 mainMenu.add(Zeichnung);
			 
		 }
		 	
	        

	   }
}

Achtung: Als ich dieses Posting erstellt habe, hatte ich das letzte Posting noch nicht gelesen.
Habe Problem das ich keine Rechtecke sehe nicht lösen können. Warum kommt der ActionListener jetzt eigentlich in FussballGUI? Kommen die Listener nicht sonst immer in eine eigene Klasse?

 

[Marco13]

Achtung: Als ich dieses Posting erstellt habe, hatte ich das letzte Posting noch nicht gelesen.Habe Problem das ich keine Rechtecke sehe nicht lösen können.
Sag' nochmal bescheid, wenn's nicht klappt.

Warum kommt der ActionListener jetzt eigentlich in FussballGUI? Kommen die Listener nicht sonst immer in eine eigene Klasse?
Wenn sie nur eine (private) Methode aufrufen, kann man sie auch anonym machen. Aber wenn man will, kann man natürlich auch eine eigene Klasse dafür erstellen...

 

[hansmann]

Hallo,
Ich weiß der Code ist nicht schön, aber man muss sich ja hocharbeiten... Es funktioniert immer noch nicht. Dabei erzeuge ich doch in der Klasse MyActionListener ein Objekt von PaintingPanel und zeichne auf diesem JPanel ein Rechteck. Das Panel übergebe ich anschließend einem Objekt von MainMenu. Wo ist jetzt mein Denkfehler???

Hier

public class PaintingPanel extends JPanel {
    List<DrawingObject> objList = new ArrayList<DrawingObject>();
    
    public void add(DrawingObject obj) {
        this.objList.add(obj);
    }
    
    public void clear() {
        this.objList.clear();
    }
    
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        for (int i=0; i<objList.size(); i++)
            objList.get(i).paintObject(g);
    }
}[/Java]

Weiß niemand etwas, oder habe ich mich schlecht ausgedrückt?

 

[Marco13]

Verständlich schon, aber man kann nicht erwarten, dass jeder nur vor einem bestimmten Thread sitzt, und so lange "Reload" drückt, bis er eine neue Antwort geben darf, oder der Thread als beendert markiert ist.

Inzwischen ist ein bißchen unklar, wie dein gesamtes Programm aussieht. An dem zuletzt geposteten Stück ist nicht "falsch", aber ... wo und wie existiert denn so ein "PaintingPanel"-Objekt?

 

[Michael...]

[Java] mainMenu.add(Zeichnung);[/code]

Wenn man zur Laufzeit Komponenten neu hinzufügt, hat das ja Auswirkungen aufs Layout
daher validate() auf der Komponente aufrufen.
Daher ja mein Vorschlag eine Komponente (in meinem Code PaintingPanel) zu verwenden, die dauerhaft im Frame verbleibt, welche die zuzeichnenden Objekte verwaltet.

 

[hansmann]

Daher ja mein Vorschlag eine Komponente (in meinem Code PaintingPanel) zu verwenden, die dauerhaft im Frame verbleibt, welche die zuzeichnenden Objekte verwaltet.

Habe das leider noch nicht umsetzen können. Werde mich damit wohl noch ein wenig mehr auseinandersetzen müssen. Trotzdem schon mal vielen Dank für die Antworten.

 

[André Uhres]

Hier mal für den Anfang ein kleines Beispiel:

import java.awt.*;
import java.awt.event.*;
import java.awt.geom.*;
import java.util.*;
import javax.swing.*;
public class CMain {
    public static void main(final String[] args) {
        Runnable gui = new Runnable() {
            @Override
            public void run() {
                MyActionListener cmd = new MyActionListener();
                MainMenu view = new MainMenu(cmd);
                cmd.setView(view);
                view.setVisible(true);
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}
class MyActionListener implements ActionListener {
    private MainMenu mm;
    private Random r = new Random();
    public void actionPerformed(final ActionEvent event) {
        String cmd = event.getActionCommand();
        if (cmd.equals("Beenden")) {
            System.exit(0);
        } else if (cmd.equals("Test")) {
            Point location = new Point(r.nextInt(300), r.nextInt(200));
            mm.createCircle(location, 50, 50);
        }
    }
    public void setView(final MainMenu mm) {
        this.mm = mm;
    }
}
class MainMenu extends JFrame {
    private final ActionListener listener;
    private final JMenu menuFile;
    private final JMenuBar menubar;
    private final JPanel panel;
    public MainMenu(final ActionListener listener) {
        super("MainMenu");
        this.listener = listener;
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(400, 300);
        setLocationRelativeTo(null);
        menuFile = createFileMenu(listener);
        menubar = new JMenuBar();
        menubar.add(menuFile);
        setJMenuBar(menubar);
        panel = new JPanel(null);
        add(panel);
    }
    private JMenu createFileMenu(final ActionListener listener) {
        JMenu menu = new JMenu("File");
        menu.setMnemonic('f');
        JMenuItem mi = addNewMenuItem("Test", 't', 't', listener);
        menu.add(mi);
        menu.add(new JSeparator());
        mi = addNewMenuItem("Beenden", 'b', 'b', listener);
        menu.add(mi);
        return menu;
    }
    private JMenuItem addNewMenuItem(final String CMenuItemName, final char cShortcut,
            final char cAccelerationKey, final ActionListener Listener) {
        JMenuItem mi = new JMenuItem(CMenuItemName);
        mi.setMnemonic(cShortcut);
        mi.setAccelerator(KeyStroke.getKeyStroke(cAccelerationKey));
        mi.addActionListener(listener);
        return mi;
    }
    public void createCircle(final Point location, final int width, final int height) {
        Shape shape = new Ellipse2D.Double(0, 0, width, height);
        ShapeComponent shapeComponent = new ShapeComponent(location, shape);
        panel.add(shapeComponent);
        shapeComponent.setForeground(Color.GREEN);
        shapeComponent.requestFocusInWindow();
    }
}
class ShapeComponent extends JComponent {
    private final Shape shape;
    private final JPopupMenu popup;
    public ShapeComponent(final Point location, final Shape shape) {
        setCursor(Cursor.getPredefinedCursor(Cursor.HAND_CURSOR));
        setToolTipText("");
        setOpaque(false);
        popup = new JPopupMenu();
        popup.add(new JMenuItem("Test 1"));
        popup.add(new JMenuItem("Test 2"));
        setComponentPopupMenu(popup);
        Rectangle bounds = shape.getBounds();
        setBounds(location.x, location.y,
                bounds.width + 2,
                bounds.height + 2);
        this.shape = shape;
        addFocusListener(new FocusListener() {
            public void focusGained(final FocusEvent e) {
                repaint();
            }
            public void focusLost(final FocusEvent e) {
                repaint();
            }
        });
        addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(final MouseEvent e) {
                getParent().setComponentZOrder(ShapeComponent.this, 0);
                requestFocusInWindow();
            }
        });
    }
    @Override
    public String getToolTipText(final MouseEvent event) {
        return "hasFocus:" + hasFocus();
    }
    @Override
    public boolean contains(final int x, final int y) {
        return shape.contains(x, y);
    }
    @Override
    protected void paintComponent(final Graphics g) {
        super.paintComponent(g);
        Graphics2D g2 = (Graphics2D) g;
        g2.setColor(getForeground());
        if(hasFocus()){
            g2.setColor(getForeground().darker().darker());
        }
        g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
        g2.fill(shape);
    }
}

 

[hansmann]

Hallo,
der Code hat mir geholfen. Ich soll in meiner Testataufgabe zwar keine Shapes verwenden, aber es war als Anregung ganz hilfreich:idea: