0

所以我正在制作这个 java 小程序，我只想先确保键输入正常工作，然后再让它变得更复杂。我不知道为什么，但是当你删除“System.out.print(needUpdating);” 它没有根据键输入正确移动矩形。谁能告诉我为什么以及如何解决它？这对我来说完全是个谜。

import java.applet.Applet;
import java.awt.Graphics;
import java.awt.Rectangle;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;    
import javax.swing.JApplet;    

public class firstApplet extends JApplet implements KeyListener, Runnable {
    final int MOVEAMOUNT = 1;       
    boolean needUpdating;       
    int x,y,dx,dy;      
    Thread runner = null;

    public void init() {
        this.setFocusable(true);            
        needUpdating = false;           
        this.requestFocusInWindow();            
        x=0;            
        y=0;            
        dx=0;           
        dy=0;           
        addKeyListener(this);
    }

    public void stop() {            
    }

    public void start() {
        runner = new Thread(this);          
        runner.start();
    }

    public void paint(Graphics g) {
        System.out.println("x= "+x+" y = "+y);          
        g.drawRect( x, y, 100, 15 );
    }

    @Override
    public void keyPressed(KeyEvent e) {                        
        int key = e.getKeyCode();           
        if(key==KeyEvent.VK_UP) {
            System.out.println("up");
            dy=MOVEAMOUNT;
        }
        else if (key==KeyEvent.VK_DOWN) {
            dy=-MOVEAMOUNT;
        }
        else if (key==KeyEvent.VK_LEFT) {
            dx=-MOVEAMOUNT;
        }
        else if (key==KeyEvent.VK_RIGHT) {
            dx=MOVEAMOUNT;
        }
        // TODO Auto-generated method stub          
        needUpdating = true;            
        System.out.println("needUpdating listening = " +needUpdating);
    }

    @Override
    public void keyReleased(KeyEvent e) {
        // TODO Auto-generated method stub
        dx=0;
        dy=0;
    }

    @Override
    public void keyTyped(KeyEvent e) {          
    }

    public void processMovement() {
        System.out.println("processing");
        x+=dx;
        y+=dy;
    }

    @Override
    public void run() {
        this.addKeyListener(this);
        while(true) {
            System.out.print(needUpdating);             
            if(needUpdating) {
                processMovement();                  
                repaint();                  
                needUpdating=false;
            }           
        }           
    }
}
4

3 回答 3

5

您的代码未正确同步。更好地说，它根本不同步。但是，System.out.println它是一种同步方法，在当今典型的 CPU 架构上，进入同步块恰好是作为原生代码级别的内存屏障实现的。这具有使对布尔值的更改对其他线程可见的效果。

结论：正确同步你的代码，“神奇”的行为就会消失。

于 2013-07-02T13:45:56.787 回答
4

此代码存在严重问题。首先，您无缘无故地实施 Runnable。其次，您的 run() 方法是一个很大的忙等待循环。第三，当然是你的 needsUpdating 变量缺乏同步。

您应该在侦听器方法中执行所需的操作，而不是使用线程不安全的 needsUpdating 变量，并且线程安全问题将消失，因为您将处于事件调度线程中。

于 2013-07-02T13:49:16.967 回答
1

您必须将您的字段设置needUpdating为volatile：

volatile boolean needUpdating;

此行为由run()方法内部的无限循环定义：JVM 缓存needUpdating字段的值。

UPD：我刚刚检查了您的代码：它可以与volatilefield 上的修饰符一起正常工作needUpdating，所以我的答案是解决方案。

UPD2：为了澄清这个问题，请查看 JLS7: 17.3 的第 17 章中的示例。睡眠和产量

于 2013-07-02T13:48:05.927 回答