Question

我对Java有点新意。在下面的forLoop中，我循环遍历arraylist的元素，我正在尝试更改位置对象。当forLoop完成时，所有元素的位置等于最后一个元素中的值。我通过这个调试，但可以弄清楚为什么会发生这种情况。

编辑：下面的函数是我初始化和填充选项的地方。它还包含为每个选项设置位置的逻辑。

protected void InitializeOptions() {
    options = new ArrayList<Button>();
    options.add(new Button("button.png", "Quick Fire"));
    options.add(new Button("button.png", "20 Questions"));
    options.add(new Button("button.png", "Decisions! Decisions!"));
    options.add(new Button("button.png", "OMG"));

    for(int i = 0; i < OPTIONCOUNT; ++i) {
        options.get(i).SetPosition(i*35, i*35);
    }
}

我的对象元素被声明为Button对象的通用列表，如下所示。我不确定这是否有所作为。

List<Button> options = new ArrayList<Button>();

编辑：我有一个类是按钮，它有两个重要的对象：PositionedTexture背景和PositionedText文本。每个对象都有一个位置的Vector2。每个类的代码都在

之下

public class PositionedTexture {
    public Texture Texture;
    public Vector2 Position;

    public PositionedTexture(String texturePath) {
        Texture = new Texture(Gdx.files.internal(texturePath));
        Position = Vector2.Zero;
    }

    public PositionedTexture(String texturePath, Vector2 position) {
        Texture = new Texture(Gdx.files.internal(texturePath));
        Position = position;
    }
}

public class PositionedTexture {
    public Texture Texture;
    public Vector2 Position;

    public PositionedTexture(String texturePath) {
        Texture = new Texture(Gdx.files.internal(texturePath));
        Position = Vector2.Zero;
    }

    public PositionedTexture(String texturePath, Vector2 position) {
        Texture = new Texture(Gdx.files.internal(texturePath));
        Position = position;
    }
}

public class Button {
    protected PositionedTexture background;
    protected PositionedText text;
    protected Vector2 center;
    protected Vector2 scale;
    protected float rotation;
    protected Rectangle rect;
    protected ShapeRenderer shapeRenderer;

    public Button(String spritePath, String btnText) {
        background = new PositionedTexture(spritePath);
        text = new PositionedText(btnText);
        rect = new Rectangle(6, 41, 498, 171);
        center = new Vector2((rect.width - rect.x)/2, (rect.height - rect.y)/2);
        scale = new Vector2(0.5f,0.5f);
        rotation = 0.0f;
        shapeRenderer = new ShapeRenderer();
    }

    public Button(String spritePath, String btnText, Vector2 pos) {
        background = new PositionedTexture(spritePath, pos);
        text = new PositionedText(btnText, pos);
        rect = new Rectangle(6, 41, 498, 171);
        center = new Vector2((rect.width - rect.x)/2, (rect.height - rect.y)/2);
        scale = new Vector2(0.5f,0.5f);
        rotation = 0.0f;
        shapeRenderer = new ShapeRenderer();
    }

    public void SetPosition(Vector2 pos) {
        background.Position = pos;
        text.Position = pos;
    }

    public void SetPosition(float x, float y) {
        background.Position.x = x;
        background.Position.y = y;
        text.Position.x = x;
        text.Position.y = y;
    }

    public void Draw(SpriteBatch batch, BitmapFont font) {
        // draw background
        batch.draw(background.Texture, background.Position.x, 
                   background.Position.y, center.x, center.y,
                   background.Texture.getWidth(), background.Texture.getHeight(),
                   scale.x, scale.y, rotation, (int)rect.x, (int)rect.y,
                   (int)rect.width, (int)rect.height, false, false);

        // draw text
        font.setColor(0, 0, 0, 1);
        font.draw(batch, text.Text, text.Position.x, text.Position.y);

        // draw collision rect
        shapeRenderer.begin(ShapeType.Line);
        shapeRenderer.setColor(1, 0, 0, 1);
        shapeRenderer.rect(background.Position.x, background.Position.y,
                           background.Texture.getWidth() * scale.x,
                           background.Texture.getHeight() * scale.y);
        shapeRenderer.end();
    }
}

Answer 1

啊，我看到了问题......它是在类PositionedTexture中。您没有为每个Button创建一个新位置。所以基本上所有按钮位置都指向Vector2.Zero。

创建一个新的Position元素是要走的路......

public class PositionedTexture {
public Texture Texture;
public Vector2 Position;

public PositionedTexture(String texturePath) {
    Texture = new Texture(Gdx.files.internal(texturePath));
    Position = Vector2.Zero;
}

public PositionedTexture(String texturePath, Vector2 position) {
    Texture = new Texture(Gdx.files.internal(texturePath));
    Position = position;
}

Answer 2

确保没有将相同的对象实例放在数组列表的所有位置 - 换句话说，一遍又一遍地添加相同的元素。这可能会导致你所看到的行为。

更改下一个元素时，将覆盖ArrayList / List中的Java前一个元素

2 个答案: