按下键即可更改球的方向

Question

我一直试图使一个球改变其方向，因为按下任何键。 如果在按下键时球正在向侧面移动，则球将开始向下移动，并且当它触摸底部时，它将向后移回向上位置...我想我写的是正确的代码，我找不到任何错误。 所以有人可以告诉我这段代码有什么问题吗？

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.geom.Ellipse2D;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class ASS2 extends JFrame {

    public static void main(String args[]) {
        ASS2 g = new ASS2();
        g.setLayout(new BorderLayout());
        g.setSize(500, 500);
        MyPanel mp = new MyPanel();
        g.add(mp);
        mp.setSize(500, 500);
        mp.setBackground(Color.black);
//mp.addKeyListener(mp);
        g.setVisible(true);
        g.setDefaultCloseOperation(EXIT_ON_CLOSE);

    }
}

class MyPanel extends JPanel implements KeyListener {

    {
        addKeyListener(this);
    }
    int xpos = 20, ypos = 200;
    int xtop = 15, ytop = 15;
    int xtemp = 250, ytemp = 250;
    int xbot = 450, ybot = 400;
    int flag = 1, flag1 = 1;

    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        Graphics2D g2d = (Graphics2D) g;
        g2d.setColor(Color.white);
        g2d.fill(new Ellipse2D.Double(xpos, ypos, 50, 50));
        if (xpos < xbot && flag == 1) {
            xpos++;
            if (xpos == xbot) {
                flag = 0;
            }
        } else if (xpos > xtop && flag == 0) {
            xpos--;
            if (xpos == xtop) {
                flag = 1;
            }
        }

        try {
            Thread.sleep(05);
        } catch (Exception e) {
        }
        repaint(1000);

    }

    public void keyPressed(KeyEvent ae) {

    }

    public void keyReleased(KeyEvent ae) {
        Object t = ae.getKeyCode();
        if (t.equals(KeyEvent.VK_DOWN)) {
            if (ypos < ybot && flag1 == 1) {
                ypos++;
                if (ypos == ybot) {
                    flag1 = 0;
                }
            } else if (ypos > ytop && flag1 == 0) {
                ypos--;
                if (ypos == ytop) {
                    flag1 = 1;
                }
            }
            repaint();
        } else if (t.equals(KeyEvent.VK_RIGHT)) {
            if (xpos < xbot && flag == 1) {
                xpos++;
                if (xpos == xbot) {
                    flag = 0;
                }
            } else if (xpos > xtop && flag == 0) {
                xpos--;
                if (xpos == xtop) {
                    flag = 0;
                }
            }
            repaint();
        }

    }
}

Answer 1

欢迎来到“为什么不应该使用KeyListener ”的奇妙世界。

基本上， KeyListener仅在侦听器所连接的组件可聚焦且具有焦点时才引发事件。

相反，您应该使用按键绑定 ，该按键绑定可让您控制触发按键事件的焦点级别。

• 不要在EDT中调用的任何方法（尤其是paint方法）中使用Thread.sleep 。
• 不要调用repaint(1000); 在任何paint方法内部或调用任何可能从paint方法内部触发repaint paint方法
• 不要在paint方法内修改状态， paint方法就是这样。
• 使用某种“更新”线程/进程负责更新游戏模型并请求更新视图。 java.swing.Timer是一个很好的简单选择。 请参阅如何使用Swing计时器
• 当Swing Timer限制您要尝试的操作并且开始探索Thread的使用时，请不要在EDT之外修改任何UI组件。 有关更多详细信息，请参见Swing中的并发。
• 始终从EDT上下文中启动程序，有关更多详细信息，请参见初始线程 。