ni3dprint Posted October 26, 2020 Share Posted October 26, 2020 I'm sure this is something stupid, but why won't this work? [autoit] Local $iNumber1 = GUICtrlCreateInput("", 50 , 80 , 40, 30, $ES_NUMBER) GUICtrlSetLimit(-1, 4) Local $iNumber2 = GUICtrlCreateInput("", 50 , 130 , 40, 30, $ES_NUMBER) GUICtrlSetLimit(-1, 4) [/autoit] etc.. Local $n=1 then in the funtion: [autoit] local $sCounterID=("$iNumber" & $n) $iCounter=GUICtrlRead($sCounterID) ConsoleWrite($sCounterID) ConsoleWrite($iCounter) [/autoit] the idea is to iterate the $n to read the various inputs ( I have 15 of them), however the GUICtrlRead simply returns 0? I know it's working $sCounterID is displaying correctly and if I replace it with $iNumber1 etc then the script returns the input value correctly. strange one? Thanks for any help! Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted October 26, 2020 Moderators Share Posted October 26, 2020 ni3dprint, Concatenating the variable name as you are doing results in a literal string - when that string is passed to GUICtrlRead it is interpreted as a number, as that is what the function expects. And a literal string which does not start with a numeric character is interpreted as 0 - hence the function fails to read the correct control. What you need to do is to use Eval like this: #include <GUIConstantsEx.au3> #include <EditConstants.au3> $hGUI = GUICreate("Test", 500, 500) Local $iNumber1 = GUICtrlCreateInput("34", 50, 80, 40, 30, $ES_NUMBER) $cButton = GUICtrlCreateButton("Test", 10, 300, 80, 30) GUISetState() While 1 Switch GUIGetMsg() Case $GUI_EVENT_CLOSE Exit Case $cButton _Test() EndSwitch WEnd Func _Test() Local $n = 1 Local $sCounterID = ("iNumber" & $n) $iCounter = GUICtrlRead(Eval($sCounterID)) ConsoleWrite($sCounterID & @CRLF) ConsoleWrite($iCounter & @CRLF) EndFunc ;==>_Test But even better would be to use an array to hold the ControlIDs of your inputs - then you can use a simple loop to iterate through them. M23 Skysnake 1 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...
ni3dprint Posted October 26, 2020 Author Share Posted October 26, 2020 Thank you for your help ! 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