Ошибка множественного определения переменной

@timofey1113 · Регистрация: 11.12.2017

У меня при компиляции вылазит ошибка множественного определения переменной.

Вот код.

C++

#ifndef GLOBALS_H
#define GLOBALS_H
 
#include <ctime>
#include <random>
 
std::mt19937 myRand(time(0));
 
const int height = 721;
const int width = 1279;
const int sizeSquare = 6;
 
#endif

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

C++

#ifndef GLOBALS_H
#define GLOBALS_H
.....
#endif

я не забыл.

@zss · 27.10.2018, 20:31

Вы этот include добавляете в несколько файлов проекта.
Т.е. в каждой единице трансляции появляются эти переменные.

Надо так:
Global.h:

C++

#ifndef GLOBALS_H
#define GLOBALS_H
 
#include <ctime>
#include <random>
 
extern std::mt19937 myRand;
 
extern const int height;
extern const int width;
extern const int sizeSquare;
 
#endif

и добавьте в проект файл
Global.cpp:

C++

#include <ctime>
#include <random>
 
std::mt19937 myRand(time(0));
 
const int height = 721;
const int width = 1279;
const int sizeSquare = 6;

@Azazel-San · 27.10.2018, 20:41

zss,
а почему не без определения?

C++

1
2
3

extern const int height;
extern const int width;
extern const int sizeSquare;

@zss · 27.10.2018, 21:01

НЕ понял вопроса.
Если мы хотим использовать переменные, объявленные в другой единице трансляции,
то их нужно объявить как extern.

@Azazel-San · 27.10.2018, 21:11

zss,
Global.h:

C++

почему так Вы написали так 
const int height = 721;
const int width = 1279;
const int sizeSquare = 6;
 
а не так
extern const int height;
extern const int width;
extern const int sizeSquare;

?
Чисто самому интересно?

@zss · 27.10.2018, 21:32

Сообщение от Azazel-San

почему так Вы написали так

Так я написал в global.cpp.
Этот файл откомпилируется в отдельной единице трансляции и в нем будут созданы эти переменные.
А в файле global.h лишь указывается, что эти переменные где-то существуют.
Поэтому-то они и объявлены как extern.

@Azazel-San · 27.10.2018, 21:42

zss, не, значит это я тупой, не могу объяснить
ок:

C++

// вот мы в "заголовке" Global.h вы в своем посте в этом заголовке написали так
const int height = 721;
const int width = 1279;
const int sizeSquare = 6;
// почему в этом заголовке нельзя написать так же, но без определения переменных
extern const int height;
extern const int width;
extern const int sizeSquare;
// это все ещё заголовок, в cpp мы их ещё раз конечно же определим

@DrOffset · 27.10.2018, 21:55

Для таких const`ов делать внешнее связывание скорее вредно, чем полезно. Так они станут неизменяемыми переменными, а могли бы остаться константами времени компиляции (компилятор мог бы их вообще не размещать в памяти). Ошибки множественного определения для const`ов не будет, т.к. по умолчанию они имеют внутреннее связывание.

@zss · 28.10.2018, 07:23

Сообщение от Azazel-San

в этом заголовке

Это не заголовок, а ОТДЕЛЬНО КОМПИЛИРУЕМЫЙ .cpp ФАЙЛ

@zss Модератор 12631 / 10128 / 6098 Регистрация: 18.12.2011 Сообщений: 27,159
	27.10.2018, 21:01	4
	НЕ понял вопроса. Если мы хотим использовать переменные, объявленные в другой единице трансляции, то их нужно объявить как extern. 0

@zss Модератор 12631 / 10128 / 6098 Регистрация: 18.12.2011 Сообщений: 27,159
	27.10.2018, 21:32	6
	Сообщение от Azazel-San почему так Вы написали так Так я написал в global.cpp. Этот файл откомпилируется в отдельной единице трансляции и в нем будут созданы эти переменные. А в файле global.h лишь указывается, что эти переменные где-то существуют. Поэтому-то они и объявлены как extern. 0

@DrOffset 16490 / 8984 / 2204 Регистрация: 30.01.2014 Сообщений: 15,603
	27.10.2018, 21:55	8
	Для таких const`ов делать внешнее связывание скорее вредно, чем полезно. Так они станут неизменяемыми переменными, а могли бы остаться константами времени компиляции (компилятор мог бы их вообще не размещать в памяти). Ошибки множественного определения для const`ов не будет, т.к. по умолчанию они имеют внутреннее связывание. 2

@zss Модератор 12631 / 10128 / 6098 Регистрация: 18.12.2011 Сообщений: 27,159
	28.10.2018, 07:23	9
	Сообщение от Azazel-San в этом заголовке Это не заголовок, а ОТДЕЛЬНО КОМПИЛИРУЕМЫЙ .cpp ФАЙЛ 0

Опции темы