Ошибка ссылки на C++ .. недействительна?

Question

Я продолжаю получать эту ссылку на Visual C++ 2010 LNK2005, говоря, что у меня есть несколько определений двух функций, которые содержатся в «Error.h». (Заголовок я сделал для отображения ошибок)

Мой проект так:

BaseImage.h 
BaseImage.cpp --> includes BaseImage.h , Error.h 
PNGImage.h --> includes BaseImage.h 
PNGImage.cpp --> includes PNGImage.h , Error.h 
main.cpp --> includes PNGImage.h 

И, конечно же, error.h:

/* 
Optional macros: 
AE_EXIT_AT_ERROR 
*/ 
#pragma once 
#include <stdexcept> 

void aeError(const char *str, int code=1) 
{ 
    throw std::runtime_error(str); 
    #ifdef AE_EXIT_AT_ERROR 
    std::exit(code); 
    #endif 
} 

void aeAssert(bool b, const char *failStr = "assertion failed") 
{ 
    if(!b) 
     aeError(failStr); 
} 

У меня есть #pragma once в каждом заголовке файл, и я попробовал добавить include guard для Error.h.

Вот результат компиляции:

1>PNGImage.obj : error LNK2005: "void __cdecl aeError(char const *,int)" (?aeError@@YAXPBDH@Z) already defined in BaseImage.obj 
1>PNGImage.obj : error LNK2005: "void __cdecl aeAssert(bool,char const *)" (?aeAssert@@YAX_NPBD@Z) already defined in BaseImage.obj 
1>C:\...\Project.exe : fatal error LNK1169: one or more multiply defined symbols found 

Может ли это быть ошибка?

Answer 1

Когда вы определяете функции в файле .h, делайте их inline. В противном случае определения функций являются частью объектного кода всех файлов .cpp, которые #include, с внешней связью.

inline void aeError(const char *str, int code=1) 
{ 
    throw std::runtime_error(str); 
    #ifdef AE_EXIT_AT_ERROR 
    std::exit(code); 
    #endif 
} 

inline void aeAssert(bool b, const char *failStr = "assertion failed") 
{ 
    if(!b) 
     aeError(failStr); 
} 

Другой вариант для вас, чтобы объявить функции в файле .h и определить их точно в одном .cpp файле.

заголовочный файл:

extern void aeError(const char *str, int code=1); 

extern void aeAssert(bool b, const char *failStr = "assertion failed"); 

Файл .cpp:

// Don't use inline and don't include the default argument values. 

void aeError(const char *str, int code) 
{ 
    throw std::runtime_error(str); 
    #ifdef AE_EXIT_AT_ERROR 
    std::exit(code); 
    #endif 
} 

void aeAssert(bool b, const char *failStr) 
{ 
    if(!b) 
     aeError(failStr); 
}