Problem with using the keyboard - completely stumped

[cwbr]

Can anyone tell me why the mouse one works, and the keyboard one doesn't (with or without quotes)? I cannot figure it out, the keyboard code here worked until I tried to do custom painting properly.
Anyway, the mouse code works, but the keyboard code does absolutely nothing.

addMouseListener(new MouseAdapter() {
	public void mousePressed(MouseEvent e) {
		int button = e.getButton();
		switch (button) {
		case MouseEvent.BUTTON1:
			pc.moveUp();
			repaint();
			break;
		case MouseEvent.BUTTON3:
			pc.moveDown();
			repaint();
			break;
		}
	}
});
addKeyListener(new KeyAdapter() {
	public void keyPressed(KeyEvent k) {
		/*int keyCode = k.getKeyCode();
		switch (keyCode) {
		case KeyEvent.VK_LEFT:
		case KeyEvent.VK_A:
			pc.moveLeft();
			repaint();
			break;
		case KeyEvent.VK_RIGHT:
		case KeyEvent.VK_D:
			pc.moveRight();
			repaint();
			break;
		case KeyEvent.VK_DOWN:
		case KeyEvent.VK_S:
			pc.moveDown();
			repaint();
			break;
		case KeyEvent.VK_UP:
		case KeyEvent.VK_W:*/
			pc.moveUp();
			repaint();
			//break;
		}
	}
/*}*/);

[Fubarable]

Well, one reason your KeyListener does nothing because it's coded that way. It's essentially little more than a shell class filled with one large

/* comment */

[pbrockway2]

the keyboard code does absolutely nothing.

The keyboard code is commented out and, as posted, it won't compile.

Rather than "works"/"doesn't work" it would be better to post an SSCCE - that is, something brief that compiles and runs together with a brief description of the actual and intended behaviour of the code.

[Edit] Sorry, it does compile - I got confused by the fact that the forum software doesn't syntax colour comments properly.

[cwbr]

I'm afraid a concise snippet of code isn't possible... for the method to do anything, at least three different classes and a few small images are needed.
I don't suppose you can think of anything that would make those two Listeners behave differently off the top of your head? The separate actions for the mouse buttons can be ignored, originally it had the same code as the keyboard one.

Apart from the fact that one is for a keyboard and one is for a mouse, they are exactly the same, so I don't know what could be preventing it from working.

Edit: I could provide the main .java file for a lookover, it's not too long, and everything outside it is guarateed to be the same for the mouse and the keyboard anyway.

[Norm]

Is the problem that the keyListener methods are not being called? Add a println to verify it is being called by printing out a message when it is called.
Or is the problem that the methods that the keyListener calls are not working as you want?

[cwbr]

Thanks for the suggestion, I tested it, and the KeyListener isn't being called. I still can't think why though.

Edit: If anyone would be willing to look over the code, I have a shortened version here: http://ubuntuone.com/1iVtgpFHdBpuPtUt9t4uQ4

[Norm]

the KeyListener isn't being called. I still can't think why though.

Does the component with the listener have the focus?

[cwbr]

I don't know how you would tell. I assume it does, since the mouse listener works, but is there a test? I could check.

Edit: I put a mouse listener and key listener on the JFrame that hold the JPanel, and neither of them register anything, so I think the focus is on the JPanel

[Fubarable]

Yep. You could always use a Swing Timer to check for focus and print out the result with a println, but regardless, you shouldn't be using a KeyListener with a Swing component. Use Key Bindings and your focus issues are fixed.

[cwbr]

Hmm, haven't heard of Key Bindings. Can you point me in the right direction and I'll have a look at them?

Edit: Found them. I'll have a look and see if that solves it. Thanks, everyone