Random number generation has flaws

Random generators should be initialised once per thread, and should not be reinitialised within functions. Seeding with the current time will give the same sequences if seeding takes place at pretty much the same time. Re-entrant versions should be used, and are not. Better random generators (faster as well as more random) are available, notably PCG http://www.pcg-random.org/