Ajuda com Threads por favor!

JavaBaby Membro desde: 13/11/2007 13:56:39 Mensagens: 92 Offline

Galera, tenho esse código pro carro andar, e oque ocorre é que ele ta andando t udo certo sozinho (simplifiquei os metodos dos cases só pra dar uma idéia). Cada tecla que o usuário aperta ele vai para o lado correto sozinho, mas oque ocorre é que cada tecla ele cria uma nova Thread, entao se aperto pra ele ir pra cima ele vai na velocidade correta, mas se aperta direita ou para baixo, ele vai na direção correta só que mais rápido e vai ficando mais e mais rápido.

Será que tem como evitar que essa thread seja criada mais de uma vez? Alguém poderia me ajudar? Tentei singleton mas nao deu muito c erto, nao sei como poder no código a instancia com o run.

Olá.

Fiz um exemplo aqui para vc. Apenas uma thread fica executando e mandando o painel se repintar.

Carro.java

import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.geom.Ellipse2D;

/**
 * Um carro.
 * 
 * @author David
 */
public class Carro {

private int x;
    private int y;

// velocidades
    private int vx;
    private int vy;

public int getVx() {
        return vx;
    }

public void setVx( int vx ) {
        this.vx = vx;
    }

public int getVy() {
        return vy;
    }

public void setVy( int vy ) {
        this.vy = vy;
    }

public int getX() {
        return x;
    }

public void setX( int x ) {
        this.x = x;
    }

public int getY() {
        return y;
    }

public void setY( int y ) {
        this.y = y;
    }

// pinta o carro
    public void paintMe( Graphics g ) {
        Graphics2D g2d = ( Graphics2D ) g;
        g2d.setRenderingHint( RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON );
        g2d.fill( new Ellipse2D.Double( getX(), getY(), 20, 20 ) );
    }

}

ThreadCarro.java

PainelCarro.java

Janela.java

import java.awt.BorderLayout;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import javax.swing.JFrame;

/**
 *
 * @author David
 */
public class Janela extends JFrame implements KeyListener {

private Carro carro;
    private ThreadCarro threadCarro;
    private PainelCarro painelCarro;

public Janela() {

// instancia os componentes
        carro = new Carro();
        painelCarro = new PainelCarro( carro );
        threadCarro = new ThreadCarro( painelCarro, true );

// configura o frame
        add( painelCarro, BorderLayout.CENTER );
        setSize( 400, 400 );
        setTitle( "Olha o Carro!" );
        setVisible( true );
        setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
        addKeyListener( this );

// cria uma nova thread e a inicia
        new Thread( threadCarro ).start();
        
    }

@Override
    public void keyTyped( KeyEvent e ) {
    }

@Override
    public void keyPressed( KeyEvent e ) {
        // atualiza os valores do carro
        if ( e.getKeyCode() == KeyEvent.VK_UP ) {
            carro.setVy( carro.getVy() - 1 );
        } else if ( e.getKeyCode() == KeyEvent.VK_DOWN ) {
            carro.setVy( carro.getVy() + 1 );
        } else if ( e.getKeyCode() == KeyEvent.VK_LEFT ) {
            carro.setVx( carro.getVx() - 1 );
        } else if ( e.getKeyCode() == KeyEvent.VK_RIGHT ) {
            carro.setVx( carro.getVx() + 1 );
        }
    }

@Override
    public void keyReleased( KeyEvent e ) {
    }

public static void main( String[] args ) {
        new Janela().setVisible( true );
    }

}

Qualquer dúvida avise.

[]´s

Debugger Membro desde: 06/02/2009 14:25:23 Mensagens: 54 Offline

Péraí, quem é que chama o método andarCarro() ? Se esse método já é executado a cada evento de tecla, então você não tem porquê de executar uma thread a cada vez que uma tecla é pressionada.
Já se você quer que ele mantenha o carro andando em uma determinada direção indefinidamente até alguém dar outro comando para mudar a direção, aí é melhor postar mais do código para tentarmos adaptá-lo.

JavaBaby Membro desde: 13/11/2007 13:56:39 Mensagens: 92 Offline

O andarCarro() é executado por um keyListener que fica na JFrame principal.

O lance é que se o cara apertar pra frete o carro deve continuar andando pra frente até ele apertar pro lado ou pra trás, entende?

aí esse método que cada vez que o keylistner dispara o evento ele cria uma Thread e acho que é isso que está errado. Teria que criar a Thread no Jframe principal?

valeu :d

[BlacK wrote:]aí esse método que cada vez que o keylistner dispara o evento ele cria uma Thread e acho que é isso que está errado. Teria que criar a Thread no Jframe principal?

O ideal é fazer como o David Postou. Dá uma estudada no código dele, está muito bom.

Notícias, artigos e o maior fórum brasileiro sobre Java