19 Jun
1993
19 Jun
'93
9:11 a.m.
A pseudorandom number generator recently proposed here, namely:
int rand1(int seedval) { return (seed * 183041 % 183319 + 1); }
needs some cleaning up. It should be something like:
unsigned long rand1(unsigned long n)
{ return ( ( ( n * 183041L) % 183319 ) + 1 ); }
where n is initally set to some seed value. However, this is
particularly weak, and quickly degenerates into a cycle,
usually of length 208, as the following program will confirm:
#include