4b0082 Posted January 13, 2015 Share Posted January 13, 2015 (edited) For some reason I can't compare $rTotal and $Total properly. Both have a return value of "0.57" and are set as double variables, but when I compare the two I still get my error message. I've tried converting the variables with Number() but that didn't seem to change anything. Local $Total = .57 Local $SubTotal = Round($Total/1.15, 2) Local $rFee = FloorEx($SubTotal * 0.1, 2) + FloorEx($SubTotal * 0.05, 2) Local $rTotal = $SubTotal + $rFee If $rTotal <> $Total Then MsgBox(0, "Broken", VarGetType($Total)) EndIf Func FloorEx($fNumber, $iDec = 3) Return StringFormat("%." & $iDec & "f", Floor($fNumber * 10 ^ $iDec) / 10 ^ $iDec) EndFunc Edited January 13, 2015 by 4b0082 Link to comment Share on other sites More sharing options...
water Posted January 13, 2015 Share Posted January 13, 2015 Function FloorEx doesn't make much sense to me. You return a formatted string and then use it in a numeric operation forcing autoit to convert the string into a number. Shouldn't you first do all the calculation and then format the result? My UDFs and Tutorials: Spoiler UDFs: Active Directory (NEW 2024-07-28 - Version 1.6.3.0) - Download - General Help & Support - Example Scripts - Wiki ExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example Scripts OutlookEX (2021-11-16 - Version 1.7.0.0) - Download - General Help & Support - Example Scripts - Wiki OutlookEX_GUI (2021-04-13 - Version 1.4.0.0) - Download Outlook Tools (2019-07-22 - Version 0.6.0.0) - Download - General Help & Support - Wiki PowerPoint (2021-08-31 - Version 1.5.0.0) - Download - General Help & Support - Example Scripts - Wiki Task Scheduler (2022-07-28 - Version 1.6.0.1) - Download - General Help & Support - Wiki Standard UDFs: Excel - Example Scripts - Wiki Word - Wiki Tutorials: ADO - Wiki WebDriver - Wiki Link to comment Share on other sites More sharing options...
4b0082 Posted January 13, 2015 Author Share Posted January 13, 2015 I needed a way to round down and that's a snippet of code I found through the forums that worked. I'm not sure of another way to do it. Link to comment Share on other sites More sharing options...
Moderators Solution Melba23 Posted January 13, 2015 Moderators Solution Share Posted January 13, 2015 4b0082,If you convert the 2 values to Hex then you will see that they are slightly different:$rTotal: 3FE23D70A3D70A3E $Total : 3FE23D70A3D70A3DThis is a result of the way in which computers store fractional number (Google "floating point arithmetic" to learn more) and not an AutoIt problem. The normal way to get over this is to use integer values - multiply the value by a large enough factor to get it as an integer for calculations and then divide by that factor when you need the actual value. 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...
4b0082 Posted January 13, 2015 Author Share Posted January 13, 2015 (edited) I tried multiplying the results by 100 then comparing them but I'm still getting double as my variable and it's still processing the error. Edit: Nevermind! I forgot to wrap Int() around the new variable. It works now, thanks! Edited January 13, 2015 by 4b0082 Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted January 13, 2015 Moderators Share Posted January 13, 2015 4b0082,It works for me: $iFactor = 100 Local $Total = .57 Local $SubTotal = Round($Total/1.15, 2) Local $rFee = Number(FloorEx($SubTotal * 0.1, 2)) + Number(FloorEx($SubTotal * 0.05, 2)) Local $rTotal = $SubTotal + $rFee ConsoleWrite($rTotal & " - " & $Total & @CRLF) ConsoleWrite(Hex($rTotal) & @CRLF & Hex($Total) & @CRLF) If Int ($rTotal * $iFactor) <> Int ($Total * $iFactor) Then ; <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< MsgBox(0, "Broken", Hex($rTotal) & @CRLF & Hex($Total)) Else MsgBox(0, "Works", ";)") EndIf Func FloorEx($fNumber, $iDec = 3) Return StringFormat("%." & $iDec & "f", Floor($fNumber * 10 ^ $iDec) / 10 ^ $iDec) EndFuncM23 4b0082 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...
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