如何排查并修复LibGDX关卡编辑器的内存泄漏与重复dispose问题

一、内存泄漏核心排查与修复 #

1. 瓦片Texture必须复用 #

检查你的Tile.java，如果每个Tile实例都新建Texture对象，必然导致内存暴涨——LibGDX的Texture是重量级GPU资源，同类型瓦片必须共享同一个Texture引用。

• 实现方式：用静态缓存池统一管理Texture，每个纹理路径只加载一次：

  public class Tile {
      private static final Map<String, Texture> TEXTURE_CACHE = new HashMap<>();
      private Texture texture;
      private final String texturePath;
  
      public Tile(String texturePath) {
          this.texturePath = texturePath;
          if (!TEXTURE_CACHE.containsKey(texturePath)) {
              TEXTURE_CACHE.put(texturePath, new Texture(Gdx.files.internal(texturePath)));
          }
          texture = TEXTURE_CACHE.get(texturePath);
      }
  
      // 全局统一清理所有瓦片纹理，不要在单个Tile的dispose中释放纹理
      public static void disposeAllTextures() {
          for (Texture tex : TEXTURE_CACHE.values()) {
              if (!tex.isDisposed()) tex.dispose();
          }
          TEXTURE_CACHE.clear();
      }
  }
  

2. 避免渲染时创建临时对象 #

检查EditorScreen.java的render()方法，若每次循环都创建Sprite、Rectangle等对象，会触发频繁GC并导致内存堆积：

• 错误示例：

  for (Tile tile : tiles) {
      Sprite sprite = new Sprite(tile.getTexture());
      sprite.setPosition(tile.getX(), tile.getY());
      sprite.draw(batch);
  }
  

• 正确做法：直接使用Batch的原生draw方法，跳过临时Sprite创建：

  batch.begin();
  for (Tile tile : tiles) {
      batch.draw(tile.getTexture(), tile.getX(), tile.getY(), tile.getWidth(), tile.getHeight());
  }
  batch.end();
  

二、重复dispose()错误定位与解决 #

1. 日志追踪调用栈 #

在所有调用dispose()的位置添加日志，直接定位重复调用的来源：

// Tile类中（注意：这里不要调用texture.dispose()，因为纹理是共享的）
@Override
public void dispose() {
    System.out.println("[DISPOSE] Tile实例: " + this.hashCode() + ", 关联纹理路径: " + texturePath);
}

// EditorScreen类中
@Override
public void dispose() {
    System.out.println("[DISPOSE] EditorScreen被调用");
    Tile.disposeAllTextures(); // 统一清理全局纹理缓存
}

运行后查看控制台日志，就能找到重复调用dispose的对象实例，顺着调用栈定位到触发代码。

2. 用引用计数管理共享资源 #

如果需要更严谨的资源管理，实现LibGDX的ReferenceCounted接口，通过引用计数控制dispose时机，彻底避免重复释放：

public class SharedTexture implements ReferenceCounted {
    private final Texture texture;
    private int refCount = 0;

    public SharedTexture(String path) {
        texture = new Texture(Gdx.files.internal(path));
    }

    @Override
    public int incRef() {
        return ++refCount;
    }

    @Override
    public int decRef() {
        refCount--;
        if (refCount == 0 && !texture.isDisposed()) {
            texture.dispose();
        }
        return refCount;
    }

    public Texture getTexture() {
        return texture;
    }
}

Tile创建时调用sharedTexture.incRef()，销毁时调用sharedTexture.decRef()，只有当所有引用都释放时才会清理纹理。

三、针对你代码的重点检查项 #

• 确认Tile.java中没有在单个Tile的dispose方法里调用texture.dispose()，所有共享纹理必须全局统一清理。
• 检查EditorScreen.java中是否存在多次切换屏幕时重复创建Tile实例但未正确清理旧实例的逻辑。
• 验证SpriteBatch是否在show()中初始化、dispose()中释放，且没有在渲染循环中重复创建。

内容的提问来源于stack exchange，提问作者EpicOweo