将换行分隔字符串转为char**数组时触发段错误

先看你遇到的问题：这段把换行分隔的字符串转成二维字符数组的代码触发了段错误，我帮你拆解下问题所在，再给出修正后的代码。

核心错误原因 #

• 内存分配完全错误：你声明的tab是char**类型（指向指针的指针，也就是二维数组的首地址），但你的malloc语句是malloc(sizeof(char) * height * width)——这只分配了能存height*width个字符的空间，而不是height个指向字符数组的指针空间。当你尝试访问tab[y][x]时，相当于在一段连续的字符空间里当作指针数组来用，必然会出现非法内存访问，触发段错误。
• 字符筛选逻辑错误：你的条件if (str[i] != '\n' || !(str[i]))逻辑搞反了——你本意是跳过换行符，同时不处理字符串末尾的'\0'，但这里用了||，导致当str[i]是'\0'时，!(str[i])为真，代码会执行赋值操作，这时候已经访问到字符串的结束位置之外了，同样会引发内存错误。另外，这个条件会把换行符也误判进去，导致每行的字符数不对。
• 未提前终止循环：如果输入字符串的长度不够height*width个有效字符（不含换行），代码会一直访问str的越界位置，引发未定义行为。

修正后的代码 #

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

char **string_to_tab(char *str, int width, int height) {
    int i = 0;
    int x = 0;
    int y = 0;
    // 第一步：分配存储行指针的数组空间
    char **tab = malloc(sizeof(char*) * height);
    if (!tab) { // 检查内存分配是否成功
        return NULL;
    }

    // 第二步：为每一行分配字符空间
    for (y = 0; y < height; y++) {
        tab[y] = malloc(sizeof(char) * (width + 1)); // 多分配1个位置存字符串结束符（可选，如果你需要每行是C字符串的话）
        if (!tab[y]) {
            // 如果某一行分配失败，要释放之前已经分配的内存，避免内存泄漏
            for (int k = 0; k < y; k++) {
                free(tab[k]);
            }
            free(tab);
            return NULL;
        }
        memset(tab[y], 0, width + 1); // 初始化内存，避免垃圾值
        x = 0;
        while (x < width && str[i] != '\0') {
            if (str[i] != '\n') { // 跳过换行符，只保留有效字符
                tab[y][x] = str[i];
                x++;
            }
            i++;
        }
        // 如果字符串提前结束，剩下的位置可以留空或者处理成你需要的默认值
    }
    return tab;
}

关键修正点说明 #

1. 正确的内存分配：先给tab分配height个char*的空间，再为每一行单独分配width（或width+1）个字符的空间，这样tab[y][x]的访问才是合法的。
2. 修复字符筛选逻辑：用str[i] != '\n'来跳过换行符，同时加上str[i] != '\0'的判断，避免访问越界。
3. 内存分配失败的处理：添加了内存分配失败的检查，并且在部分分配失败时释放已分配的内存，防止内存泄漏。
4. 可选的字符串结束符：如果需要每行是标准的C字符串（可以用printf("%s", tab[y])输出），就多分配1个字节存'\0'，并用memset初始化。

调用示例验证 #

用你给出的测试字符串：

char *str = "+--+\n| |\n| |\n+--+";
char **matrix = string_to_tab(str, 4, 4);

现在matrix的每一行会正确存储对应的字符：

• matrix[0]是"+--+"
• matrix[1]是"| |"
• matrix[2]是"| |"
• matrix[3]是"+--+"

注意：使用完matrix后，记得要手动释放内存，避免泄漏：

for (int y = 0; y < 4; y++) {
    free(matrix[y]);
}
free(matrix);

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