如何用srand和rand持续生成新数？循环调用函数触发段错误求助

嘿，刚接触C语言遇到这种问题太正常了，我来帮你拆解下代码里的几个关键错误，以及怎么修复：

1. 随机数生成的核心错误（导致段错误的直接原因） #

你的代码里对srand()的使用完全不符合规范：

• 未初始化变量作为种子：randrow和randcol是局部变量，声明后没有赋值，它们的内存里是随机的垃圾值。用这种不确定的值作为srand()的种子，会导致随机数生成器的状态异常，甚至触发内存访问错误（也就是你遇到的Segmentation fault）。
• 重复调用srand()：srand()是用来初始化随机数生成器的，只需要在程序启动时调用一次就够了。每次生成随机数前都调用它，会破坏随机数序列的随机性，甚至因为种子的异常值导致崩溃。

2. 形参传递的逻辑错误 #

你函数里的row和col是值传递的形参，相当于main函数里变量的“副本”。你在函数里给它们赋值，只会修改这个副本，main函数里的原始变量根本不会收到新值——这也是一个隐藏的逻辑bug，即使没有段错误，你的代码也达不到预期效果。

修复后的完整代码 #

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

#define SIZE 4

// 改用指针参数，让函数能修改main里的变量
void generate_player2_move(char board[][SIZE], int *row, int *col) { 
    // 生成0~3的整数（对应4行/列的数组索引）
    *row = rand() % 4;
    *col = rand() % 4;
}

int main() {
    // 只在程序启动时初始化一次随机种子，用当前时间保证每次运行种子不同
    srand((unsigned int)time(NULL));
    
    char board[SIZE][SIZE];
    int row, col;
    
    // 循环调用测试
    for (int i = 0; i < 10; i++) {
        generate_player2_move(board, &row, &col);
        printf("第%d次生成：行=%d，列=%d\n", i+1, row, col);
    }
    
    return 0;
}

关键修复点说明 #

• 把srand()移到main函数开头，用time(NULL)作为种子：这样每次运行程序时，种子都是当前的时间戳，保证生成的随机数序列不会重复。
• 将函数的row和col改为指针参数：通过*row和*col赋值，就能直接修改main函数里的原始变量值。
• 移除了函数里多余的srand()调用和未初始化的变量：彻底消除了导致段错误的根源。

至于为什么第一次循环正常、第二次崩溃：第一次运行时，未初始化的randrow可能刚好是一个不会触发错误的值，但第二次循环时，垃圾值的状态发生了变化，导致srand()或rand()的内部操作触发了非法内存访问，最终出现段错误。

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