C语言库函数strstr无法匹配指定字符串问题排查

咱们来一步步拆解你代码里的问题，这几个小细节组合起来就导致了那些奇怪的匹配结果：

1. needle数组初始化的致命错误 #

你写的char needle[4] = "done";是个典型的字符串初始化坑——字符串字面量"done"其实是5个字符：'d'、'o'、'n'、'e'，再加上字符串必须的终止符\0。但你只给数组分配了4个字节的空间，这就导致\0没地方放，needle变成了一个没有终止符的“畸形字符串”。

strstr函数完全依赖\0来判断字符串的边界，没有这个终止符，它会一直读取needle数组后面的随机内存数据，搜索行为彻底失控——这就是为什么有时候能匹配到，有时候完全没反应的核心原因。

修复方案：
要么给数组足够的空间：

char needle[5] = "done"; // 留足存储\0的位置

要么更简单，直接用字符串指针（编译器会自动处理终止符）：

char *needle = "done";

2. 修改argv参数的未定义行为 #

你在代码里直接修改argv[i][j]的内容：argv[i][j] = tolower(argv[i][j]);，这是踩了C语言的一个隐形坑：命令行参数对应的字符串是只读内存。虽然有些编译器不会立刻报错，但这种操作属于「未定义行为」——可能会破坏字符串的终止符，甚至触发内存访问错误。

比如你输入单个参数"done"时，修改这个只读字符串的内容，很可能会覆盖掉它的\0，让strstr根本识别不出这是一个合法的字符串；而多参数场景下，第一个参数的内存布局更敏感，修改后直接导致匹配失效，后面的参数碰巧没触发这个问题而已。

修复方案：
先把argv[i]的内容复制到一个可修改的缓冲区里，再做转小写操作：

// 分配足够的内存存储参数内容（+1是给\0留位置）
char *temp_str = malloc(strlen(argv[i]) + 1);
if (temp_str == NULL) {
    perror("内存分配失败");
    return 1;
}
strcpy(temp_str, argv[i]);

// 转小写操作在temp_str上进行
int length = strlen(temp_str);
for (j = length - 1; j >= 0; j--) {
    temp_str[j] = tolower((unsigned char)temp_str[j]);
}

// 用temp_str来执行搜索
char *pch = strstr(temp_str, needle);
if (pch) {
    printf("DONE!");
}

// 用完记得释放内存
free(temp_str);

（另外，给tolower传unsigned char是为了避免处理ASCII码之外的字符时出现未定义行为，算是个好习惯）

3. 额外小提示 #

你的倒序遍历转小写其实没必要，正序遍历也能实现同样的功能，不过这不是导致匹配异常的原因，只是个小优化点。

为什么不同测试场景结果不一样？ #

• 输入hi there done时，第三个参数的内存位置刚好让needle的随机内存数据没干扰到匹配，所以能正常输出；
• 输入单个done时，修改只读内存+needle无终止符的双重问题，直接让strstr失效；
• 第一个参数的done无法匹配，也是因为修改只读内存导致字符串结构被破坏，加上needle的问题，搜索彻底失败。

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