Weird problem with if and then

[PcExpert]

Hi all,

Autoit is telling me that I need to put a then statement after an if statement. But the if, then and endif statements are already there. When I just run the script or run it once compiled it works just fine, it only gives the error when the program is ran from another autoitscript. Why.

RunWait(@TempDir & "\str.exe")

Gives this error:

Line -1:

Error: "If" statements must have a "Then" keyword.

Thanks! Edited February 7, 2008 by PcExpert

[whim]

Well... with this little information all I can suggest is

to recheck all your loops, not only the If/Then ones.

In my experience another unmatched loop inside or around an

If/Then can provoke this error.

[PcExpert]

As I said, it works. But it only doesnt when called by another autoitscript. So I dont think its an error in my loop.

Edited February 7, 2008 by PcExpert

[BrettF]

Is that the full code? And is Str.exe the name of the compiled script?

Edited February 7, 2008 by Bert

[PcExpert]

No the full code from the main program, that calls str.exe is this part:

GUICtrlSetState($store, $GUI_DISABLE)
RunWait(@TempDir & "\str.exe")
GUICtrlSetState($store, $GUI_Enable)

thanks

Edited February 8, 2008 by PcExpert

[PcExpert]

*bump*

[whim]

You could try to run the source of str.au3 via AutoIt3.exe,

then it should at least tell you at which line it errors ?

[SadBunny]

I suspect you forgot to copy a working version of str.exe to your @TempDir. The str code works fine.

/edit: yes, I compiled the str.au3 code to an exe and ran it from another script, just like you said Are you sure you don't mean @scriptdir instead of @tempdir?

Edited February 8, 2008 by SadBunny

[PcExpert]

The main script first does a fileinstall() and then runs str.exe from @tempdir where the file is also being installed, when the user clicks a button the str.exe is run.

[PsaltyDS]

The main script first does a fileinstall() and then runs str.exe from @tempdir where the file is also being installed, when the user clicks a button the str.exe is run.

You didn't try to slip @ScriptDir or some other macro or variable into the FileInstall() source field, did you? I don't see this FileInstall() in the code you posted.

[PcExpert]

The fileinstall part:

FileInstall("store.exe", @TempDir & "\str.exe")

Run str.exe

Case $msg = $store
GUICtrlSetState($store, $GUI_DISABLE)
RunWait(@TempDir & "\str.exe")
GUICtrlSetState($store, $GUI_Enable)

[SadBunny]

I would change two things about the fileinstall part:

1. change "store.exe" to "C:\mystuff\store.exe" (full path)

2. use the third parameter (the overwrite flag). It makes sure that the FileInstall overwrites an already present file, so that if you have an old version at the old location it gets overwritten with the new one.

That would lead to:

FileInstall("c:\mystuff\store.exe", @TempDir & "\str.exe",1)

[PcExpert]

Thanks, it works now.