JPanel draw graphic on key event

[Rolf83]

Ok I feel mighty stupid, but now I managed to solve my problem, ofcourse it was very simple....

import java.awt.Graphics;
import java.awt.Image;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import javax.swing.JPanel;

public class drawtest extends JPanel implements KeyListener {

private static final long serialVersionUID = 1L;
    int width, height;
    int kx, ky;


  public drawtest() {
      kx = 10;
      ky = 10;
      addKeyListener( this );
  }
  
  public void paintComponent(Graphics g) {
	  g.drawLine( kx, ky, kx, ky );
  }
  
	public void keyPressed(KeyEvent e) {

                    if (e.getKeyChar()=='d'){kx ++;}
	    if (e.getKeyChar()=='s'){ky ++;}
	    if (e.getKeyChar()=='a'){kx --;}
	    if (e.getKeyChar()=='w'){ky --;}    
	    repaint();
	    e.consume();
	}

	public void keyReleased(KeyEvent e) {}
	public void keyTyped(KeyEvent e) {}

}

This does the trick for me, and I call it from my JFrame like this:

	        drawtest drawtest= new drawtest();
	        jContentPane.add(drawtest, BorderLayout.CENTER);
	        drawtest.setFocusable(true);

[camickr]

I don't have the patience to read through the entire posting so I'll just make one comment and suggestion.

It is obvious you copied the code from an Applet. The code you copied is used for AWT and should not be used for Swing. Mainly, you should never override of invoke the update() method.

For standard Swing painting read the Swing tutorial Lesson: Performing Custom Painting (The Java™ Tutorials > Creating a GUI With JFC/Swing) for working examples and explanations.

[Rolf83]

I totally agree, thats why I solved it and posted the solution without using the update function, just don't know how to write solved in the headline... If you read the the last post before you posted you will see my solution... Sorry