Sign in to follow this  
Followers 0
oasis375

If Min and Max are the same value then Random() will return 0. What?

3 posts in this topic

https://www.autoitscript.com/autoit3/docs/functions/Random.htm
If Min and Max are the same value then Random() will return 0.
Conceptually, a random function (using integers) is a choice between options. For example, "pick a number between 1 and 10". Random(1,10,1)
If we have only two options, we can still choose. Random(1,2,1). Finally, if our options are reduced to only one, and we must choose, the only solution is to select that single option.
But the AutoIt Random() function simply says that a single option is wrong, and sets the @error flag to non-zero. If someone offers you to drink a) coffee or B) coffee, and you must choose, the answer isn't "I don't know".

I coded a script that made an error before I found out this behavior. I'm going to put an example.

#include <Array.au3>
;Vending machine available beverages
$aDrinks[5] = ["coffee", "water", "juice", "beer", "tea"]
For $n = 1 To 5
    $qty = UBound($aDrinks)
    $r = Random(1, $qty, 1)
    ;If $r = 0 Then $r = 1 ;fix
    ConsoleWrite("I choose: " & $aDrinks[$r - 1] & @CRLF)
    _ArrayDelete($aDrinks, $r - 1)
Next

This machine has a shrinking catalog. The only solution is to use the commented fix.

Share this post


Link to post
Share on other sites



oasis375,

 

The only solution is to use the commented fix.

Since July this year (Beta version 3.3.13.10) Random has been modified and if both Max & Min are the same it will now return that value. :)

M23


Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______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

 

Share this post


Link to post
Share on other sites

That's what I call a quick fix. :) 

1 person likes this

Share this post


Link to post
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
Sign in to follow this  
Followers 0