Set color to label variable based of number

[autosm]

If $PingNum = 125 OR $PingNum < 125 Then
    GuiCtrlSetColor($PingNum, $COLOR_GREEN)
    
Else
    GUICtrlSetColor($PingNum, 0xff0000)
    EndIf

I have tried everything, using $COLOR_RED instead but nothing seems to work when I have $PingNum be over 125. It just doesn't want to come up as the color red when the variable is over 125.

Edited January 7, 2015 by autosm

[SmOke_N]

Wait, what is $PingNum?  You're using it as a controlid.

Edit:

BTW, if you're unsure or can't explain correctly (because your code makes little sense), then you're going to need to provide a working recreation script that demonstrates your issue.  Not a conditional statement that really says nothing.  BTW

If $PingNum <= 125 Then

is the same as your "Or" statement. Edited January 7, 2015 by SmOke_N

[autosm]

I ping an ip adress and the ms gets set to $PingNum.

Local $iPing = Ping("IP ADRESS THAT IS SET HERE", 1850)

If $iPing Then ; If a value greater than 0 was returned then display the following message.

GUICtrlSetData ($PingNum, $iPing)

Else

MsgBox($MB_SYSTEMMODAL, "", "An error occurred. If this conitues the region severs might be down/busy.")

EndIF

 

I have a few of those set my code based of of what radio button is selected. Everything works fine but my major issue is that when controlID PingNum is over 126 it doesnt set the color to red.

[jguinch]

$Label = GUICtrlCreateLabel("", 10, 10, 200, 25)
; [...]
$ping = Ping($host, 2000)
If $ping >= 125 Then
    $color = 0x00ff00
Else
    $color = 0xff0000
EndIf
GUICtrlSetBkColor($label, $color)

[TheSaint]

@autosm - have a close look at the first code you posted.

You have intermixed the ID or handle of your label, with your ping variable.

This is why, it is always best to post a larger portion of code, than just some snippet.

We can see instantly then, that you have made such a glaring mistake.