How to make my program consume less CPU without hindering its execution?

Question

How to make my program consume less CPU without hindering its execution?

Navigation

#1 by (2 votes)
#2 by (1 votes)
#3 by (-1 votes)

4

I have a program that reads the memory of a computer process continuously (with while (true) ), but this ends up requiring a lot of CPU, arriving at 20% used, my question is, how to decrease CPU usage without lose performance in the program?

I tried Thread.Sleep(10) , it worked, CPU usage dropped to 1%, but even though it was only 10 thousandths, program performance dropped dramatically.

while (Vars.GlowActive) //Essa variavel sempre vai ser true
    if (Vars.GlowAlways || Vars.GetKeyState(Vars.GlowButton) == 0)
    {
        for (int num = 1; num < 24; num++)
        {
            int entity = mem.Read<int>(Vars.bClient + Vars.EntList + (num * 0x10));
            ehealth = mem.Read<int>(entity + Vars.Health);

            int Glow = mem.Read<int>(entity + Vars.GlowIndex);
            int EntTeam = mem.Read<int>(entity + Vars.Team);
            if (EntTeam == Vars.MyTeam)
            {
                if (Vars.glowteamenabled)
                {
                    if (Vars.TeamRainbow)
                        TeamColor();
                    color[0] = Vars.glow_team_r / 255;
                    color[1] = Vars.glow_team_g / 255;
                    color[2] = Vars.glow_team_b / 255;
                    DrawGlow(Glow, color);
                }
            }
            else
            {
                if (Vars.glowenemyenabled)
                {
                    if (Vars.EnemyRainbow)
                        EnemyColor();
                    else if (Vars.glowhealth)
                    {
                        Vars.glow_enemy_r = 255 - 2.55f * ehealth;
                        Vars.glow_enemy_g = 2.55f * ehealth;
                        Vars.glow_enemy_b = 0;
                    }
                    color[0] = Vars.glow_enemy_r / 255;
                    color[1] = Vars.glow_enemy_g / 255;
                    color[2] = Vars.glow_enemy_b / 255;
                    DrawGlow(Glow, color);
                }
            }
        }
    }
    else if (one && Vars.BeepEnable)
    {
        Console.Beep(3000, 200);
        one = !one;
    }
}

Summary:

mem.Read reads the memory of a process.

DrawGlow() writes in memory.

desempenho c# thread

asked by anonymous 13.07.2017 / 23:56

3 answers

1

Not everything or anything

A program that runs non-stop consumes more CPU. A program that sleeps always loses performance. How about doing both, but not always?

        while ( Vars.GlowActive )
        {
            if ( Vars.GlowAlways || Vars.GetKeyState( Vars.GlowButton ) == 0 )
            {
                for ( int num = 1 ; num < 24 ; num++ ) // Ver observação
                {
                }

                // Já que teve uma execuçao válida, não cairia no elseif abaixo
                // então explicitamente tenta executar de novo
                continue;
            }
            else if ( one && Vars.BeepEnable )
            {
                Console.Beep( 3000 , 200 );
                one = !one;
            }
            // Se chegou aqui, não tinha dados a processar.
            // Dar uma folga a espera de novos dados.
            System.Threading.Thread.Yield();
        }

Note

That number 24 seems kind of magical. See if it is requirement of the API you are using. But if it's just an attempt to balance load between rounds and beeps, it would be best to take that for , and move the beep test into the while.

15.07.2017 / 17:35

-1

Decrease the amount of If's in your program, so I've seen you can use an and in some if's, this should improve a bit.

15.07.2017 / 15:09

What is the correct way to use RunOnUiThread () in Xamarin.Android? What is the main difference between the Knuth-Morris-Pratt and Boyer-Moore algorithms

score 2 · Accepted Answer

Try to decrease the duration of sleep

Thread.Sleep(1);

Or use a Timer

var timer =  new  System.Timers.Timer();
timer.Interval = 10;
timer.Elapsed += (ctx, arg) => /*chama a sua funcao aqui*/;
timer.AutoReset = true;
timer.Start();