具有多个相同 ArrayList 元素的 Java 有效方法

Question

我正在使用 Libgdx 在 Java 中编写游戏，并且有一个关于如何在不滥用垃圾收集器的情况下在 ArrayList 中拥有同一对象的多个实例的问题。这是我的主要游戏状态的代码。我还是新手，所以我认为我的编码习惯非常糟糕:

package com.frog.game;

import java.util.ArrayList;

import com.badlogic.gdx.Gdx;
import com.badlogic.gdx.graphics.g2d.BitmapFont;
import com.badlogic.gdx.graphics.g2d.SpriteBatch;
import com.badlogic.gdx.graphics.g2d.freetype.FreeTypeFontGenerator;
import com.badlogic.gdx.math.MathUtils;
import com.frog.entities.Frog;
import com.frog.entities.Grass;
import com.frog.entities.LilyPad;
import com.frog.gamestates.GameState;
import com.frog.managers.GameStateManager;
import com.frog.managers.Jukebox;
import com.frog.managers.Save;
import com.frog.managers.SimpleDirectionGestureDetector;
import com.frog.managers.SpeedManager;
import com.frog.game.Game;



public class PlayState extends GameState {
private SpriteBatch sb;
private Frog frog;
private BitmapFont font;
private ArrayList<LilyPad> lilypads;
private Grass grass;

private float hopY;
public boolean tmp;
private SpeedManager speedManager;
private float speed;
private float[] placement;

public PlayState(GameStateManager gsm) {
    super(gsm);
}


@SuppressWarnings("deprecation")
public void init() {
    speedManager = new SpeedManager();
    lilypads = new ArrayList<LilyPad>();
    sb = new SpriteBatch();
    frog = new Frog();
    frog.init();
    grass = new Grass();
    grass.init();
    hopY = Game.SIZE * 0.8f;
    placement = new float[] {
            0,
            Game.SIZE,
            Game.SIZE * 2
        };

    addPad();

    FreeTypeFontGenerator gen = new FreeTypeFontGenerator(
            Gdx.files.internal("PressStart2P-Regular.ttf")
            );
    font = gen.generateFont((Game.WIDTH / 10));
    gen.dispose();

    Gdx.input.setInputProcessor(new SimpleDirectionGestureDetector(new SimpleDirectionGestureDetector.DirectionListener() {

        @Override
        public void onUp() {
            // TODO Auto-generated method stub
        }

        @Override
        public void onRight() {
            frog.moveRight();

        }

        @Override
        public void onLeft() {
            frog.moveLeft();

        }

        @Override
        public void onDown() {

        }
    }));

}

这是我用来在屏幕顶部添加另一个 lilypad 的方法。我让它随机出现在三个地方之一。由于这些 lilypad 的主要功能是向下滚动屏幕，因此 lilypad 的实例会不停地从数组中添加和删除。我知道这是致命的，因为每次我添加一个新的 lilypad 时，我都必须为它运行 init()，否则我会得到一个 nullpointerexception。 init() 实例化一堆对象以使 lilypad 渲染，例如纹理、默认 Y 值等。有什么方法可以为整个数组列表运行一次 init() 方法，即使我不断添加并删除它们？我考虑过在屏幕底部包裹相同的睡莲，所以我只需要 7 个，然后我就不必再添加或删除了，但必须为此重做一大块代码，并将其视为最后的手段。游戏已经运行得很流畅，只是偶尔会有一些我想避免的几乎不明显的卡顿。

private void addPad() {
        lilypads.add(new LilyPad(placement[MathUtils.random(0, 2)], 300, false));
        lilypads.get(lilypads.size() - 1).init();
}

public void update(float dt) {


    speed = speedManager.speed(dt);
    speedManager.update(dt);

//      update grass
    if(!grass.shouldStop()) {
        grass.update(dt, speed);
        frog.introPlay(speed);
    }

//              update lily pads
    for(int i = 0; i < lilypads.size(); i++) {
        lilypads.get(i).update(dt, speed);

这是我调用 addPad() 方法的地方。它基本上是说如果最后添加的垫在屏幕上完全可见，请添加下一个。

        if(lilypads.get(i).getY() < (Game.HEIGHT - Game.SIZE) && lilypads.get(i).past()) {
            addPad();
        }

//          hop frog

        if(lilypads.get(i).getY() < hopY  && lilypads.get(i).past2()) {
//              play hop
            if(lilypads.get(i).getX() > frog.getX() - Game.SIZE / 2 && lilypads.get(i).getX() < frog.getX() + Game.SIZE / 2){
                frog.incrementScore();
                Jukebox.play("hop");
                lilypads.get(i).splash();
                frog.play(speed);
            } else {
                Jukebox.stopAll();
                Save.gd.setTempScore(frog.getScore());
                gsm.setState(GameStateManager.GAMEOVER);
                return;
            }
            return;
        }

        if(lilypads.get(i).getY() < (-Game.SIZE)){
            lilypads.remove(i);
        }
    }

}

public void draw() {

//      draw grass
    if(!grass.shouldStop()) {
        grass.draw(sb);
    }

//      draw pads

    for(int i = 0; i < lilypads.size(); i++){
        lilypads.get(i).draw(sb, true);
    }

//      draw frog
    frog.draw(sb, speed, true);

    sb.setColor(1, 1, 1, 1);
    sb.begin();
    font.draw(sb, Long.toString(frog.getScore()), 
            Game.WIDTH * 0.92f -font.getBounds(Long.toString(frog.getScore())).width, Game.HEIGHT * 0.958f);
    sb.end();
}

public void handleInput() {

}

public void dispose() {
    sb.dispose();
    Jukebox.stopAll();
}

感谢支持

Answer 1

为此使用 libGDX 的自定义集合。你可以找到所有这些 here .

在你的情况下你想使用 Array .它们经过优化，不会导致垃圾收集器启动，例如通过重新使用迭代器。

使用标准的ArrayList，渲染方法中的以下代码将在每一帧中创建一个新的Iterator。

for(LilyPad lilyPad : lilipads){
    lilypad.draw(sb, true);
}

一般来说，尽可能避免使用 new。 This wiki article可能会帮助你，它解释了池化的工作原理。

另一个避免 GC 的常见技巧，尤其是在使用 Vector2 或 Vector3 时，是保持 private final Vector2 tmp = new Vector2() 和总是只使用 tmp.set(x, y) 来改变它。此原则也可以应用于您自己创建的用于保存某些数据的任何其他自定义类。