Shell Execute

[Masen]

when i press Home button on my program it just closes the window.

not sure where im going wrong here.

 

#include <ButtonConstants.au3>
#include <EditConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
#include <MsgBoxConstants.au3>



$Button1 = GUICtrlCreateButton("Home", 320, 72, 75, 25, $WS_GROUP)

While 1
$nMsg = GUIGetMsg()
Switch $nMsg
    Case $GUI_EVENT_CLOSE
                     Case $Button1

Exit

Case $Form1_1
EndSwitch
WEnd

Func Home ()

 ShellExecute ("Masen's Headquarter Interface.exe","", "C:\Users\masen\Desktop\HQ-General\Masen's Headquarter Interface.exe" ,"open","@SW_MAXIMIZE")


    EndFunc

 

[Melba23]

Masen,

The third parameter is the working directory, not the full path and name of the executable. Try changing the line to read:

ShellExecute ("Masen's Headquarter Interface.exe","", "C:\Users\masen\Desktop\HQ-General" ,"open","@SW_MAXIMIZE")

M23

[Starf0x]

Try this:

While 1
    $nMsg = GUIGetMsg()
    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
        Case $Button1
            Call("Run")
    EndSwitch
WEnd

Func Run()
    ShellExecute ("Masen's Headquarter Interface.exe","", "C:\Users\masen\Desktop\HQ-General" ,"open", @SW_MAXIMIZE)
EndFunc

You've added Exit after you clicked the "Home" Button.

Cheers, Starf0x

[Masen]

Thx i didnt see that exit there and i added

and changed the shell execute code

and it worked

 

 

While 1
$nMsg = GUIGetMsg()

Switch $nMsg
    Case $GUI_EVENT_CLOSE
        Exit

        Case $Button1
             Home()



EndSwitch
WEnd

Func Home ()

 ShellExecute ("Masen's Headquarter Interface.exe","", "C:\Users\masen\Desktop\HQ-General\" ,"open","@SW_MAXIMIZE")


    EndFunc

Quote

Masen,

The third parameter is the working directory, not the full path and name of the executable. Try changing the line to read:

ShellExecute ("Masen's Headquarter Interface.exe","", "C:\Users\masen\Desktop\HQ-General" ,"open","@SW_MAXIMIZE")

 

1 hour ago, Starf0x said:

Try this:

While 1
    $nMsg = GUIGetMsg()
    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
        Case $Button1
            Call("Run")
    EndSwitch
WEnd

Func Run()
    ShellExecute ("Masen's Headquarter Interface.exe","", "C:\Users\masen\Desktop\HQ-General" ,"open", @SW_MAXIMIZE)
EndFunc

You've added Exit after you clicked the "Home" Button.

Cheers, Starf0x

 

Edited January 26, 2017 by Masen

[Starf0x]

I don't know which editor you use, but if it is scite, press Ctrl+t in the code window.

It cleans it a bit.

Cheers, Starf0x

[Masen]

i do what do u mean cleans it?

[Skysnake]

Hi

Tidy (the Ctrl+T (clean)) comment referred to above will make your code look nice by aligning tabs etc.  Try it.

I do not think naming a function "Run" - which is also an internal AutoIt function name, is a good idea, rather use something like _Run() or myRun()

Skysnake