使用事件监听器添加到arraylist [Java]

Question

我有一个带有三角形的游戏循环，可随机在屏幕上移动。 我想添加一个按钮，单击该按钮将添加另一个随机移动的三角形。 我正在使用事件监听器，无论我放在哪里，我仍然会遇到异常

这是我的代码

import javax.swing.*;
import drawing.Canvas;
import tools.Utils;
import triangle.DynamicTriangle;

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;

public class TriangleProgram {
    private JFrame frame; 
    private Canvas canvas; 
    private JPanel lowerPanel; 
    private JButton addTriangleButton; 
    private ArrayList<DynamicTriangle> triangles;

    public void gameLoop() {
        int deltaTime = 20;

        triangles.add(new RandomTriangleA(canvas, 150, 150));

        for (DynamicTriangle dynamicTriangle : triangles) {
            dynamicTriangle.drawTriangle();
        }
        while (true)
        {
            for (DynamicTriangle dynamicTriangle : triangles) {
                dynamicTriangle.undrawTriangle();
            }

            for (DynamicTriangle dynamicTriangle : triangles) {
                dynamicTriangle.update(100);
                dynamicTriangle.wrapPosition();
            }

            for (DynamicTriangle dynamicTriangle : triangles) {
                dynamicTriangle.drawTriangle();
            }

            Utils.pause(deltaTime);
            canvas.repaint();
        }

    }

    public TriangleProgram() {
        frame = new JFrame();
        frame.setTitle("Canvas");
        frame.setSize(800, 600);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);

        canvas = new Canvas();
        frame.add(canvas, BorderLayout.CENTER);

        lowerPanel = new JPanel();
        lowerPanel.setLayout(new FlowLayout());
        frame.add(lowerPanel, BorderLayout.SOUTH);

        triangles = new ArrayList<DynamicTriangle>();

        addTriangleButton = new JButton("Add Triangle"); 
        lowerPanel.add(addTriangleButton);
        frame.revalidate();

        addTriangleButton.addActionListener(new ButtonListener());

        gameLoop();

    }

    public static void main(String[] args) {
        System.out.println("Running TriangleProgram...");
        new TriangleProgram();
    }

    class ButtonListener implements ActionListener {

        @Override
        public void actionPerformed(ActionEvent ae) {
            System.out.println(triangles.size());
            triangles.add(new RandomTriangleA(canvas, 150, 150));
        }

    }

}

我希望每次按下按钮时都会在画布上添加一个新的三角形，而是获得堆栈跟踪：

Running TriangleProgram...
1
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index -1 out-of-bounds for length 0
    at java.base/jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
    at java.base/jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
    at java.base/jdk.internal.util.Preconditions.checkIndex(Preconditions.java:248)
    at java.base/java.util.Objects.checkIndex(Objects.java:372)
    at java.base/java.util.ArrayList.remove(ArrayList.java:517)
    at java.base/java.util.Collections$SynchronizedList.remove(Collections.java:2424)
    at drawing.Canvas.removeMostRecentLine(Canvas.java:162)
    at triangle.Triangle.undrawTriangle(Triangle.java:131)
    at TriangleProgram.gameLoop(TriangleProgram.java:29)
    at TriangleProgram.<init>(TriangleProgram.java:69)
    at TriangleProgram.main(TriangleProgram.java:75)

Answer 1

        triangle.add(new Triangle(canvas, 150, 150));

那么你在这里甚至都没有声明triangle所以不确定这个代码是如何运行来给你一个例外的。 我假设你的意思是triangles你也可以（？）压缩你的循环。

编辑：当你声明列表三角形时，为什么它是DynamicTriangle的列表，但是当你初始化并添加列表时它是Triangle 。

不确定DynamicTriangle是Triangle子类还是只是拼写错误但是如果是这样的话它就不应该编译。

如果这是一个拼写错误，那么让你的ArrayList<DynamicTriangle> triangles改为使用Triangle 。

如果DyanmicTriangle是Triangle的子类，那么你想要将三角形声明并初始化为ArrayList<Triangle>() ，当你添加时你做.add(new DynamicTriangle(...