C++结合SFML开发时全局变量main.obj重复定义解决方法

你遇到的[Variable] already defined in main.obj重定义错误，根源是两个不规范的写法：

• 在setup.cpp中使用了#include "main.cpp"。C++项目中所有.cpp实现文件会被编译器单独编译为独立的目标文件（.obj），#include本质是纯文本插入操作，引入main.cpp相当于把main.cpp中定义的所有全局变量、函数在setup.cpp里又完整写了一遍，链接阶段两个目标文件存在同名的全局变量定义，就会触发重定义报错。
• 如果直接把全局变量定义写在头文件中，当多个.cpp文件引入这个头文件时，同样会在每个生成的obj文件中产生一份同名变量定义，也会报完全相同的错误。

C++的单定义规则要求全局变量在整个程序中只能有一处定义，跨文件访问时仅需在访问前做声明即可，按以下步骤修改：

1. 立刻删除setup.cpp里的#include "main.cpp"代码，永远不要用#include引入.cpp后缀的实现文件。
2. 新建公共头文件global.h，用extern关键字声明所有需要跨文件访问的全局变量，注意头文件里只做声明，不要给变量赋值：

// global.h
#pragma once
#include <SFML/Graphics.hpp>

extern sf::RenderWindow window;
extern int unsortedArray[];
extern int arrLen;
extern int height;
extern float unit;
extern bool done;

3. 保留main.cpp中已有的全局变量定义，在main.cpp顶部引入global.h即可，原有main函数逻辑不需要改动。
4. 在所有需要访问这些全局变量的文件（比如setup.cpp）中，只需要引入global.h，就可以正常读写全局变量，不会触发重定义错误，修改后的setup.cpp头部如下：

// setup.cpp
#include "setup.h"
#include "bar.h"
#include "global.h"

// 原有的Setup::visualizeArray函数逻辑不需要改动

• 你当前计算单位宽度的代码float unit = 800 / arrLen;存在整数除法精度丢失问题，两个int类型做除法会先得到整数结果再转float，建议修改为float unit = 800.0f / arrLen;得到正确的浮点宽度值。
• visualizeArray函数中不要在循环内每次绘制一个柱形就调用window.display()，SFML单帧内只需要调用一次display，应该等所有柱形都绘制完成后再统一调用display刷新屏幕，否则会出现画面闪烁、渲染效率低的问题。
• 全局变量耦合度高，后续功能迭代可以考虑把窗口、数组、排序状态封装到专门的管理类中，减少全局变量的使用。

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