search

apjones6 / DiceRoller

RPG Dice Roller App for Windows Phone 8
1 stars 0 forks source link

The pool result is not correctly random #8

Closed apjones6 closed 11 years ago

apjones6 commented 11 years ago

A new Random object is created each PoolResult, so the seeds are very similar and it does not accumulate a proper distribution.

Also, it may be worth saving the random object until the application is properly closed, so that frequently restoring will not also compromise random distribution

apjones6 commented 11 years ago

It seems that it probably was random, and it's just misleading how statistics tends towards the center distribution. Not done any work for restoring random instance when deactivated or tombstoned, as very likely unnecessary. New instance each app start is sufficient.