Floppy Posted October 23, 2011 Share Posted October 23, 2011 Hello, this script is driving me crazy. I have an array of numbers and I want to have the greater. $job_soldi_magg = 0 For $z = 1 To (UBound($job_soldi)-1) If $job_soldi[$z] > $job_soldi_magg Then $job_soldi_magg = $job_soldi[$z] Next MsgBox(0, 'Maggiore finale', $job_soldi_magg) The array $job_soldi is this: [0]| [1]|0.15 [2]|18.06 [3]|4.88 [4]|1.12 [5]|0.21 [6]|0.09 [7]|0.09 [8]|0.09 [9]|0.06 [10]|0.39 [11]|0.18 [12]|0.24 [13]|0.03 [14]|0.78 [15]|0.18 [16]|0.24 [17]|0.84 [18]|0.30 [19]|0.24 [20]|0.12 The greater number is 18.06, but the script returns 4.88. Can someone explain me why this happens? Thank you! Link to comment Share on other sites More sharing options...
Yashied Posted October 23, 2011 Share Posted October 23, 2011 You are comparing a strings instead of float numbers. My UDFs: iKey | FTP Uploader | Battery Checker | Boot Manager | Font Viewer | UDF Keyword Manager | Run Dialog Replacement | USBProtect | 3D Axis | Calculator | Sleep | iSwitcher | TM | NetHelper | File Types Manager | Control Viewer | SynFolders | DLL Helper Animated Tray Icons UDF Library | Hotkeys UDF Library | Hotkeys Input Control UDF Library | Caret Shape UDF Library | Context Help UDF Library | Most Recently Used List UDF Library | Icons UDF Library | FTP UDF Library | Script Communications UDF Library | Color Chooser UDF Library | Color Picker Control UDF Library | IPHelper (Vista/7) UDF Library | WinAPI Extended UDF Library | WinAPIVhd UDF Library | Icon Chooser UDF Library | Copy UDF Library | Restart UDF Library | Event Log UDF Library | NotifyBox UDF Library | Pop-up Windows UDF Library | TVExplorer UDF Library | GuiHotKey UDF Library | GuiSysLink UDF Library | Package UDF Library | Skin UDF Library | AITray UDF Library | RDC UDF Library Appropriate path | Button text color | Gaussian random numbers | Header's styles (Vista/7) | ICON resource enumeration | Menu & INI | Tabbed string size | Tab's skin | Pop-up circular menu | Progress Bar without animation (Vista/7) | Registry export | Registry path jumping | Unique hardware ID | Windows alignment More... Link to comment Share on other sites More sharing options...
water Posted October 23, 2011 Share Posted October 23, 2011 (edited) You do a string compare in your script. Try this: $job_soldi_magg = 0 For $z = 1 To UBound($job_soldi)-1 If Number($job_soldi[$z]) > $job_soldi_magg Then $job_soldi_magg = Number($job_soldi[$z]) Next MsgBox(0, 'Maggiore finale', $job_soldi_magg) Edited October 23, 2011 by water My UDFs and Tutorials: Spoiler UDFs:Active Directory (NEW 2022-02-19 - Version 1.6.1.0) - Download - General Help & Support - Example Scripts - WikiExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example ScriptsOutlookEX (2021-11-16 - Version 1.7.0.0) - Download - General Help & Support - Example Scripts - WikiOutlookEX_GUI (2021-04-13 - Version 1.4.0.0) - DownloadOutlook Tools (2019-07-22 - Version 0.6.0.0) - Download - General Help & Support - WikiPowerPoint (2021-08-31 - Version 1.5.0.0) - Download - General Help & Support - Example Scripts - WikiTask Scheduler (NEW 2022-07-28 - Version 1.6.0.1) - Download - General Help & Support - Wiki Standard UDFs:Excel - Example Scripts - WikiWord - Wiki Tutorials:ADO - WikiWebDriver - Wiki Link to comment Share on other sites More sharing options...
Floppy Posted October 23, 2011 Author Share Posted October 23, 2011 LOL thanks! 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