C#生成不重复随机字母数组时程序冻结,求原因分析
C#程序冻结问题分析
作业需求
- 创建一个5×5的
string[,]数组用于存储字母 - 遍历数组,为每个单元格存入不重复的随机字母
- 使用嵌套
for循环输出该二维数组 - 通过
while循环让用户重复输入坐标(x,y)和字母修改对应位置内容:仅当字母未在数组中存在时才替换,否则提示用户;每次修改后输出数组,输入-1作为x或y时退出程序
用户代码
string[,] grid = new string[5,5]; Random rnd = new Random(); char randomChar = (char)rnd.Next('a','z'); int check = 0; int leave = 0; for (int x = 0; x < 5; x++){ for (int y = 0; y < 5; y++){ grid[x,y] = randomChar.ToString(); for (int i = 0; i < 5; i++){ for (int j = 0; j < 5; j++){ if (grid[i,j] == grid[x,y]){ check = 1; } } } if (check == 1){ y--; check = 0; } } } Console.WriteLine("Input a -1 when asked for row or column to exit program."); while(leave == 0){ for (int x = 0; x < 5; x++){ for (int y = 0; y < 5; y++){ Console.WriteLine(grid[x,y]); } } Console.WriteLine("Enter what row you want to alter: "); int row = Convert.ToInt32(Console.ReadLine()); Console.WriteLine("Enter what column you want to alter: "); int column = Convert.ToInt32(Console.ReadLine()); if (row == -1 || column == -1){ leave = 1; } Console.WriteLine("Enter what letter you want to enter: "); string input = (Console.ReadLine()).ToLower(); for (int x = 0; x < 5; x++){ for (int y = 0; y < 5; y++){ if (input == grid[x,y]){ Console.WriteLine("This letter already exists in the grid!"); } else{ grid[row,column] = input; } } } }
冻结原因分析
1. 随机字母生成逻辑错误导致死循环
你只在初始化阶段生成了一次随机字母randomChar,所有单元格都会被赋值同一个字母。当第一个单元格赋值后,后续单元格检查重复时,check必然被设为1,触发y--,但因为没有重新生成新的随机字母,下一次循环还是会给当前单元格赋值同一个字母,再次触发重复检查和y--,陷入无限循环,程序卡在数组初始化阶段无法继续。
2. 重复检查逻辑错误
初始化时的重复检查遍历了整个5×5数组,包括未赋值的单元格(默认值为null),但你已经给当前grid[x,y]赋值,所以只要数组中存在任何一个和它相同的元素(包括自身),就会触发check=1,直接导致y--,即使是第一个单元格,也会因为和自身相等而进入死循环。
3. 其他潜在逻辑问题(虽非当前冻结原因,但需修复)
- 用户输入-1触发退出后,代码仍会执行后续的输入字母逻辑,可能引发索引越界错误
- 修改数组时的嵌套循环逻辑错误:只要有一个单元格和输入字母不同,就会执行
grid[row,column] = input,导致同一输入被重复赋值;若数组中存在目标字母,提示信息会被打印多次
修复关键点
- 将随机字母生成代码移到内层循环中,确保每次赋值前都生成新的随机字母
- 检查重复时,仅遍历已经赋值过的单元格(而非整个数组),避免和未赋值的
null或自身比较 - 触发退出逻辑后,使用
break提前终止当前循环,跳过后续输入操作 - 修改数组时,先遍历整个数组完成重复检查,再根据结果决定是否替换,避免重复赋值和多次提示
内容的提问来源于stack exchange,提问作者Aiden Scheben
相关产品推荐
相关产品推荐

