Trying to add function causing "incorrect number of parameters"

[TalesFromTheScript]

Sorry if this is something really simple, but I am almost a complete beginner with AutoIt3.

 

This works and saves the window position to an ini file with $nIniXPos and $nIniYPos

GUICreate("My GUI", 600, 400, -1, -1, Default, $nIniXPos, $nIniYPos)

 

This works and makes the GUI always on top by using $WS_EX_TOPMOST

GUICreate("My GUI", 600, 400, -1, -1, Default, $WS_EX_TOPMOST)

 

On other GUI's that I am not adding the always on top function to, they work when
adding the function to save window position, for example this works:

GuiCreate("My GUI", 600, 400, $nIniXPos, $nIniYPos)

 

Taking that function away also works:

GuiCreate("My GUI", 600, 400)

 

But if I try to use both of those functions together (always on top with save window position)
then I get "Incorrect number of parameters in function call":

GUICreate("My GUI", 600, 400, -1, -1, Default, $WS_EX_TOPMOST, $nIniXPos, $nIniYPos)

I have tried removing "-1" and the GUI doesn't display, same if "Default" is removed.
I tried swapping the functions around so $WS_EX_TOPMOST is last, to no avail.

I am stumped now, hence asking here
 

[Danp2]

15 minutes ago, TalesFromTheScript said:

This works and saves the window position to an ini file with $nIniXPos and $nIniYPos

I think you are misinterpreting this. It seems to me you are passing X & Y values for the exStyle and parent parameters.

 

[TalesFromTheScript]

That "-1, -1" is needed when I use "$WS_EX_TOPMOST" but I was just including "-1, -1" there because it still works having those there while leaving out $WS_EX_TOPMOST.

To use $WS_EX_TOPMOST it also needs -1, -1 which is three functions in itself.

I can't grasp why you can't just add more functions, it's like there's a limit?

How can I simply append another function to it and have it saving the window position while also having the GUI always on top?

Cheers.

 

[Musashi]

16 minutes ago, TalesFromTheScript said:

But if I try to use both of those functions together (always on top with save window position)
then I get "Incorrect number of parameters in function call":

First of all 'Welcome to the forum' .

AutoIt has an excellent help. Have a look at it for GUICreate ,then you will surely figure out the problem yourself .
A little spoiler : Most functions expect parameters. At the beginning are the ones that have to be passed. After that are the so called optional parameters. These can be passed, but don't have to (then they are automatically filled with default values).

So you always have to take care to keep the prescribed order.

The following must be considered for the optional parameters :  If e.g. the second optional parameter is set, then you cannot simply omit the first one. For this parameter the keyword default often has to be used.

[TalesFromTheScript]

I had already looked at the reference before posting here.

I have no idea how to fix it and have tried putting -1 and Default in just about every combination possible, nothing fixes it.

I tried adding one more "-1" but I can't get it to work.

GUICreate("My GUI", 600, 400, -1, -1, -1, $WS_EX_TOPMOST, $nIniXPos, $nIniYPos)

I guess I'll just have to leave out the always on top functionality and put up with it, or see if it gets answered.

Cheers.

Edited February 25, 2021 by TalesFromTheScript

[Danp2]

Why do you think this line "saves the window position to an ini file with $nIniXPos and $nIniYPos"?

GUICreate("My GUI", 600, 400, -1, -1, Default, $nIniXPos, $nIniYPos)

Like I stated before, you are using the command incorrectly by passing the wrong values for the exstyle and parent parameters.

[TalesFromTheScript]

9 minutes ago, Danp2 said:

Why do you think this line "saves the window position to an ini file with $nIniXPos and $nIniYPos"?

 

It does. I just posted one line from a bigger script that also includes this before GUICreate:

Global $sMyIniLocation = "MyFolder\WinPos.ini"
Global $nIniXPos = Int(IniRead($sMyIniLocation, 'MAINWINPOS', 'XPOS', Int(@DesktopWidth/2.5)))
Global $nIniYPos = Int(IniRead($sMyIniLocation, 'MAINWINPOS', 'YPOS', Int(@DesktopHeight/2.5)))

Func _myWM_MOVE($hWnd, $nMsg, $wParam, $lParam)
    Local $aWPos = WinGetPos($hWnd)
    If IsArray($aWPos) Then
        IniWrite($sMyIniLocation, 'MAINWINPOS', 'XPOS', $aWPos[0])
        IniWrite($sMyIniLocation, 'MAINWINPOS', 'YPOS', $aWPos[1])
        $nIniXPos = $aWPos[0]
        $nIniYPos = $aWPos[1]
    EndIf
    Return
 EndFunc

 

And this after GUICreate:

GUIRegisterMsg(0x03, "_myWM_MOVE")
GUISetState()

I know the save window position part works because it's working perfectly fine in other GUI's that do not also have the always on top function added. The one GUI that does have the always on top function added, cannot have the window position function added because then all I get is an error "Incorrect number of parameters".

Either function on its own does work.

 

Edited February 25, 2021 by TalesFromTheScript

[Danp2]

@TalesFromTheScript No, it doesn't. The code you just posted reads from and updates the INI file. You don't need (or want) those two additional parameters on your GuiCreate because it could cause unexpected behavior.

[TalesFromTheScript]

1 hour ago, Danp2 said:

@TalesFromTheScript No, it doesn't. The code you just posted reads from and updates the INI file. You don't need (or want) those two additional parameters on your GuiCreate because it could cause unexpected behavior.

OK cheers.

I don't know how else to add them and since it works fine on other GUI's I assumed that's the way it would be done.

$nIniXPos and $nIniYPos is needed somewhere because it doesn't work if they are taken out.

Edited February 26, 2021 by TalesFromTheScript

[Musashi]

48 minutes ago, TalesFromTheScript said:

I don't know how else to add them and since it works fine on other GUI's I assumed that's the way it would be done.

This assumption, as @Danp2 has already tried to explain to you with great patience, is nonetheless wrong.

You pass the value $nIniXPos to parameter7=exStyle, and the value $nIniYPos to parameter8=parent.
This may fulfill the syntactical requirements of the function, but definitely not the functional ones.

Edited February 26, 2021 by Musashi
typo

[TalesFromTheScript]

I'll just pay someone on Fiverr to fix it since no one here has any intention of answering it.

Edited February 26, 2021 by TalesFromTheScript
KMA

[JLogan3o13]

Your question has been answered several times, by people that no the language inside and out - you simply cannot seem to comprehend what they are telling you. Perhaps scripting is not for you. And since you want to take your ball and go home in a fit, this thread can be closed.

Edited February 26, 2021 by JLogan3o13