Игра зависает через неопределённое время жрёт процессор и выключается

@2romanio005 · Регистрация: 31.07.2022

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

Здравствуйте, это моё первое приложение на WINAPI. Под конец столкнулся с проблемой зависания. Игра "Бомбер" запускается и работает отлично, но через несколько игр неожиданно зависает (причину не определил). Отладчик WS2019 не останавливается ни на каких исключениях. Посмотрел через Deleaker, утечек не наблюдаю.

Так как код довольно большой опишу его словами:
есть массив(map) который хранит ячейки(части) карты - class Cell. У Cell есть указатель на class Item, который является базовым для большинства элементов игры(Wall, Barrier, Void...).

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
class Cell
{
public:
    Cell();
 
    ~Cell();
 
    char get_type();
    Item* get_keep();
 
    void set(Item* keep, char type);
 
private:
    char type;
    Item* keep;
};
 
class Item {
public:
    Item() {}
 
    ~Item() {
        int a = 0;  // для отладки
    }
 
    void virtual Draw(HDC hdc, std::pair<float, float> place) {}
 
    void virtual Break(std::pair<int, int> map_cord);
};

Чтобы изменить содержимое карты используется метод Cell::set(Item*, char) :

C++
1
map[map_cord.first][map_cord.second].set(new Wall(), 'W');  // или другой класс

Ещё есть класс игрок, который не привязан к Cell, а ходит поверх карты.
Прорисовка и движения происходят по таймеру (25 мс). Для отрисовки используется двойная буферизация.

Думаю зависание как-то связано с delete в коде снизу, так как если их закоментировать, то зависания не наблюдал, но появляются утечки.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
void Cell::set(Item* keep, char type)
{
    if (this->type != type) {
        if (this->keep != nullptr) {
            delete (this->keep);
        }
        this->keep = keep;
        this->type = type;
    }
    else {
        delete keep;
        keep = nullptr;
    }
}

Пожалуйста помогите решить проблему. Также приветствуются советы как лучше можно было написать код, и где узкие места.
Приведу ссылку для удобства на GitHab с полным кодом. Если нужно, то могу отправить код и сюда, это мой первый пост и я не знаю можно ли приводить ссылки когда код большой (иначе получится очень много текста).

GitHab: https://github.com/2romanio005/Domber

@CoderPC · 31.07.2022, 18:09

2romanio005, в архиве сюда?

@2romanio005 · 31.07.2022, 18:17 **[ТС]**

Архив с кодом:

@VTsaregorodtsev · 31.07.2022, 19:57

Сообщение от 2romanio005

C++
1
2
3
4
    void set(Item* keep, char type);
 
private:
    char type;

Блин, ну нафига такое? Чтобы потом случайно поиметь проблему из-за очепятки при задании символа? Синтаксически код при очепятке (когда например, будет 'w' вместо 'W') останется корректным - но его семантическая правильность помрёт (если в коде затем не сделаешь обработку независимо от регистра). И не отловишь затем быстро подобную ошибку.
Делай вне класса enum с нужным числом состояний - и затем в классе поле этого перечислимого типа (вместо char). Состояния енума будут в затем в коде возникать в качестве именованных констант - поэтому все синтаксические очепятки будут сразу же ловиться компилятором.

@2romanio005 · 31.07.2022, 20:11 **[ТС]**

VTsaregorodtsev, спасибо! Согласен с вами. А можно как-нибудь обойтись вообще без этой переменной(она используется только для понимания какой именно класс лежит в указателе)?

Алексей1153 · 31.07.2022, 20:35

2romanio005, попробуй переписать игру, заменив сырые указатели на умные (std::unique_ptr / std::shared_ptr)

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

Сообщение от 2romanio005

А можно как-нибудь обойтись вообще без этой переменной(она используется только для понимания какой именно класс лежит в указателе)?

dynamic_cast в помощь

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

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

enum

тогда уж лучше enum class

Добавлено через 1 минуту
2romanio005, русские комментарии в гитхабе побились

Добавлено через 12 минут
2romanio005,

void (*ReadI)(HDC & I_, std::string way) = [](HDC& I_, std::string way) {

->

C++
1
auto ReadI = [](HDC& I_, std::string way) {

@2romanio005 · 31.07.2022, 21:02 **[ТС]**

nick]VTsaregorodtsev[/nick],

Сообщение от Алексей1153

попробуй переписать игру, заменив сырые указатели на умные (std::unique_ptr / std::shared_ptr)

Спасибо за ваши ответы. Почему сырые указатели так плохи?
Я могу это сделать, но:
1) это не решит/плохо решит подобные проблемы в будущем
2) я не горю желанием уходить в абстракцию, не поняв базы

Сообщение от Алексей1153

dynamic_cast в помощь

Спасибо, почитаю.

Сообщение от Алексей1153

void (*ReadI)(HDC & I_, std::string way) = [](HDC& I_, std::string way) {
-> auto ReadI = [](HDC& I_, std::string way) {

Почему я должен так делать? auto - сахар, которым я не очень хочу пользоваться.

@Ygg · 31.07.2022, 21:35

У меня зависло после одновременной установки нескольких бомб и взрыва одной из них. Хотя иногда подобные действия не приводят к зависаниям.

C++
1
2
3
4
5
6
void OneStep() {
    NowTime++;
 
    while (ActiveBomb.size() != 0 && (*(ActiveBomb.begin())).first <= NowTime) { //<--- зависло в этом цикле
        (*((*(ActiveBomb.begin())).second)).Detonation();
    }

Содержимое вектора ActiveBomb, первая бомба 'битая'

Code
1
2
3
4
-       ActiveBomb  { size=3 }  
+       [0] (2154, 0x0105e040 {map_cord=(-572662307, -572662307) radius=-572662307 timeout=-572662307 ...}) 
+       [1] (2181, 0x0105e598 {map_cord=(3, 3) radius=4 timeout=2181 ...})  
+       [2] (3607, 0x0105e598 {map_cord=(3, 3) radius=4 timeout=2181 ...})

C++
1
2
3
void Bomb::Detonation()
{
    if (this->IBroken) return; //<--- тут выходит из функции

Содержимое полей битого объекта 'бомба':

Code
1
2
3
4
5
6
7
-       this    
+       Item    {...}   
+       map_cord    (-572662307, -572662307)    
        radius  -572662307  
        timeout -572662307  
        IBroken true (221)  
+       creater 0xdddddddd

@2romanio005 · 31.07.2022, 22:01 **[ТС]**

Ygg, Спасибо! Скорее всего из ActiveBomb удаляется не та бомба (хотя странно), завтра буду тестить!

Алексей1153 · 31.07.2022, 22:44

Сообщение от 2romanio005

1) это не решит/плохо решит подобные проблемы в будущем

почему ? От сырых указателей поиметь проблем намного легче.
Но, впрочем, ошибка может быть и не из-за лапши с указателями

Сообщение от 2romanio005

Почему я должен так делать? auto - сахар, которым я не очень хочу пользоваться.

не должен, конечно же. Придумывай каждый раз и пиши вручную тип-простыню по 5 минут

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

C++
1
2
3
4
5
6
7
8
9
    //ok
    void(*f1)()=[]()
    {
    };
    
    //error
    void(*f2)()=[&]()
    {
    };

@CoderPC · 01.08.2022, 07:34

2romanio005, если закомментить delete[] ... и зависание исчезнет, видимо выходите за границы массива при его использовании

@2romanio005 · 01.08.2022, 14:02 **[ТС]**

Ygg, Всё оказалось довольно просто, как обычно. В сете ActiveBomb хранились все активные бомбы, чтобы отслеживать какая взорвётся первой. И я не очищал этот контейнер когда начиналась новая игра, в нём оставались указатели на удалённые Bomb с предыдущей игры. После этого всё зависало. Если же я убирал delete в Cell::set() то ActiveBomb хранил в себе указатели на не удалённые бомбы, что не вызывало зависаний, но вызывало неправильную работу кода(различные баги) ну и появлялись утечки.
Всем большое спасибо, и хорошего настроения!

Алексей1153 · 01.08.2022, 14:12

2romanio005,

Сообщение от 2romanio005

1) это не решит/плохо решит подобные проблемы в будущем

я угадал, значит

Новые блоги и статьи Все статьи Все блоги /
Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .	[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .
делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .

@CoderPC Заблокирован
	31.07.2022, 18:09
	2romanio005, в архиве сюда? 1

@2romanio005 1 / 1 / 0 Регистрация: 31.07.2022 Сообщений: 17
	31.07.2022, 20:11 [ТС]
	VTsaregorodtsev, спасибо! Согласен с вами. А можно как-нибудь обойтись вообще без этой переменной(она используется только для понимания какой именно класс лежит в указателе)? 0

@2romanio005 1 / 1 / 0 Регистрация: 31.07.2022 Сообщений: 17
	31.07.2022, 22:01 [ТС]
	Ygg, Спасибо! Скорее всего из ActiveBomb удаляется не та бомба (хотя странно), завтра буду тестить! 0

@CoderPC Заблокирован
	01.08.2022, 07:34
	2romanio005, если закомментить `delete[] ...` и зависание исчезнет, видимо выходите за границы массива при его использовании 0

@2romanio005 1 / 1 / 0 Регистрация: 31.07.2022 Сообщений: 17
	01.08.2022, 14:02 [ТС]
	Ygg, Всё оказалось довольно просто, как обычно. В сете ActiveBomb хранились все активные бомбы, чтобы отслеживать какая взорвётся первой. И я не очищал этот контейнер когда начиналась новая игра, в нём оставались указатели на удалённые Bomb с предыдущей игры. После этого всё зависало. Если же я убирал delete в Cell::set() то ActiveBomb хранил в себе указатели на не удалённые бомбы, что не вызывало зависаний, но вызывало неправильную работу кода(различные баги) ну и появлялись утечки. Всем большое спасибо, и хорошего настроения! 1

Игра зависает через неопределённое время жрёт процессор и выключается

Решение