Увеличение используемой оперативной памяти

@fannylife · Регистрация: 04.08.2010

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

Доброго времени суток.
На C# была написана простая однокнопочная диалоговая форма.
При использовании программа постоянно увеличивает объём используемой оперативной памяти, медленно но верно. Дело явно где-то в цикле, там больше ничего и нету )) вот только где, мб кто подскажет?

C#

        private void button3_Click(object sender, EventArgs e)
        {
            System.Diagnostics.Process[] ProcessList;
            while (true)
            {
                Thread.Sleep(5000);
                ProcessList = Process.GetProcesses();
                bool l2serverflag = false;
                bool l2npcflag = false;
                foreach (System.Diagnostics.Process pr in ProcessList)
                {
                    if (pr.ProcessName == "l2server")
                    {
                       
                        l2serverflag = true;
                        foreach (System.Diagnostics.Process pr2 in ProcessList)
                        {
                            if (pr2.ProcessName == "l2npc")
                            {
                                l2npcflag = true;
                                
                                
                            }
                        }
                        ProcessList = null;
                        
                    }
 
                }
 
                if ((l2serverflag == true)&&(l2npcflag == false))
                {
                    Process.Start(ABC.filenameL2npc);
                }
 
                if ((l2serverflag == false) && (l2npcflag == false))
                {
                    Process.Start(ABC.filenameL2server);
                    Thread.Sleep(60000);
                    Process.Start(ABC.filenameL2npc);
                }
 
                if ((l2serverflag == false) && (l2npcflag == true))
                {
                    foreach (Process proc in Process.GetProcessesByName("l2npc"))
                    {
                        proc.Kill();
                    }
                    Thread.Sleep(5000);
                    Process.Start(ABC.filenameL2server);
                    Thread.Sleep(60000);
                    Process.Start(ABC.filenameL2npc);
                }
                
            }
        }

@kirill29 · 04.08.2010, 15:40

Я у вас выхода из цикла не увидел - следовательно цикл бесконечный. Вот и расход памяти.

@fannylife · 04.08.2010, 15:58 **[ТС]**

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

Добавлено через 11 минут
Может быть то, что я хочу сделать реализуется как-то по другому, более правильно?

@kirill29 · 04.08.2010, 16:05

Освобождением памяти занимается сборщик мусора. Вообще-то он автоматом может освободить ресурсы, занятые каким-либо процессом. Но можно его вызывать и самому. Почитайте про класс GC.
Но, имхо, думаю, что у вас не оптимально работает код.

@Mikant · 04.08.2010, 16:32

будь проще

C#

        private void L2CheckLoop() {
            System.Diagnostics.Process[] ProcessList;
 
            for (; ; Thread.Sleep(10000)) {
 
                Process[] servers = Process.GetProcessesByName("l2server");
                Process[] npcs = Process.GetProcessesByName("l2npc");
 
                // уничтожить все лишние серваки
                if (servers.Length > 1) {
                    for (int i = 1; i < servers.Length; i++)
                        servers[i].Kill();
                    Thread.Sleep(10000);
                }
 
                // уничтожить все лишние серваки нюпсов
                if (npcs.Length > 1) {
                    for (int i = 1; i < npcs.Length; i++)
                        npcs[i].Kill();
                    Thread.Sleep(5000);
                }
 
                // если собсна и не было сервака
                if (servers.Length == 0) {
                    Process.Start(ABC.filenameL2server);
                    Thread.Sleep(60000);
                }
 
                // если собсна и не было сервака нюпсов
                if (npcs.Length == 0) {
                    Process.Start(ABC.filenameL2npc);
                    Thread.Sleep(5000);
                }
 
            }

Добавлено через 51 секунду

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

Но, имхо, думаю, что у вас не оптимально работает код

совсем не оптимально... вообще говоря, он вообще не работает)

Добавлено через 4 минуты
можно в конец цикла пихнуть

C#

                for (int i = 1; i < servers.Length; i++)
                    servers[i].Dispose();
                for (int i = 1; i < npcs.Length; i++)
                    npcs[i].Dispose();

для красоты

Добавлено через 1 минуту
и НЕ надо это делать в потоке GUI !

@fannylife · 04.08.2010, 16:50 **[ТС]**

Вставил GC.Collect() в конец цикла, и правда перестал расти расход оперативки =) за этот совет отдельное спасибо

Потом увидел вариант Mikant, мне понравилось

Просто раз уж мы вдаёмся в подробности... Мне надо сделать так, чтобы если упал серв, но не упал нпц, то уронить нпц, поднять серв а следом уже поднимать серв. Для того и вводил флаги. Возможно как-то сделать это по другому?

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

и НЕ надо это делать в потоке GUI !

я нупс ) имеется ввиду, что это действие от нажатия кнопки? А как тогда делать? Вызывать по кнопке функцию, которая описана в program.cs?

Добавлено через 8 минут
Ага туплю =)
вот так же будет работать?

C#

if ((servers.Length == 0)&&(npcs.Lenght == 1)) 
{
                    //тут надо убить процесс нпц, как? не разобрался ))
                    Process.Start(ABC.filenameL2server);
                    Thread.Sleep(60000);
                    Process.Start(ABC.filenameL2npc);
                    
}

@Mikant · 04.08.2010, 17:13

ну подправь

C#

        private void L2CheckLoop() {
            System.Diagnostics.Process[] ProcessList;
 
            for (; ; Thread.Sleep(10000)) {
 
                Process[] servers = Process.GetProcessesByName("l2server");
                Process[] npcs = Process.GetProcessesByName("l2npc");
 
                // уничтожить все лишние серваки
                if (servers.Length > 1) {
                    for (int i = 1; i < servers.Length; i++)
                        servers[i].Kill();
                    Thread.Sleep(10000);
                }
 
                // уничтожить все лишние серваки нюпсов
                if (npcs.Length > 1) {
                    for (int i = 1; i < npcs.Length; i++)
                        npcs[i].Kill();
                    Thread.Sleep(5000);
                }
 
                bool npcKilled = false;
                // если собсна и не было сервака
                if (servers.Length == 0) {
                    if (npcKilled = npcs.Length > 0)
                        for (int i = 1; i < npcs.Length; i++)
                            npcs[i].Kill();
                    Process.Start(ABC.filenameL2server);
                    Thread.Sleep(60000);
                }
 
                // если собсна и не было сервака нюпсов
                if (npcs.Length == 0 || npcKilled) {
                    Process.Start(ABC.filenameL2npc);
                    Thread.Sleep(5000);
                }
 
            }
        }

никогда не используй GC.Collect вручную!!!!!!!!!!!!!!!!!

@fannylife · 04.08.2010, 17:23 **[ТС]**

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

никогда не используй GC.Collect вручную!!!!!!!!!!!!!!!!!

А почему? чем грозит? Как правильно использовать?
p.s. да, я доставучий

@Mikant · 04.08.2010, 17:30

английский знаешь?
http://blogs.msdn.com/b/ricom/... 71829.aspx

@fannylife · 04.08.2010, 18:29 **[ТС]**

знаю но туго =)
можно в двух словах, почему нельзя и как нужно?

@Mikant · 04.08.2010, 18:35

а ты просто мой код пытался выполнить? ну что, увеличивается "оперативная память"?

@fannylife · 05.08.2010, 08:50 **[ТС]**

Ага. всё равно растёт потребление, примерно по 500кб за проход цикла. Это очень много.
и да, этот код я в конец цикла вставил, не забыл

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

C#

for (int i = 1; i < servers.Length; i++)
 servers[i].Dispose();
 for (int i = 1; i < npcs.Length; i++)
 npcs[i].Dispose();

Что значила фраза, что не надо этого делать в гуи? Я попробовал вынести эту функцию отдельно в файле Form1.cs и потом вызвать её по нажатию кнопки, эффект был тот же - окно программы виснет, хоть она и продолжает выполняться. Мб надо было вставить функцию в program.cs и сделать её public?

@kuroiryuu · 05.08.2010, 10:18

вот еще одно решение
(вроде память не прибавляется)

C#

while (true)
{
    Thread.Sleep(5000);
    System.Diagnostics.Process[] processList = Process.GetProcesses();
    bool l2Serverflag = false;
    bool l2Npcflag = false;
    foreach (System.Diagnostics.Process pr in processList)
    {
        if (pr.ProcessName == "l2server")
        {
            l2Serverflag = true;
        }
 
        if (pr.ProcessName == "l2npc")
        {
            l2Npcflag = true;
        }
    }
 
    if (l2Serverflag && !l2Npcflag)
    {
        Process.Start(ABC.FilenameL2Npc);
    }
 
    if (!l2Serverflag && !l2Npcflag)
    {
        Process.Start(ABC.FilenameL2Server);
        Thread.Sleep(60000);
        Process.Start(ABC.FilenameL2Npc);
    }
 
    if (!l2Serverflag && l2Npcflag)
    {
        foreach (Process proc in Process.GetProcessesByName("l2npc"))
        {
            proc.Kill();
        }
        Thread.Sleep(5000);
        Process.Start(ABC.FilenameL2Server);
        Thread.Sleep(60000);
        Process.Start(ABC.FilenameL2Npc);
    }
}

@Mikant · 05.08.2010, 17:23

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

500кб за проход цикла

- false!

какая ж разница, в каком она файле живет?? просто в отдельный поток всё это уноси

@fannylife · 07.08.2010, 19:47 **[ТС]**

Всё получилось, спасибо

@Mikant 1318 / 991 / 127 Регистрация: 08.12.2009 Сообщений: 1,299
	05.08.2010, 17:23	14
	Сообщение от fannylife 500кб за проход цикла - false! какая ж разница, в каком она файле живет?? просто в отдельный поток всё это уноси 1

@kirill29 2095 / 1260 / 170 Регистрация: 01.02.2009 Сообщений: 2,842
	04.08.2010, 15:40	2
	Я у вас выхода из цикла не увидел - следовательно цикл бесконечный. Вот и расход памяти. 1

@fannylife 4 / 4 / 0 Регистрация: 04.08.2010 Сообщений: 20
	04.08.2010, 15:58 [ТС]	3
	в этом и задумка, программа должна постоянно висеть в памяти и контролировать наличие запущенных других двух процессов. Почему при каждом проходе цикла количество используемой оперативной памяти увеличивается? Возможно ли как то освобождать в конце цикла использованную ненужную больше память? Добавлено через 11 минут Может быть то, что я хочу сделать реализуется как-то по другому, более правильно? 0

@kirill29 2095 / 1260 / 170 Регистрация: 01.02.2009 Сообщений: 2,842
	04.08.2010, 16:05	4
	Освобождением памяти занимается сборщик мусора. Вообще-то он автоматом может освободить ресурсы, занятые каким-либо процессом. Но можно его вызывать и самому. Почитайте про класс GC. Но, имхо, думаю, что у вас не оптимально работает код. 1

@fannylife 4 / 4 / 0 Регистрация: 04.08.2010 Сообщений: 20
	04.08.2010, 17:23 [ТС]	8
	Сообщение от Mikant никогда не используй GC.Collect вручную!!!!!!!!!!!!!!!!! А почему? чем грозит? Как правильно использовать? p.s. да, я доставучий 0

@Mikant 1318 / 991 / 127 Регистрация: 08.12.2009 Сообщений: 1,299
	04.08.2010, 17:30	9
	английский знаешь? http://blogs.msdn.com/b/ricom/... 71829.aspx 1

@fannylife 4 / 4 / 0 Регистрация: 04.08.2010 Сообщений: 20
	04.08.2010, 18:29 [ТС]	10
	знаю но туго =) можно в двух словах, почему нельзя и как нужно? 0

@Mikant 1318 / 991 / 127 Регистрация: 08.12.2009 Сообщений: 1,299
	04.08.2010, 18:35	11
	а ты просто мой код пытался выполнить? ну что, увеличивается "оперативная память"? 1

@fannylife 4 / 4 / 0 Регистрация: 04.08.2010 Сообщений: 20
	07.08.2010, 19:47 [ТС]	15
	Всё получилось, спасибо 0