Swing JMenuButtons mit Funktionen aus Array erstellen

[Blagg Dogg]

hiho,

ich möchte ein programm schreiben, das mir zufällige konstellationen aus personen, orten, handlungen etc. liefert. in der menüleiste habe ich für jedes feld (personen, orten, handlungen etc) ein menu, wo ich die jeweiligen sachen aus, oder abwählen kann - also ob sie mit in die auswahl aufgenommen werden sollen. dann soll aus einem eintrag "Magier" "-> Magier" werden und das jeweilige wort in die jeweilige arraylist (hier: activePersonen) mit aufgenommen werden. bei erneutemm klicken soll es wieder rückgängig gemacht werden. allerdings funktioniert das so leider nicht... ich weiß auch, wieso nicht, aber ich weiß nicht, wie ich das hinbekomme... (

jemand ne idee?

package de.sack;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.Random;

import javax.swing.*;

public class RandomStory extends JFrame {
	
	// Einige Werte zu Testzwecken
	protected String[] personen={"Dieb", "Magier", "Jäger"};
	protected String[] orte={"Waldhütte"};
	protected String[] jahreszeiten={"Winter", "Frühling", "Sommer", "Herbst"};
	protected String[] tageszeiten={"Nacht", "Morgen", "Mittag", "Abend"};
	protected String[] handlungen={"mord"};
	
	
	protected ArrayList<String> activePersonen;
	protected ArrayList<String> activeOrte;
	protected ArrayList<String> activeJahreszeiten;
	protected ArrayList<String> activeTageszeiten;
	protected ArrayList<String> activeHandlung;
	
	int i; //Zählervariable;
	JMenuItem temp;
	String tempStr;
	
	public RandomStory(){

		activePersonen = new ArrayList<String>();
		activeOrte = new ArrayList<String>();
		activeJahreszeiten = new ArrayList<String>();
		activeTageszeiten = new ArrayList<String>();
		activeHandlung = new ArrayList<String>();
		
		setTitle("Creative Stories (c) 2012");
		setSize(new Dimension(800,200));
		setLocationRelativeTo(null);
		final Random r = new Random();

		
		
		/*************Menü-Leiste**************************/
		JMenuBar leiste = new JMenuBar();
		
		
		// Hauptmenü
		JMenu main = new JMenu("Programm");
		JMenuItem close = new JMenuItem("Beenden");
		close.addActionListener(new ActionListener() {
			
			@Override
			public void actionPerformed(ActionEvent arg0) {
				// TODO Auto-generated method stub
				System.exit(0);
				
			}
		});
		main.add(close);
		leiste.add(main);
		
		// Personen
		JMenu personenMenu = new JMenu("Personen");
		
		for (i=0; i<personen.length; i++){
			temp = new JMenuItem(personen[i]);
			personenMenu.add(temp);
			tempStr = personen[i];
			temp.addActionListener(new ActionListener() {
			
				@Override
				public void actionPerformed(ActionEvent arg0) {
					
					
//					if(!activePersonen.contains(personen[i])) {
//						activePersonen.add(personen[i]);
//						temp.setText("-> "+tempStr); 
						// führt zu einer ArrayIndexOutOfBoundsException
//					} else {
//						activePersonen.remove(personen[i]);
//						temp.setText(personen[i]);						
//					}
					
				}
			});
		}
		
		leiste.add(personenMenu);
		
		
		// weitere Menüs für Orte, Handlungen etc...
		
		
		
		

		setJMenuBar(leiste);
		/***************Ende Menü-Leiste*******************/
		
		final JLabel anzeige = new JLabel("Willkommen");
		getContentPane().add(anzeige, BorderLayout.NORTH);
	
		
		JButton generator = new JButton("Generate");
		generator.setSize(50, 10);
		generator.addActionListener(new ActionListener() {
			
			@Override
			public void actionPerformed(ActionEvent arg0) {
				//anzeige.setText("Works");
				
			}
		});
		add(generator, BorderLayout.SOUTH);
		this.getRootPane().setDefaultButton(generator);
				
		
		//setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		
	}
	
	public static void main(String [] args){
		new RandomStory().setVisible(true);
		
	}

}

 

[Denny88]

Hi Blagg Dogg,

ich kann dir auch sagen warum da eine IndexOutOfBoundException kommt. Das Problem ist, dass beim ActionListener hinter "temp" und "i" nicht das steckt, was du erwartest. Wenn actionPerformed ausgeführt wird, dann ist temp das in der for-Schleife zuletzt zugewiesene Objekt und i hat immer die Länge der ArrayList und kann daher nicht als Index verwendet werden.

Mein Vorschlag ist, CheckBoxMenuItem(s) zu verwenden, wenn du auf den "->" verzichten kannst. Dann kannst du das mal so versuchen:

         for (i=0; i<personen.length; i++){
            temp = new JCheckBoxMenuItem(personen[i]);
            personenMenu.add(temp);
            temp.addActionListener(new ActionListener() {
            
                @Override
                public void actionPerformed(ActionEvent arg0) {
                    
                  JCheckBoxMenuItem srcObj = (JCheckBoxMenuItem)(arg0.getSource());
                  if(srcObj.getState()) {
                      activePersonen.add(srcObj.getText());
                  } else {
                      activePersonen.remove(srcObj.getText());                 
                  }
                  //Testausgabe von activePersonen
                  System.out.println(activePersonen);
                }
            });
        }

Man kann auch statt "if(srcObj.getState())" alternativ auch mit "if(!activePersonen.contains(srcObj.getText()))" überprüfen, ob der Text in der ArrayList befindet. Oder sollte man das sogar sicherheitshalber? Ich weiß es ehrlich gesagt nicht, ob das nötig ist Aber so geht es auf jeden Fall und ich hoffe das hilft dir weiter.

LG
Daniel

 

[Blagg Dogg]

thx... verstehe es noch nicht ganz, aber es geht

noch eine frage: ist es möglich, dass sich das menü nach auswählen einer checkbox nicht schließt?

 

[Denny88]

Ja, natürlich geht das ... Sorry, hätte da auch ein bischen im voraus mitdenken können. Verwende statt CheckBoxMenuItem einfach CheckBox und dann klappt das schon

LG
Daniel

 

[Denny88]

Also,

beim Aufruf von ActionPerformed() im ActionListener wird geschaut was "jetzt" zu diesem Zeitpunkt in i und temp steht. Zu diesen Zeitpunkt ist die Schleife ja schon durchgelaufen und i ist demnach personen.lentght + 1 , da ich i ja um ein größer sein muss, damit die Bedingung für die Schleife nicht mehr erfüllt ist. Deshalb wird immer eine IndexOutOfBoundException geworfen. temp wird auch immer das zuletzt zugewiesene Objekt aus personen beinhalten, da hier auch nach der Schleife erst nachgeschaut wird, was dahinter steckt. Aber das ist eigentlich eher nebensächlich. Das Kernproblem war, dass du jedem Objekt einen ActionListener hattest, der personen verwendete. Da aber i zum Zeitpunkt des Zugriffs in allen Listenern "gleich" ist, da ja erst jetzt nachgeschaut was in i steht, hättest du nie verschiedene Namen aus personen in activePersonen kopieren können, sondern immer nur den i-ten, egal auf welchen Namen du im Menü geklickt hättest. Wegen temp wird sich nur immer der Text des letzten Elementes in deinem Menu ändern. Deine Idee war ja sicher, dass er schon innerhalb der Schleife nachgeschaut wird, was in personen steht und dies dann im ActionListener zu verwenden. Dies ist aber leider nicht so :noe:

Zum neuen Code:
Es ist eigentlich simpel. Die Zuweisung der ActionListener hat ja über die for-Schleife ganz gut geklappt.
In ActionPerformed holt er sich dann mit arg0.getSource() das Objekt, welches zu dem ActionListener gehört, der momentan ausgeführt wird. Da es sich bei dem Objekt um ein JComboMenuItem handelt, kann man sich von ihm einfach mit srcObj.getText() seinen Text holen. Vorher muss aber noch gecastet werden, damit das Objekt die Methode getText() verwenden kann. Mit srcObj.getState() kann man überprüfen, ob ein Haken gesetzt (true) oder entfernt (false) wurde.

Ich hoffe das war einigermaßen verständlich Ich kann es dir aber auch am MI in der FH nochmal erklären, wenn du magst

LG
Daniel