0

出于某种原因，当我的精灵碰到左边的边缘时，它不会反弹。该程序运行良好，但后来我更改了一些对它完全没有影响的东西，它停止了工作。

//Sprite class

public class Sprite {
private String sprite = "sprite-adult.fw.png";

private int speed;
private int dx;
private int dy;
private int x;
private int y;
private Image image;

public Sprite() {
    ImageIcon ii = new ImageIcon(getClass().getResource(sprite));
    image = ii.getImage();

            speed=6;
            dx=speed;
            dy=speed;

    x = 40;
    y = 60;
}
public void move() {
    toggleRebound();

    x += dx;
    y += dy;
}
public void toggleRebound() {
    if(x == 1366) 
        dx = negate(dx);

    if(y == 768) 
        dy = negate(dy);

    if(x == 0) 
        dx = negate(dx);

    if(y == 0) 
        dy = negate(dy);    
}
public int negate(int x) {
    return x*-1;
}
public int getX() {
    return x;
}
public int getY() {
    return y;
}
public Image getImage() {
    return image;
}
}


//SType class

public class SType extends JFrame{
public SType() {
    add(new Board());

    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setSize(1366,768);
    setLocationRelativeTo(null);
    setTitle("S - Type");
    setVisible(true);
    setResizable(false);
}
public static void main(String[] args) {
    new SType();
}
}

//Board class

public class Board extends JPanel implements ActionListener{
private Sprite sprite;
private Timer timer;

public Board() {
    setFocusable(true);
    setBackground(new Color(39,124,36));
    setDoubleBuffered(true);

    sprite = new Sprite();

    timer = new Timer(5,this);
    timer.start();
}
public void paint(Graphics g) {
    super.paint(g);

    Random rand = new Random(5398);
    for(int x=0;x<1000;x++) {
        g.setColor(new Color(22,98,19));
        g.drawOval(rand.nextInt(1368), rand.nextInt(768), 1, 20);
    }

    Graphics2D g2d = (Graphics2D)g;
    g2d.drawImage(sprite.getImage(),sprite.getX(),sprite.getY(),this);

    Toolkit.getDefaultToolkit().sync();
}
public void actionPerformed(ActionEvent arg0) {
    sprite.move();
    repaint();
}
}
4

2 回答 2

2

第一个回答者是正确的，这不应该影响任何事情。尽管在这个例子中它并不真正相关，但在构造函数中初始化私有实例变量而不是在它们的声明中是一种很好的形式。

private int speed;
private int dx;
private int dy;

public Sprite()
{
    speed=6;
    dx=speed;
    dy=speed;
}
于 2013-07-11T20:30:23.153 回答
1

这里有两个教训要学习

首先，您遇到的真正问题是您正在通过与零（或硬编码的左/上尺寸）完全相等来检查反弹。这意味着，如果您一次移动超过一个像素（正如您在上面的代码中所做的那样），您可能会错过边界并穿过它。

一般来说，检查小于等于的限制通常是一种好习惯，尤其是当一次“移动”多个限制时。

例如，如果您想编写一个数组类，您不会使用((idx == 0) or (idx == length)). 你会检查((idx <= 0) or (idx >= length))，而不是。

与精灵位置相同。

其次，检查你的假设总是有用的，尤其是当你的期望被违反时。在这种情况下，您确信一个非常简单的更改已经破坏了您的程序。

但那不是真的。这不可能是真的。在任何情况下都不是这样：

private int dx = 6;
private int dy = 6;

与此不同：

private int speed = 6
private int dx = speed;
private int dy = speed;

因此，您的两个测量点（工作，非工作）之间发生了其他变化。完全有可能你以某种方式掩盖了这种额外的变化——也许是编译错误，或者你只是在查看程序的其他方面，而不是让它反弹。但不管是什么原因，其他事情发生了变化。

在这种特殊情况下，您可能还更改了起点或速度值。

但不管是什么原因，当你遇到一个不可能的点时，你必须回去重新检查你的假设。

但无论如何，这就是问题所在：您的精灵“移动”太快，以至于您当前的反弹无法捕捉到它。更改为小于等于测试，将是我的建议。

你不正确。做出那个改变，而且只有那个改变，并没有导致你报告的行为改变。

最有可能的是，您实际上进行了不止一次更改，但仅在此之后测试了反弹行为。

于 2013-07-11T20:25:45.633 回答