[jira] [Commented] (OFBIZ-11020) Use Random.nextDouble() instead of Math.random()

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-11020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17028091#comment-17028091 ]

Michael Brohl commented on OFBIZ-11020:
---------------------------------------

Hi [~bd2019us] ,

thanks for your contribution.

Your patch does not apply directly for current trunk. Also, is there a reason why you replaces Math.random only partially?

See CommonEvents:
{code:java}
            for (int i = 0; i < circlesToDraw; i++) {
                Random rand = new Random();
                int circleRadius = (int) (rand.nextDouble() * height / 2.0);
                int circleX = (int) (Math.random() * width - circleRadius);
                int circleY = (int) (Math.random() * height - circleRadius);
                g.drawOval(circleX, circleY, circleRadius * 2, circleRadius * 2);
            }
{code}

> Use Random.nextDouble() instead of Math.random()
> ------------------------------------------------
>
>                 Key: OFBIZ-11020
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-11020
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>            Reporter: bd2019us
>            Assignee: Michael Brohl
>            Priority: Trivial
>         Attachments: OFBIZ-11020.patch
>
>
> When using Math.random instead of Random.nextDouble, there is a slight performance cost. Due to Math.random invoking an instance of Random This cost can be alleviated by directly using Random.nextDouble. This also allows for more control over the randomization in the future.

--
This message was sent by Atlassian Jira
(v8.3.4#803005)