[MATH-1307] Create a base class for all RNGs

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Won't Fix
• Labels: api inheritance
• Assignee: Gilles Sadowski
• Reporter: Gilles Sadowski
• Created: 26/Dec/15 17:34
• Updated: 04/Feb/16 01:32
• Resolved: 04/Feb/16 01:32

Description

I proposed to create a base class which the existing abstract classes AbstractRandomGenerator and BitsStreamGenerator will extend.

This would allow to define nextBytes(byte[]) at the base class level.

The code for that method is almost identical in the two hierarchies: they only differ in a call to either nextInt() or next(32) respectively; the latter is however the same as the former, in disguise, and is not subject to change given the type of return value.

As a corollary, the new base class can be the unique place where to add utilities such as the one proposed in MATH-1306.

Update: AbstractRandomGenerator and BitsStreamGenerator are both obsoleted by the class proposed in this report.

Issue Links

• is depended upon by: MATH-1306 Add public void nextBytes(byte[] bytes, int position, int length) method into the RandomGenerator interface
• is superceded by: MATH-1319 Refactor the package "o.a.c.m.random"
• relates to: MATH-1308 Deprecate and remove "AbstractRandomGenerator"