Need to slow rectangle movement

Question

I am having a problem with threading. I am trying to recreate pong where there are 2 Action Listeners for 2 players. The 2 paddles move at the speed that I want them to move at. However, the "ball"(Rectangle object with an oval painted over it) is moving way too fast. I've tried slowing it down using a second thread in the Ball class, but that doesn't seem to work. Any help would be appreciated. Here's my code:

public class BattleBallz extends JFrame implements Runnable {
AL keyListen1 = new AL();
AL2 keyListen2 = new AL2();
Image dbi;
Graphics dbg;

int x1,x2;
int ballX, ballY, direction;
Ball b1;

public BattleBallz(){
    setTitle("Battle Ballz");
    setSize(350, 400);
    setResizable(false);
    setVisible(true);
    setLocationRelativeTo(null);
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    addKeyListener(keyListen1);
    addKeyListener(keyListen2);
    x1 = 150;
    x2 = 150;
    int direction = 0 + (int)(Math.random() * ((2 - 0) + 1));
    b1 = new Ball(direction);
    Thread b = new Thread(b1);
    b.start();
}

public void paint(Graphics g){
    dbi = createImage(getWidth(), getHeight());
    dbg = dbi.getGraphics();
    paintComponent(dbg);
    g.drawImage(dbi, 0, 0, this);
}

public void paintComponent(Graphics g){
    Rectangle p1 = new Rectangle(x1,375,60,10);
    g.setColor(Color.blue);
    g.fillRect(x1, 375, 60, 10);
    Rectangle p2 = new Rectangle(x2,45,60,10);
    g.setColor(Color.red);
    g.fillRect(x2,45,60,10);

    b1.paintComponent(g, p1, p2);

    repaint();
}


@Override
public void run() {
    try {
        while(true){
            move();
            Thread.sleep(5);
        }
    } catch (Exception e) {
        // TODO: handle exception
    }

}

public void move(){
    x1 += keyListen1.getXdirection();
    x2 += keyListen2.getXdirection();

    if (x1<=0){
        x1 =0;
    }
    if (x1>=305){
        x1 = 305;
    }
    if (x2<=0){
        x2=0;
    }
    if (x2>=305){
        x2=305;
    }
}
}

public class Ball extends JPanel implements Runnable {

boolean up,down,right,left;
int x = 150, y = 150 ;

public Ball(int direction){
    if(direction ==1){
        down = true;
    }
    if (direction ==0){
        up = true;
    }
}

public void paintComponent(Graphics g, Rectangle p1, Rectangle p2){
    super.paintComponents(g);
    if(down){
        y++;
        Rectangle ball = new Rectangle(x,y,49,49);
        g.fillOval(x, y, 50, 50);
        if (y>=385 || ball.intersects(p1)){
            down = false;
            up = true;
        }
    }
    if (up){
        y--;
        Rectangle ball = new Rectangle(x,y,49,49);
        g.fillOval(x, y, 50, 50);
        if (y<=10 || ball.intersects(p2)){
            up = false;
            down = true;
        }
    }
}

@Override
public void run() {
    // TODO Auto-generated method stub
    try {
        while (true){
            Thread.sleep(30);
        }
    } catch (Exception e) {
        // TODO: handle exception
    }
}
}

Answer 1

1. use Swing Timer instead of plain Thread, you have an issue with Concurency in Swing, code posted to block Event Dispatch Thread, locked by Thread.sleep, endless block

2. use paintComponent but put there JPanel, 1st. code line inside should be super.paintComponent, otherwise paint is cumulated, override getPreferredSize for JPanel

3. use KeyBindings instead of KeyListener