JAVA 贪吃蛇游戏错误。蛇走了3步就停了下来

Question

这是我在这里的第一个查询。我正在构建一个运行完美的蛇游戏，但 body 只移动三步并暂停。以下是游戏玩法的代码，我有主要方法，但它绝对没问题。 Image : before pressing right arrow. ………… Image : after pressing right arrow

主要方法:

public class Snake {

public static void main(String[] args){

    Gameplay gplay = new Gameplay();
    JFrame obj = new JFrame();
    obj.setTitle("Super SNake");
    obj.setBounds(10, 10, 905, 700);
    obj.setBackground(Color.getHSBColor(192, 68, 66));
    obj.setResizable(false);
    obj.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    obj.add(gplay);
    obj.setVisible(true);
}

}

游戏代码:

public class Gameplay extends JPanel implements KeyListener, ActionListener{

private int[] snakeXlength = new int[750];
private int[] snakeYlength = new int[750];
private int lengthsnake = 3;
private int moves = 0;

private boolean left = false;
private boolean right = false;
private boolean up = false;
private boolean down = false;

private ImageIcon umouth;
private ImageIcon lmouth;
private ImageIcon rmouth;
private ImageIcon dmouth;

private ImageIcon body;
private ImageIcon imagetitle;

private Timer t;
private int delay = 100;

public Gameplay(){

    addKeyListener(this);
    setFocusable(true);
    setFocusTraversalKeysEnabled(false);
    t = new Timer(delay,this);
    t.start();

}

public void paint(Graphics g){
    if(moves==0){

        snakeXlength[2] = 50;
        snakeXlength[1] = 75;
        snakeXlength[0] = 100;

        snakeYlength[2] = 100;
        snakeYlength[1] = 100;
        snakeYlength[0] = 100;
    }


    //title border
    g.setColor(Color.black);
    g.drawRect(24, 10, 851, 55);
    //title
    imagetitle = new ImageIcon( getClass().getResource("title.jpg"));
    imagetitle.paintIcon(this, g, 25, 11);
    //play area
    g.setColor(Color.black);
    g.drawRect(24, 74, 851, 577);
    //bg for play area
    g.setColor(Color.black);
    g.fillRect(25, 75, 850, 575);
    rmouth = new ImageIcon( getClass().getResource("rmouth.png") );
    //rmouth = new ImageIcon("game/rmouth.png");
    rmouth.paintIcon(this, g, snakeXlength[0], snakeYlength[0]);

    for(int a=0; a<lengthsnake; a++){
        if(a==0 && right){
        rmouth = new ImageIcon( getClass().getResource("rmouth.png") );
        //rmouth = new ImageIcon("rmouth.png");
        rmouth.paintIcon(this, g, snakeXlength[a], snakeYlength[a]);
        }
        if(a==0 && left){
        lmouth = new ImageIcon( getClass().getResource("lmouth.png") );
        //lmouth = new ImageIcon("lmouth.png");
        lmouth.paintIcon(this, g, snakeXlength[a], snakeYlength[a]);
        }
        if(a==0 && up){
        umouth = new ImageIcon( getClass().getResource("umouth.png") );
        //umouth = new ImageIcon("umouth.png");
        umouth.paintIcon(this, g, snakeXlength[a], snakeYlength[a]);
        }
        if(a==0 && down){
        dmouth = new ImageIcon( getClass().getResource("dmouth.png") );
        //dmouth = new ImageIcon("dmouth.png");
        dmouth.paintIcon(this, g, snakeXlength[a], snakeYlength[a]);
        }
        if(a!=0){
        body = new ImageIcon( getClass().getResource("body.png") );
        //body = new ImageIcon("body.png");
        body.paintIcon(this, g, snakeXlength[a], snakeYlength[a]);
        }
        }
        g.dispose();
}

@Override
public void keyTyped(KeyEvent e) {

}

@Override
public void keyPressed(KeyEvent e) {
     if(e.getKeyCode()== KeyEvent.VK_RIGHT){
    moves++;
    right= true;
    if(!left){
     right=true;
    }
    else{
    right= false;
    left= true;
    }
}

if(e.getKeyCode()== KeyEvent.VK_LEFT){
 moves++;
left= true;
if(!right){
left=true;
}
else{
 left= false;
 right= true;
}
up=false;
down=false;
}
if(e.getKeyCode()== KeyEvent.VK_UP){
moves++;
up= true;
if(!down){
up=true;
}
else{
up= false;
down= true;
}

left=false;
right=false;
}

if(e.getKeyCode()== KeyEvent.VK_DOWN){
moves++;
down= true;
if(!up){
down=true;
}
else{
down= false;
up= true;
}

left=false;
right=false;
}

}

@Override
public void keyReleased(KeyEvent e) {

}

@Override
public void actionPerformed(ActionEvent e) {
    t.start();

if(right){

for(int r= lengthsnake-1; r>=0; r--){
snakeYlength[r+1]= snakeYlength[r];
}
for(int r=lengthsnake; r>0; r--){
if(r==0){
 snakeXlength[r]= snakeXlength[r]+25;
}
else{
 snakeXlength[r]= snakeXlength[r-1];
}

if(snakeXlength[r]> 850){
 snakeXlength[r]= 25;
}
}

repaint();
}

    if(left){

    for(int r= lengthsnake-1; r>=0; r--){
snakeYlength[r+1]= snakeYlength[r];
}
for(int r=lengthsnake; r>0; r--){
if(r==0){
 snakeXlength[r]= snakeXlength[r] -25;
}
else{
 snakeXlength[r]= snakeXlength[r-1];
}

if(snakeXlength[r]< 25){
 snakeXlength[r]= 850;
}
}

repaint();
    }
    if(up){

    for(int r= lengthsnake-1; r>=0; r--){
snakeXlength[r+1]= snakeXlength[r];
}
for(int r=lengthsnake; r>0; r--){
if(r==0){
 snakeYlength[r]= snakeYlength[r]+25;
}
else{
 snakeYlength[r]= snakeYlength[r-1];
}

if(snakeYlength[r]< 625){
 snakeYlength[r]= 75;
}
}

repaint();
}
    if(down){

    for(int r= lengthsnake-1; r>=0; r--){
snakeXlength[r+1]= snakeXlength[r];
}
for(int r=lengthsnake; r>0; r--){
if(r==0){
 snakeYlength[r]= snakeYlength[r] +25;
}
else{
 snakeYlength[r]= snakeYlength[r-1];
}

if(snakeYlength[r] > 625){
 snakeYlength[r]= 75;
}
}

repaint();
    }

    }

}

希望您能帮助我解决我的问题。我不知道我错在哪里。蛇移动，但在 3 步后停了下来，它仍在改变头部的方向，但仅在那个位置。这是我在堆栈溢出中的第一个查询。希望以上数据能够帮助大家理解这个问题。请帮助我找到解决方案。

Answer 1

所以，有很多事情“错误”，但我将从最明显的问题开始。

仅查看右方向处理程序...

for (int r = lengthsnake; r > 0; r--) {
    if (r == 0) {
        snakeXlength[r] = snakeXlength[r] + 25;
    } else {
        snakeXlength[r] = snakeXlength[r - 1];
    }

    if (snakeXlength[r] > 850) {
        snakeXlength[r] = 25;
    }
}

在此循环中，r 不可能等于 0，这意味着 0 元素永远不会改变。

它应该更像......

for (int r = lengthsnake; r > 0; r--) {
    snakeXlength[r] = snakeXlength[r - 1];

    if (snakeXlength[r] > 850) {
        snakeXlength[r] = 25;
    }
}
snakeXlength[0] = snakeXlength[0] + 25;
if (snakeXlength[0] > 850) {
    snakeXlength[0] = 25;
}

我怀疑所有其他“方向”处理程序都有同样的问题。

下一步...

@Override
public void actionPerformed(ActionEvent e) {
    t.start();
    //...

在它自己的 ActionListener 中启动计时器可能不是最好的主意。 Swing Timer 会自动重复。

下一步...

public void paint(Graphics g) {
    if (moves == 0) {
    //...
    g.dispose();
}

一堆坏主意。

首先，作为一般建议，您应该避免覆盖 paint。相反，您应该专注于重写 paintComponent。

您还应该调用 paint 方法的 super，以维护绘制链。

您永远不应该处置不是您自己创建的Graphics上下文，它可能会对需要在组件之后绘制的其他内容产生不利影响。

相反，更像是......

protected void paintComponent(Graphics g) {
    super.paintComponent(g);
    if (moves == 0) {
    //...
}

下一步...

我什至放弃了讨论 KeyListener - 相反，您应该使用 How to Use Key Bindings

下一步...

你永远不会释放任何关键状态，这将是一个很大的问题，头部的方向实际上应该与关键状态和绘画过程分开管理

建议...

使用不同的数据结构来管理你的蛇。如果您只需要线性访问，则可以找到类似 LinkedList 的东西(如果您需要随机访问，则可以找到 ArrayList )。关键是，您可以轻松地在 List 的“头部”插入一个新元素，而不必将所有旧元素“复制”到一个位置。

我会再创建两个元素。一种用于管理元素的“类型”，另一种用于管理元素的位置。

public enum SnakePartType {
    LEFT_HEAD, RIGHT_HEAD, DOWN_HEAD, UP_HEAD, BODY
}

public class SnakePart {
    private SnakePartType type;
    private Point location; // You could use separate x/y properties, but Point already does it

    public SnakePart(SnakePartType type, Point location) {
        this.type = type;
        this.location = location;
    }

    // Getters for type and location
}

这样，您的 paint 进程就不需要了解关键状态，该状态会在绘制 channel 之间发生变化，并且可以依赖模型中的信息