acmecd Posted July 11, 2015 Share Posted July 11, 2015 Hi, I would like to know whether it is Autoit error or not. If one of the values is 0, then the IF function is always positive, and the answer is incorrect. AutoIt Version: 3.3.12.0 Local $BiteMain=MainParameters(1,0)Func MainParameters($ParamArrayCol="A", $ParamArrayRow="B") If $ParamArrayCol="A" Or $ParamArrayRow="B" Then MsgBox(0,"Info","Err.") Return False Else Return "Ok" EndIfEndFunc Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted July 11, 2015 Moderators Share Posted July 11, 2015 acmecd,In future, please do not post general help questions in the "Examples" section - I have now moved it for you.You need to make sure comparisons are made between similar datatypes or you get strange results - as you have seen. In this case the lefthand side of the comparisons is a number so AutoIt converts the righthand side to that type as well - and converting non-numeric strings to numbers sets them to 0, so the comparison is correct. You need to force string comparisons by using the String function or the == operator like this:Local $BiteMain = MainParameters(1, 0) Func MainParameters($ParamArrayCol = "A", $ParamArrayRow = "B") If String($ParamArrayCol) = "A" Or $ParamArrayRow == "B" Then MsgBox(0, "Info", "Err.") Return False Else MsgBox(0, "Info", "OK") Return "Ok" EndIf EndFunc ;==>MainParametersAll 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 Link to comment Share on other sites More sharing options...
acmecd Posted July 11, 2015 Author Share Posted July 11, 2015 Thank you very much, because so much time passed before I understood where is the error !!! Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted July 11, 2015 Moderators Share Posted July 11, 2015 acmecd,Glad I could 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 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