Jump to content

Trouble Comparing Two Variables


4b0082
 Share

Go to solution Solved by Melba23,

Recommended Posts

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 by 4b0082
Link to comment
Share on other sites

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 2022-02-19 - Version 1.6.1.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 (NEW 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

  • Moderators
  • Solution

4b0082,

If you convert the 2 values to Hex then you will see that they are slightly different:

$rTotal: 3FE23D70A3D70A3E
$Total : 3FE23D70A3D70A3D
This 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

Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png 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 columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Link to comment
Share on other sites

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 by 4b0082
Link to comment
Share on other sites

  • Moderators

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)
EndFunc
M23

Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png 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 columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Link to comment
Share on other sites

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 account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...