GUI CPU USAGE

[florynyus]

ok everyone i have a litlle problem at my gui , is using like 10, 12% processor and idk why..i mean i use a lot of  Sleep and idk were is the problem

mygui2.au3

[Zedna]

Problem is this line (probably):

  _GUICtrlIpAddress_Set($hIPAddress, @IPAddress1)

You update GUI in each loop no matter if value changes. Use some global variable and compare @IPAddress1 to that value, if it differs then set @IPAddress1 to that variable and update GUI by _GUICtrlIpAddress_Set()

[florynyus]

Problem is this line (probably):

  _GUICtrlIpAddress_Set($hIPAddress, @IPAddress1)

You update GUI in each loop no matter if value changes. Use some global variable and compare @IPAddress1 to that value, if it differs then set @IPAddress1 to that variable and update GUI by _GUICtrlIpAddress_Set()

ok i`ll try

[Zedna]

Also change set of "If" tests

                   IF GUICtrlRead($combo) = "Chiavetta Internet" And _IsPressed("01", "user32.dll") then
                 IF GUICtrlRead($combo) = "Check Yahoo Statuss" And _IsPressed("01", "user32.dll") then

...

To Switch command
 

[florynyus]

Also change set of "If" tests

                   IF GUICtrlRead($combo) = "Chiavetta Internet" And _IsPressed("01", "user32.dll") then

                 IF GUICtrlRead($combo) = "Check Yahoo Statuss" And _IsPressed("01", "user32.dll") then

...

To Switch command

 

ok thx

[florynyus]

so IF is using so much processor? i`am change right now with switch  and i`ll tell you guys the results

[florynyus]

ok guys i changed all IF  with Switch and processor usage has decreased at max 5% and then at the end of  while whas Sleep(10)
        WEnd and i set it to Sleep(150)  Wend and nou i have 0 and max 1%  THX for help guys!!!!

[BrewManNH]

BTW, you can remove the GUIGetMsg line from the script, you're not using it anyways so it's just taking up processing time unnecessarily.

[florynyus]

BTW, you can remove the GUIGetMsg line from the script, you're not using it anyways so it's just taking up processing time unnecessarily.

Ok i will thx

[Zedna]

Problem is this line (probably):

  _GUICtrlIpAddress_Set($hIPAddress, @IPAddress1)

You update GUI in each loop no matter if value changes. Use some global variable and compare @IPAddress1 to that value, if it differs then set @IPAddress1 to that variable and update GUI by _GUICtrlIpAddress_Set()

 

Like this:

...
Global $ip_prev

;WHILE =====================================================================================================================================================
while 1
 
  $ip = @IPAddress1
  If $ip <> $ip_prev Then
      ;SET @IPAddress1 TO _GUICtrlIpAddress_Create =============================================================================================================
      _GUICtrlIpAddress_Set($hIPAddress, $ip)
      ;END OF SET ============================================================================================================================================
      $ip_prev = $ip
  EndIf

  ...
 WEnd

And important question:

Does your @IPAddress1 change during running of your script? If not then call _GUICtrlIpAddress_Set($hIPAddress, @IPAddress1) only once before your main loop.

Also remove line with GUIGetMsg() because it's not needed in OnEvent mode.

Edited January 17, 2014 by Zedna