C++项目中‘function already has a body’错误的原因及解决方法

通用原因 #

• 函数（包括构造、析构、普通成员函数）在头文件的类定义中已经提供了函数体（哪怕是空的{}），同时又在.cpp源文件中再次实现该函数的函数体，造成重复定义。
• 头文件未添加保护机制（如#ifndef/#define/#endif或#pragma once），导致头文件被多次包含，间接引发函数重复定义问题。
• 同一非inline函数在多个源文件中被重复实现。

你的代码问题及解决步骤 #

你的代码触发错误的直接原因是：头文件里的构造函数User(string username)和析构函数~User()都写了空函数体{}，相当于已经完成了函数定义，而你又在.cpp文件里重新实现了这两个函数，导致编译器判定重复定义。

解决方法如下：

1. 修改头文件的类定义，移除构造函数和析构函数的空函数体，只保留函数声明：

class User {  

private:
    string *name; 
    vector<Post> posts; 
    unsigned int *num_comments; 

public:
    User(string username);
    ~User();
    Post& createPost(string content); 
     
    vector<Post*> getPosts(); 
    int getnumcomments();
    string getName(); 
};

2. 保留.cpp文件中的构造函数实现，同时建议补充析构函数的内存释放逻辑（避免内存泄漏）：

User::User(string username) {
    this->name = &username; 
    num_comments = new unsigned int(0);
}

User::~User() {
    delete num_comments;
    // 注意：当前name指针指向的是构造函数形参的地址，函数结束后形参销毁，name会变成野指针，后续访问会触发未定义行为
}

额外的代码优化建议 #

你当前构造函数中this->name = &username;存在严重问题：形参username在构造函数执行完毕后会被销毁，name指针会变成野指针。建议修改为以下两种方式之一：

• 将头文件中的string *name;改为string name;，构造函数中直接赋值：this->name = username;
• 如果坚持使用指针，改为动态分配内存：

// 构造函数中
this->name = new string(username);
// 析构函数中补充
delete name;

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