kor Posted August 7, 2012 Share Posted August 7, 2012 why does this code work If _IsPressed("0D") Then ; if enter is pressed For $i = 9 To 31 $aHandles[$i] = GUICtrlGetHandle($aControls[$i]) ; read controls into a new array (not sure this is necessary) Next Switch _WinAPI_GetFocus() Case $aHandles[9], $aHandles[11], $aHandles[13], $aHandles[16], $aHandles[18], $aHandles[20], $aHandles[22], $aHandles[24], $aHandles[27], $aHandles[29], $aHandles[31] GUICtrlSendToDummy($cDummy) EndSwitch While _IsPressed("0D") Sleep(10) WEnd EndIf but this code doesn't? If _IsPressed("0D") Then ; has enter been pressed For $i = 9 To 31 $aHandles[$i] = GUICtrlGetHandle($aControls[$i]) Next Switch _WinAPI_GetFocus() Case $aHandles[9] To $aHandles[31] GUICtrlSendToDummy($cDummy) EndSwitch While _IsPressed("0D") Sleep(10) WEnd EndIf I'm trying to reduce the case statement down into something easier rather than naming each and every control. Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted August 7, 2012 Moderators Share Posted August 7, 2012 kor,It does not work because, unlike Autoit ControlIDs, handles are not consecutive integers and so using To to act on a range does not work.What you could do is something like this (not tested): If _IsPressed("0D") Then ; has enter been pressed ; Why do this on every pass - why not do it just the once before the loop? For $i = 9 To 31 $aHandles[$i] = GUICtrlGetHandle($aControls[$i]) Next ; Get the focused handle $hActive = _WinAPI_GetFocus() ; Now loop through the array to see if there is a match For $i = 9 To 31 If $aHandles[$i] = $hActive Then ; If there is then action something GUICtrlSendToDummy($cDummy) EndIf Next ; Wait until key is released While _IsPressed("0D") Sleep(10) WEnd EndIfM23 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...
kor Posted August 7, 2012 Author Share Posted August 7, 2012 Took your example and came up with this. It works nicely. thanks! If _IsPressed("0D") Then ; has enter been pressed For $i = 9 To 31 If GUICtrlGetHandle($aControls[$i]) = _WinAPI_GetFocus() Then GUICtrlSendToDummy($cDummy) Next While _IsPressed("0D") Sleep(10) WEnd EndIf 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