Buttons

[Pandacyber]

So i am having a problem making buttons run files when they are pressed. 

Below are the codes for the buttons. When i try to open my gui now it ONLY runs the mousemove file (which is my file for moving the mouse) and no longer opening the GUI. Please help.

 

; Create the GUI.

#include <GUIConstantsEx.au3>

#include "C:UserscollierDocumentsautoitmakemousemove.au3"

 

GuiCreate("HITS Palla", 400, 400)

 

 Local $button1 = GUICtrlCreateButton("Start", 120, 320, 85, 25)

    Local $button2 = GUICtrlCreateButton("Close", 210, 320, 85, 25)

 

    ; Display the GUI.

    GUISetState()

 

    Local $iPID = 0

 

    ; Loop until the user exits.

    While 1

        Switch GUIGetMsg()

            Case $GUI_EVENT_CLOSE, $idClose

                ExitLoop

 

            Case $button1

                ; start button

                $button1 = Run("C:UserscollierDocumentsautoitmakemovemouse.au3")

 

            Case $button2

                ; cancel button

                Exitloop

 

 

        EndSwitch

    WEnd

 

 

Tell me what i am doing wrong.

Edited October 15, 2014 by Pandacyber

[Pandacyber]

basically what i need to know is how to make buttons run a file

[JLogan3o13]

It's hard to tell from your script where the issue is (no error checking, and you're calling another script from within your script). Can you explain what application or window you're trying to manipulate? You may be able to control it without having to call your movemouse.au3 script.

[BrewManNH]

Your major problem is you're trying to Run an AutoIt script, they are not runnable because they're not executable. You need to use ShellExecute to run a script, but that only works if you have AutoIt installed on the computer you're trying to run the script on. Second problem is that your Run command was overwriting your $button1 variable, used to hold the control ID of the button, with the PID returned by the Run command, so the button would only work once.

[Pandacyber]

 

#include <ButtonConstants.au3>

#include <GUIConstantsEx.au3>

#include <ProgressConstants.au3>

#include <StaticConstants.au3>

#include <WindowsConstants.au3>

 

 

$Form1 = GUICreate("Form1", 196, 279, 373, 211)

$Button1 = GUICtrlCreateButton("Start", 16, 136, 161, 33)

$Button2 = GUICtrlCreateButton("Cancel", 16, 184, 161, 33)

GUISetState(@SW_SHOW)

 

 

 

While 1

$nMsg = GUIGetMsg()

Switch $nMsg

 Case $GUI_EVENT_CLOSE

Case $nmsg = $Button1

ShellExecute("C:UserscollierDocumentsautoitmakemousemove.au3") 

Case $nmsg = $Button2

Exit

 

Exit

 

 

EndSwitch

wend

Alright this is what i got but it keeps saying that it cannot find my au3 file

Edited October 16, 2014 by Pandacyber

[Spider001]

Your major problem is you're trying to Run an AutoIt script, they are not runnable because they're not executable. You need to use ShellExecute to run a script, but that only works if you have AutoIt installed on the computer you're trying to run the script on.

 

.au3 is not runnable

Edited October 16, 2014 by Spider001

[Pandacyber]

.exe wont work either for some reason. It says its .exe is not compatible with my windows computer because of the x64 bit

[BrewManNH]

I just noticed something else, you're using the Switch/Case statements incorrectly. You have it written in the syntax of a Select statement, a Switch works like this.

While 1

    $nMsg = GUIGetMsg()

    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
        Case $Button1 ; <<<<<<<<<<<<<<<<<<
            ShellExecute("C:\Users\collier\Documents\autoitmake\mousemove.au3")
        Case $Button2 ; <<<<<<<<<<<<<<<<<<
            Exit
    EndSwitch

WEnd

[Pandacyber]

OMG THANKYOU!
It works now and greatly appreciate the help

[BrewManNH]

Glad to hear it, and sorry I didn't notice it sooner.