Ошибка при работе с объектом класса в конструкторе другого класса

@Sinkha · Регистрация: 04.07.2015

Имеется данный код: https://github.com/ubelian/glg... ter/glgame

В файле Food.cpp в конструкторе класса Food::Food() на 19 строке закомментировано обращение объекта snake класса Snake к методу getHeadCP();. Эта строка вызывает некую ошибку в процессе работы программы, которую понять я не могу. Если snake.getHeadCP() написать не в конструкторе, а в любом другом методе, то все работает нормально. Можно было бы подумать, что объект food класса Food создается раньше чем snake, но это не так. Прошу помощи

nd2 · 29.10.2016, 19:34

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

Эта строка вызывает некую ошибку

И как это выглядит?

@Sinkha · 29.10.2016, 19:49 **[ТС]**

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

И как это выглядит?

Привет. Выскакивает ошибка "Прекращена работа программы "glgame.exe" ".

nd2 · 29.10.2016, 19:59

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

Выскакивает ошибка "Прекращена работа программы "glgame.exe" ".

Больше ничего не пишет? В чём собираешь? Debug?

Добавлено через 4 минуты
Глобальные объекты Food есть?

@Sinkha · 29.10.2016, 20:12 **[ТС]**

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

Глобальные объекты Food есть?

Есть. Я давал уже ссылку сверху на исходный код, может поможет. Не пойму в чем дело. По любому же ошибка банальная

@castaway · 29.10.2016, 20:15

Sinkha, у вас неправильный подход. Объекты классов Food и Snake определены как глобальные в разных единицах трансляции (в разных cpp-файлах). Стандарт языка не гарантирует порядок инициализации объектов в таких случаях.

nd2 · 29.10.2016, 20:16

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

Есть.

Если есть, то где гарантия, что он создаётся (а значит и конструктор вызывается) позже, чем глобальный snake?

@Sinkha · 29.10.2016, 20:38 **[ТС]**

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

Sinkha, у вас неправильный подход. Объекты классов Food и Snake определены как глобальные в разных единицах трансляции (в разных cpp-файлах). Стандарт языка не гарантирует порядок инициализации объектов в таких случаях.

Это объясняет уже пару вещей. А как правильно объеденить все эти определения в 1ой единице трансляции?

@castaway · 29.10.2016, 20:44

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

Это объясняет уже пару вещей. А как правильно объеденить все эти определения в 1ой единице трансляции?

Вы не поняли. Вам вообще следует избавиться от глобальных объектов.

@Sinkha · 29.10.2016, 20:52 **[ТС]**

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

Вы не поняли. Вам вообще следует избавиться от глобальных объектов.

Но мне нужно использовать snake в другом классе. Как мне быть?

@castaway · 29.10.2016, 20:56

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

Но мне нужно использовать snake в другом классе. Как мне быть?

Что вам мешает передать объект одного класса в конструктор/метод другого?

@Sinkha · 29.10.2016, 21:21 **[ТС]**

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

Что вам мешает передать объект одного класса в конструктор/метод другого?

Кстати да. Но я задавался первоначально целью по максимуму все инкапсулировать, чтобы подключил хедеры и РАЗ! и все заработало.

IGPIGP · 29.10.2016, 21:32

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

Но я задавался первоначально целью по максимуму все инкапсулировать, чтобы подключил хедеры и РАЗ! и все заработало.

Но Вы включаете хедер в хедер, а не на перекрест хедер одного в cpp другого. Советов не слушаете и говорите, что если включать не так как у Вас сделано, то не работает. Но Вы пробовали не те варианты, что надо. А вариантов много.

@Sinkha · 29.10.2016, 21:50 **[ТС]**

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

Но Вы включаете хедер в хедер, а не на перекрест хедер одного в cpp другого. Советов не слушаете и говорите, что если включать не так как у Вас сделано, то не работает. Но Вы пробовали не те варианты, что надо. А вариантов много.

Я просто был не в курсе что стандарт не гарантирует создание объекта в заданом порядке, если объявлять глоб переменные в разных файлах. При дебаге выяснилось, что объект snake в конструкторе food в момент вызова конструктора не существует. А перекресное включение нужно было для того чтобы food.cpp не говорил, мол snake не определен, так как он им пользуется.

@castaway 4978 / 3085 / 456 Регистрация: 10.11.2010 Сообщений: 11,164 Записей в блоге: 10
	29.10.2016, 20:56	11
	Сообщение от Sinkha Но мне нужно использовать snake в другом классе. Как мне быть? Что вам мешает передать объект одного класса в конструктор/метод другого? 0

IGPIGP Комп_Оратор) $Эксперт по математике/физике$ 8733 / 4466 / 599 Регистрация: 04.12.2011 Сообщений: 13,315 Записей в блоге: 16
	29.10.2016, 21:32	13
	Сообщение от Sinkha Но я задавался первоначально целью по максимуму все инкапсулировать, чтобы подключил хедеры и РАЗ! и все заработало. Но Вы включаете хедер в хедер, а не на перекрест хедер одного в cpp другого. Советов не слушаете и говорите, что если включать не так как у Вас сделано, то не работает. Но Вы пробовали не те варианты, что надо. А вариантов много. 0

@Sinkha 3 / 3 / 2 Регистрация: 04.07.2015 Сообщений: 187
	29.10.2016, 21:50 [ТС]	14
	Сообщение от IGPIGP Но Вы включаете хедер в хедер, а не на перекрест хедер одного в cpp другого. Советов не слушаете и говорите, что если включать не так как у Вас сделано, то не работает. Но Вы пробовали не те варианты, что надо. А вариантов много. Я просто был не в курсе что стандарт не гарантирует создание объекта в заданом порядке, если объявлять глоб переменные в разных файлах. При дебаге выяснилось, что объект snake в конструкторе food в момент вызова конструктора не существует. А перекресное включение нужно было для того чтобы food.cpp не говорил, мол snake не определен, так как он им пользуется. 0

nd2 3426 / 2805 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	29.10.2016, 19:34	2
	Сообщение от Sinkha Эта строка вызывает некую ошибку И как это выглядит? 0

@Sinkha 3 / 3 / 2 Регистрация: 04.07.2015 Сообщений: 187
	29.10.2016, 19:49 [ТС]	3
	Сообщение от nd2 И как это выглядит? Привет. Выскакивает ошибка "Прекращена работа программы "glgame.exe" ". 0

nd2 3426 / 2805 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	29.10.2016, 19:59	4
	Сообщение от Sinkha Выскакивает ошибка "Прекращена работа программы "glgame.exe" ". Больше ничего не пишет? В чём собираешь? Debug? Добавлено через 4 минуты Глобальные объекты Food есть? 0

@Sinkha 3 / 3 / 2 Регистрация: 04.07.2015 Сообщений: 187
	29.10.2016, 20:12 [ТС]	5
	Сообщение от nd2 Глобальные объекты Food есть? Есть. Я давал уже ссылку сверху на исходный код, может поможет. Не пойму в чем дело. По любому же ошибка банальная 0

@castaway 4978 / 3085 / 456 Регистрация: 10.11.2010 Сообщений: 11,164 Записей в блоге: 10
	29.10.2016, 20:15	6
	Sinkha, у вас неправильный подход. Объекты классов `Food` и `Snake` определены как глобальные в разных единицах трансляции (в разных cpp-файлах). Стандарт языка не гарантирует порядок инициализации объектов в таких случаях. 0

nd2 3426 / 2805 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	29.10.2016, 20:16	7
	Сообщение от Sinkha Есть. Если есть, то где гарантия, что он создаётся (а значит и конструктор вызывается) позже, чем глобальный snake? 0

@Sinkha 3 / 3 / 2 Регистрация: 04.07.2015 Сообщений: 187
	29.10.2016, 20:38 [ТС]	8
	Сообщение от castaway Sinkha, у вас неправильный подход. Объекты классов Food и Snake определены как глобальные в разных единицах трансляции (в разных cpp-файлах). Стандарт языка не гарантирует порядок инициализации объектов в таких случаях. Это объясняет уже пару вещей. А как правильно объеденить все эти определения в 1ой единице трансляции? 0

@castaway 4978 / 3085 / 456 Регистрация: 10.11.2010 Сообщений: 11,164 Записей в блоге: 10
	29.10.2016, 20:44	9
	Сообщение от Sinkha Это объясняет уже пару вещей. А как правильно объеденить все эти определения в 1ой единице трансляции? Вы не поняли. Вам вообще следует избавиться от глобальных объектов. 0

@Sinkha 3 / 3 / 2 Регистрация: 04.07.2015 Сообщений: 187
	29.10.2016, 20:52 [ТС]	10
	Сообщение от castaway Вы не поняли. Вам вообще следует избавиться от глобальных объектов. Но мне нужно использовать snake в другом классе. Как мне быть? 0

@Sinkha 3 / 3 / 2 Регистрация: 04.07.2015 Сообщений: 187
	29.10.2016, 21:21 [ТС]	12
	Сообщение от castaway Что вам мешает передать объект одного класса в конструктор/метод другого? Кстати да. Но я задавался первоначально целью по максимуму все инкапсулировать, чтобы подключил хедеры и РАЗ! и все заработало. 0

Опции темы