Why does this not work? multiple clock times in statment

[Vegar]

$h = 18 or 20 or 22
$m = 15 or 30 or 45
$S = 00 or 30

While 1
   If @hour=$h and @min=$m and @sec=$s  Then
   sleep (1000)
   _somefunc()
   Else
   Sleep(10)
   EndIf
Wend

Func _somefunc()
    MsgBox(0,"",@HOUR&"h "&@MIN&"m "&@SEC&"s "&@msec&"ms")
EndFunc

hmm

[Jos]

Because this is not doing what you assume it is doing. It will resolve $h as true first when doing "18 or 20 or 22" and then you compare @hour with true.

Run this in Scite and look at the output pane for the result.

$h = 18 or 20 or 22
ConsoleWrite('@@ Debug(' & @ScriptLineNumber & ') : $h = ' & $h & @crlf & '>Error code: ' & @error & @crlf) ;### Debug Console

Edited February 14, 2012 by Jos

[Vegar]

hmm, yeah i see ... "@@ Debug(5) : $h = True"

But what is the best/ easyest solution to this? ..

sorry, i have'nt been scripting for a while

[fett8802]

It took me a long time before I finally figured out that I could group If statements like mathematical statements (to an extent), for example:

If (@HOUR = 18 Or @HOUR = 20 Or @HOUR = 22) And (@MIN = 15 Or @MIN = 30 Or @MIN = 45) And (@SEC = 00 Or @SEC = 30) Then

This If statement will become true in 18 different ways - which should be what you are wanting. It's amazing what specific things you can filter when you group If statements in parenthesis. According to the help file:

The expression can contain the boolean operators of AND, OR, and NOT as well as the logical operators <, <=, >, >=, =, ==, and <> grouped with parentheses as needed.

- Fett

Edited February 14, 2012 by fett8802

[Vegar]

Superb, worked perfect, Thank you!