bartekd Posted May 28, 2009 Share Posted May 28, 2009 (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? expandcollapse popup#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 May 28, 2009 by bartekd Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted May 28, 2009 Moderators Share Posted May 28, 2009 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 Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
oMBRa Posted May 28, 2009 Share Posted May 28, 2009 (edited) Use GuiSetOnEvent Edited May 28, 2009 by oMBRa Link to comment Share on other sites More sharing options...
bartekd Posted May 28, 2009 Author Share Posted May 28, 2009 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. Link to comment Share on other sites More sharing options...
bartekd Posted May 28, 2009 Author Share Posted May 28, 2009 Thanks, is there something I can put in my code that would fix this? Link to comment Share on other sites More sharing options...
GEOSoft Posted May 28, 2009 Share Posted May 28, 2009 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!" Link to comment Share on other sites More sharing options...
bartekd Posted May 28, 2009 Author Share Posted May 28, 2009 But the thing is it returns what I want it to, then sticks for a little while. Link to comment Share on other sites More sharing options...
GEOSoft Posted May 28, 2009 Share Posted May 28, 2009 $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!" Link to comment Share on other sites More sharing options...
bartekd Posted May 28, 2009 Author Share Posted May 28, 2009 thanks, that works Link to comment Share on other sites More sharing options...
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now