C语言解析行返回动态字符串数组的实现问题求解

你代码跑不通和内部字符串指针失效没关系，堆上通过malloc/calloc分配的内存在主动free之前会一直有效，真正的问题有三个：

• C语言函数参数是值传递，你在函数内部执行words = malloc(81*sizeof(char*));修改的只是words参数的局部副本，函数返回后这个副本直接销毁，外部传入的指针根本拿不到你分配的数组地址，等于白分配内存。
• 跳空格逻辑存在越界bug：while (isspace(*p++));没有提前判断字符串结束位置，当指针走到末尾\0时依然会先解引用再自增，直接访问非法内存触发异常。
• 内存逻辑冗余且泄漏：每次给单词缓冲区分配和整行等长的内存完全没必要，逐字符调用strncat拼接效率极低；循环退出前还多分配了一次未使用的单词缓冲区，没有释放造成内存泄漏，也没有处理行首、行尾、连续空格的边界场景。

两种常用实现可以按需选择：

方案1：遵循原有设计，外部传入预分配的指针数组 #

这种方式不需要在函数内分配指针数组内存，调用方提前准备足够长度的char*数组即可，适配你已知单行最大长度81字符的场景。

#include <ctype.h>
#include <string.h>
#include <stdlib.h>

// 注意：调用方需要保证传入的words数组长度足够存下所有单词指针+末尾NULL标记
void parseLine(char *line, char **words){
    char *p = line;
    int wordIndex = 0;
    while (*p != '\0'){
        // 先判结束再跳空格，避免越界，强转unsigned char避免isspace触发未定义行为
        while (*p != '\0' && isspace((unsigned char)*p)){
            p++;
        }
        if (*p == '\0') break; // 跳过末尾空格后直接结束，不生成空单词
        
        // 标记单词起止位置，直接计算长度，不用逐字符拼接
        char *wordStart = p;
        while (*p != '\0' && !isspace((unsigned char)*p)){
            p++;
        }
        int wordLen = p - wordStart;
        // 只分配当前单词需要的内存，不浪费空间
        char *currentWord = (char*)calloc(wordLen + 1, sizeof(char));
        memcpy(currentWord, wordStart, wordLen);
        words[wordIndex++] = currentWord;
    }
    words[wordIndex] = NULL; // 末尾置空，方便调用方遍历判断结束
}

调用示例：

int main(){
    char testLine[] = "test1 test2 test3";
    char *words[81]; // 预分配足够指针位，81字符长度的行最多生成41个单词，冗余量足够
    parseLine(testLine, words);
    
    // 遍历使用单词
    for(int i=0; words[i] != NULL; i++){
        printf("%s\n", words[i]);
        free(words[i]); // 用完逐个释放单词内存
    }
    return 0;
}

方案2：函数内部分配指针数组，直接返回给调用方 #

如果不想让调用方提前准备数组，可以让函数直接返回分配好的数组指针：

#include <ctype.h>
#include <string.h>
#include <stdlib.h>

char** parseLine(char *line){
    // 分配指针数组，预留足够冗余
    char **words = (char**)malloc(81 * sizeof(char*));
    char *p = line;
    int wordIndex = 0;
    while (*p != '\0'){
        while (*p != '\0' && isspace((unsigned char)*p)) p++;
        if (*p == '\0') break;
        
        char *wordStart = p;
        while (*p != '\0' && !isspace((unsigned char)*p)) p++;
        int wordLen = p - wordStart;
        char *currentWord = (char*)calloc(wordLen + 1, sizeof(char));
        memcpy(currentWord, wordStart, wordLen);
        words[wordIndex++] = currentWord;
    }
    words[wordIndex] = NULL;
    return words;
}

调用示例：

int main(){
    char testLine[] = "test1 test2 test3";
    char **words = parseLine(testLine);
    
    for(int i=0; words[i] != NULL; i++){
        printf("%s\n", words[i]);
        free(words[i]); // 先释放每个单词的内存
    }
    free(words); // 再释放指针数组本身的内存
    return 0;
}

• 调用isspace类字符判断函数时，一定要把参数强转为unsigned char，否则当char为有符号类型、字符值为负时会触发未定义行为。
• 所有堆上分配的内存，用完必须对应free，避免内存泄漏。
• 处理字符串时优先通过指针偏移计算起止位置、用memcpy拷贝，比逐字符拼接效率高很多，也不容易出现缓冲区溢出问题。

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