使用for循环填充vector后Enemy精灵无法渲染的问题排查

你遇到的问题是Enemy类的默认拷贝/移动行为导致Sprite关联的Texture失效：

• 循环中创建的newEnemy是局部变量，每次循环结束后会被销毁。
• 执行enemies.push_back(newEnemy)时，会拷贝newEnemy到vector中，但默认拷贝构造函数只会复制Sprite的状态（包括它指向原newEnemy的Texture的指针）。
• 原newEnemy销毁后，其Texture被释放，vector中拷贝的Enemy的Sprite指向的是无效纹理，因此无法渲染。
• 手动创建时，原Enemy对象（如Enemy e1;）在main函数生命周期内不会销毁，所以Texture始终有效，Sprite能正常渲染。

方案1：直接在vector中构造Enemy（推荐） #

使用emplace_back直接在vector的内存空间中构造Enemy对象，完全避免拷贝操作：

// 替换原循环代码
for (int i = 0; i < 5; i++) {
    enemies.emplace_back(); // 直接在vector内构造Enemy
    Enemy& newEnemy = enemies.back();
    newEnemy.setPos({ randint(10 + newEnemy.getSize().x, gbl::winWidth - newEnemy.getSize().x), 100 });
}

方案2：为Enemy类实现正确的拷贝构造函数和赋值运算符 #

手动处理拷贝逻辑，确保Sprite关联当前对象的Texture：

// 在Enemy.hpp中声明拷贝构造函数和赋值运算符
class Enemy {
    // ... 原有成员 ...
public:
    Enemy(const Enemy& other);
    Enemy& operator=(const Enemy& other);
    // ... 原有成员函数 ...
};

// 在Enemy.cpp中实现
Enemy::Enemy(const Enemy& other) 
    : texture(other.texture), sprite(other.sprite), speed(other.speed), offscreen(other.offscreen) {
    sprite.setTexture(texture); // 重新关联到当前对象的Texture
}

Enemy& Enemy::operator=(const Enemy& other) {
    if (this != &other) {
        texture = other.texture;
        sprite = other.sprite;
        sprite.setTexture(texture); // 重新关联纹理
        speed = other.speed;
        offscreen = other.offscreen;
    }
    return *this;
}

方案3：使用智能指针存储Enemy #

通过std::unique_ptr存储Enemy对象，避免拷贝实体：

// 将vector类型修改为智能指针容器
std::vector<std::unique_ptr<Enemy>> enemies;

// 替换原循环代码
for (int i = 0; i < 5; i++) {
    auto newEnemy = std::make_unique<Enemy>();
    newEnemy->setPos({ randint(10 + newEnemy->getSize().x, gbl::winWidth - newEnemy.getSize().x), 100 });
    enemies.push_back(std::move(newEnemy));
}

// 渲染循环修改为指针调用
for (auto& enemy : enemies) {
    enemy->update();
    if (enemy->getPos().y + enemy->getSize().y > gbl::winHeight + enemy->getSize().y) {
        enemy->setOffscreen();
    }
    enemy->draw(window);
}

• 统一随机数生成逻辑：你现在在main和Enemy.cpp中各定义了一个randint函数，建议将其移到全局头文件（如Global.hpp）中，并将随机数生成器设为静态，避免每次调用重新初始化：

  float randint(float min, float max) {
      static std::random_device rd;
      static std::mt19937 gen(rd());
      std::uniform_real_distribution<float> dis(min, max);
      return dis(gen);
  }
  

• 检查纹理加载：可以在Enemy构造函数中添加加载失败的判断，避免因路径错误导致的隐形问题：

  Enemy::Enemy() {
      if (!texture.loadFromFile("assets/images/Enemy.png")) {
          std::cerr << "Failed to load Enemy texture!" << std::endl;
          // 可以设置一个默认纹理或终止程序
      }
      sprite.setTexture(texture);
      speed = 0.1f;
      offscreen = false;
  }
  

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