1

我遇到了这个奇怪的问题，我不确定是什么原因造成的。有时这个问题甚至不存在。从我的猜测来看，这是一个 Java 内存问题或某种线程问题。

我有一个Ship和船射击Bullets如果我按住Space键船射击子弹。我将子弹设置为每 200 毫秒发射一次。有时他们射击得很好并且以相同的速度移动！其他时候，他们射击他们以不同的速度移动。什么可能导致这种情况？

package JGame.Actions;

import JGame.GameObject.GameObject;
import javax.swing.AbstractAction;

public class MoveAction extends Action implements Runnable{

    protected GameObject obj;
    protected int endX = 0, endY = 0;
    protected int moveAmount = 0;
    protected Thread thread;

    public void moveToY(GameObject obj, int y, int amount, AbstractAction complete){
        this.obj = obj;
        this.endY = y;
        this.moveAmount = amount;
        this.complete = complete;
        thread = new Thread(this);
        thread.start();
    }

    public void run(){
        try{
            boolean run = true;
            while(run){
                int objY = obj.getY();
                if(objY > this.endY){
                    obj.setY(obj.getY() - 1);
                }else if(objY < this.endY){
                    obj.setY(obj.getY() + 1);
                }else{
                    run = false;
                    this.actionComplete();
                }
                thread.sleep(moveAmount);
            }
        }catch(Exception e){
        }
    }
}

动作完成：

package JGame.Actions;

import javax.swing.AbstractAction;

public class Action {
    protected boolean actionComplete = false;
    protected AbstractAction complete;

    public void actionComplete(){
        complete.actionPerformed(null);
    }
}

在我的代码中，我称moveToY这是一个非常简单的调用，但有时Bullets以不同的速度移动（错误），而其他人则以相同的速度移动（正确）。我不知道提到子弹移动时有时会减慢一两秒钟然后加速到正确的速度是否会有所帮助。

编辑：主线程

以下是我与paintComponent的主线程

@Override
public void run(){
    try{
        while(true){
            // Check for key press events
            Iterator actions = KeyboardMap.map.entrySet().iterator();
            while(actions.hasNext()){
                Map.Entry ap = (Map.Entry)actions.next();
                Mapping mp = (Mapping)ap.getValue();
                if(mp.pressed){
                    mp.run();
                }
            }

            // Check for click mouse events
            Iterator actions2 = MouseMap.map.entrySet().iterator();
            while(actions2.hasNext()){
                Map.Entry ap = (Map.Entry)actions2.next();
                Mapping mp = (Mapping)ap.getValue();
                if(mp.pressed){
                    mp.run();
                }
            }

            for(GameObject go : gameObjects){
                if(!go.getLeaveScreen()){
                    int goWidth = go.getWidth();
                    int goHeight = go.getHeight();
                    int goX = go.getX();
                    int goY = go.getY();
                    int gameWidth = Game.width;
                    int gameHeight = Game.height;
                    if(goX + goWidth >= gameWidth){
                        go.setX(gameWidth - goWidth);
                    }
                    if(goX <= 0){
                        go.setX(0);
                    }
                    if(goY + goHeight >= gameHeight){
                        go.setY(gameHeight - goHeight);
                    }
                    if(goY <= 0){
                        go.setY(0);
                    }
                }
            }
            this.repaint();
            Thread.sleep(roomSpeed);
        }
    }catch(Exception e){
    }
}

public void paintComponent(Graphics g){
    try{
        g.drawImage(bg, 0, 0, this);
        for(int i = 0; i < gameObjects.size(); i++){
            GameObject go = gameObjects.get(i);
            g.drawImage(go.getSprite(), go.getX(), go.getY(), this);
        }
    }catch(Exception e){
    }
}
4

3 回答 3

5

好吧，我猜您有大量并发线程正在运行（一个通过移动子弹），并且您希望每个线程在精确的moveAmount毫秒后唤醒。您不能有这样的保证，因为线程调度程序允许每个线程一次运行一段时间，因此您可能会遇到故障。

另一个问题是您似乎在事件调度线程之外对 Swing 组件执行修改，这显然是Swing 的线程策略所禁止的。

于 2012-12-25T18:15:24.297 回答
1

更改代码中最重要的事情是不要使用Thread.sleep. 使用Swing Timer，您将安排它每moveAmount毫秒执行一次，作为奖励，代码在 Event Dispatch Thread 上执行，您无需付出任何努力。

于 2012-12-25T18:23:21.597 回答
0

了解多线程的使用以及如何正确制作游戏的处理结构。对于您的情况，最重要的信息：您应该只使用一个线程来处理/移动所有子弹，然后渲染它们。

于 2012-12-25T18:20:43.230 回答