C#掷骰子程序生成重复随机数修复及返回全部掷骰结果问询
问题根源
Random类默认以系统时钟为种子初始化,你在RullaTärning方法内每次调用都新建Random实例,循环调用间隔远小于系统时钟更新精度,导致所有实例种子相同,生成的随机数完全一致。- 你已经在Main方法中创建了全局的
Random实例并作为参数传入RullaTärning,但方法内完全没有使用这个传入的对象,属于逻辑浪费。 - 额外问题:
Random.Next(min, max)的第二个参数是开区间,原代码写Next(1, 6)只会生成1-5的整数,永远无法掷出6点。 - 平均值计算用整数除法会截断小数部分,结果精度不准。
修复方案
第一步:修改RullaTärning方法
删除方法内新建Random的代码,直接使用传入的全局实例,同时修正随机数范围:
static int RullaTärning(Random slumpObjekt) { int slumpa = slumpObjekt.Next(1, 7); return slumpa; }
第二步:(可选)优化平均值计算逻辑
将case 2中的平均值计算改为支持小数的版本,修改后代码:
case 2: if (tärningar.Count <= 0) { Console.WriteLine("\n\tDet finns inga sparade tärningsrull! "); } else { double medelvärde = tärningar.Average(); Console.WriteLine("\n\tRullade tärningar: "); foreach (int tärning in tärningar) { Console.WriteLine("\t" + tärning); } Console.WriteLine("\n\tMedelvärdet på alla tärningsrull: " + medelvärde.ToString("F1")); } break;
修复后所有掷骰结果都会正常生成不同的随机值,全部保存在tärningar列表中,选择菜单2即可查看所有结果和平均值。
内容的提问来源于stack exchange,提问作者Strand
相关产品推荐
相关产品推荐

