Как избавиться от multiple definition?

@elch10 · Регистрация: 27.04.2015

Author24 — интернет-сервис помощи студентам

Есть header:

C++

//base.hpp
#ifndef BASE_HPP
#define BASE_HPP
 
struct Coord {
    int x, y;
};
 
bool isValidCoord(const Coord& c) 
{
    return true;
}

Он включается во многие другие хидеры. Но проблема в том, что у меня есть циклические ссылки, т.е. один файл включает другой, а другой включает первый. Выглядит все примерно так:

C++

//Player.hpp
#ifndef PLAYER_HPP
#define PLAYER_HPP
 
#include "base.hpp"
 
//что то
#endif

C++

//Figure.hpp
#ifndef FIGURE_HPP
#define FIGURE_HPP
 
#include "Player.hpp"
 
//что-то
#endif

C++

//State.hpp
#ifndef STATE_HPP
#define STATE_HPP
 
#include "Resources.hpp"
#include "Game.hpp"
 
//что то
#endif

C++

//Game.hpp
#ifndef GAME_HPP
#define GAME_HPP
 
#include "State.hpp"
 
//что то
#endif

C++

//Resources.hpp
#ifndef RESOURCES_HPP
#define RESOURCES_HPP
 
#include "base.hpp"
#include "Figure.hpp"
#include "Player.hpp"
 
//что то
#endif

После компиляции линкер выдает:

Код

build/Debug/GNU-Linux/Game.o: In function `isValidCoord(Coord const&)':
/home/elchin/CppApplication_1/base.hpp:42: multiple definition of `isValidCoord(Coord const&)'
build/Debug/GNU-Linux/Figure.o:/home/elchin/CppApplication_1/base.hpp:42: first defined here
build/Debug/GNU-Linux/Player.o: In function `isValidCoord(Coord const&)':
/home/elchin/CppApplication_1/base.hpp:42: multiple definition of `isValidCoord(Coord const&)'
build/Debug/GNU-Linux/Figure.o:/home/elchin/CppApplication_1/base.hpp:42: first defined here
build/Debug/GNU-Linux/Resources.o: In function `isValidCoord(Coord const&)':
/home/elchin/CppApplication_1/base.hpp:42: multiple definition of `isValidCoord(Coord const&)'
build/Debug/GNU-Linux/Figure.o:/home/elchin/CppApplication_1/base.hpp:42: first defined here
build/Debug/GNU-Linux/State.o: In function `isValidCoord(Coord const&)':
/home/elchin/CppApplication_1/base.hpp:42: multiple definition of `isValidCoord(Coord const&)'
build/Debug/GNU-Linux/Figure.o:/home/elchin/CppApplication_1/base.hpp:42: first defined here

Т.е.я понимаю, что происходит несколько определений, но я не пойму почему инклуд гуарды не спасают. Как можно разрешить эту проблему?

@GbaLog- · 29.08.2017, 11:43

Сообщение от elch10

C++

1 2	//State.hpp #include "Game.hpp"

Сообщение от elch10

C++

1 2	//Game.hpp #include "State.hpp"

знаете, как директива препроцессора include работает?

Байт · 29.08.2017, 12:01

Один из способов

C++

1	inline bool isValidCoord(const Coord& c)

Другой - вынести реализацию из хедера в CPP
И. да, повторю за предыдущим оратором

Сообщение от GbaLog-

знаете, как директива препроцессора include работает?

Сообщение от elch10

не пойму почему инклуд гуарды не спасают

В каждом из cpp- модулей (и объектных o- модулях) будет реализация этой функции. Линковщик в замешательстве - какую из них использовать?

ValeryS · 29.08.2017, 12:08

если возникнет вопрос с переменными
то для них есть extern

Добавлено через 4 минуты

Сообщение от elch10

но я не пойму почему инклуд гуарды не спасают.

потому что Си файлы компилируются в разное время и при компиляции одного компилятор ничего не знает про другой
создаются куча объектных файлов не связанных друг с другом
а вот линковшик видит все объектные файлы и

Сообщение от Байт

Линковщик в замешательстве

Байт · 29.08.2017, 12:08

Сообщение от ValeryS

если возникнет вопрос с переменными
то для них есть extern

Тут немного другая ситуация... Да и с extern-переменными в хедерах не так все просто...

ValeryS · 29.08.2017, 12:12

Сообщение от Байт

Тут немного другая ситуация

ну да другая

переменная объявлена сто раз

Сообщение от Байт

Да и с extern-переменными в хедерах не так все просто

а че там сложного
в одном Си файле

C++

int x;

в заголовочном

C++

1	extern int x;

@elch10 · 29.08.2017, 12:43 **[ТС]**

Сообщение от Байт

Другой - вынести реализацию из хедера в CPP

Да действтельно вынес в cpp - заработало.

Сообщение от ValeryS

потому что Си файлы компилируются в разное время и при компиляции одного компилятор ничего не знает про другой
создаются куча объектных файлов не связанных друг с другом
а вот линковшик видит все объектные файлы и

Возникает другой вопрос, а почему тогда вынесение реализации в cpp файл не выдает ту же ошибку, ведь впринципе ничего не меняется?

Байт · 29.08.2017, 13:09

Сообщение от elch10

ведь впринципе ничего не меняется?

Еще как меняется! Прочитайте внимательно предыдущие посты в этой теме.

ValeryS · 29.08.2017, 18:53

Сообщение от elch10

ведь впринципе ничего не меняется?

очень даже меняется
макрос include просто подставляет текст заголовочного файла в Си файл
в результате в каждом Си файле есть

Сообщение от elch10

C++

bool isValidCoord(const Coord& c) 
{
 return true;
}

линкер видит стопитсот функций с именем isValidCoord и сходит с ума
а в случае реализации в Си файле функция только одна, в заголовочном только объявление функции а не реализация

Байт · 29.08.2017, 22:21

И придется повторить вопрос еще разок

Сообщение от GbaLog-

знаете, как директива препроцессора include работает?

А она работает чрезвычайно просто. Без всякой мистики.
Больше я этот вопрос повторять не намерен. Имеющий уши - да слышит! Имеющий хотя бы один глаз - да видит! Имеющий хоть капельку мозгов - да думает!

Как избавиться от multiple definition?

Решение

Решение

ValeryS Модератор 8941 / 6708 / 920 Регистрация: 14.02.2011 Сообщений: 23,649
	29.08.2017, 12:08	4
	Сообщение было отмечено elch10 как решение Решение если возникнет вопрос с переменными то для них есть `extern` Добавлено через 4 минуты Сообщение от elch10 но я не пойму почему инклуд гуарды не спасают. потому что Си файлы компилируются в разное время и при компиляции одного компилятор ничего не знает про другой создаются куча объектных файлов не связанных друг с другом а вот линковшик видит все объектные файлы и Сообщение от Байт Линковщик в замешательстве 2

Байт Диссидент 27709 / 17325 / 3812 Регистрация: 24.12.2010 Сообщений: 38,979
	29.08.2017, 12:08	5
	Сообщение от ValeryS если возникнет вопрос с переменными то для них есть extern Тут немного другая ситуация... Да и с extern-переменными в хедерах не так все просто... 0

@elch10 40 / 21 / 3 Регистрация: 27.04.2015 Сообщений: 176
	29.08.2017, 12:43 [ТС]	7
	Сообщение от Байт Другой - вынести реализацию из хедера в CPP Да действтельно вынес в cpp - заработало. Сообщение от ValeryS потому что Си файлы компилируются в разное время и при компиляции одного компилятор ничего не знает про другой создаются куча объектных файлов не связанных друг с другом а вот линковшик видит все объектные файлы и Возникает другой вопрос, а почему тогда вынесение реализации в cpp файл не выдает ту же ошибку, ведь впринципе ничего не меняется? 0

Байт Диссидент 27709 / 17325 / 3812 Регистрация: 24.12.2010 Сообщений: 38,979
	29.08.2017, 13:09	8
	Сообщение от elch10 ведь впринципе ничего не меняется? Еще как меняется! Прочитайте внимательно предыдущие посты в этой теме. 1

Байт Диссидент 27709 / 17325 / 3812 Регистрация: 24.12.2010 Сообщений: 38,979
	29.08.2017, 22:21	10
	И придется повторить вопрос еще разок Сообщение от GbaLog- знаете, как директива препроцессора include работает? А она работает чрезвычайно просто. Без всякой мистики. Больше я этот вопрос повторять не намерен. Имеющий уши - да слышит! Имеющий хотя бы один глаз - да видит! Имеющий хоть капельку мозгов - да думает! 0