I don't see where you're adding the KeyListener to a component. You should either add it to your JFrame or to a focusable component inside the JFrame.

But anyway, this sounds like a job for key...