将元素添加到 ArrayList [关闭]答案

【问题标题】：Adding Elements to an ArrayList [closed]将元素添加到 ArrayList [关闭]
【发布时间】：2017-10-05 22:20:53
【问题描述】：

我尝试编写一个应该创建圆柱体的类。但是当我尝试绘制我的ArrayList 时，其中有 180 个具有相同值的对象。当我尝试将 Quads 添加到我的 ArrayList 时，我不明白那里出了什么问题。

public class Planet {
    public static ArrayList<Quad> createRing(int x, int y, int z, int radius) {
        ArrayList<Quad> quads = new ArrayList<Quad>();
        for (int i = 0; i < 360; i++) {
            float x1 = (float) (Math.sin(Math.toRadians(i)) * radius);
            float y1 = (float) (Math.cos(Math.toRadians(i)) * radius);
            i++;
            float x2 = (float) (Math.sin(Math.toRadians(i)) * radius);
            float y2 = (float) (Math.cos(Math.toRadians(i)) * radius);

            quads.add(new Quad(new Vector3f(x1, y1, 4), new Vector3f(x2, y2, 4), new Vector3f(x2, y2, 0),
                    new Vector3f(x1, y1, 0)));
        }
        return quads;
    }
}

----QUAD.java

import org.lwjgl.opengl.GL11;
import org.lwjgl.util.vector.Vector3f;

public class Quad {
//have to be non-static
static Vector3f Cord1; 
static Vector3f Cord2;
static Vector3f Cord3;
static Vector3f Cord4;
//have to be non-static
public Quad(Vector3f cord1,Vector3f cord2,Vector3f cord3,Vector3f cord4) {
Cord1 = cord1;
Cord2 = cord2;
Cord3 = cord3;
Cord4 = cord4;
}
public void draw() {
//GL11.glBegin(GL11.GL_QUADS);
GL11.glColor3f(1, 0, 0);
GL11.glVertex3f(Cord1.x, Cord1.y, Cord1.z);
GL11.glVertex3f(Cord2.x, Cord2.y, Cord2.z);
GL11.glVertex3f(Cord3.x, Cord3.y, Cord3.z);
GL11.glVertex3f(Cord4.x, Cord4.y, Cord4.z);
}
public void out() {

System.out.println(Cord1.x+" "+ Cord1.y+" "+ Cord1.z);
System.out.println(Cord2.x+" "+ Cord2.y+" "+ Cord2.z);
System.out.println(Cord3.x+" "+ Cord3.y+" "+ Cord3.z);
System.out.println(Cord4.x+" "+ Cord4.y+" "+ Cord4.z);
}
}

【问题讨论】：

您是否尝试过System.out.println(eins); System.out.println(zwei); 等并检查它们是否相同？您确定每次都分配不同的值吗？
我通过更改 for 循环中的值进行了测试，每次绘制时我都会得到圆柱体的另一部分。但我不能一次画出整个圆柱体。
Quad 是您定义的类吗？它的成员是static 吗？请添加该类的代码。（如果它们是静态的，则将它们设为非静态）。

标签： java arraylist lwjgl

【解决方案1】：

public class Quad {
static Vector3f Cord1;
static Vector3f Cord2;
static Vector3f Cord3;
static Vector3f Cord4;
public Quad(Vector3f cord1,Vector3f cord2,Vector3f cord3,Vector3f cord4) {
Cord1 = cord1;
Cord2 = cord2;
Cord3 = cord3;
Cord4 = cord4;
}

您已将CordN 变量设为static，因此每个字段只有一个值，在类的所有实例之间共享。因此，每次创建该类的新实例时，之前的值都会被覆盖。

删除static 修饰符。

另外，请注意，创建成员变量final 是一种很好的做法，除非您绝对不需要它们。此处似乎并非如此，因此将它们设为final - 这会自动发现您的问题，因为您无法在构造函数中重新分配static final 字段。

【讨论】：