KeyListener Graphics Problem

[Makesfolkslose]

Hi there,
I'm pretty new to Java but interested in eventually making games of some kind. Right now I'm struggling with the KeyListener. I'm trying to set up basic WASD-like movement, but the graphics don't seem to respond. KeyPressed is supposed to change the x-coord of a component, but no visual movement happens. However, the System.out.println returns changing coordinates, which means that the Listener itself is working, right? Any help would be appreciated, thanks.

public class MoveViewer
{
    static JFrame frame=new JFrame("Move Viewer");
    static JPanel panel=new JPanel();
    static int yPos=10;
    static int xPos=10;
    static MoveComponent mc=new MoveComponent(xPos,yPos);
    public static void main(String[] args)
    {
	frame.setSize(600,600);
	frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	panel.add(mc);
	panel.setLayout(new OverlayLayout(panel));
	frame.add(panel);
	panel.setFocusable(true);
	panel.requestFocus();
class MoveListener implements KeyListener
{
    public void keyTyped(KeyEvent e)
    {
    }
    public void keyPressed(KeyEvent e)
    {
	int key=e.getKeyCode();
	if(key==KeyEvent.VK_RIGHT)
	    {
		xPos=xPos+50;
		mc=new MoveComponent(xPos,yPos);
		System.out.println(""+xPos+", "+yPos);
		panel.repaint();
	    }
    }
    public void keyReleased(KeyEvent e)
    {
    }
}
MoveListener ml=new MoveListener();
panel.addKeyListener(ml);
frame.setVisible(true);
    }
}

[Norm]

What does panel's paintComponent method do?
What code is supposed to cause the movement?

What is supposed to happen to the object pointed to by mc in the listener?

[camickr]

mc=new MoveComponent(xPos,yPos);

This DOES NOT add a component to the GUI. All it does is create a new Object in memory.

If you want to create a new component then you would do:

mc=new MoveComponent(xPos,yPos);
panel.add( mc );
panel.revalidate();
panel.repaint();

However even that code is wrong because now you would have 2 components on the panel.

The proper way to do this is to just invoke mc.setLocation(...) to move the existing component to a new location.