C语言生成每行无重复元素随机二维数组的代码问题排查

原代码核心错误 #

• 对比逻辑完全颠倒：你还没有给matrix[i][j]赋值，该位置是栈内存的随机脏值，用脏值和新生成的随机数对比没有任何意义。且你遍历了整行7个元素，实际上只需要对比当前行已经赋值完成的前j个元素即可。
• 没有重试机制：检测到重复后你直接跳过了当前位置的赋值，j会正常自增进入下一个位置的处理，最终会导致该行多个位置残留未初始化的脏值。
• 缺少随机数种子初始化：rand()函数如果不初始化种子，每次运行生成的随机序列都是固定的。

修复后完整代码 #

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

#define MAX_ROWS 5
#define NUM_PER_ROW 7

int doStuff(int count) 
{
    int matrix[MAX_ROWS][NUM_PER_ROW];
    int num=0;
    // 初始化随机数种子，全局已经初始化过的话可以删掉这行
    srand((unsigned int)time(NULL));

    // 填充用户输入的行
    for(int i=0; i < count; i++)
    {
        for(int j=0; j < NUM_PER_ROW; j++) 
        {
            printf(" > ");
            scanf("%d", &num);
            matrix[i][j] = num;
        }
    }

    // 自动填充剩余行
    for(int i = count; i < MAX_ROWS; i++) 
    {
        for(int j = 0; j < NUM_PER_ROW; j++) 
        {  
            int flag = 0;
            // 循环生成随机数直到拿到当前行不存在的值
            do {
                flag = 0;
                num = rand() % 40 + 1;
                // 仅对比当前行已经填好的前j个元素
                for(int k=0; k < j; k++) 
                {
                    if(matrix[i][k] == num) 
                    {
                        flag = 1;
                        break;
                    }
                }
            } while(flag);
            
            // 拿到有效值再赋值打印
            matrix[i][j] = num;
            printf("%5d ", num);
        }
        printf(" 第%d行生成完成\n", i+1);
    }
    return 0;
}

修改点说明 #

• 用do-while循环包裹随机数生成和重复校验逻辑，只要检测到重复就重新生成新的随机数，保证每个位置都能拿到有效值
• 重复校验的遍历范围改为0 ~ j-1，只对比当前行已经赋值完成的元素，判断逻辑正确
• 补充了随机数种子初始化，保证每次运行生成的随机矩阵不同
• 调整了赋值和打印的顺序，保证打印的内容和存入矩阵的内容一致

内容的提问来源于stack exchange，提问作者laurimikhael