c - 在 C 中实现 strrchr 时复制字符串的问题

Question

对于家庭作业，我必须在 string.h 中实现 22 个函数。我正在努力实现 strrchr，但在尝试复制生成的匹配字符串时遇到了一些问题。

这是我目前尝试实现它的尝试:

char *new_strrchr(const char *str, int c) {
    const char *str_copy = str;
    int i, index = -1;
    char *result;
    for(i = 0; *str_copy != '\0'; ++str_copy, ++i){
        if(*str_copy == c) {
            index = i;
        }
    }
    if(index == -1) {
        return NULL;
    }
    while(index < strlen(str))
    {
        *result = str[index];
        result++;
        index++;
    }
    return result;
}

出于某种原因，result 发生了一些奇怪的行为。我将举例说明。

char str1[] = "abcHdefHello World";
char search = 'H';
char *result;
result = new_strrchr(str1, search);
printf("Result: %s\n", result);

这应该打印出 Hello World，因为它跳过了初始的“H”。但是，它会打印出 HHdefHello World。

最初，我认为这与我如何实现查找最后一个字符的实际算法有关，但当我这样做时 printf("%c",str[index]); ，在 while 循环中，我的输出最终是 Hello World。所以，我可以肯定地说我在正确的索引处，并且我正在输出正确的字符。

我猜它与 result 有关，但我不知道它可能是什么。我什至尝试使用 *result = 0 在末尾添加一个空字符，但最终只是删除了整个字符串。

Answer 1

你有undefined behavior在您的代码中:变量 result 未初始化，但您使用了它。

未初始化的局部(非静态)变量的值不确定，实际上看起来是随机的。这意味着您不知道 result 指向何处。取消引用并添加到它是导致未定义行为的原因。

我认为你需要read more about the actual strrchr function因为它不进行复制，而只是返回指向传递的字符串中最后一个匹配字符的指针。