2 Listener wiedersprechen sich.

[Guest]

Hi,

ich habe zwei Komponenten des Typs JCheckBoxMenuItem die die selbe aufgabe haben.

Wenn eine von den beiden Komponenten angeklickt wird, sollen beide Komponenten des selben Status erhalten.

Es handelt sich für den User scheinbar um ein und die selbe Komponente, an 2 verschiedenen Plätzen.

Problem ist nun das jede Box das Event der anderen auslöst, bzw mit meiner jetzigen Lösung das deselektieren nicht mehr geht, weil ja eine Box zum Zeitpunkt des deselektierens noch selektiert ist.

Mein Listener:

    private void autoSaveItem2ActionPerformed(java.awt.event.ActionEvent evt) {                                              

        if ( autoSaveItem.getState() || autoSaveItem2.getState() )
            SmallMethods.setAutoSave(true);
        else
            SmallMethods.setAutoSave(false);        
        
    }

Meine Methode setAutoSave:

    /**
     * Sets the autoSave on or off (true or false).
     * @param booleanValue True or false
     */
    public static void setAutoSave(boolean booleanValue) {
        autoSave = booleanValue;
        if( booleanValue ) {
            MainFrame.saveItem.setEnabled   (false);
            MainFrame.autoSaveItem.setState (true);
            MainFrame.autoSaveItem2.setState(true);
        }
        else {
            MainFrame.saveItem.setEnabled   (true);
            MainFrame.autoSaveItem.setState (false);
            MainFrame.autoSaveItem2.setState(false);
        }
    }

Vorher hatte ich anstatt eines ActionListeners einen ItemListener. Ich sehe 2 möglichkeiten, welche ich beide nicht umsetzen kann:

1. In der jetzigen Variante if ( autoSaveItem.getState() || autoSaveItem2.getState() ) durch sowas wie if ( evt.getComponent.getState() ) zu ersetzen.

2. In der Itemlistener-Methode abfragen ob das Ereignis durch den Benutzer oder eine Methode ausgelöst wurde.

Wie gesagt weis ich bei beidem nicht wie ich es umsetzen kann.

Ich könnte natürlich auch eine eigene Klasse myOwnButton (erbt von JCheckBoxMenuItem) erstellen, aber das ist mir ehrlichgesagt zu viel aufwand, dieses Objekt dann auch noch in den Netbeans-GUI-Builder als Bean zu intigrieren, sollte es einfacher gehen :autsch:

Vielen Dank

Gruß

 

[Marco13]

Klingt als ob das genau das richtige Einsatzgebiet für eine Action wäre, die du verwenden kannst, um (mit ein und der selben Action) die BEIDEN MenuItems zu erstellen:
http://java.sun.com/docs/books/tutorial/uiswing/misc/action.html

 

[20mithrandir]

Wie wärs mit event.getSource() abfragen?

 

[Guest]

Wie wärs mit event.getSource() abfragen?

Klar geht das, aber von event.getSource() kann ich nicht .getState() abfragen.

----

Ein Fall für Actions? Was zum Henker habe ich denn gemacht? Oó

 

[20mithrandir]

Also ganz schnell zusammengeschustert:

import java.awt.Container;
import java.awt.FlowLayout;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;

import javax.swing.JCheckBox;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class CheckboxTest extends JFrame implements ItemListener
{
	JCheckBox	chk1, chk2;

	public CheckboxTest()
	{
		super( "CheckboxTest" );

		this.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );

		this.chk1 = new JCheckBox( "Checkbox Nr. 1" );
		this.chk1.addItemListener( this );

		this.chk2 = new JCheckBox( "Checkbox Nr. 2" );
		this.chk2.addItemListener( this );

		Container pane = this.getContentPane();

		JPanel chkPanel = new JPanel( new FlowLayout() );
		chkPanel.add( this.chk1 );
		chkPanel.add( this.chk2 );

		pane.add( chkPanel );

		this.setSize( 320, 200 );
	}
	
	public void itemStateChanged( ItemEvent evt )
	{
		Object src = evt.getSource();
		
		System.out.println( src );
		
		if ( this.chk1.equals( src ) )
		{
			this.chk2.setSelected( chk1.isSelected() );
		}
		else if ( this.chk2.equals( src ) )
		{
			this.chk1.setSelected( chk2.isSelected() );
		}
		else
		{
			System.out.println( "Something's wrong..." );
		}		
	}

	public static void main( String[] args )
	{
		new CheckboxTest().setVisible( true );
	}
}

Was funktioniert daran nicht? Sollte auch mit den MenuItems gehen...

 

[Guest]

Sicher das es geht?

Ich habe jetzt equivalent zu deinem Code folgendes:

    private void autoSaveItemItemStateChanged(java.awt.event.ItemEvent evt) {                                              

        if ( evt.getSource() == autoSaveItem ) {
            SmallMethods.setAutoSave(autoSaveItem.getState());
            autoSaveItem2.setState  (autoSaveItem.getState());
        }
        else {
            SmallMethods.setAutoSave(autoSaveItem2.getState());
            autoSaveItem.setState   (autoSaveItem2.getState());            
        }
        
    }

    /**
     * Sets the autoSave on or off (true or false).
     * @param booleanValue True or false
     */
    public static void setAutoSave(boolean booleanValue) {
        autoSave = booleanValue;
        if( booleanValue ) {
            MainFrame.saveItem.setEnabled (false);
        }
        else MainFrame.saveItem.setEnabled(true);
    }

Allerdings wird das Immer 2 mal aufgerufen. Da ja der Status verändert wird. Hau mal bei deinem Code ein

System.out.println("Ohoh");

rein. Dann siehst dus.

 

[Guest]

Edit:\

Das eben war eine veraltete Methode. Sorry.. die neue sieht natürlich so aus:

    /**
     * Sets the autoSave on or off (true or false).
     * @param booleanValue True or false
     */
    public static void setAutoSave(boolean booleanValue) {
        autoSave = booleanValue;
        if( booleanValue ) {
            autoSave = true;
        }
        else autoSave = false;
    }

 

[20mithrandir]

Ich muss gestehen, dass mir deine Code-Auszüge nicht unbedingt sehr viel verraten. Aber die MenuItems funktionieren bei mir identisch:

import java.awt.Container;
import java.awt.FlowLayout;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;

import javax.swing.JCheckBox;
import javax.swing.JCheckBoxMenuItem;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JPanel;

public class CheckboxTest extends JFrame implements ItemListener
{
	JCheckBox	chk1, chk2;
	
	JCheckBoxMenuItem	men1, men2;

	public CheckboxTest()
	{
		super( "CheckboxTest" );

		this.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
		
		
		this.men1 = new JCheckBoxMenuItem( "Menuitem Nr. 1" );
		this.men1.setSelected( true );
		this.men1.addItemListener( this );
		
		this.men2 = new JCheckBoxMenuItem( "Menuitem Nr. 2" );
		this.men2.setSelected( true );
		this.men2.addItemListener( this );

		this.chk1 = new JCheckBox( "Checkbox Nr. 1" );
		this.chk1.addItemListener( this );

		this.chk2 = new JCheckBox( "Checkbox Nr. 2" );
		this.chk2.addItemListener( this );
		

		
		JMenuBar mb = new JMenuBar();
		JMenu menu = new JMenu( "Menu" );
		menu.add( men1 );
		menu.add( men2 );
		mb.add( menu );
		
		this.setJMenuBar( mb );
		
		Container pane = this.getContentPane();

		JPanel chkPanel = new JPanel( new FlowLayout() );
		chkPanel.add( this.chk1 );
		chkPanel.add( this.chk2 );

		pane.add( chkPanel );

		this.setSize( 320, 200 );
	}
	
	public void itemStateChanged( ItemEvent evt )
	{
		Object src = evt.getSource();
		
		System.out.println( src );
		
		if ( this.chk1.equals( src ) )
		{
			this.chk2.setSelected( chk1.isSelected() );
		}
		else if ( this.chk2.equals( src ) )
		{
			this.chk1.setSelected( chk2.isSelected() );
		}
		else if ( this.men1.equals( src ) )
		{
			this.men2.setSelected( men1.isSelected() );
		}
		else if ( this.men2.equals( src ) )
		{
			this.men1.setSelected( men2.isSelected() );
		}
		else
		{
			System.out.println( "Something's wrong..." );
		}		
	}

	public static void main( String[] args )
	{
		new CheckboxTest().setVisible( true );
	}
}

Objekt-Vergleiche würde ich übrigens immer mit der equals() Methode machen.

 

[Guest]

Danke für deine Hilfe, doch hast du den Code jeh ausgeführt?

Füge in Zeile 82 (also vor men1.setSelected()) das ein:

 System.out.println("Ouch!");

Nun Starte das Prog und betrachte die Konsole. Du bekommst IMMER ein Ouch ausgegeben wenn du ein MenuItem anklickst.

Und genau diesen Nebeneffekt möchte ich doch vermeiden.

 

[20mithrandir]

Was bedeutet "jedes Mal"? Natürlich kommst du jedes Mal wenn du eine Selektion änderst so für beide Checkboxen / MenuItems in die Listener-Methode. Das war ja doch auch so gedacht, dass beide immer denselben selected-Wert haben, oder?

 

[Guest]

Was bedeutet "jedes Mal"? Natürlich kommst du jedes Mal wenn du eine Selektion änderst so für beide Checkboxen / MenuItems in die Listener-Methode. Das war ja doch auch so gedacht, dass beide immer denselben selected-Wert haben, oder?

Klar. Nur möchte ich eben nicht, dass das setStatus (von Hand) ein Event auslöst, bzw das Event genau das erkennen kann.

Warum?

Die MenuItems haben natürlich auch eine Funktion. Bei Selektieren/Deselektieren rufen sie methoden auf und diese würden bei mir momentan 2x ausgelöst

 

[20mithrandir]

Das müsste sich ja so abfragen lassen:

else if ( this.men1.equals( src ) )
{
	if ( this.men1.isSelected() == this.men2.isSelected() )
	{
		System.out.println( "Men2 hat sich geändert..." );
	}
	
	this.men2.setSelected( men1.isSelected() );
}
else if ( this.men2.equals( src ) )
{
	if ( this.men1.isSelected() == this.men2.isSelected() )
	{
		System.out.println( "Men1 hat sich geändert..." );
	}
	
	this.men1.setSelected( men2.isSelected() );
}

 

[Guest]

LoL natürlich XD Da hätte ich auch drauf kommen müssen. Dankeschön.

Merkwürdig ist, dass ganze beim ersten Funktioniert und beim zweitem nicht :autsch:

ein Klick auf autosaveItem führt die Methode SmallMethods.setAutoSave einmal aus und
ein Klick auf autosaveItem2 führt die Methode "" zweimal aus Oo

Bahnhof? :?: Ich verstehs nicht.

In der Methode erscheint ein JOptionPane, deshalb weis ich wie oft sie ausgeführt wird.

    private void autoSaveItemItemStateChanged(java.awt.event.ItemEvent evt) {                                              

        if ( evt.getSource() == autoSaveItem ) {
            if( autoSaveItem.getState() == autoSaveItem2.getState() && autoSaveItem.getState() )
                SmallMethods.setAutoSave(autoSaveItem.getState());
            autoSaveItem2.setState  (autoSaveItem.getState());
        }
        else if ( evt.getSource() == autoSaveItem2 ) {
            if( autoSaveItem.getState() == autoSaveItem2.getState() && autoSaveItem2.getState() )
                SmallMethods.setAutoSave(autoSaveItem2.getState());
            autoSaveItem.setState   (autoSaveItem2.getState());            
        }
        
    }

Mein &&-Zusatz stellt sicher, dass die Komponente aktiviert wurde und nicht deaktiviert