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

@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 30.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .	Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .
Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .	Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .

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

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

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

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

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

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

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

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

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