为什么我的ArrayList没有循环遍历所有值?

时间:2015-11-19 18:23:48

标签: java for-loop arraylist

我试图制作一个小应用程序,在窗框周围弹跳球。当我在列表中添加超过1个球时;它没有像预期的那样循环遍历它们。

这是我的代码:

主要类别:

public class Main extends JFrame {

private static final long serialVersionUID = 1L;
private List<Ball> balls = new ArrayList<Ball>();
private List<Ball> tempballs = new ArrayList<Ball>();
private static Timer t;

public void addBall(Ball b) {
    tempballs.add(b);
}

public void initUI() {
    this.setSize(500, 500);
    this.setDefaultCloseOperation(3);
    this.setVisible(true);
}

private BufferStrategy bs;
private Random r = new Random();
public void paint() {
    if (!tempballs.isEmpty()) {
        balls.addAll(tempballs);
        tempballs = new ArrayList<Ball>();
    }
    int i = 0;
    System.out.println(balls.size());
    for (Ball b : new ArrayList<Ball>(balls)) {
        i++;
        System.out.println(i);

        if ((bs = this.getBufferStrategy()) == null) {
            this.createBufferStrategy(2);
            return;
        }

        if (bs.contentsLost() || bs.contentsRestored()) {
            return;
        }
        if (b.y >= this.getHeight() - 100) {
            b.ydirection = -r.nextDouble() * 5;
        }
        if (b.y < 20) {
            b.ydirection = r.nextDouble() * 5;
        }
        if (b.x >= this.getWidth() - 100) {
            b.xdirection = -r.nextDouble() * 5;
        }
        if (b.x < 0) {
            b.xdirection = r.nextDouble() * 5;
        }

        b.x += b.xdirection;
        b.y += b.ydirection;

        if (b.xdirection > 0)
            b.xdirection += 0.1;
        else
            b.xdirection += -0.1;

        if (b.ydirection > 0)
            b.ydirection += 0.1;
        else
            b.ydirection += -0.1;

        Graphics g = bs.getDrawGraphics();

        g.fillOval((int) b.x, (int) b.y, 100, 100);

        bs.show();

        g.dispose();

        bs.dispose();

    }
    i = 0;

}

public static void main(String[] args) {
    try {
        final Main m = new Main();
        m.addMouseListener(new Mouse(m));
        m.initUI();
        t = new Timer();
        TimerTask tt = new TimerTask() {

            @Override
            public void run() {
                m.paint();
            }
        };
        t.schedule(tt, Calendar.getInstance().getTime(), 20);
    } catch (ConcurrentModificationException e) {
        e.printStackTrace();
    }
}

}

这是Ball类:

public class Ball {

private Random r = new Random();
public double y, x, ydirection, xdirection;
public Ball(int x, int y) {
    this.y = y;
    this.x = x;
    ydirection = r.nextGaussian() * 5;
    xdirection = r.nextGaussian() * 5;
}

}

和鼠标监听器:

public class Mouse implements MouseListener {
Main m;
public Mouse(Main m) {
    this.m = m;
}

@Override
public void mouseClicked(MouseEvent e) {
    m.addBall(new Ball(e.getX(), e.getY()));
    System.out.println("cl");

}
@Override
public void mouseEntered(MouseEvent arg0) {

}

@Override
public void mouseExited(MouseEvent arg0) {

}

@Override
public void mousePressed(MouseEvent arg0) {

}

@Override
public void mouseReleased(MouseEvent arg0) {

}

}

其他详情:

  • 它只循环遍历列表中的第一项,但列表大小会增加。
  • 我使用的是java 6,但如果需要,我会更改版本。

2 个答案:

答案 0 :(得分:1)

循环内部有return个语句。如果执行到达任何return语句,则循环结束。

弄清楚,如果您输入这些条件并返回列表中的第一个值。

if ((bs = this.getBufferStrategy()) == null) {
            this.createBufferStrategy(2);
            return;
        }

        if (bs.contentsLost() || bs.contentsRestored()) {
            return;
        }

答案 1 :(得分:1)

如果您的循环没有按预期运行,请尝试找出它被取消的原因。循环中有两个return语句可能会导致此行为。在返回之前创建sysout以确定哪一个是原因。然后找出为什么你的回报是真的。为了深入挖掘,您可以使用IDE的调试模式并将断点放在有趣的行上,或者使用步骤模式一次运行一行代码。

除此之外,你可以将if放在循环之前,当你在paint()函数中时(你正在使用可能会改变它们的UI线程),它们检查的值不应该改变。