AutoHotkey Community

[rgblair11]

I'm trying to learn about program flow by creating a simple "guess the number game" (i.e. define a random number, have user guess, display higer or lower).

Here is the code I'm starting with. Problem is, no mater what the guess is, MsgBox Higher displays and then the "guess" dialog starts again. I realize I'm doing something complety wrong, but I can't figure it out. Here is the code:

InputBox, min, Enter Minimum Value, Enter Minimum Value,
Inputbox, max, Enter Maximum Value, Enter Maximum Value
random, num, %min%, %max%
MsgBox, The Number is %num% ; used for testing, I want to know what the number is
GUESS:
InputBox, guess, Guess, Enter Guess Between %min% and %max%
If %guess% < %num%
{
MsgBox, Higher
Gosub, GUESS
}
else if %guess% > %num%
{
MsgBox, Lower
Gosub, Guess
}

[jordis]

rgblair11,

the problem is in these lines:
If %guess% < %num%
else if %guess% > %num%

you should get rid of the %s in guess, like:
If guess < %num%
else if guess > %num%

jordi

[BoBo]

IfEqual, var, value
same
if var = value

taken from the help.

As I know you use the percentage char in cases where literals could be used as well e.g. with the Send command

[Chris]

Also, you should not do "Gosub Guess" in this case. You should either use "Goto Guess" or enclose the contents in a loop as shown below: