C++程序编译出现LNK2005/LNK1169重复定义链接错误求助

先看看你遇到的链接错误信息：

Error 1 error LNK2005: "class bus bs" (?bs@@3Vbus@@A) already defined in bus.obj c:\Users\tahir\documents\visual studio 2013\Projects\Project17\Project17\main.obj Project17
Error 2 error LNK2005: "class ticket tick" (?tick@@3Vticket@@A) already defined in bus.obj c:\Users\tahir\documents\visual studio 2013\Projects\Project17\Project17\main.obj Project17
Error 3 error LNK2005: "class ticket tick" (?tick@@3Vticket@@A) already defined in bus.obj c:\Users\tahir\documents\visual studio 2013\Projects\Project17\Project17\ticket.obj Project17
Error 4 error LNK2005: "class bus bs" (?bs@@3Vbus@@A) already defined in bus.obj c:\Users\tahir\documents\visual studio 2013\Projects\Project17\Project17\ticket.obj Project17
Error 5 error LNK1169: one or more multiply defined symbols found c:\users\tahir\documents\visual studio 2013\Projects\Project17\Debug\Project17.exe 1 1 Project17

问题根源 #

这是C++里非常典型的链接错误，原因很直接：你在bus.h和ticket.h的末尾直接定义了全局类实例bs和tick，比如：

// bus.h里的错误写法
class bus {
    // ... 类内容 ...
}bs; // 这里直接定义了全局实例bs

当main.cpp、bus.cpp、ticket.cpp这些源文件都包含bus.h或ticket.h时，每个源文件在编译阶段都会生成一个bs和tick的副本。链接器把这些目标文件（.obj）合并成可执行文件时，就会发现同一个符号被定义了多次，自然就抛出LNK2005错误。

哪怕你加了#pragma once也没用——它只能防止同一个头文件被同一个源文件重复包含，但解决不了多个源文件包含同一个头文件时的全局变量重复定义问题。

修复方案 #

核心原则：头文件只做声明，定义放在源文件里，具体分三步：

1. 修改头文件，把全局实例的定义改成声明

在头文件里用extern关键字声明全局实例，告诉编译器“这个东西在其他地方定义了，你先用着”：

修改后的bus.h：

#pragma once
class bus {
    int busno;
    int noofkidsseats;
    int noofwomenseats;
    int noofmenseats;
    int noofspecialseats;
    int noofvvip;
    char *busname;
    char *startpoint;
    char *destination;
public:
    bus();
    void input();
    int rbusno();
    int rnoofkidsseats();
    int rnoofwomenseats();
    int rnoofmenseats();
    int rnoofspecialseats();
    int rnoofvvip();
    void display();
};
// 只声明，不定义
extern bus bs;

修改后的ticket.h：

#pragma once
class ticket {
    int resno;
    int age;
    int tokids;
    int towomen;
    int tomen;
    int tospecial;
    int tovvip;
    int noofkidsseats;
    int noofwomenseats;
    int noofmenseats;
    int noofspecialseats;
    int noofvvip;
    char *pname;
    char *status;
public:
    ticket();
    void reservation();
    int returnresno();
    void cancellation();
    void print();
};
// 只声明，不定义
extern ticket tick;

2. 在对应的源文件里定义全局实例

在bus.cpp的末尾添加全局实例的定义（整个程序只会有这一个副本）：

// bus.cpp末尾
bus bs;

在ticket.cpp的末尾添加：

// ticket.cpp末尾
ticket tick;

3. 正常包含头文件即可

所有需要使用bs或tick的源文件（比如main.cpp），只需要包含对应的头文件，不需要额外定义，直接使用就行。

为什么这样能解决问题？ #

• extern声明只是告诉编译器符号的类型和名字，不会分配内存；
• 只有在.cpp文件里的定义才会实际分配内存，而且整个程序里只有这一个定义；
• 链接器在合并目标文件时，只会找到一个bs和tick的定义，自然就不会报重复定义的错误了。

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