GUI Error

[IcantCodeHelp]

Can someone explain the Autoit error to me I'm not sure what I did wrong 

#Include <MsgBoxConstants.au3>
Local $PanMan = InputBox("Title Im on top", "Prompt Im at bottom", "Im the PassCode Part", "","M2" _
             - 1, -1, 0, 0)
Sleep("1000", $PanMan)
;MsgBox($MB_SYSTEMMODAL, "", $PanMan)
;MsgBox($MB_OK, "", $PanMan)

Here's a gif I've created (To give you a better perspective)

I put 1000 so that the sleep time could be 1000
But I don't know what the error is telling me, hopefully you can make it sound more simple

 

[Subz]

Sleep only has one parameter it should just be Sleep(1000), which is 1 second although unsure why you need the sleep.

[IcantCodeHelp]

9 minutes ago, Subz said:

Sleep only has one parameter it should just be Sleep(1000), which is 1 second although unsure why you need the sleep.

I need sleep, to change the value of it by using the input GUI

And thank you

Edited January 1, 2021 by IcantCodeHelp

[Subz]

Sorry that doesn't make sense, inputbox will pause until the user has clicked the OK button, so there is no need for the sleep, your parameters are also incorrect it should be something like:

Local $sValue = InputBox("Title Im on top", "Prompt Im at bottom", "", " M2")
MsgBox(4096, "", $sValue)

 

[IcantCodeHelp]

Thank you,

And if you were wondering-

I was trying to make what ever was typed in for $sValue the the amount of Milliseconds for sleep
for example something like this

 

[JLogan3o13]

Where did you get that Sleep accepted more than one param? Certainly not by looking at it in the help file...

[Dan_555]

You can simply use 

Sleep ($PanMan)

or better:

#Include <MsgBoxConstants.au3>
Local $PanMan = InputBox("Title Im on top", "Prompt Im at bottom", "5000", "","M2" _
             - 1, -1, 0, 0)

Sleep((IsNumber($PanMan)=1) ? $PanMan : 3000)           ;Check if the $PanMan is a number, then sleep for that amount, or if not, sleep for 3 seconds.
MsgBox($MB_SYSTEMMODAL, "", $PanMan)

 

[TheXman]

32 minutes ago, Dan_555 said:

You can simply use 

Sleep ($PanMan)

or better:

#Include <MsgBoxConstants.au3>
Local $PanMan = InputBox("Title Im on top", "Prompt Im at bottom", "5000", "","M2" _
             - 1, -1, 0, 0)

Sleep((IsNumber($PanMan)=1) ? $PanMan : 3000)           ;Check if the $PanMan is a number, then sleep for that amount, or if not, sleep for 3 seconds.
MsgBox($MB_SYSTEMMODAL, "", $PanMan)

 

@Dan_555

The data type of the return value of InputBox() is a string, regardless of the value.  Therefore, in your second example, IsNumber($PanMan) will always return FALSE.  IsNumber() checks the variable's data type not its value.  It does not check if the value is numeric.

 

Edited January 1, 2021 by TheXman

[IcantCodeHelp]

Thank you all for your help I'm understanding now

[Dan_555]

Thanks for mentioning it, ... @TheXman

When i tested the code, it worked. Anyway this should work now:

 

Local $PanMan = int(InputBox("Title Im on top", "Prompt Im at bottom", "5000", "","M2" _
             - 1, -1, 0, 0))

Sleep((IsNumber($PanMan)=1) ? $PanMan : 3000)           ;Check if the $PanMan is a number, then sleep for that amount, or if not, sleep for 3 seconds. MsgBox($MB_SYSTEMMODAL, "", $PanMan)

should work

[Nine]

Not quite.  By putting the int() in front of the InputBox, you are loosing the possibility to test @error.   And then not able to check if user has cancelled...

Use instead StringIsInt().