My repaint() method flickers

Question

Hi, can someone help me enhance my GUI so that it won't flicker? I tried to use BufferStrategy but it still flickers. I used the code below to override the repaint method of the Canvas that I am trying to draw. Also, I'm not really sure if the repaint() method was the right method to override.
public void repaint() {
          super.repaint();
          this.createBufferStrategy(2);
          BufferStrategy bufferStrategy = this.getBufferStrategy();
          Graphics g = bufferStrategy.getDrawGraphics();
          g.setColor(Color.BLACK);
                //x and y are changed by a different Thread every 1 millisecond
          g.drawOval(x, y, 20, 20);
          g.dispose();
          bufferStrategy.show();
     }I also have another Thread that repaints my Canvas.
public class Painter implements Runnable {
     private LandAreaCanvas area;
     public Painter(LandAreaCanvas area) {
          this.area = area;
     public void run() {
          while (true) {
               area.repaint();
}So in summary, my application has 3 Threads running
1. Main class with main method that initializes all GUI components
2. Painter class that repaints forever
3. Calculator class that changes the value of x and y in the repaint method of the Canvas class.
Thanks a lot!

Answer 1

Actually you should do your custom painting by overriding paintComponent(Graphics). Which should not flicker since Swing uses double buffering by default.
See [the tutorial|http://java.sun.com/docs/books/tutorial/uiswing/painting/].
On edit looks like you might be using AWT, so above advice might not be relevant. [The linked article|http://java.sun.com/products/jfc/tsc/articles/painting/] in the tutorial still can be of help though.
Edited by: WalterLaan on May 7, 2009 2:13 AM