ошибка привязки встроенной функции redefinition

Question

У меня есть класс, экспортированный в dll. И получил встроенные функции в этом классе экспортируемого:

header.h

class MODULE_EXPORT A 
{ 
public: 
    int GetInt(){ return iSomeInt; } 
}; 

Когда я включить этот заголовочный файл в отдельном модуле. Я получил ошибку LNK2005, что означает: GetInt() уже определен.

Если я поместил определение функции в файл .cpp. Ошибка НЕТ.

GetInt является встроенной функцией, если я определяю ее таким образом в файле заголовка, правильно? так почему ошибка переустановки связи? Я использую компилятор vC++. (Visual Studio 2010).

EDIT:

#pragma once 

уже был добавлен в файл заголовка. Забыл упомянуть об этом.

Answer 1

По существу для определения функции в файле заголовка отображается для каждого блока компиляции, и они пытаются скомпилировать его.

Однако функция уже определена в dll.

Решение будет размещать определение в файле cpp, как вы отметили, или использовать ключевое слово static, поэтому определение доступно только для единицы компиляции, в которой он определен, а не те, которые его включают.

Также функции инкрустации - это только подсказка о производительности для компилятора, она может не включать ее. И для этой функции я не могу себе представить, что они являются преимуществами производительности, учитывая ее простоту, поэтому я бы рекомендовал переместить ее в файл .cpp.

Answer 2

Это классическая проблема начинающего C/C++: поскольку вы включаете свой .h в несколько мест, вы получаете несколько экземпляров одного и того же метода. Самый интуитивный и, следовательно, несколько правильный метод решения этого вопроса - это метод в файле .cpp. Во всяком случае, ваш компилятор, вероятно, включит его.

Прежде всего, поскольку вы действительно подразумеваете, что это встроенный, используйте ключевое слово inline для метода.

Затем используйте надлежащие включенные охранники (ищите #pragma once или #ifndef __FILE_NAME_H__ или аналогичные в начале файла .h).

Тогда определение вашего метода является его декларацией и должно заканчиваться ;.

Кроме того, обратите внимание, что ваш метод не изменяет свой класс, так что вы должны использовать ключевое слово const:

#ifndef __Header_h__ 
#define __Header_h__ 
class MODULE_EXPORT A { 
    private: 
    int internalint; 
    public: 
    int getInt() const; 
}; 
#endif 

.cpp

int A::getInt() const { 
    return internalint; 
}