Выделение большого количества оперативной памяти

@Sankoh_Tew · Регистрация: 16.11.2019

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

Я уже год разрабатываю свою игру на C# и только спустя это время, когда программа уже имеет довольно большой объём, я столкнулся с неприятной проблемой, а именно, при почти каждом действии в игре она "съедает" некоторое количество оперативной памяти. И всё бы ничего, но после завершения действия память обратно не возвращается. Тем самым, постоянно открывая новую форму в игре, прибавляется примерно 300МБ. При достижении отметки в 2ГБ, игра вылетает.
Пробовал команду

C#
1
*компонент*.Dispose();

, но она "съедает" памяти больше, чем освобождает. Также команда

C#
1
GC.Collect();

ничего не делает.

Пожалуйста, помогите найти команду, которая освобождала уже не используемую оперативную память.

@SeIZVeIZ · 16.11.2019, 10:51

Надо не искать команду, а смотреть Ваш код, больше вероятность что это вы не очищаете за собой ресурсы и система не может их потому собрать.

@Sankoh_Tew · 16.11.2019, 11:00 **[ТС]**

Поясните, пожалуйста, какой именно кусок кода надо показать?
Я пользовался командой Dispose(), но она не помогает.

@SeIZVeIZ · 16.11.2019, 11:08

По хорошему нужно иметь весь проект чтобы найти утечки, ибо никогда не знаешь где она может быть, но то что она у Вас есть это 100%, принудительные вызовы сбора мусора не помогут Вам, т.к. у Вас, по тому что вы описали, остаются ссылки на объекты. Судя из описания они остаются после открытия формы некой, т.е. как минимум уже можно смотреть данную форму с чем она работает, как она работает и прочее.

@Sankoh_Tew · 16.11.2019, 11:31 **[ТС]**

Код открытия формы:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public void инвентарьToolStripMenuItem1_Click(object sender, EventArgs e)
        {
            
            
            if (pbrad.Value > 0)
            {
                dlg = new frinv();
                dlg.Owner = this;
                dlgb = true;
                dlg.ShowDialog();
            }
            else
            {
                MessageBox.Show("текст.", this.Text);
                инвентарьToolStripMenuItem1.Enabled = false;
            }
        }

Код куска инвентаря, занимающего самое большое количество ОП:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
public void CONT()
        {
            frg = this.Owner as frgame;
            frs = new frshop();
            Random k = new Random();
            int kont = k.Next(1, 471);
 
            if (kont <= 50)
            {
                if (99 - frg.kol1 >= 10)
                {
                    frg.kol1 = frg.kol1 + 10;
                    lbkol1.Text = frg.kol1.ToString();
                }
                else
                {
                    frg.mon = frg.mon + (10 - (99 - frg.kol1)) * 25;
                    frg.lbmon.Text = frg.mon.ToString();
                    MessageBox.Show("Компенсация за Хлеб", "Новый предмет!");
                }
                if (frg.kol1 <= 0)
                {
                    pb1.Visible = false;
                    pb1.Enabled = false;
                    lbname1.Visible = false;
                    lbname1.Enabled = false;
                    label1.Visible = false;
                    label1.Enabled = false;
                    lbkol1.Enabled = false;
                    lbkol1.Visible = false;
                }
                else
                {
                    if (frg.kol1 > 0)
                    {
                        pb1.Visible = true;
                        pb1.Enabled = true;
                        lbname1.Visible = true;
                        lbname1.Enabled = true;
                        label1.Visible = true;
                        label1.Enabled = true;
                        lbkol1.Enabled = true;
                        lbkol1.Visible = true;
                    }
                }
                MessageBox.Show("Хлеб х 10", "Новый предмет!");
            }
//Дальше около 50-ти подобных строк.

@Usaga · 16.11.2019, 11:36

Sankoh_Tew, не нужно гадать. Нужно посмотреть на дамп кучи, чтобы понять, какие объекты там оседают.

Выделение большого количества оперативной памяти

Снимаете дамп. Потом несколько раз выполняете дейтсвие, что занимает память. Потому снова снимаете дамп и смотрите количество каких объектов увеличилось. Потом уже ищите причину.

@Usaga · 16.11.2019, 11:37

Но почти наверняка проблема в этом:

@Sankoh_Tew · 16.11.2019, 12:11 **[ТС]**

Так без этих команд не будут работать основные функции.
А снимки вот:

@IamRain · 16.11.2019, 12:36

А зачем для диалоговой формы знать родителя? Это так важно?
Можно вообще убрать из формы ссылку на диалоговое окно и создавать через локальную переменную.

C#
1
dlg = new frinv();

Заменить на

C#
1
var dlg = new frinv();

И Owner-а не указывать. + Удалить соответствующее поле класса dlg. Отрефакторить.

@Usaga · 16.11.2019, 12:43

Sankoh_Tew, судя по дампу, оперативка забивается формами. И причина в том, что формы друг на друга ссылаются.

Dispose() ничего тут не изменит. Это вообще не про сборку мусора. У вас рукотворная утечка памяти. Какого лешего у вас формы друг на друга ссылаются? Зачем?

@Sankoh_Tew · 16.11.2019, 13:05 **[ТС]**

Они ссылаются для изменения переменных игры(код формы dlg меняет значения переменных формы frg)
А если не указать Ownera, то при открытии этой формы выдаёт ошибку:

@Usaga · 16.11.2019, 13:10

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

Они ссылаются для изменения переменных игры(код формы dlg меняет значения переменных формы frg)

Такой хрени не должно быть. "Переменные игры" должны храниться отдельно от форм. Заведите класс под такие переменные. Или даже группу классов. Создайте экземпляр класса в главной форме (или даже раньше). Передавайте ссылку на этот объект всем формам, которым от него что-то нужно.

@Sankoh_Tew · 16.11.2019, 13:37 **[ТС]**

Можете объяснить как? А то я плохо разбираюсь в классах.

@Usaga · 16.11.2019, 13:41

Sankoh_Tew, что "как"? Я, вроде бы, объяснил идею. Вы не умеете класс создать?

@Sankoh_Tew · 16.11.2019, 13:48 **[ТС]**

Класс создал, но вот как ссылку на переменные в нём давать не знаю.

@Usaga · 16.11.2019, 13:53

Sankoh_Tew, какие ссылки на переменные? Все данные игры нужно держать в таких классах, а не в формах.

@Sankoh_Tew · 16.11.2019, 14:02 **[ТС]**

Я создал класс

C#
1
2
3
4
/*не знаю что тут*/ class GAME
        {
            //переменные, скопированные с главной формы
        }

Затем мне начали высвечиваться ошибки...

@Usaga · 16.11.2019, 14:05

Sankoh_Tew, вы методом тыка работаете?

@Sankoh_Tew · 16.11.2019, 14:10 **[ТС]**

Я не разбираюсь в классах.

@Usaga · 16.11.2019, 14:12

Sankoh_Tew, тогда вам в любой доступный учебник по C#. К примеру, в этот.

Ответ на ваш вопрос в этой теме вы получили. А "я сделал бездумное действие и ничего не заработало" уже за рамки темы выходит.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@SeIZVeIZ 911 / 796 / 329 Регистрация: 08.02.2014 Сообщений: 2,391
	16.11.2019, 10:51
	Надо не искать команду, а смотреть Ваш код, больше вероятность что это вы не очищаете за собой ресурсы и система не может их потому собрать. 0

@Sankoh_Tew 0 / 0 / 0 Регистрация: 16.11.2019 Сообщений: 10
	16.11.2019, 11:00 [ТС]
	Поясните, пожалуйста, какой именно кусок кода надо показать? Я пользовался командой Dispose(), но она не помогает. 0

@SeIZVeIZ 911 / 796 / 329 Регистрация: 08.02.2014 Сообщений: 2,391
	16.11.2019, 11:08
	По хорошему нужно иметь весь проект чтобы найти утечки, ибо никогда не знаешь где она может быть, но то что она у Вас есть это 100%, принудительные вызовы сбора мусора не помогут Вам, т.к. у Вас, по тому что вы описали, остаются ссылки на объекты. Судя из описания они остаются после открытия формы некой, т.е. как минимум уже можно смотреть данную форму с чем она работает, как она работает и прочее. 0

@Usaga 14138 / 9362 / 1350 Регистрация: 21.01.2016 Сообщений: 35,201
	16.11.2019, 11:36
	Sankoh_Tew, не нужно гадать. Нужно посмотреть на дамп кучи, чтобы понять, какие объекты там оседают. Снимаете дамп. Потом несколько раз выполняете дейтсвие, что занимает память. Потому снова снимаете дамп и смотрите количество каких объектов увеличилось. Потом уже ищите причину. 0

@Usaga 14138 / 9362 / 1350 Регистрация: 21.01.2016 Сообщений: 35,201
	16.11.2019, 11:37
	Но почти наверняка проблема в этом: 0

@Sankoh_Tew 0 / 0 / 0 Регистрация: 16.11.2019 Сообщений: 10
	16.11.2019, 12:11 [ТС]
	Так без этих команд не будут работать основные функции. А снимки вот: Миниатюры 0

@Usaga 14138 / 9362 / 1350 Регистрация: 21.01.2016 Сообщений: 35,201
	16.11.2019, 12:43
	Sankoh_Tew, судя по дампу, оперативка забивается формами. И причина в том, что формы друг на друга ссылаются. Dispose() ничего тут не изменит. Это вообще не про сборку мусора. У вас рукотворная утечка памяти. Какого лешего у вас формы друг на друга ссылаются? Зачем? 0

@Sankoh_Tew 0 / 0 / 0 Регистрация: 16.11.2019 Сообщений: 10
	16.11.2019, 13:05 [ТС]
	Они ссылаются для изменения переменных игры(код формы dlg меняет значения переменных формы frg) А если не указать Ownera, то при открытии этой формы выдаёт ошибку: Миниатюры 0

@Sankoh_Tew 0 / 0 / 0 Регистрация: 16.11.2019 Сообщений: 10
	16.11.2019, 13:37 [ТС]
	Можете объяснить как? А то я плохо разбираюсь в классах. 0

@Usaga 14138 / 9362 / 1350 Регистрация: 21.01.2016 Сообщений: 35,201
	16.11.2019, 13:41
	Sankoh_Tew, что "как"? Я, вроде бы, объяснил идею. Вы не умеете класс создать? 0

@Sankoh_Tew 0 / 0 / 0 Регистрация: 16.11.2019 Сообщений: 10
	16.11.2019, 13:48 [ТС]
	Класс создал, но вот как ссылку на переменные в нём давать не знаю. 0

@Usaga 14138 / 9362 / 1350 Регистрация: 21.01.2016 Сообщений: 35,201
	16.11.2019, 13:53
	Sankoh_Tew, какие ссылки на переменные? Все данные игры нужно держать в таких классах, а не в формах. 0

@Usaga 14138 / 9362 / 1350 Регистрация: 21.01.2016 Сообщений: 35,201
	16.11.2019, 14:05
	Sankoh_Tew, вы методом тыка работаете? 0

@Sankoh_Tew 0 / 0 / 0 Регистрация: 16.11.2019 Сообщений: 10
	16.11.2019, 14:10 [ТС]
	Я не разбираюсь в классах. 0

@Usaga 14138 / 9362 / 1350 Регистрация: 21.01.2016 Сообщений: 35,201
	16.11.2019, 14:12
	Sankoh_Tew, тогда вам в любой доступный учебник по C#. К примеру, в этот. Ответ на ваш вопрос в этой теме вы получили. А "я сделал бездумное действие и ничего не заработало" уже за рамки темы выходит. 1