首先，你的代码触发Segmentation Fault（段错误）的核心原因是函数内部错误地重新定义了length指针并初始化为空指针，导致解引用空指针，同时还有其他逻辑问题需要修正。下面一步步拆解问题并给出修复方案：

1. 导致段错误的直接原因 #

在longestStrInAr函数的for循环内部，你写了：

int max = 0, *length = 0;

这里的*length = 0是重新定义了一个局部的length指针变量，并且把它初始化为空指针（NULL）。当你后续执行if (max > *length)时，就是在对空指针进行解引用操作——这是C语言中绝对禁止的行为，直接触发了段错误。

更糟的是，这个局部变量完全覆盖了函数参数传入的length指针，彻底偏离了需求中通过指针返回最长字符串长度的设计。

2. 其他需要修正的逻辑问题 #

• 未初始化指针p：函数里的char *p;没有初始值，如果所有字符串长度都不满足max > *length（虽然题目规定size>0，但逻辑上仍存在野指针风险），返回p时会引发未定义行为。
• 手动计数字符串长度冗余且易出错：没必要自己写循环计数，直接用标准库的strlen函数更简洁可靠。
• 初始值设置逻辑混乱：没有先将第一个字符串作为默认最长项，后续比较逻辑的起点不清晰。

修复后的完整代码 #

#include <stdio.h>
#include <string.h>
#define N 20
char *longestStrInAr(char str[N][40], int size, int *length);

int main() {
    int i, size, length;
    char str[N][40], *result;
    char dummychar;
    printf("Enter array size: \n");
    scanf("%d", &size);
    scanf("%c", &dummychar); // 吃掉输入size后的换行符
    for (i = 0; i < size; i++) {
        printf("Enter string %d: \n", i + 1);
        fgets(str[i], 40, stdin);
        char *p = strchr(str[i], '\n');
        if (p != NULL)
            *p = '\0'; // 去除fgets读取的换行符
    }
    result = longestStrInAr(str, size, &length);
    printf("longest: %s \nlength: %d\n", result, length);
    return 0;
}

char *longestStrInAr(char str[N][40], int size, int *length) {
    // 初始化最长字符串为数组第一个元素，长度同步初始化
    char *longest = str[0];
    *length = strlen(str[0]);
    
    // 从第二个元素开始逐个比较
    for (int i = 1; i < size; i++) {
        int current_len = strlen(str[i]);
        // 仅当当前字符串更长时，更新最长项和长度
        if (current_len > *length) {
            *length = current_len;
            longest = str[i];
        }
        // 长度相等时保持最先出现的，无需额外处理
    }
    return longest;
}

修复要点说明 #

• 移除了函数内部重新定义的length指针，直接使用传入的参数指针存储长度，避免空指针解引用。
• 初始化longest指针为数组第一个字符串，同时将*length设为第一个字符串的长度，彻底规避野指针问题。
• 用strlen替代手动循环计数，减少代码冗余和出错概率。
• 从第二个元素开始遍历比较，严格符合“返回最先出现的最长字符串”的需求。

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