Как избавиться от замусоривания cpp файлов дубликатами деклараций функций и переменных?

@programina · Регистрация: 23.10.2011

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

Как избавиться от замусоривания cpp файлов дубликатами деклараций функций и переменных?

У меня есть например несколько файлов:
opengl.h

C++

#ifndef OPENGL_H
#define OPENGL_H
 
extern int a;
extern void foo();
 
#endif // OPENGL_H

main.cpp

C++

#include "opengl.h"
int a;
 
int main()
{
  a = 5;
  return 0;
}

opengl.cpp

C++

#include "opengl.h"
int a;
 
void foo()
{
  a = 9;
}

Видно, что int a объявляется не только в opengl.h, но еще и в main.cpp и opengl.cpp. Как сделать, чтобы можно было один раз написать int a?

@TheCalligrapher · 17.11.2014, 09:08

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

Как сделать, чтобы можно было один раз написать int a?

Ну вообще-то должно быть два раза: объявление и определение. Для того, чтобы получить глобальную переменную (если она вам действительно нужна), в заголовочном файле должно быть написано 'extern int a;' (объявление), а в одном-единственном файле реализации - просто 'int a;' (определение). В какой именно файл вы поместите это определение - ваш выбор. Все.

А то, что у вас определение 'int a;' повторяется в нескольких файлах реализации - это грубая ошибка. В языке С++ такое запрещается. Поэтому ответ на ваш вопрос прост - просто выкосите нафиг лишние определения из файлов реализации и оставьте лишь одно объявление в заголовочном файле и одно определение в каком-то файле реализации.

Ситуация полностью аналогична ситуации с 'foo'. Вы же не пихаете определение функции 'foo' в несколько файлов реализации? Вот и определение переменной тоже пихать не надо.

@programina · 17.11.2014, 09:27 **[ТС]**

Так ведь если я уберу int a например в main.cpp, то будет ошибка, значит нужно в каждый файл cpp писать int a.

@TheCalligrapher · 17.11.2014, 09:29

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

Так ведь если я уберу int a например в main.cpp, то будет ошибка

С чего бы это вдруг? Какая ошибка? Приведите пример.

Если это файлы одной программы, то нет, никакой ошибки не будет.

@programina · 17.11.2014, 09:30 **[ТС]**

А если этих файлов cpp штук 500, то 500 int a по одному в каждом + 1 extern int a в заголовке.

@TheCalligrapher · 17.11.2014, 09:36

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

А если этих файлов cpp штук 500, то 500 int a по одному в каждом

На колу мочало, начинай сначала... Я же ясно сказал - так делать нельзя ни в коем случае. 'int a;' должно присутствовать только в одном файле реализации. Все.

Хоть у вас 500 файлов, хоть миллион, во всей программе у вас будет открытым текстом написано ровно два 'int a;': один в заголовке как 'extern int a;' и еще один в одном из файлов реализации, как 'int a;'.

@programina · 17.11.2014, 09:37 **[ТС]**

А если не один int a, а еще int b, int c, int d и т.д. штук 200, то получится 200*500 = 100000 переменных.

@TheCalligrapher · 17.11.2014, 09:39

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

А если не один int a, а еще int b, int c, int d и т.д. штук 200,

Вы, я вижу, не читатель, а писатель.

В любом случае, если у вас в программе 200 глобальных переменных вида 'int a' россыпью, то проблему надо править в несколько другой консерватории...

@programina · 17.11.2014, 09:40 **[ТС]**

TheCalligrapher, мне кажется так не будет работать, в файле реализации, где нет int a; должно выдать ошибку. Вчера по крайней мере выдавало, сегодня вечером еще раз проверю.

@TheCalligrapher · 17.11.2014, 09:41

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

TheCalligrapher, мне кажется так не будет работать, в файле реализации, где нет int a; должно выдать ошибку.

Вы несете чепуху. С чего бы это вдруг оно будет выдавать ошибку?

Наоборот, если вы начнете пихать 'int a' в каждый файл реализации, то получите от линкера ошибку множественного определения. (Некоторые реализации разрешают подобное, если определение не содержит инициализатора. Но это тем не менее не меняет того факта, что это ошибка.)

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

Вчера по крайней мере выдавало,

Вчера вы что-то делали совсем не так, как описываете сейчас.

@programina · 17.11.2014, 09:43 **[ТС]**

В любом случае, если у вас в программе 200
глобальных переменных вида 'int a', то проблему
надо править в несколько другой консерватории...

Не надо, мне нужно много глобальных переменных, чтобы они виделись во всех файлах.

@TheCalligrapher · 17.11.2014, 09:46

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

мне нужно много глобальных переменных, чтобы они виделись во всех файлах.

Именно для этого вы помещаете 'extern int a;' в заголовочный файл. Именно это делает переменную 'a' видимой во всех файлах (не забывайте только включать заголовочный файл везде, где надо). Больше ничего делать не надо.

@programina · 17.11.2014, 09:47 **[ТС]**

Вчера вы что-то делали совсем не так, как
описываете сейчас

Возможно. Пишу по памяти, компилятора по рукой нет

Добавлено через 1 минуту
Вечером точно скажу.

ValeryS · 17.11.2014, 09:58

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

Видно, что int a объявляется не только в opengl.h, но еще и в main.cpp и opengl.cpp. Как сделать, чтобы можно было один раз написать int a?

для глобальных переменных, я обычно создаю
файл Global.c

C

#include "Global.h"
int a;
int b;
.......

и файл Global.h

C

#ifndef GLOBAL_H
#define GLOBAL_H
extern int a;
extern int b;
.......
#endif

потом где нужно подключаю Global.h

@programina · 17.11.2014, 10:06 **[ТС]**

для глобальных переменных, я обычно создаю
файл Global.c

Попробую сделать так, если не получится так как пишет TheKaligrapher

@TheCalligrapher · 17.11.2014, 10:24

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

Попробую сделать так, если не получится так как пишет TheKaligrapher

Гм... И я, и ValeryS написали абсолютно одно и то же.

@mss307 · 17.11.2014, 10:34

TheCalligrapher,

Не по теме:

нефига, ТС выбрал ValeryS )) хотя с согласен с твоим постом.

@TheCalligrapher · 17.11.2014, 10:38

Не по теме:

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

нефига, ТС выбрал ValeryS ))

Посмотрим, что он запоет, когда его припашут определять "200*500 = 100000 переменных" россыпью в 500 файлах.

ValeryS · 17.11.2014, 11:01

Не по теме:

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

когда его припашут

А некому меня припахивать:)
сам создаю ТЗ, сам решаю, сам у себя принимаю, сам себя матерю:)
Заказчику нужно чтобы железяка "моргала", а что там внутри ему не интересно

@Kuzia domovenok · 17.11.2014, 11:09

Регистрация: 23.10.2011
Сообщений: 4,439

... и ты, всё это время программируя, ни разу не получал ошибку множественного определения переменных?

@TheCalligrapher Вездепух 11695 / 6374 / 1724 Регистрация: 18.10.2014 Сообщений: 16,066
	17.11.2014, 09:46	12
	Сообщение от programina мне нужно много глобальных переменных, чтобы они виделись во всех файлах. Именно для этого вы помещаете 'extern int a;' в заголовочный файл. Именно это делает переменную 'a' видимой во всех файлах (не забывайте только включать заголовочный файл везде, где надо). Больше ничего делать не надо. 1

@programina 2062 / 618 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	17.11.2014, 09:47 [ТС]	13
	Вчера вы что-то делали совсем не так, как описываете сейчас Возможно. Пишу по памяти, компилятора по рукой нет Добавлено через 1 минуту Вечером точно скажу. 0

@programina 2062 / 618 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	17.11.2014, 10:06 [ТС]	15
	для глобальных переменных, я обычно создаю файл Global.c Попробую сделать так, если не получится так как пишет TheKaligrapher 0

@TheCalligrapher Вездепух 11695 / 6374 / 1724 Регистрация: 18.10.2014 Сообщений: 16,066
	17.11.2014, 10:24	16
	Сообщение от programina Попробую сделать так, если не получится так как пишет TheKaligrapher Гм... И я, и ValeryS написали абсолютно одно и то же. 1

@TheCalligrapher Вездепух 11695 / 6374 / 1724 Регистрация: 18.10.2014 Сообщений: 16,066
	17.11.2014, 09:08	2
	Сообщение от programina Как сделать, чтобы можно было один раз написать int a? Ну вообще-то должно быть два раза: объявление и определение. Для того, чтобы получить глобальную переменную (если она вам действительно нужна), в заголовочном файле должно быть написано 'extern int a;' (объявление), а в одном-единственном файле реализации - просто 'int a;' (определение). В какой именно файл вы поместите это определение - ваш выбор. Все. А то, что у вас определение 'int a;' повторяется в нескольких файлах реализации - это грубая ошибка. В языке С++ такое запрещается. Поэтому ответ на ваш вопрос прост - просто выкосите нафиг лишние определения из файлов реализации и оставьте лишь одно объявление в заголовочном файле и одно определение в каком-то файле реализации. Ситуация полностью аналогична ситуации с 'foo'. Вы же не пихаете определение функции 'foo' в несколько файлов реализации? Вот и определение переменной тоже пихать не надо. 1

@programina 2062 / 618 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	17.11.2014, 09:27 [ТС]	3
	Так ведь если я уберу int a например в main.cpp, то будет ошибка, значит нужно в каждый файл cpp писать int a. 0

@TheCalligrapher Вездепух 11695 / 6374 / 1724 Регистрация: 18.10.2014 Сообщений: 16,066
	17.11.2014, 09:29	4
	Сообщение от programina Так ведь если я уберу int a например в main.cpp, то будет ошибка С чего бы это вдруг? Какая ошибка? Приведите пример. Если это файлы одной программы, то нет, никакой ошибки не будет. 1

@programina 2062 / 618 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	17.11.2014, 09:30 [ТС]	5
	А если этих файлов cpp штук 500, то 500 int a по одному в каждом + 1 extern int a в заголовке. 0

@TheCalligrapher Вездепух 11695 / 6374 / 1724 Регистрация: 18.10.2014 Сообщений: 16,066
	17.11.2014, 09:36	6
	Сообщение от programina А если этих файлов cpp штук 500, то 500 int a по одному в каждом На колу мочало, начинай сначала... Я же ясно сказал - так делать нельзя ни в коем случае. 'int a;' должно присутствовать только в одном файле реализации. Все. Хоть у вас 500 файлов, хоть миллион, во всей программе у вас будет открытым текстом написано ровно два 'int a;': один в заголовке как 'extern int a;' и еще один в одном из файлов реализации, как 'int a;'. 1

@programina 2062 / 618 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	17.11.2014, 09:37 [ТС]	7
	А если не один int a, а еще int b, int c, int d и т.д. штук 200, то получится 200*500 = 100000 переменных. 0

@TheCalligrapher Вездепух 11695 / 6374 / 1724 Регистрация: 18.10.2014 Сообщений: 16,066
	17.11.2014, 09:39	8
	Сообщение от programina А если не один int a, а еще int b, int c, int d и т.д. штук 200, Вы, я вижу, не читатель, а писатель. В любом случае, если у вас в программе 200 глобальных переменных вида 'int a' россыпью, то проблему надо править в несколько другой консерватории... 1

@programina 2062 / 618 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	17.11.2014, 09:40 [ТС]	9
	TheCalligrapher, мне кажется так не будет работать, в файле реализации, где нет int a; должно выдать ошибку. Вчера по крайней мере выдавало, сегодня вечером еще раз проверю. 0

@TheCalligrapher Вездепух 11695 / 6374 / 1724 Регистрация: 18.10.2014 Сообщений: 16,066
	17.11.2014, 09:41	10
	Сообщение от programina TheCalligrapher, мне кажется так не будет работать, в файле реализации, где нет int a; должно выдать ошибку. Вы несете чепуху. С чего бы это вдруг оно будет выдавать ошибку? Наоборот, если вы начнете пихать 'int a' в каждый файл реализации, то получите от линкера ошибку множественного определения. (Некоторые реализации разрешают подобное, если определение не содержит инициализатора. Но это тем не менее не меняет того факта, что это ошибка.) Сообщение от programina Вчера по крайней мере выдавало, Вчера вы что-то делали совсем не так, как описываете сейчас. 1

@programina 2062 / 618 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	17.11.2014, 09:43 [ТС]	11
	В любом случае, если у вас в программе 200 глобальных переменных вида 'int a', то проблему надо править в несколько другой консерватории... Не надо, мне нужно много глобальных переменных, чтобы они виделись во всех файлах. 0

@mss307 118 / 118 / 41 Регистрация: 14.12.2013 Сообщений: 352
	17.11.2014, 10:34	17
	TheCalligrapher, Не по теме: нефига, ТС выбрал ValeryS )) хотя с согласен с твоим постом. 0

@TheCalligrapher
	17.11.2014, 10:38 #18
	Не по теме: Сообщение от mss307 нефига, ТС выбрал ValeryS )) Посмотрим, что он запоет, когда его припашут определять "200*500 = 100000 переменных" россыпью в 500 файлах. 0

ValeryS
	17.11.2014, 11:01 #19
	Не по теме: Сообщение от TheCalligrapher когда его припашут А некому меня припахивать:) сам создаю ТЗ, сам решаю, сам у себя принимаю, сам себя матерю:) Заказчику нужно чтобы железяка "моргала", а что там внутри ему не интересно 0

@Kuzia domovenok 4064 / 3318 / 924 Регистрация: 25.03.2012 Сообщений: 12,493 Записей в блоге: 1
	17.11.2014, 11:09	20
	Регистрация: 23.10.2011 Сообщений: 4,439 ... и ты, всё это время программируя, ни разу не получал ошибку множественного определения переменных? 0