java - 如何防止按住某个键重复操作？

Question

我正在尝试在 Java FX“游戏”中执行一个简单的操作，每次按下空格键时， pig 的图像都会向上“跳跃”。这是关键事件处理程序的代码和我用来实际执行操作的动画计时器。

关键处理程序:

  ArrayList<String> in = new ArrayList<String>();

    s.setOnKeyPressed(
        new EventHandler<KeyEvent>()
           {

            public void handle(KeyEvent e)
            {
                String code = e.getCode().toString();

                 if ( !in.contains(code) ){
                 in.add( code );                          
                 }

            }
        });

      s.setOnKeyReleased(
        new EventHandler<KeyEvent>()
        {
            public void handle(KeyEvent e)
            {                   
                String code = e.getCode().toString();
                in.remove( code );
            }
        });

动画定​​时器:

   new AnimationTimer()
{
    double q = 200;
    public void handle(long currentNanoTime)
    {
        double t = (currentNanoTime - startNanoTime) / 4000000.0;    

       if(in.contains("SPACE")){

            q -= 20;

        }

        double y = q + t;

        if(y >= 520){
            gc.drawImage(background1, 0, 0, 1160, 740);
            gc.drawImage(pig, 90, 520, 125, 100);

        }else{
        gc.drawImage(background1, 0, 0, 1160, 740);
        gc.drawImage(pig, 90, y, 125, 100);
        }

    }
}.start();

如您所见，我让动画计时器简单地使“ pig ”沿 y 轴逐渐下降，当按下空格键时，它会稍微向上插入。

问题是，如果按住空格键， pig 只会不停地向上飞。我希望防止这种情况发生，以便必须反复敲击空格键，而不仅仅是按住它。所以我只想按空格键一次“跳跃”。我尝试解决的任何方法都没有奏效。我该怎么做？

Answer 1

编辑: 我修改了答案。最初的解决方案使用了一个计数器，它可以防止按下的键在一段时间内产生任何影响。不幸的是，这不是这个问题的目的。 :) 当前的解决方案更直接，仅使用简单的 boolean 锁。

在回答这个问题之前，这里有一些烦人的提示:我建议使用 Map<KeyCode, Boolean>而不是 List<String>存储有关当前按下的键的信息。它将在可读性方面简化您的代码，同时提升其性能。接下来，创建一个专用对象来存储有关 pig 的信息(哈哈!)可能是个好主意。最后，使用常量而不是硬编码文字值是一种很好的做法。

另外请注意，您实际上不需要存储有关是否按下空格键的信息，然后从计时器线程中引用它。仅当您希望通过按住空格键来控制 pig 时才需要这样做。但是由于您希望它只在按下空格键时跳转，您可以告诉 pig 直接从处理程序切换到“跳转”状态。当然，这不会解决您的问题，因为 onKeyPressed长时间按住某个键时，会重复调用处理程序。但我认为值得一提。 :)

现在，回答这个问题。如果您想快速修复当前的解决方案并忽略所有“良好实践”废话，请只关注 jumpLock Pig 的领域类(class)。诀窍是不断告诉 pig 像你现在做的那样反复跳跃，但要确保 pig 只有在 jumpLock 时才会服从。允许它这样做。

注意:以下解决方案假设您将使用固定间隔(例如每 30 毫秒)更新游戏状态。但正如最后指出的那样，可以轻松修改此解决方案以使用基于 FPS 的计时器。

以下类包含您在将来调整游戏时可能希望更改的常量:

public final class Settings {

    private Settings() {
    }

    public static final double BOOST_VELOCITY = 10.0;
    public static final double GRAVITY = 0.3;
}

这个类代表 pig 。 x和 y字段存储有关 pig 当前位置的信息。 velocityX和 velocityY是分别包含有关 pig 在 X 轴和 Y 轴上的方向和“速度”信息的 vector 。 jumpLock是一个简单的 boolean 标志，实际上是您问题的解决方案。每当用户进行跳跃时，此锁将设置为 true .它将一直保持这种状态，直到它被告知释放锁，这将在用户释放空格键时发生。

public final class Pig {

    private double x;
    private double y;
    private double velocityX;
    private double velocityY;
    private boolean jumpLock;

    public Pig() {
        // ...
    }

    public void timeChanged() {
        x += velocityX;
        y += velocityY; 
        velocityY -= Settings.GRAVITY;
    }

    public void jumpBoost() {
        if (!jumpLock) {
            velocityY = Settings.BOOST_VELOCITY;
            jumpLock = true;
        }
    }

    public void releaseLock() {
        jumpLock = false;
    }

    public double getX() {
        return x;
    }

    public double getY() {
        return y;
    }

您的处理程序可能看起来像这样。注意 Map<KeyCode, Boolean>用于存储有关当前按下的键的信息。它的性能优于 List<String>在这个情况下。同时添加 @Override即使在覆盖抽象方法时，注释也是一种很好的做法:

final Map<KeyCode, Boolean> keyboard = new HashMap<>();
keyboard.put(KeyCode.SPACE, false);

scene.setOnKeyPressed(
    new EventHandler<KeyEvent>() {
        @Override
        public void handle(KeyEvent e) {
            if (e.getCode() == KeyCode.SPACE) {  
                keyboard.put(e.getCode(), true);
                // You could alternately call pig.jumpBoost()
                // directly from this handler and not having to 
                // deal with the 'keyboard' map at all
                // as illustrated with by pig.releaseLock()
                // in the next handler
            }
        }
    });

scene.setOnKeyReleased(
    new EventHandler<KeyEvent>() {
        @Override
        public void handle(KeyEvent e) {     
            if (e.getCode() == KeyCode.SPACE) {              
                keyboard.put(e.getCode(), false);
                pig.releaseLock(); // IMPORTANT!!!
            }
        }
    });

最后，必须重复执行以下代码片段。此解决方案假定此代码将以固定间隔执行，例如每 30 毫秒一次。如果您使用基于 FPS 的计时器(意味着执行之间会有不规则的间隔)，您应该将自上次更新以来耗时作为参数传递给 timeChanged()。方法，并在该方法中与它相乘。

pig.timeChanged();    
if (keyboard.get(KeyCode.SPACE)) {
    pig.jumpBoost();
}    
// Note that pig.releaseLock() could be called in else 
// branch here and not in the onKeyReleased handler.
// Choose whatever solution suits you best.

// + draw image of the pig on pig.getX() and pig.getY() coordinates

希望我做对了。写这篇文章时我几乎睡着了，一开始误解了这个问题。但我真的需要赢得一些声誉点数才能被允许对目前对我来说很重要的问题发表评论。哈哈! :D:D