代码有时有效，但有时无效（内存或线程问题）

Question

我遇到了这个奇怪的问题，我不确定是什么原因造成的。有时这个问题甚至不存在。根据我的猜测，这是 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){
    }
}

Answer 1

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

另一个问题是你似乎在事件调度线程之外对 Swing 组件执行修改，Swing's threading policy 明确禁止这样做。

Answer 2

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

Answer 3

更改代码最重要的一点是不要用Thread.sleep 模拟时间。使用Swing Timer，您将安排它每moveAmount 毫秒执行一次，作为奖励，代码在事件调度线程上执行，您无需付出任何努力。