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

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

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

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

Вот код.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
#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++
1
2
3
4
#ifndef GLOBALS_H
#define GLOBALS_H
.....
#endif

я не забыл.

@zss · 27.10.2018, 20:31

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

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
#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++
1
2
3
4
5
6
7
8
#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++
1
2
3
4
5
6
7
8
9
почему так Вы написали так 
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++
1
2
3
4
5
6
7
8
9
// вот мы в "заголовке" 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 ФАЙЛ

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

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

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