C++多文件类方法调用报错：重复定义与未识别问题求助

问题描述 #

我正在为学校项目开发POS系统，跨类文件调用方法时遇到问题：在AppUI.cpp、EditBook.cpp和SecuritySys.cpp顶部定义全局对象AppUI UI、SecuritySys SecSysFunc、EditBook BookFunc时，会触发“重复定义UI、SecSysFunc、BookFunc”的编译错误；但如果不定义这些全局对象，代码中如BookFunc.AddBook()这类类方法调用会无法被识别。

相关代码如下：

main.cpp #

#include <iostream>
#include <windows.h>
int main()
{
    AppUI UI;
    SecuritySys SecSysFunc;
    EditBook BookFunc;

    UI.MainMenu();
}

AppUI.cpp #

#include <iostream>
#include <windows.h>
#include <iomanip>
#include <string>
#include <cstring>
#include <algorithm>
#include <unistd.h>
#include <stdio.h>

#include "SecuritySys.h"
#include "AppUI.h"
#include "EditBook.h"

AppUI UI;
SecuritySys SecSysFunc;
EditBook BookFunc;

using namespace std;

void AppUI::MainMenu()
{
    //variable declarations
    int opt;

    MenuStart:

    system("CLS");
    TitleHeader();
    setTxtColor(10);
    PageTitle("Main Menu");
    string Menu[] = {"Add Books", "Search Books", "View Books", "Delete Book", "Exit"};
    cout << "1.\tAdd Books" << endl;
    cout << "2.\tSearch Books" << endl;
    cout << "3.\tView Books" << endl;
    cout << "4.\tDelete Books" << endl << endl;
    cout << "0.\tExit";

    cout << "\n\nEnter Option: ";
    cin >> opt;

    switch(opt)
    {
        case 1:
            BookFunc.AddBook();
            break;

        case 2:
            BookFunc.AddBook();
            break;

        case 3:
            BookFunc.AddBook();
            break;

        case 4:
            BookFunc.AddBook();
            break;

        case 0:
            SecSysFunc.Logout();
            break;

        default:
            cout << "Invalid option!";
            sleep(1);
            goto MenuStart;
    }
}

AppUI.h #

#ifndef APPUI_H
#define APPUI_H


class AppUI
{
    public:
        void MainMenu();
        void AddBook();
        void SearchBook();

    private:
};

#endif // APPUI_H

EditBook.cpp #

#include <iostream>
#include <windows.h>
#include <iomanip>
#include <string>
#include <cstring>
#include <algorithm>
#include <unistd.h>
#include <stdio.h>

#include "SecuritySys.h"
#include "AppUI.h"
#include "EditBook.h"

AppUI UI;
SecuritySys SecSysFunc;
EditBook BookFunc;

using namespace std;

void EditBook::AddBook()
{
    system("CLS");
    UI.AddBook();
}

SearchByTitle(string searchTxt)
{
    cout << "Enter search by title code here";
    system("pause");
    UI.MainMenu();
}

EditBook.h #

#ifndef EDITBOOK_H
#define EDITBOOK_H


class EditBook
{
    public:
        void AddBook();

        void SearchBook();

        void ViewBooks();

        void DeleteBooks();

    protected:

    private:
};

#endif // EDITBOOK_H

SecuritySys.cpp #

#include <iostream>
#include <windows.h>
#include <iomanip>
#include <string>
#include <cstring>
#include <algorithm>
#include <unistd.h>
#include <stdio.h>

#include "SecuritySys.h"
#include "AppUI.h"
#include "EditBook.h"

AppUI UI;
SecuritySys SecSysFunc;
EditBook BookFunc;

using namespace std;
void SecuritySys::Login()
{
    UI.MainMenu();
}

void SecuritySys::Logout()
{
    exit(0);
}

SecuritySys.h #

#ifndef SECURITYSYS_H
#define SECURITYSYS_H


class SecuritySys
{
    public:
        void Login();
        void Logout();

    private:

};

#endif

错误原因 #

你现在的核心问题是全局对象重复定义：每个.cpp文件里都直接定义了UI、SecSysFunc、BookFunc这三个全局对象。编译阶段每个cpp会生成独立的目标文件，到链接阶段，编译器发现多个目标文件里都有同名的全局符号，就会抛出“重复定义”的错误。

解决方法 #

这里给你两种可行的方案，按需选择：

方案1：用extern声明共享全局对象（快速修复） #

只在一个文件里定义全局对象，其他需要使用的文件用extern声明，表示“这个对象在别的地方已经定义了，我只是拿来用”：

1. 保留main.cpp里的对象定义（也可以选其他核心文件，比如单独建一个globals.cpp）：

   AppUI UI;
   SecuritySys SecSysFunc;
   EditBook BookFunc;
   

2. 在AppUI.cpp、EditBook.cpp、SecuritySys.cpp的顶部，把原来的定义替换成extern声明：

   extern AppUI UI;
   extern SecuritySys SecSysFunc;
   extern EditBook BookFunc;
   

这样链接时只会找到唯一的全局对象定义，冲突就解决了。

方案2：摒弃全局对象，用依赖注入（更规范的写法） #

全局对象会导致代码耦合度高，后期维护麻烦，更专业的做法是把需要的对象作为参数传递给方法：

1. 修改AppUI.h中MainMenu的声明，添加前置声明并接收对象引用：

   class EditBook; // 前置声明，避免头文件循环引用
   class SecuritySys;
   
   class AppUI
   {
       public:
           void MainMenu(EditBook& bookFunc, SecuritySys& secFunc);
           void AddBook();
           void SearchBook();
   
       private:
   };
   

2. 修改AppUI.cpp的MainMenu实现，用传入的引用代替全局对象：

   void AppUI::MainMenu(EditBook& bookFunc, SecuritySys& secFunc)
   {
       int opt;
       MenuStart:
       system("CLS");
       TitleHeader();
       setTxtColor(10);
       PageTitle("Main Menu");
       cout << "1.\tAdd Books" << endl;
       cout << "2.\tSearch Books" << endl;
       cout << "3.\tView Books" << endl;
       cout << "4.\tDelete Books" << endl << endl;
       cout << "0.\tExit";
   
       cout << "\n\nEnter Option: ";
       cin >> opt;
   
       switch(opt)
       {
           case 1:
               bookFunc.AddBook();
               break;
           case 2:
               bookFunc.SearchBook();
               break;
           case 3:
               bookFunc.ViewBooks();
               break;
           case 4:
               bookFunc.DeleteBooks();
               break;
           case 0:
               secFunc.Logout();
               break;
           default:
               cout << "Invalid option!";
               sleep(1);
               goto MenuStart;
       }
   }
   

3. 修改EditBook.h的AddBook声明，添加前置声明并接收AppUI引用：

   class AppUI; // 前置声明
   
   class EditBook
   {
       public:
           void AddBook(AppUI& ui);
           void SearchBook();
           void ViewBooks();
           void DeleteBooks();
   
       protected:
       private:
   };
   

4. 修改EditBook.cpp的AddBook实现，同时补全SearchByTitle的返回类型：

   void EditBook::AddBook(AppUI& ui)
   {
       system("CLS");
       ui.AddBook();
   }
   
   void SearchByTitle(string searchTxt)
   {
       cout << "Enter search by title code here";
       system("pause");
       UI.MainMenu();
   }
   

5. 最后在main.cpp里调用时传递对象：

   int main()
   {
       AppUI UI;
       SecuritySys SecSysFunc;
       EditBook BookFunc;
   
       UI.MainMenu(BookFunc, SecSysFunc);
   }
   

这种方式彻底消除了全局对象，代码结构更清晰，耦合度更低。

内容的提问来源于stack exchange，提问作者Lourd Nathaniel Gonzalez