KeyListener

I have a JFrame to which I have added JButtons.

Code :

---

win=new JFrame("MyFrame");
win.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
win.setBounds(400,200,335,630);win.setResizable(false);
win.setFocusable(true);
 
Container c=win.getContentPane();
c.setBackground(Color.black);c.setFocusable(true);
btnA = new JButton("A");
btnA.setForeground(Color.white);
btnA.setBackground(Color.black);
 
c.add(btnA);

---

Where do I add the KeyListener to? win or c? I've tried both and the code never calls keyPressed(), keyReleased() or keyTyped()...

Any ideas on what I'm doing wrong?

The component needs the focus to be able to receive key strokes.
key binding is a more reliable way to receive key strokes.

Post a small, complete program that compiles, executes and shows the problem. Make sure the code is properly formatted with indented statements that show nesting logic.

Norm,
Here's a simple example. I have breakpoints in the debugger on all three key functions and they never get called:

Code :

---

import java.awt.Color;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
 
import javax.swing.JButton;
import javax.swing.JFrame;
 
 
public class MyClass extends JFrame implements ActionListener, KeyListener
{
	static JFrame win;
	static JButton btnAbout;
 
	public static void main(String[] args)
	{
		win=new JFrame("MyFrame");
		win.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		win.setBounds(400,200,300,200);
		win.setResizable(false);
		win.setFocusable(true);
 
		Container c=win.getContentPane();
		c.setBackground(Color.black);
		c.setLayout(null);
		c.setFocusable(true);
		Insets insets = c.getInsets();
		c.addKeyListener(new MyClass());
 
		btnAbout = new JButton("About");
		btnAbout.setPreferredSize(new Dimension(80,30));
		btnAbout.setForeground(Color.white);
		btnAbout.setBackground(Color.blue);
		btnAbout.setBounds(insets.left+100, insets.top+80,80,30);
 
		c.add(btnAbout);
		win.setVisible(true);
}
 
	@Override
	public void keyPressed(KeyEvent e) {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void keyReleased(KeyEvent e) {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void keyTyped(KeyEvent e) {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void actionPerformed(ActionEvent e) {
		// TODO Auto-generated method stub
 
	}
}

---

Norm,
Everytime I try and post the small complete program, the Forum deletes the post...

The forum's software/server has problems occasionally. You now have 3 copies here

How do you know if a key listener method is called? None of them call println to print out a message.

Where is the GUI being shown that has the focus and the listener added to it?

Norm,
As you should be able to see in the code, I tried to set the KeyListener to the Container. I have a breakpoint in the debugger in all three key routines and it never gets there. I have also tried to set the KeyListener to the Frame with the same results. The only place I can get it to work is by setting the keyListener to the JButton. I'd rather not do that as I want any keypress in the application to get trapped...

Quote:

---

Where is the GUI being shown that has the focus and the listener added to it?

---

What does the constructor for MyClass do to make any GUI that you can see?

Norm,
Not sure what you mean. MyClass creates a JButton adds it to the Frame Container and displays the Frame with the button on it. That much works just fine. And as I mentioned if I assign the KeyListener to the JButton instead of the Frame or Container, it functions correctly as well...

Quote:

---

MyClass creates a JButton adds it to the Frame

---

Sorry, I do NOT see a constructor for the MyClass class.
Are you talking about what is done in the main() method?


Suggestion: move all the GUI creation code from the main() method to a constructor.

@Norm: his posts with code were "moderated" and thus only showing to us moderators. To show them, we must approve them with Moderator Tools. I've done this with one of his posts so that it now shows to all.