утечка памяти

@Eclipze · Регистрация: 28.01.2013

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

Доброго времени суток, магистры и гуру программирования. Помогите решить вопрос. Задавал другие по этой теме, но разобравшись понял, что дело именно в утечке памяти.
Почти неделю писал игру Морской бой с компьютером. и вот как я это делал:
Сами поля это двумерный массив Panel( 1 Panel - одна ячейка поля) и того 200 Panel, которые создаются динамически по циклу. Так же есть 2 массива кораблей 10 и 10, представляют собой PictureBox, и служат для графического отображения подбитых кораблей. Ну и по мелочи: label - ы, которые отображают ход игры и т.д.
Для заполнения массивов и и многих других расчетов (автоматическая расстановка кораблей, выделение зоны вокруг корабля и т.д) используются функции в которых довольно много используется циклов и вложеных циклов. Так же при игре компьютера один циклы может выполняться очень много раз. Так как все завязано на контролах( для графического отображения происходящего), то и обращение к этим контролам идет чуть ли ни из каждой функции.
Несколько функций, например для заполнения полей Panel - ми, работают в отдельном потоке через backgroundWorker.
Примерно так. Получилось очень даже красиво, но вот с каждой игрой и даже с каждым ходом, кол-во оперативы вырастает в чуть ли не в геометрической прогрессии, пока не вылетет ошибка "недостаточно памяти".

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

@Aloir · 28.02.2013, 00:37

вообще странно, если учесть что в C# автоматический собирает мусор, попробуйте принудительно после отрисовки каждого хода "собирать мусор"

а как ты отрисовываешь ходы?

@adm_loro · 28.02.2013, 00:43

Dispose() тебя спасет. Вообще тебе нужно в QA идти на .net утечку замутить своими руками, да и еще OutOfMemoryException получить. - Силён.

Добавлено через 4 минуты
Aloir, дружище не стоит давать таких советов. Он грамаздит контролы и оставляет на них ссылки везде, сборщику просто нечего собирать(и мне его даже жаль). Простой Dispose руками на ненужном объекте и уже все в шоколаде. просто ради смеха скинь маленький, но самый суровый кусок кода.

@Winhttp22 · 28.02.2013, 00:46

Eclipze, сложно гадать, надо видеть код.

@adm_loro · 28.02.2013, 01:09

Winhttp22,
чего тут гадать? почему gc не может собрать мусор? потому что его нет.
Почему нет мусора?
Потому что есть коллекция Controls из нее ни чего не удаляется или удаляется, но не так как нужно. Все contol финализируемые, т.е. если даже забить на IDisposable, если на объект нет доступной ссылки и памяти уже совсем нет, то gc пройдет по всем поколениям и очистит память, дернув финалайз, но этого не происходит, т.к. ссылки доступны.
Могу утверждать на 100% это очень херовый код и рефакторинг тут не поможет.

@Eclipze · 28.02.2013, 10:40 **[ТС]**

Вот например функция случайной расстановки кораблей.

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
void Ships(int[,] Pole)
        {
            int x, y, kx, ky;
            bool B;
 
            Init(Pole);
            Random rnd = new Random();
            for (int N = 3; N >= 0; N--)
            {
 
                for (int M = 0; M <= 3 - N; M++)
                {
                    do
                    {
                        x = rnd.Next(11);
                        y = rnd.Next(11);
                        kx = rnd.Next(2);
                        if (kx == 0) ky = 1;
                        else ky = 0;
                        B = true;
                        for (int i = 0; i <= N; i++)
                        {
                            if (x + kx * i > 9)
                            {
                                int q = 0;
                            }
                            if (y + ky * i > 9)
                            {
                                int q = 0;
                            }
                            if (!Freedom(x + kx * i, y + ky * i, Pole)) B = false;
                        }
                        if (B)
                            for (int i = 0; i <= N; i++)
                            {
                                Pole[x + kx * i, y + ky * i] = 1;
                                pan[x + kx * i, y + ky * i].BackColor = Color.Gray;
                                pan[x + kx * i, y + ky * i].Tag = (N + 1).ToString() + M.ToString();
                            }
                    }
                    while (!B);
                }
            }
        }

Добавлено через 3 минуты
Я совсем недавно начал писать на C# и многого еще не знаю, так что, слова о том, что мой код очень плохой это так и есть, но на ошибках и учатся, вот я и хочу найти все недочеты.

Добавлено через 2 минуты
Вот наверно, одна из самых суровых функций)

Кликните здесь для просмотра всего текста

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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
public void SelectionArea(int Ship,bool True)
        {
            string Owner = Cell.Parent.Name;
            if (Owner == "playingField2")
            {
                if (True == true)
                {
                    Sound(2);
                    for (int i = 0; i < 10; i++)
                    {
                        for (int j = 0; j < 10; j++)
                        {
                            if (pan2[i, j].Tag.ToString() == Ship.ToString())
                            {
                                if (i + 1 < 10 && pan2[i + 1, j].Tag == "0") pan2[i + 1, j].Tag = "-2";
                                if (i - 1 > -1 && pan2[i - 1, j].Tag == "0") pan2[i - 1, j].Tag = "-2";
                                if (j + 1 < 10 && pan2[i, j + 1].Tag == "0") pan2[i, j + 1].Tag = "-2";
                                if (j - 1 > -1 && pan2[i, j - 1].Tag == "0") pan2[i, j - 1].Tag = "-2";
                                if (i + 1 < 10 && j + 1 < 10) pan2[i + 1, j + 1].Tag = "-2";
                                if (i - 1 > -1 && j - 1 > -1) pan2[i - 1, j - 1].Tag = "-2";
                                if (i - 1 > -1 && j + 1 < 10) pan2[i - 1, j + 1].Tag = "-2";
                                if (i + 1 < 10 && j - 1 > -1) pan2[i + 1, j - 1].Tag = "-2";
                            }
                        }
                    }
                }
                else
                {
                    for (int i = 0; i < 10; i++)
                    {
                        for (int j = 0; j < 10; j++)
                        {
                            if (pan2[i, j].Name == Cell.Name)
                            {
                                if (i + 1 < 10 && j + 1 < 10) pan2[i + 1, j + 1].Tag = "-2";
                                if (i - 1 > -1 && j - 1 > -1) pan2[i - 1, j - 1].Tag = "-2";
                                if (i - 1 > -1 && j + 1 < 10) pan2[i - 1, j + 1].Tag = "-2";
                                if (i + 1 < 10 && j - 1 > -1) pan2[i + 1, j - 1].Tag = "-2";
                            }
                        }
                    }
                }
            }
 
                if (Owner == "playingField1")
                {
                    if (True == true)
                    {
                        Sound(2);
                        for (int i = 0; i < 10; i++)
                        {
                            for (int j = 0; j < 10; j++)
                            {
                                if (pan[i, j].Tag.ToString() == Ship.ToString())
                                {
                                    if (i + 1 < 10 && pan[i + 1, j].Tag == "0") pan[i + 1, j].Tag = "-2";
                                    if (i - 1 > -1 && pan[i - 1, j].Tag == "0") pan[i - 1, j].Tag = "-2";
                                    if (j + 1 < 10 && pan[i, j + 1].Tag == "0") pan[i, j + 1].Tag = "-2";
                                    if (j - 1 > -1 && pan[i, j - 1].Tag == "0") pan[i, j - 1].Tag = "-2";
                                    if (i + 1 < 10 && j + 1 < 10) pan[i + 1, j + 1].Tag = "-2";
                                    if (i - 1 > -1 && j - 1 > -1) pan[i - 1, j - 1].Tag = "-2";
                                    if (i - 1 > -1 && j + 1 < 10) pan[i - 1, j + 1].Tag = "-2";
                                    if (i + 1 < 10 && j - 1 > -1) pan[i + 1, j - 1].Tag = "-2";
                                }
                            }
                        }
                    }
                    else
                    {
                        for (int i = 0; i < 10; i++)
                        {
                            for (int j = 0; j < 10; j++)
                            {
                                if (pan[i, j].Name == Cell.Name)
                                {
                                    if (i + 1 < 10 && j + 1 < 10) pan[i + 1, j + 1].Tag = "-2";
                                    if (i - 1 > -1 && j - 1 > -1) pan[i - 1, j - 1].Tag = "-2";
                                    if (i - 1 > -1 && j + 1 < 10) pan[i - 1, j + 1].Tag = "-2";
                                    if (i + 1 < 10 && j - 1 > -1) pan[i + 1, j - 1].Tag = "-2";
                                }
                            }
                        }
                       
                    }
                }
                if (Owner == "playingField1")
                {
                    for (int i = 0; i < 10; i++)
                    {
                        for (int j = 0; j < 10; j++)
                        {
                            if (pan[i, j].Tag.ToString() == "-2")
                            {
                                pan[i, j].BackColor = Color.FromArgb(56, 52, 49);
                                pan[i, j].Enabled = false;
                            }
                        }
                    }
                   
                }
                if (Owner == "playingField2")
                {
                    for (int i = 0; i < 10; i++)
                    {
                        for (int j = 0; j < 10; j++)
                        {
                            if (pan2[i, j].Tag.ToString() == "-2")
                            {
                                pan2[i, j].BackColor = Color.FromArgb(56, 52, 49);
                                pan2[i, j].Enabled = false;
                            }
                        }
                    }
                   
                }
 
        }

Добавлено через 3 минуты
к сожалению целиком проект скинуть не могу, но могу скинуть exe-шник, может как-то это поможет понять где у меня именно косяки)

Добавлено через 25 минут
Один косяк я все же нашел сам)) У меня почти на каждое действие в игре воспроизводится звук, через AudioVideoPlayback. Т.е есть функция, которой при определенном действии передается int и она выбирает что именно воспроизвести. Буду ковырять ее, возможно основная проблема именно в этом)

@kodv · 28.02.2013, 11:18

Не по теме:

Вот и выросло поколение, которое, благодаря GC, даже не в курсе, что память в программе должна еще и освобождаться.

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

@adm_loro · 28.02.2013, 11:28

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

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

@Eclipze · 28.02.2013, 11:37 **[ТС]**

Я нашел весь косяк, методом тыка.
Чувствую себя полным нубом, но у меня был срок всего неделя (завтра дедлайн).
Вообщем получается следующая ситуация:
Есть такая функция

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void Sound(int State)
{
switch(State)
{
case 1:{
var audio = new Audio("путь до файла в каталоге программы", true);
audio.Play();
}break;
case 2:{
var audio = new Audio("путь до файла в каталоге программы", true);
audio.Play();
}break;
case 3:{
var audio = new Audio("путь до файла в каталоге программы", true);
audio.Play();
}break;
}
}

Ну и получается, что при каждом вызове этой функции создается новый экземпляр Audio, вот и уходит память на все эти экземпляры. При отключении этой функции, все работает на ура и оператива не утекает. Вопрос такой, как построить такую функцию, чтобы решить данную проблему?) Заранее извеняюсь, но надо сегодня доделать и оттестировать. А еще надо доработать игру ИИ, он немного туповат еще)

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

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

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

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

Я бы с радостью скинул целиком проект, и если получится то обязательно скину, так как любая критика это всегда хорошо.Но "коперайт" (хотя и не знаю зачем). Вообщем это мой первый такой проект в программировании, до этого были небольшие базы данных на Дульфях и немного различных медицинских калькуляторов и БД на C#.
А тут нашлась работа "программистом"=)) И вот дали такое тестовое задание) Босс очень любит эту игру)
Может показаться ситуация смешной, но где в наше время можно найти компанию, которая возьмет в штат программиста на самообучение=)

@kodv · 28.02.2013, 12:09

Eclipze, MSDN отказывается выдавать информацию о таком классе для C#, как Audio. Что вы подключали, чтобы его использовать? Может быть целесообразнее использовать для воспроизведения звуков что то более родное для C#, такое, как System.Media.SoundPlayer?

@Eclipze · 28.02.2013, 12:16 **[ТС]**

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

Eclipze, MSDN отказывается выдавать информацию о таком классе для C#, как Audio. Что вы подключали, чтобы его использовать? Может быть целесообразнее использовать для воспроизведения звуков что то более родное для C#, такое, как System.Media.SoundPlayer?

Как это так, это всего навсего AudioVideoPlayback! А насчет System.Media.SoundPlayer он не может воспроизводить 2 звука одновреммено, только если в разных потоках, но и то не уверен.

@adm_loro · 28.02.2013, 12:52

юзать DirectX только для пиканья? это сурово.

Добавлено через 9 минут
Eclipze, http://msdn.microsoft.com/en-u... s.85).aspx

Добавлено через 4 минуты
вынеси в поля класса, в свиче вызывай Play.

C#
1
2
3
private readonly Audio _audio1;
private readonly Audio _audio2;
private readonly Audio _audio3;

или оберни в using(var audio = new Audio(fileName, true)){ audio.Play(); }

@Eclipze · 28.02.2013, 14:47 **[ТС]**

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

using(var audio = new Audio(fileName, true)){ audio.Play(); }

Так не работает)

@adm_loro · 28.02.2013, 19:53

покажи.

@Eclipze · 01.03.2013, 10:46 **[ТС]**

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public void Sound(int state)
        {
            switch (state)
            {
                case 0:
                    {
                        using (var audio = new Audio(Application.StartupPath + "путь", true))
                        { audio.Play(); }
                    } break;
                case 1://Звук ранения
                    {
                        using (var audio = new Audio(Application.StartupPath + "путь", true)))
                        { audio.Play(); }
                    } break;
                case 2://Звук потопления корабля
                    {
                        using (var audio = new Audio(Application.StartupPath + "путь", true)))
                        { audio.Play(); }
                    } break;

Звук есть если делать отладку. Видимо я совсем не так сделал. Похоже, что звук воспроизводится, но тут же происходит Dispose, так как в using обернул.

@Aloir · 01.03.2013, 12:04

попробуй сделать, переменную не локальную, а вынести в класс ее(инициализация в конструкторе), и передать только путь воспроизведения.

@kodv · 01.03.2013, 12:12

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

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

вынеси в поля класса, в свиче вызывай Play. Код C#1
2
3
4
5
private readonly Audio _audio1;
private readonly Audio _audio2;
private readonly Audio _audio3;

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

PS: Интересен смысл сочетания метода Play() и создание объекта со вторым параметром true (автовоспроизведение после создания). С какой целью было сделано именно так?

@bupal · 01.03.2013, 12:25

Попробуй после проигрывания файла делать что то типа

C#
1
2
3
audio.Stop();
audio.Dispose();
audio = null;

@Eclipze · 01.03.2013, 14:27 **[ТС]**

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

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

Если можно, опишите подробнее, как правильно это реализовать.

@kodv · 05.03.2013, 06:18

Eclipze, Я не ставил себе Managed DirectX (или как он называется), поэтому только на ощупь могу ... В смысле, возможно ошибки, но принцип я предполгаю такой:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
public void Sound(int state)
{
// Какой то код до создания объекта Audio
    Audio audio = new Audio(Application.StartupPath + "путь"); // нах true, запустим потом, вручную
    audio.Ending += new EventHandler(audio_Ending);
    audio.Play();
// Какой то код после создания объекта Audio
}
 
private void audio_Ending(object sender, EventArgs e)
{
    ((IDisposable)sender).Dispose();
}

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .

@Eclipze 29 / 29 / 8 Регистрация: 28.01.2013 Сообщений: 211

	утечка памяти 27.02.2013, 23:34. Показов 3948. Ответов 24 Метки нет (Все метки) Доброго времени суток, магистры и гуру программирования. Помогите решить вопрос. Задавал другие по этой теме, но разобравшись понял, что дело именно в утечке памяти. Почти неделю писал игру Морской бой с компьютером. и вот как я это делал: Сами поля это двумерный массив Panel( 1 Panel - одна ячейка поля) и того 200 Panel, которые создаются динамически по циклу. Так же есть 2 массива кораблей 10 и 10, представляют собой PictureBox, и служат для графического отображения подбитых кораблей. Ну и по мелочи: label - ы, которые отображают ход игры и т.д. Для заполнения массивов и и многих других расчетов (автоматическая расстановка кораблей, выделение зоны вокруг корабля и т.д) используются функции в которых довольно много используется циклов и вложеных циклов. Так же при игре компьютера один циклы может выполняться очень много раз. Так как все завязано на контролах( для графического отображения происходящего), то и обращение к этим контролам идет чуть ли ни из каждой функции. Несколько функций, например для заполнения полей Panel - ми, работают в отдельном потоке через backgroundWorker. Примерно так. Получилось очень даже красиво, но вот с каждой игрой и даже с каждым ходом, кол-во оперативы вырастает в чуть ли не в геометрической прогрессии, пока не вылетет ошибка "недостаточно памяти". Подскажите куда копать, где и чем искать утечку памяти? к сожалению исходники выложить не могу, да и в них слишком много кода, чтобы искать в каждой функции. 1

@Aloir 225 / 225 / 38 Регистрация: 17.12.2010 Сообщений: 713
	28.02.2013, 00:37
	вообще странно, если учесть что в C# автоматический собирает мусор, попробуйте принудительно после отрисовки каждого хода "собирать мусор" а как ты отрисовываешь ходы? 1

@adm_loro Заблокирован
	28.02.2013, 00:43
	Dispose() тебя спасет. Вообще тебе нужно в QA идти на .net утечку замутить своими руками, да и еще OutOfMemoryException получить. - Силён. Добавлено через 4 минуты Aloir, дружище не стоит давать таких советов. Он грамаздит контролы и оставляет на них ссылки везде, сборщику просто нечего собирать(и мне его даже жаль). Простой Dispose руками на ненужном объекте и уже все в шоколаде. просто ради смеха скинь маленький, но самый суровый кусок кода. 1

@Winhttp22 169 / 132 / 29 Регистрация: 16.02.2013 Сообщений: 867
	28.02.2013, 00:46
	Eclipze, сложно гадать, надо видеть код. 1

@adm_loro Заблокирован
	28.02.2013, 01:09
	Winhttp22, чего тут гадать? почему gc не может собрать мусор? потому что его нет. Почему нет мусора? Потому что есть коллекция Controls из нее ни чего не удаляется или удаляется, но не так как нужно. Все contol финализируемые, т.е. если даже забить на IDisposable, если на объект нет доступной ссылки и памяти уже совсем нет, то gc пройдет по всем поколениям и очистит память, дернув финалайз, но этого не происходит, т.к. ссылки доступны. Могу утверждать на 100% это очень херовый код и рефакторинг тут не поможет. 0

@kodv 1449 / 1121 / 347 Регистрация: 11.04.2011 Сообщений: 2,621
	28.02.2013, 11:18
	Не по теме: Вот и выросло поколение, которое, благодаря GC, даже не в курсе, что память в программе должна еще и освобождаться. Eclipze, Почитайте вот эту статью. Вполне возможно, подчеркнете для себя много чего полезного. 1

@adm_loro Заблокирован
	28.02.2013, 11:28
	ну если хочешь учится скинь проект в архиве. в коде прокомментирую, что нужно исправить. или у тебя коперайт? а на издевки не обращай внимания, тебе когда-нибудь придется работать в команде, а там если не будешь с юмором к стебу подходить будет - плохо. 0

@kodv 1449 / 1121 / 347 Регистрация: 11.04.2011 Сообщений: 2,621
	28.02.2013, 12:09
	Eclipze, MSDN отказывается выдавать информацию о таком классе для C#, как Audio. Что вы подключали, чтобы его использовать? Может быть целесообразнее использовать для воспроизведения звуков что то более родное для C#, такое, как System.Media.SoundPlayer? 0

@adm_loro Заблокирован
	28.02.2013, 19:53
	покажи. 0

@Aloir 225 / 225 / 38 Регистрация: 17.12.2010 Сообщений: 713
	01.03.2013, 12:04
	попробуй сделать, переменную не локальную, а вынести в класс ее(инициализация в конструкторе), и передать только путь воспроизведения. 0