SDL2拆分LTexture类后无法引用main全局gRenderer变量问题咨询

这是C++全局变量跨编译单元访问的典型问题：你在main.cpp中定义的SDL_Renderer* gRenderer默认只在main.cpp内部可见，拆分出来的LTexture.cpp属于独立的编译单元，默认无法直接访问这个变量。

方案1：快速适配（适合当前代码最小改动） #

只需要在你的预编译头pch.h中添加一行extern声明即可：

extern SDL_Renderer* gRenderer;

extern关键字的作用是告知编译器，这个变量是在其他编译单元中定义的，链接阶段会自动匹配到main.cpp中你定义的gRenderer，这样LTexture.cpp中就可以正常使用这个全局变量了。

方案2：更合理的设计（推荐，适合长期维护） #

全局变量滥用会导致后续维护难度大幅提升，更建议你通过参数传递的方式传入渲染器，避免依赖全局变量：

1. 修改LTexture类的方法声明，给需要用到渲染器的方法加SDL_Renderer*入参，或者在LTexture的构造函数中传入渲染器存为类成员变量。
   示例（给方法加参数）：

// LTexture.h中修改声明
bool loadFromFile(std::string path, SDL_Renderer* renderer);
void render(int x, int y, SDL_Renderer* renderer);

2. 对应修改LTexture.cpp的实现，把原代码中用到gRenderer的地方替换为传入的参数即可。调用时直接从main.cpp把创建好的gRenderer传参进去就行。
   这种方式代码耦合度更低，后续如果需要多渲染器的场景也可以直接适配。

针对你当前的代码，有几个适合C++初学者的优化点：

• 空指针统一用nullptr代替NULL，C++11起nullptr是专门的空指针类型，比本质是整数0的NULL更安全，不会出现类型匹配错误。
• 补充错误判空逻辑：当前loadFromFile方法中调用IMG_Load后没有判断返回值，如果图片路径错误加载失败，直接访问loadedSurface的成员会直接崩溃，建议加判空：

  SDL_Surface* loadedSurface = IMG_Load(path.c_str());
  if(loadedSurface == nullptr) {
      printf("图片加载失败：%s，错误信息：%s\n", path.c_str(), IMG_GetError());
      return false;
  }
  

• 色键值改成可选参数：当前你把色键写死为青色(0,0xff,0xff)，可以改成loadFromFile的可选参数，需要的时候可以自定义色键，灵活度更高。
• 类的get方法加const修饰：比如int getWidth() const;，保证方法不会修改类的成员变量，符合C++的const正确性规范，也能避免很多误改的问题。

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