Thread: Combining JMenuItem and a KeyListener

I have a JMenuItem "Find" in the Edit Menu.
I want to add a shortcut key to the JMenuItem.

For Find for example i want to use Ctrl + F

Here is my 'Action' for 'Find' which can be used via Edit->Find. It works.

Find = new AbstractAction(){
		public void actionPerformed(ActionEvent e){
 
                    String word = JOptionPane.showInputDialog(new MainWindow(),"","Find",PLAIN_MESSAGE);
                     new WordSearcher(textArea,word);
                 }
};

WordSearcher() is a class i am using to search the word

Now i want to add a KeyListener for ctrl+F
for doing the same purpose.
Even this one works.

 
private KeyListener k1 = new KeyAdapter()
	{       
		public void keyPressed(KeyEvent e)
		{   
                   if(e.isControlDown())
                        {
 
 
                                    if(e.getKeyCode()== KeyEvent.VK_F)
                                    {
                                        String word = JOptionPane.showInputDialog(new MainWindow(),"","Find",PLAIN_MESSAGE);
                                       new WordSearcher(textArea,word);
 
                                    }
 
 
                        }
                 }
}

But the problem here is i have to write the same code twice.
Is there some way by which I can Use the already written Action Find in the KeyListener.

You can set the accelerator of the JMenuItem. An example of doing this can be found at
How to Use Menus (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)

Originally Posted by copeg

You can set the accelerator of the JMenuItem. An example of doing this can be found at
How to Use Menus (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)

I found the solution!!!

Dont noe how it worked but it did

private KeyListener k1 = new KeyAdapter()
	{       
		public void keyPressed(KeyEvent e)
		{   
                   if(e.isControlDown())
                        {
 
 
                                    if(e.getKeyCode()== KeyEvent.VK_F)
                                    {
 
                                     Find.actionPerformed(null); 
                                    }
 
 
                        }
                 }
}

I found the solution!!!

The code you posted does not indicate you even tried what I suggested in post #2 which would be the more appropriate way to create a menu shortcut. Using a KeyListener can result in a host of other issues (one of which would be Component focus). Why ask for advice and then seemingly ignore the advice you get?

Originally Posted by copeg

The code you posted does not indicate you even tried what I suggested in post #2 which would be the more appropriate way to create a menu shortcut. Using a KeyListener can result in a host of other issues (one of which would be Component focus). Why ask for advice and then seemingly ignore the advice you get?

Sorry Sir, but no insults meant. I did read the link you gave me. But the issue with Find.setMnemonic(KeyEvent.VK_F); was that it was working only with ALT. I wanted the key to work with CNTRL. Thats it.

Originally Posted by amitection

But the issue with Find.setMnemonic(KeyEvent.VK_F); was that it was working only with ALT. I wanted the key to work with CNTRL. Thats it.

Setting the mnemonic and setting the accelerator are two different things, and my post alluded to setting the accelerator:

myMenuItem.setAccelerator(KeyStroke.getKeyStroke(K eyEvent.VK_F, InputEvent.CTRL_MASK));

And the API for setAccelerator:

Sets the key combination which invokes the menu item's action listeners without navigating the menu hierarchy.