Добавление в массив с прослушивателем событий [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(...