Java变量未在计时器上更新

Java变量未在计时器上更新,java,actionlistener,Java,Actionlistener,我试图创建一个x和y坐标递增的移动对象,方法是在movingGame类中设置一个计时器,在另一个类中启动一个actionlistener,该类在原始类中运行一个方法,该方法运行代码以递增x和y变量,并且为了检查值,打印出x和y。但是,x和y不会上升,就好像结果没有被记录一样。如果我在打印结果之前增加它们,则为1,表明它从原始值适当增加。如果我在打印值后增加,它将显示值没有差异。这是我的密码: 移动游戏类: import javax.swing.JFrame; import javax.swing

我试图创建一个x和y坐标递增的移动对象,方法是在movingGame类中设置一个计时器,在另一个类中启动一个actionlistener,该类在原始类中运行一个方法,该方法运行代码以递增x和y变量,并且为了检查值,打印出x和y。但是,x和y不会上升,就好像结果没有被记录一样。如果我在打印结果之前增加它们,则为1,表明它从原始值适当增加。如果我在打印值后增加,它将显示值没有差异。这是我的密码:

移动游戏类:

import javax.swing.JFrame;
import javax.swing.Timer;

public class movingGame extends JFrame {

    public int x;
    public int y;

    void moving() {
        Timer timer = new Timer(100,new ActionPerformer());
        timer.start(); 
    }

    public void timeToDraw() {
        //This is where it is supposed to increment.
        x++;
        y++;
        System.out.println("y: "+y);
        System.out.println("x: "+x);
        //If I put x++ and y++ here, it would give a value of 0.
    };

    public static void main(String[] args){
        movingGame d = new movingGame();
        d.setVisible(true);
        d.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        d.setSize(1000, 666);
        d.setExtendedState(MAXIMIZED_BOTH); 
        d.moving();
    };
}
ActionPerformer类:

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class ActionPerformer implements ActionListener {

    @Override
    public void actionPerformed(ActionEvent e) {
        movingGame m = new movingGame();
        m.timeToDraw();
    }
}

总之,我的问题是,在运行该方法之后,x和y值保持不变,并且更改仅显示在该方法内部,而仅显示在特定的运行中。感谢您的帮助。

每次执行动作时,您都会创建一个新的
移动游戏对象。尝试在
actionPerformed
方法之外创建对象

每次执行动作时,您都会创建一个新的
MovingGame
对象。尝试在
actionPerformed
方法之外创建对象

您正在
actionPerformed()
方法中创建一个新的移动游戏。相反,您应该传递对在
main
方法中创建的游戏的引用。类似的东西

public class ActionPerformer implements ActionListener {
    private movingGame game;

    public ActionPerformer(movingGame mg) {
        this.game = mg;
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        this.game.timeToDraw();
    }
}
然后

Timer timer = new Timer(100, new ActionPerformer(this));

您正在
actionPerformed()
方法中创建一个新的MovingGame。相反,您应该传递对在
main
方法中创建的游戏的引用。类似的东西

public class ActionPerformer implements ActionListener {
    private movingGame game;

    public ActionPerformer(movingGame mg) {
        this.game = mg;
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        this.game.timeToDraw();
    }
}
然后

Timer timer = new Timer(100, new ActionPerformer(this));

不要忘了告诉计时器不要“压缩”调用…@AmirAfghani没错,因此
timer.setCoalesce(false)
将在多个事件排队等待触发的情况下阻止计时器仅执行一个事件。OP可以在这里阅读更多内容:别忘了告诉计时器不要“压缩”调用…@AmirAfghani没错,因此
timer.setCoalesce(false)
将在多个事件排队等待触发的情况下阻止计时器仅执行一个事件。OP可以在这里阅读更多内容:惯例是在Java中大写类名。所以在这种情况下,
movingame
应该是
movingame
,就像它是
ActionPerformer
ActionListener
ActionEvent
JFrame
,等等。惯例是在Java中大写类的名称。所以在这种情况下,
movingame
应该是
movingame
,就像它是
ActionPerformer
ActionListener
ActionEvent
JFrame
,等等。