remin Posted October 21, 2013 Posted October 21, 2013 (edited) I'm a bit confused about accelerator keys and checkboxes. 1) See my code below: Is this the correct way how I put them in the script? (First: reading the ini file to see if the checkbox is enabled then setting the checkbox state before "GUISetState(@SW_SHOW" Second: After having created the GUI, if the status of the checkbox changes disable or enable the accelerator keys (Case $CB3) Third: when the GUI is closed, write checkbox status in ini) 2) I've read in a reply of Melba in another topic that in order to disable the accelerator keys (in my case {enter}) It is needed to Send a CR to the control "ControlSend($hGUI, "", $hFocus, @CR)" I don't know why this is, I haven't understand it very well and I don't know where to put it in my case. (but it seems to work fine in my code without it) 3) In my code below I have inserted only 2 buttons (search and images) In reality I have a lot more buttons. The code between ";Focus and Accelerator key" and ";END Focus and Accelerator key" is almost the same in all Cases. The only thing different is the value of $focus. The value of $focus is the Case name. Is there a way to put this code in an external function to avoid multiplying the code many times? (I noted in the help file that there is not a GUI ID in GUICtrlSetState and GUISetAccelerators) expandcollapse popup#include <ButtonConstants.au3> #include <GUIConstantsEx.au3> #include <WindowsConstants.au3> #include <String.au3> #include <GuiButton.au3> #include <Constants.au3> #include <EditConstants.au3> HotKeySet("^4", "SearchMenu") While 1 Sleep(10000) WEnd Func SearchMenu() ;========GUI============= $MyFile = "d:\temp\RW-autoit.ini" Global $url $Form5= GUICreate("Search Menu", 345, 140, @DesktopWidth*0.65, @DesktopHeight*0.21) $RWSearch = GuiCtrlCreateEdit("", 16, 10, 270, 50) $textGoogle = GUICtrlCreateLabel("Google:", 20, 90, 100, 20) $bsearch = GUICtrlCreateButton("Search", 110, 90, 60, 20) $bimages = GUICtrlCreateButton("Images", 175, 90, 60, 20) $CB3 = GUICtrlCreateCheckbox("Enter=Send (^Enter=Enter)", 110, 120, 145, 20) If FileExists($MyFile) Then $State = IniRead($MyFile,"CheckBox","CB3",0) If $State == 1 Then GUICtrlSetState(($CB3), BitOr($GUI_ENABLE, $GUI_CHECKED)) Else GUICtrlSetState(($CB3), BitOr($GUI_ENABLE, $GUI_UNCHECKED)) EndIf $Focus = IniRead($MyFile,"Focus","Button",0) Else GUICtrlSetState(($CB3, BitOr($GUI_ENABLE, $GUI_CHECKED)) $Focus = "bsearch" EndIf GUISetState(@SW_SHOW) ;========END GUI========== Local $aAccelKeys[1][2] = [["{ENTER}", eval($focus)]] If _GUICtrlButton_GetCheck($CB3) Then GUISetAccelerators($aAccelKeys) Global $EnterSubmit = true Else GUISetAccelerators(0) Global $EnterSubmit = false EndIf ; Set the Default Key GUICtrlSetState(eval($focus), $GUI_DEFBUTTON) While 1 $nMsg = GUIGetMsg() Switch $nMsg Case $GUI_EVENT_CLOSE ; write status checkboxes in ini If FileExists($MyFile) Then If _GUICtrlButton_GetCheck($CB3) Then IniWrite($MyFile, "CheckBox", "CB3", 1) Else IniWrite($MyFile, "CheckBox", "CB3", 0) EndIf EndIf ; write focus in ini IniWrite($MyFile, "Focus", "Button", $Focus) GuiDelete($Form5) ExitLoop Case $CB3 If _GUICtrlButton_GetCheck($CB3) Then GUISetAccelerators($aAccelKeys) Global $EnterSubmit = true Else GUISetAccelerators(0) Global $EnterSubmit = false EndIf Case $bsearch $RWSearch1 = GUICtrlRead($RWSearch) $RWSearch2 = StringRegExpReplace($RWSearch1, "(\s+)", "+") ;Focus and Accelerator key $focus = "bsearch" GUICtrlSetState(eval($focus), $GUI_DEFBUTTON) If $EnterSubmit then GUISetAccelerators($aAccelKeys) Else GUISetAccelerators(0) Endif ;End Focus and Accelerator key $url = "https://www.google.com/search?q=" & $RWSearch2 SearchMenuExec() Case $bimages $RWSearch1 = GUICtrlRead($RWSearch) $RWSearch2 = StringRegExpReplace($RWSearch1, "(\s+)", "+") ;Focus and Accelerator key $focus = "bsearch" GUICtrlSetState(eval($focus), $GUI_DEFBUTTON) If $EnterSubmit then GUISetAccelerators($aAccelKeys) Else GUISetAccelerators(0) Endif ;End Focus and Accelerator key $url = "http://images.google.com/images?hl=en&q=" & $RWSearch2 SearchMenuExec() EndSwitch WEnd EndFunc ;==>SearchMenu Func SearchMenuExec() ClipPut($url) ShellExecute($url) EndFunc ;==>SearchMenuExec Edited October 21, 2013 by remin
Moderators Melba23 Posted October 21, 2013 Moderators Posted October 21, 2013 remin, I've read in a reply of Melba in another topic that in order to disable the accelerator keys (in my case {enter}) - It is needed to Send a CR to the controlCan you point me to where you found that because it is wrong and I cannot remember having said such a thing. How you are setting/cancelling the Accelerator keys is fine - just pass a non-array value to the function. 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
remin Posted October 21, 2013 Author Posted October 21, 2013 remin, Can you point me to where you found that because it is wrong and I cannot remember having said such a thing. How you are setting/cancelling the Accelerator keys is fine - just pass a non-array value to the function. M23 Sure '?do=embed' frameborder='0' data-embedContent>> (the last post) Can you please tell me what you mean with pass a non-array value to the function?
Moderators Melba23 Posted October 21, 2013 Moderators Posted October 21, 2013 remin,That code is used because the OP wanted to to actually use the associated Accel key when another control had focus. The code looks to see which control has keyboard focus and then if required disables the Accel function, passes an ENTER to the control and then re-enables the Accel function:Case $cEnter ; Get control with focus $hFocus = _WinAPI_GetFocus() If $hFocus = GUICtrlGetHandle($cInput) Then ; If input MsgBox(0, "Test", "Enter Pressed In Input") Else ; If not we need to disable the Accel key GUISetAccelerators(0) ; Send a CR to the control ControlSend($hGUI, "", $hFocus, @CR) ; And then reenable the Accel key GUISetAccelerators($aAccelKeys) EndIfAs you can see, the Accel keys are disabled/re-enabled in the 2 GUISetAccelerators lines - the ControlSend line has nothing to do with it. A "non-array" value is exactly that - anything which is not an array. I tend to use a simple numeric value of 0 as you can see from the above code. 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
remin Posted October 21, 2013 Author Posted October 21, 2013 (edited) Melba23, I told you I'm confused about accelerators. You told me to add a Non-array value in your 1st replay, but I added one (or am I wrong?): GUISetAccelerators(0) Melba, Have you seen also my 3rd question? ps: does accelerators not work with $WS_SIZEBOX? Edited October 21, 2013 by remin
remin Posted October 21, 2013 Author Posted October 21, 2013 How you are setting/cancelling the Accelerator keys is fine - just pass a non-array value to the function. Oh.. sorry, I didn't understand this phrase in the beginning. I thought I had to add a non-array value to the function but your way of writing is a confirmation of what I did. About my 3rd question: I'm curious to know if I can use GUICtrlSetState and GUISetAccelerators in an external function.. there is no way to indicate the Gui-name in both functions ....
Moderators Melba23 Posted October 21, 2013 Moderators Posted October 21, 2013 remin,Yes, you can use a function - this is how I might do it (although I have not tested the code as I have no idea what you are doing): expandcollapse popup#include <ButtonConstants.au3> #include <GUIConstantsEx.au3> #include <WindowsConstants.au3> #include <String.au3> #include <GuiButton.au3> #include <Constants.au3> #include <EditConstants.au3> HotKeySet("^4", "SearchMenu") Global $url,$Focus, $EnterSubmit ; Do not declare Global variables inside functions <<<<<<<<<<<< While 1 Sleep(10) ; 10 is quite sufficient <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< WEnd Func SearchMenu() ;========GUI============= $MyFile = "d:\temp\RW-autoit.ini" $Form5 = GUICreate("Search Menu", 345, 140, @DesktopWidth * 0.65, @DesktopHeight * 0.21) $RWSearch = GUICtrlCreateEdit("", 16, 10, 270, 50) $textGoogle = GUICtrlCreateLabel("Google:", 20, 90, 100, 20) $bsearch = GUICtrlCreateButton("Search", 110, 90, 60, 20) $bimages = GUICtrlCreateButton("Images", 175, 90, 60, 20) $CB3 = GUICtrlCreateCheckbox("Enter=Send (^Enter=Enter)", 110, 120, 145, 20) If FileExists($MyFile) Then $State = IniRead($MyFile, "CheckBox", "CB3", 0) If $State = 1 Then ; == means case-sensitive string comparison, so not needed here <<<<<<<<<<<<<<<<<<<<<<< GUICtrlSetState(($CB3), BitOR($GUI_ENABLE, $GUI_CHECKED)) Else GUICtrlSetState(($CB3), BitOR($GUI_ENABLE, $GUI_UNCHECKED)) EndIf $Focus = IniRead($MyFile, "Focus", "Button", 0) Else GUICtrlSetState($CB3, BitOR($GUI_ENABLE, $GUI_CHECKED)) $Focus = "bsearch" EndIf GUISetState(@SW_SHOW) ;========END GUI========== Local $aAccelKeys[1][2] = [["{ENTER}", Eval($Focus)]] ;If _GUICtrlButton_GetCheck($CB3) Then If GUICtrlRead($CB3) = $GUI_CHECKED Then ; Use native functions if possible <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< GUISetAccelerators($aAccelKeys) $EnterSubmit = True Else GUISetAccelerators(0) $EnterSubmit = False EndIf ; Set the Default Key GUICtrlSetState(Eval($Focus), $GUI_DEFBUTTON) While 1 $nMsg = GUIGetMsg() Switch $nMsg Case $GUI_EVENT_CLOSE ; write status checkboxes in ini If FileExists($MyFile) Then If GUICtrlRead($CB3) = $GUI_CHECKED Then IniWrite($MyFile, "CheckBox", "CB3", 1) Else IniWrite($MyFile, "CheckBox", "CB3", 0) EndIf EndIf ; write focus in ini IniWrite($MyFile, "Focus", "Button", $Focus) GUIDelete($Form5) ExitLoop Case $CB3 If GUICtrlRead($CB3) = $GUI_CHECKED Then GUISetAccelerators($aAccelKeys) $EnterSubmit = True Else GUISetAccelerators(0) $EnterSubmit = False EndIf Case $bsearch $RWSearch1 = GUICtrlRead($RWSearch) $RWSearch2 = StringRegExpReplace($RWSearch1, "(\s+)", "+") _Focus_Accel("bsearch", $aAccelKeys) ; Call a function with suitable parameters <<<<<<<<<<<<<<<<< $url = "https://www.google.com/search?q=" & $RWSearch2 SearchMenuExec() Case $bimages $RWSearch1 = GUICtrlRead($RWSearch) $RWSearch2 = StringRegExpReplace($RWSearch1, "(\s+)", "+") _Focus_Accel("bImages", $aAccelKeys) ; Call a function with suitable parameters <<<<<<<<<<<<<<<<< $url = "http://images.google.com/images?hl=en&q=" & $RWSearch2 SearchMenuExec() EndSwitch WEnd EndFunc ;==>SearchMenu Func SearchMenuExec() ClipPut($url) ShellExecute($url) EndFunc ;==>SearchMenuExec Func _Focus_Accel($Focus, $aAccelKeys) $Focus = "bsearch" GUICtrlSetState(Eval($Focus), $GUI_DEFBUTTON) If $EnterSubmit Then GUISetAccelerators($aAccelKeys) Else GUISetAccelerators(0) EndIf EndFuncI have added a few comments as well - please ask if any of them are unclear. 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
remin Posted October 22, 2013 Author Posted October 22, 2013 (edited) Thank you for your reply Melba It seems that something doesn't work. Without the function (_Focus_Accel) it works fine. With the function not (neither the Accelerator Keys nor the focus are updated). ........ You wrote in your reply that you didn't understand what I'm trying to do. What I would like to do with the checkbutton is: - when the checkbutton CB3 (Enter=Send) is checked I would like to view an inputbox of 1 line $RWSearch = GUICtrlCreateInput("", 16, 10, 270, 25) (with accelerators enabled and focus active on last used button) - when the checkbutton CB3 (label: Enter=Send) is unchecked I would like to view an editbox of 3 lines $RWSearch = GUICtrlCreateEdit("", 16, 10, 270, 65) (with accelerators disabled and focus active on last used button I hope I made myself clear Edited October 22, 2013 by remin
Moderators Solution Melba23 Posted October 22, 2013 Moderators Solution Posted October 22, 2013 remin,I think I understand now. I would code something like that rather differently. Most importantly I would create the GUI just once and then hide/show it as required - that way you do not need an ini file as the various values can be stored within the script itself. I have also shown how you can use a 2D array to have as many buttons as you want - although you will have to develop an algorithm to position them within your GUI, as well as making the GUI big enough to hold them. Take a look at this and see if it does what you want - then we can then look to developing the button algorithm:expandcollapse popup#include <GUIConstantsEx.au3> #include <WindowsConstants.au3> #include <Constants.au3> #include <EditConstants.au3> HotKeySet("^4", "SearchMenu") ; These need to be Global as we retain their values between function calls Global $RWSearch, $Focus = 9999 ; Create an array to hold button data: ["Title", ControlID, "url to use"] Global $aButton[3][3] = [[2, 0, 0], _ ["Search", -1, "https://www.google.com/search?q="], _ ["Images", -1, "http://images.google.com/images?hl=en&q="]] $Form5 = GUICreate("Search Menu", 345, 140, @DesktopWidth * 0.65, @DesktopHeight * 0.21) $textGoogle = GUICtrlCreateLabel("Google:", 20, 90, 100, 20) For $i = 1 To $aButton[0][0] $aButton[$i][1] = GUICtrlCreateButton($aButton[$i][0], 45 + ($i * 65), 90, 60, 20) Next $CB3 = GUICtrlCreateCheckbox("Enter=Send (^Enter=Enter)", 110, 120, 145, 20) GUISetState(@SW_HIDE) While 1 Sleep(10) WEnd Func SearchMenu() ; Check state of checkbox If GUICtrlRead($CB3) = $GUI_CHECKED Then ; Create correct sized input $RWSearch = GUICtrlCreateInput("", 16, 10, 270, 25, $ES_WANTRETURN) ; Set accel key If $Focus <> 9999 Then Local $aAccelKeys[1][2] = [["{ENTER}", $Focus]] GUISetAccelerators($aAccelKeys) EndIf Else ; Create correct sized input $RWSearch = GUICtrlCreateEdit("", 16, 10, 270, 65, BitOr($ES_WANTRETURN, $WS_VSCROLL, $ES_AUTOVSCROLL)) ; Clear Accel key GUISetAccelerators(0) EndIf ; Set the Default Key GUICtrlSetState($Focus, $GUI_DEFBUTTON) ; Show GUI GUISetState(@SW_SHOW) While 1 $nMsg = GUIGetMsg() Switch $nMsg Case $GUI_EVENT_CLOSE ; Delete search box GUICtrlDelete($RWSearch) ; Hide GUI GUISetState(@SW_HIDE) ExitLoop Case $CB3 ; Delete current edit control GUICtrlDelete($RWSearch) ; Creat new edit control and adjust Accel keys as required If GUICtrlRead($CB3) = $GUI_CHECKED Then ; Create correct sized input $RWSearch = GUICtrlCreateInput("", 16, 10, 270, 25, $ES_WANTRETURN) If $Focus = 9999 Then ; No focus set so uncheck GUICtrlSetState($CB3, $GUI_UNCHECKED) Else ; Set accel key Local $aAccelKeys[1][2] = [["{ENTER}", $Focus]] GUISetAccelerators($aAccelKeys) EndIf Else ; Create correct sized input $RWSearch = GUICtrlCreateEdit("", 16, 10, 270, 65, BitOr($ES_WANTRETURN, $WS_VSCROLL, $ES_AUTOVSCROLL)) ; Clear accel key GUISetAccelerators(0) EndIf Case Else ; Look through the buttons For $i = 1 To $aButton[0][0] If $nMsg = $aButton[$i][1] Then ; Found a button $RWSearch1 = GUICtrlRead($RWSearch) $RWSearch2 = StringRegExpReplace($RWSearch1, "(\s+)", "+") ; Set focus for next round $Focus = $aButton[$i][1] $url = $aButton[$i][2] & $RWSearch2 MsgBox($MB_SYSTEMMODAL, "Actioning", $aButton[$i][0] & @CRLF & @CRLF & $url) ;SearchMenuExec($url) ; No point in looking further ExitLoop EndIf Next EndSwitch WEnd EndFunc ;==>SearchMenu Func SearchMenuExec($url) ClipPut($url) ShellExecute($url) EndFunc ;==>SearchMenuExecLet me know how it works - and if you have any questions. 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
remin Posted October 22, 2013 Author Posted October 22, 2013 (edited) Thank you very much Melba! You've dedicated a lot of time to my script! I resolved all my questions. The only thing I didn't use is the dynamic button creator. Most Urls's of my Cases are constructed and there is a lot more text then the single Url. Just a few remarks: -------- While 1 Sleep(10) WEnd Can I change it to Sleep(1000) or Sleep(10000)? Higher values takes less CPU I've noted. (I still don't understand very well what it does) -------- I haven't added (for now) ExitLoop to all Case. It seems to work fine without. But I added to Case $GUI_EVENT_CLOSE: GuiDelete($Form5) ExitLoop Is it correct to do it without ExitLoop in all other CASE -------- Now when my Search Script is ready I'll add it to my general script file with other scripts, isn't it? There will no conflicts I believe because the search script is invoked by hotkey isn't it? -------- You've told me to write the global keys at the top of the file. I have many hotkeys in my general script file. Do I have to declare the global keys below the hotkeys or can it also be between the hotkeys? Is this correct? <includes.......> HotKeySet("^4", "SearchMenu") Global $url,$Currenttab ..... HotKeySet("^5", "emptyclipb") HotKeySet("^+5", "empty2ndclipb") -------- You wrote in a previous reply: $State = IniRead($MyFile, "CheckBox", "CB3", 0) If $State = 1 Then ; == means case-sensitive string comparison, so not needed here <<<<<<<<<<<<<<<<<<<<<<< I still don't understand exactly what you mean by this. -------- My convertion from AHK to AI is almost done One thing I haven't converted till now: My autotext lines :?*:\\em::myname@domain.com etc.. This isn't possible in autoit isn't it? -------- Melba, thank you again for all you did. I'm glad I switched to autoit. Have a nice day. Edited October 22, 2013 by remin
Moderators Melba23 Posted October 22, 2013 Moderators Posted October 22, 2013 remin, I'm glad I switched to autoitGood. I am glad you like the script - and it is my pleasure to help you. Now to all the questions: Can I change it to Sleep(1000) or Sleep(10000)?Sleep(10) is the lowest script pause you can have and is quite suffcient to give the CPU time to breathe and personally I have never seen for than 1% CPU use on my machine when using that value. Certainly you can use a bigger value - but I doubt if would make that much difference to this script. I haven't added (for now) ExitLoop to all CaseYou do not need to. The only reason that it is present when we look for the buttons is that once we have found the pressed button there is no point looking any further. So we use ExitLoop to exit from the For $i = 1 To $aButton[0][0] loop. As the other Case codes do not use loops there is no need to exit them. There will no conflicts I believe because the search script is invoked by hotkey isn't it?I have no idea as I have not seen the other script. Do I have to declare the global keys below the hotkeys or can it also be between the hotkeys?Autoit is pretty tolerant about where you place things, but I suggest that you try and structure your code in a sensible way - it helps debugging and is a boon when you return to a script after some time. I always try to structure my scripts like this:Directives Include files Options HotKeys Global declarations Main code Functions HandlersObviously you may wish to do something different, but coding in a structured manner, whatever it is, makes life much easier, believe me! case-sensitive string comparison, so not needed here <<<<<<<<<<<<<<<<<<<<<<<I still don't understand exactly what you mean by thisAutoIt normally compares using the = operator. If both sides are numeric it does a numeric comparison; if both sides are strings it does a case-insensitive ASCII comparison; if there is a mix of datatypes it can go very wrong, so force the 2 sides into type you want using String or Number. If you use the == operator then AutoIt forces both sides into strings and does a case-sensitive ASCII comparison - this takes longer than a simple compare and so you should only use it if you really require such a comparison. In this case you are comparing 2 numbers and so there is no need. :?*:em::myname@domain.comI have no idea what an "autotext" line is supposed to do - so I cannot comment on whether AutoIt can do whatever it is. If you explain what you are trying to do we can probably come up with a solution. 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
remin Posted October 22, 2013 Author Posted October 22, 2013 (edited) Good. I am glad you like the script - and it is my pleasure to help you. Thank you Sleep(10) is the lowest script pause you can have and is quite suffcient to give the CPU time to breathe and personally I have never seen for than 1% CPU use on my machine when using that value. Certainly you can use a bigger value - but I doubt if would make that much difference to this script. The higher the sleep value the slower the response of the script isn't it? AutoIt normally compares using the = operator. If both sides are numeric it does a numeric comparison; if both sides are strings it does a case-insensitive ASCII comparison; if there is a mix of datatypes it can go very wrong, so force the 2 sides into type you want using String or Number. If you use the == operator then AutoIt forces both sides into strings and does a case-sensitive ASCII comparison - this takes longer than a simple compare and so you should only use it if you really require such a comparison. In this case you are comparing 2 numbers and so there is no need. I still don't get it totally. My code: $State = IniRead($MyFile,"CheckBox",$cbarray[$i],0) If $State == 1 Then GUICtrlSetState(eval($cbarray[$i]), BitOR($GUI_ENABLE, $GUI_CHECKED)) Else GUICtrlSetState(eval($cbarray[$i]), BitOR($GUI_ENABLE, $GUI_UNCHECKED)) EndIf Do i have to change it like this (is that what you mean?) If IniRead($MyFile,"CheckBox",$cbarray[$i],0) Then Else Endif I have no idea what an "autotext" line is supposed to do - so I cannot comment on whether AutoIt can do whatever it is. If you explain what you are trying to do we can probably come up with a solution. Sorry Autotext is the name of ahk text expanding. p.e. in this case :?*:em::myname@domain.com Typing "em" will expand to the email address "myname@domain.com" Thanks again ps: In my search script I will use 3 global arrays with long strings inside. I tried to declare the arrays above the scripts like this: Global $BClass[3] etc and then use the array within the function like this: $BClass[3] = ["text", "text", "text"] but that doesn't seems to work. What did I wrong Melba? . Edited October 22, 2013 by remin
Moderators Melba23 Posted October 22, 2013 Moderators Posted October 22, 2013 remin, The higher the sleep value the slower the response of the script isn't it?But only in MessageLoop mode where you should just use the built-in pause in GUIGetMsg - that waits up to 12-15ms when the CPU is very busy while the return almost instantaneous if it is idle. In OnEvent mode, and here when only HotKeys are used, the Sleep period has no effect. This is one reason I would not comment on the interaction between your current script and the one with which you wish to merge it - until I see exactly what is involved I am not prepared to say whether problem might arise. [=/==] I still don't get it totallyIn this case as you are comparing numbers, you do not need a case-sensitive string comparison. All you need do is replace the == operator with =:If $State == 1 Then ; should become If $State = 1 ThenAutotext is the name of ahk text expandingI see. In Autoit you have 2 options: - 1. Use a variable to contain the expanded text:; Declare the content $sEM = "myname@domain.com" ; And use it like this $url = $sEM & "\section\item.html"$url will then hold myname@domain.comsectionitem.html-2. Use the Abbrev manager in SciTE4AutoIt3 to create an abbreviation which you can expand as you type. Look under the <Help - SciTE Help - SciTE4AutoIt3 - User Abbreviations> and <Help - SciTE Help - Extra Utilities - Abbrev Manager> to learn more. If you do not have the full SciTE4AutoIt3 package I highly recommend that you download it from here - you get lots of helpful utilities to help you code in AutoIt. M23 remin 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
remin Posted October 22, 2013 Author Posted October 22, 2013 (edited) I'll read more about GUIGetMsg In this case as you are comparing numbers, you do not need a case-sensitive string comparison. All you need do is replace the == operator with =: Oh.. I see the error I've made Thank you also for the autotext info. I'll study it a bit more. I added one more thing in my previous post but you were too fast answering ------------ In my search script I will use 3 global arrays with long strings inside. I tried to declare the arrays above the scripts like this: Global $BClass[3] etc and then use the array within the function like this: $BClass[3] = ["text", "text", "text"] but that doesn't seems to work. What did I wrong Melba? ------------ Remin . Edited October 22, 2013 by remin
Moderators Melba23 Posted October 22, 2013 Moderators Posted October 22, 2013 remin, you were too fast answeringI will try to do worse in future. You need to declare the arrays and their contents at the same time - otherwise you can only assign individual elements:; This will work Global $BClass[3] = ["text", "text", "text"] ; As will this if you use the Beta Global $BClass[] = ["text", "text", "text"] ;~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ; Or you need to declare the array at the beginning of the script Global $BClass[3] ; And then assign the elements individually $BClass[0] = "text" $BClass[1] = "text" $BClass[2] = "text"Al clear? 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
remin Posted October 22, 2013 Author Posted October 22, 2013 You need to declare the arrays and their contents at the same time - otherwise you can only assign individual elements: Or just declare them in the function self? Melba, thank you very much for the time you've dedicated to my script. Thanks for all
Moderators Melba23 Posted October 22, 2013 Moderators Posted October 22, 2013 remin, Or just declare them in the function self?You can always redeclare arrays - but you still need to either declare the contents at the same time or assign the individual elements separately. As always, my pleasure to help. 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
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