在指针指向的字符串数组中查找字符串的C语言代码警告求助

我来帮你梳理下这段代码里的问题，这些问题正是导致警告和运行异常的核心原因：

存在的核心问题 #

• 缺少必要头文件：你使用了strcmp函数，但没有包含<string.h>头文件，编译器会报「隐式声明函数strcmp」的警告。
• 未分配内存：char (*ptr)[50];是一个指向长度为50的char数组的指针，但你没给它分配内存空间，直接用ptr[i]访问会触发非法内存访问，属于严重的未定义行为。
• 错误使用printf输出strcmp结果：strcmp返回整数（0表示两字符串相等，负数/正数表示字符串大小关系），直接把这个整数传给printf既不符合参数要求，也没法输出你想要的查找结果。
• 冗余头文件：<malloc.h>在标准C中已经被<stdlib.h>包含，不需要单独引入。

修正后的代码 #

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

int main() {
    int n, i;
    char (*ptr)[50]; // 指向长度为50的char数组的指针
    char search[50];
    int found = 0; // 标记是否找到目标字符串

    printf("How many students?\n");
    // 读取n后处理输入合法性，避免后续读取字符串出错
    if (scanf("%d", &n) != 1 || n <= 0) {
        printf("Invalid input!\n");
        return 1;
    }
    // 分配内存：n个长度为50的char数组
    ptr = malloc(n * sizeof(*ptr));
    if (ptr == NULL) {
        printf("Memory allocation failed!\n");
        return 1;
    }

    // 读取学生姓名
    for (i = 0; i < n; i++) {
        printf("Enter the name of student %d:\n", i + 1);
        // 限制49个字符防止缓冲区溢出
        scanf("%49s", ptr[i]);
    }

    // 读取要查找的姓名
    printf("Enter a name to search:\n");
    scanf("%49s", search);

    // 查找并输出结果
    for (i = 0; i < n; i++) {
        if (strcmp(ptr[i], search) == 0) {
            printf("Found '%s' at index %d\n", search, i);
            found = 1;
            // 如果只需要找第一个匹配项，可在这里加break;
        }
    }

    if (!found) {
        printf("'%s' not found in the list\n", search);
    }

    // 释放动态分配的内存，避免内存泄漏
    free(ptr);
    return 0;
}

关键修改说明 #

1. 添加<string.h>头文件：让编译器识别strcmp函数，消除隐式声明的警告。
2. 内存分配与检查：用malloc为ptr分配对应大小的内存，同时检查分配结果，避免内存分配失败导致的异常。
3. 正确处理strcmp结果：通过判断返回值是否为0来确定字符串相等，输出清晰的查找提示。
4. 内存释放：使用完动态内存后用free释放，避免内存泄漏。
5. 输入合法性检查：添加对输入n的判断，确保是有效正整数，避免后续逻辑出错。

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