Sign in to follow this  
Followers 0
bartekd

GUI Won't close

9 posts in this topic

#1 ·  Posted (edited)

When i try to close this gui after I hit the 'Go' button, it doesn't want to close. Is there something that I can add into the script below, or change to make me able to close the gui?

#include <GUIConstants.au3>
$Form1 = GUICreate("PcAnywhere-Agents Bartek - June-2009", 150, 100, 193, 225)
$EXT = GUICtrlCreateInput("", 30, 12, 80, 21)
$EXTToWK = GUICtrlCreateButton("GO",  30, 42, 80, 21)
GUISetState(@SW_SHOW)

While 1
    $nMsg = GUIGetMsg()
    Switch $nMsg
        Case $GUI_EVENT_CLOSE
            Exit
Case $EXTToWK
    $IDNumber = GUICtrlRead($EXT)
    GetID($IDNumber)
        EndSwitch
    WEnd
            Func GetID($ID)
            $FileName="c:\Computers_EXT.xls"
            if not FileExists($FileName) then
            Msgbox (0,"Excel Test","Can't find database")
            Exit
            endif
            $oExcel = ObjGet($FileName); Get an Excel Object from an existing filename
            if IsObj($oExcel) then
            $OEvent=ObjEvent("AutoIt.Error","nothing"); Equal to VBscript's On Error Resume Next
            For $cell in $oExcel.ActiveSheet.Range("A1:A1000")
            If $cell.Value = $ID Then 
            $ID = $cell.Value
            $Row = $cell.Row
            $Model = $oExcel.Activesheet.Cells($Row,"B").Value
            GUICtrlSetData($ext, $Model)
            Endif 
            Next 
            else
            Msgbox (0,"Excel Test","Error: Could not open database")
            endif
            EndFunc
Edited by bartekd

Share this post


Link to post
Share on other sites



bartekd,

Works for me.

When you post code, please use Code tags. Put [code ] before and [/code ] after your posted code (but omit the trailing space - it is only there so the tags display here). Thanks in advance.

M23


Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

#3 ·  Posted (edited)

Use GuiSetOnEvent

Edited by oMBRa

Share this post


Link to post
Share on other sites

I tried adding exitfunc and GuiSetOnEvent but that didn't work.

This is the problem.

If I run the program, I can close it without a problem. it is only after I hit the Go button and it searches a excel file and then try to close the program, I have to keep clicking the 'X' in order to close the program. Do you know what it could be. It seems like the loop is stuck somewhere.

Share this post


Link to post
Share on other sites

Thanks, is there something I can put in my code that would fix this?

Share this post


Link to post
Share on other sites

Take a look at what can be done to exit the For / Next loop in your function. That's where is is hanging up. Nothing will happen while you are in that loop.


George

Question about decompiling code? Read the decompiling FAQ and don't bother posting the question in the forums.

Be sure to read and follow the forum rules. -AKA the AutoIt Reading and Comprehension Skills test.***

The PCRE (Regular Expression) ToolKit for AutoIT - (Updated Oct 20, 2011 ver:3.0.1.13) - Please update your current version before filing any bug reports. The installer now includes both 32 and 64 bit versions. No change in version number.

Visit my Blog .. currently not active but it will soon be resplendent with news and views. Also please remove any links you may have to my website. it is soon to be closed and replaced with something else.

"Old age and treachery will always overcome youth and skill!"

Share this post


Link to post
Share on other sites

But the thing is it returns what I want it to, then sticks for a little while.

Share this post


Link to post
Share on other sites

$Model = $oExcel.Activesheet.Cells($Row,"B").Value
If $Model Then
    GUICtrlSetData($ext, $Model)
    Return
EndIf


George

Question about decompiling code? Read the decompiling FAQ and don't bother posting the question in the forums.

Be sure to read and follow the forum rules. -AKA the AutoIt Reading and Comprehension Skills test.***

The PCRE (Regular Expression) ToolKit for AutoIT - (Updated Oct 20, 2011 ver:3.0.1.13) - Please update your current version before filing any bug reports. The installer now includes both 32 and 64 bit versions. No change in version number.

Visit my Blog .. currently not active but it will soon be resplendent with news and views. Also please remove any links you may have to my website. it is soon to be closed and replaced with something else.

"Old age and treachery will always overcome youth and skill!"

Share this post


Link to post
Share on other sites

thanks, that works

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0