在没有明显原因的情况下获得“多重定义”

时间:2017-05-10 09:18:03

标签: c++ multiple-definition-error

问题从标题中可以清楚地看出来。这是代码:

TimeTask.hpp:

#pragma once
#include <locale>
#include <codecvt>
#include <chrono>

namespace TimeTask {

static std::wstring_convert<std::codecvt_utf8_utf16<wchar_t> > converter;

std::tm * to_date(
    int day, int month, int year,
    int hours = -1, int minutes = -1, int seconds = -1
) {
    time_t rawTime = time(0);
    std::tm * tInfo = localtime(&rawTime);
    tInfo->tm_mday  = day;
    tInfo->tm_mon   = month - 1;
    tInfo->tm_year  = year - 1900;

    if(hours >= 0)
        tInfo -> tm_hour = hours;
    if(seconds >= 0)
        tInfo -> tm_sec = seconds;
    if(minutes >= 0)
        tInfo -> tm_min = minutes;

    return tInfo;
}

std::string from_time_point(const std::chrono::system_clock::time_point &arg) {
    time_t time = std::chrono::system_clock::to_time_t(arg);
    return ctime(&time);
}

}

Task.hpp:

#pragma once

#include <string>
#include "TimeTask.hpp"

class Task
{
public:
    Task(
        const std::basic_string<wchar_t>&,
        const bool&,
        const std::chrono::system_clock::time_point&
    );
    Task();

    std::basic_string<wchar_t>                  description;
    bool                                        executed;
    std::chrono::system_clock::time_point       date;

    std::wstring towString();
};

Task.cpp:

#include "Task.hpp"
#include <sstream>    

Task::Task(
    const std::basic_string<wchar_t>& description,
    const bool& executed,
    const std::chrono::system_clock::time_point& date
):
    description(description),
    executed(executed),
    date(date) { }

Task::Task():
    Task( L"Task", false, std::chrono::system_clock::now() ) { }

std::wstring Task::towString() {
    std::wostringstream retval(L"");

    retval << L"Task:\t"
           << description
           << L"\nPerformed:\t"
           << std::boolalpha << executed;    

    retval << TimeTask::converter.from_bytes(TimeTask::from_time_point(date));

    return retval.str();
}

main.cpp中:

#include    <iostream>
#include    <type_traits>
#include    <vector>
#include    <list>
#include    "Task.hpp"

int main () {
    Task t1(L"Lezione programmazione",
            false,
            std::chrono::system_clock::from_time_t(
                std::mktime(TimeTask::to_date( 5,5,2017)) ));

    Task t2(L"Lezione programmazione",
            false,
            std::chrono::system_clock::from_time_t(
                std::mktime(TimeTask::to_date( 1,10,2017)) ));

    Task t3(L"Lezione programmazione",
            false,
            std::chrono::system_clock::from_time_t(
                std::mktime(TimeTask::to_date( 1,17,2017 )) ));

    std::list   <Task>    agenda1 = { t1, t2, t3 };
    std::vector <Task>    agenda2 = { t1, t2, t3 };

    for( auto it : agenda1 )
        std::cout << TimeTask::converter.to_bytes(it.towString());

    return 0;
}

我在这里做错了什么?我没有看到to_datefrom_time_point的任何多重定义(我的编译器函数,g++正在抱怨)。

1 个答案:

答案 0 :(得分:5)

您应将这些功能标记为inline。因为您的头文件不仅声明,而且还定义了此函数。当从多个翻译单元中包含时,这两个翻译单元都会编译该函数的实例,并且在链接期间会出错。

有关详细信息,请参阅单定义规则([basic.def.odr]):

  
      
  1. 每个程序应该只包含每个非内联函数或变量的一个定义,该函数或变量在废弃语句之外的程序中使用;无需诊断。   该定义可以在程序中明确显示,可以在标准或用户定义的库中找到,或者(在适当的时候)隐式定义(参见[class.ctor],[class.dtor]和[class.copy] ])。   内联函数或变量应在每个翻译单元中定义,在翻译单元中,它在废弃的陈述之外使用。
  2.         

    /../

         
        
    1. 可以有多个类类型的定义,枚举类型,带外部链接的内联函数([dcl.inline]),带外部链接的内联变量([dcl.inline]),类模板,非静态函数模板,类模板的静态数据成员,类模板的成员函数,或模板特化,在程序中未指定某些模板参数([temp.spec],[temp.class.spec])定义出现在不同的翻译单元中,并且定义满足以下要求。
    2.