Skip to content

Bikeshed the name #33

@tabatkins

Description

@tabatkins

#31 points out, correctly, that since the function is a constructor, it needs to start with a capital letter. Let's open up the naming discussion more broadly, tho.

The current name (seededPRNG) came from @erights's complaint during a plenary that we should be clear this is a PRNG, not "true" randomness. (See #3). However, Math.SeededPRNG() is annoying to type; notably, the four letters of PRNG alternate between hands on a qwerty keyboard, so you have to constantly press and release alternating Shift keys, which is annoying and easy to make a mistake with. Even the initial capital S is on the opposite hand from the capital P, making the alternation even worse.

Other suggestions I find reasonable:

  • just Math.PRNG(). Still has the alternating-hands problem, but at least it's six characters shorter and loses one capital.
  • Math.SeededRandom(). Less capitals, and the notion of "seeded" already necessarily implies a PRNG.
  • something else?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions