If inside if Topic is solved

[klb1111]

Hello,

can't figure out why the second if is working (message box appears) while it shouldn't start.
It shouldn't start because second if should return false.
Could you help please?

Code: Select all

T1:
FormatTime, TimeString, T10, Time
If (TimeString >= 1400)
{
if (2>=1)
{
MsgBox The current time and date (time first) is %TimeString%.
}
}
return

[nnnik]

The if inside returns true because you checked if 2 is larger or the same as 1.

[Scr1pter]

Also, it's useful to add some tabs inside of your code.
Especially for beginners (I don't know your programming knowledge), it helps you keep the overview.
For example (just to focus on the structure) :

Code: Select all

if ...
{
  while ..
  {
    if
    {
  
    }
  }
}
return

Regards

[klb1111]

The if inside returns true because you checked if 2 is larger or the same as 1.

I've messed it with "1>=2" condition, thank you.
Actually my real check is:

Code: Select all

if (WinExist ("ahk_class #32770"))

but it returns True even if such window does not exist.
Is there a way to get list of all existing windows with their ahk_class value?

[klb1111]

Also, it's useful to add some tabs inside of your code.
Especially for beginners (I don't know your programming knowledge), it helps you keep the overview.
...

That looks really convenient, thank you.
My programming knowledge is very very basic

[swagfag]

Code: Select all

if (WinExist ("ahk_class #32770"))

Is a blank var concatenated with a string literal, and not a function call as you might have intended. Delete the space.

[klb1111]

Code: Select all

if (WinExist ("ahk_class #32770"))

Is a blank var concatenated with a string literal, and not a function call as you might have intended. Delete the space.

It still returns True while it shouldn't do so. Just can't figure out why.

[klb1111]

UPD.

In this code

Code: Select all

T1:

FormatTime, TimeString, T10, Time
If (TimeString >= 1400)

{
   if WinExist ("ahk_class #32770")
   {
   MsgBox % "The active window's ID is " . WinExist("A")
   }

}

return

Second IF statement which is: if WinExist ("ahk_class #32770")
runs "MsgBox % "The active window's ID is " . WinExist("A")" string
even if there is no window with "#32770" class.

I've used NirSoft winlister to check:


As you can see there's no window with such class but the result of the second IF statement is TRUE.

[swagfag]

Code: Select all

if WinExist ("ahk_class #32770")

this isnt a function call. delete the space:

Code: Select all

if WinExist("ahk_class #32770")

[klb1111]

Code: Select all

if WinExist ("ahk_class #32770")

this isnt a function call. delete the space:

Code: Select all

if WinExist("ahk_class #32770")

Got it. Many thanks!