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

@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#. К примеру, в этот.

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

Новые блоги и статьи Все статьи Все блоги /
Валидация и контроль данных табличной части документа перед записью Maks 22.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в КА2. Задача: контроль и валидация данных табличной части документа перед записью с учетом регламента компании. . .	Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .
Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .	Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .

@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 14321 / 9411 / 1356 Регистрация: 21.01.2016 Сообщений: 35,479
	16.11.2019, 11:36
	Sankoh_Tew, не нужно гадать. Нужно посмотреть на дамп кучи, чтобы понять, какие объекты там оседают. Снимаете дамп. Потом несколько раз выполняете дейтсвие, что занимает память. Потому снова снимаете дамп и смотрите количество каких объектов увеличилось. Потом уже ищите причину. 0

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

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

@Usaga 14321 / 9411 / 1356 Регистрация: 21.01.2016 Сообщений: 35,479
	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 14321 / 9411 / 1356 Регистрация: 21.01.2016 Сообщений: 35,479
	16.11.2019, 13:41
	Sankoh_Tew, что "как"? Я, вроде бы, объяснил идею. Вы не умеете класс создать? 0

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

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

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

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

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