Question

我写了以下代码片段，但它无法正常工作。谁能帮我？我哪里错了？

char *xstrrev(char *string)
{
    int len=0,i=0;
    char *reverse;
    while(*string!='\0')
    {
        len++;
        string++;
    }
    len=len-1;
    while(len>=0)
    {
      *reverse=string[len];
       reverse++;
       len--;
    }
    *reverse='\0';
    return reverse; 
}

int main()
{
    char name[10];
    scanf("%s",name);
    printf("%s",xstrrev(name)); 
    return 0;
}

我无法在主函数

中返回整个字符串

Answer 1

问题以

开始（并结束）

  *reverse=string[len];

其中reverse未初始化。这会调用undefined behavior。

您需要初始化reverse以使其指向有效内存位置，然后才能取消引用指针。

由于您希望从函数中_return_新的_string_并在调用者中使用它，您应该使用内存分配器函数（如malloc()）来分配内存并初始化reverse在成功检查malloc()调用之后返回的指针。完成使用后，您还需要处理free()分配的内存。

之后，根据您的逻辑，您正在进行reverse++;，最后，您将返回reverse，因此请考虑返回的确切值。您将返回指向字符串 end 的指针，而不是指向 start 的指针。您需要保留reverse的实际开始的副本并返回。

Answer 2

三个问题：

[1] reverse未指向有效的内存区域。

[2]返回reverse将返回字符串reverse的最后一个字节地址。所以需要存储reverse的开头并返回reverse的开头。

[3] string在计算长度时被完全解析，因此，原始字符串的开始需要保存并随后使用

char *xstrrev(char *string)
{
    char *original_string = string;
    int len=0,i=0;
    //[1] char *reverse;
    while(*string!='\0')
    {
        len++;
        string++;
    }

    //[3] till here "string" is completely parsed, so it points to past the end.

    char *reverse = (char*)malloc(len + 1);
    char *reverse_to_be_returned = reverse;
    len=len-1;
    while(len>=0)
    {
        //[3] *reverse=string[len];
        *reverse = original_string[len]; 
        reverse++;
        len--;
    }
    *reverse='\0';
    //[2] return reverse; 
    return reverse_to_be_returned;
}

Answer 3

您需要为char *reverse分配空间以指向。这可以通过malloc或strdup来完成。

基本上：

malloc()在堆上分配请求的内存，最后返回一个void*指针。

您还需要检查对malloc()的任何调用的返回指针，因为如果发生错误，有时它可以返回NULL。

为了安全起见，您还需要free最后请求的内存。

此外，由于您要返回一个指针，您需要确保它指向字符串的开头而不是结尾。

您的代码可能如下所示：

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

#define NAMESTRLEN 10

char *xstrrev(char *string);

int
main(void) {
    char name[NAMESTRLEN+1];
    char *result;

    printf("Enter string: ");
    scanf("%10s", name);

    result = xstrrev(name);
    printf("Reversed string = %s\n", result);

    free(result);

    return 0;
}

char
*xstrrev(char *string) {
    char *result;
    size_t slen, count = 0;
    int i;

    slen = strlen(string);

    result = malloc(slen+1); /* +1 for nullbyte */
    if (!result) {
        printf("Cannot allocate space for string.\n");
        exit(EXIT_FAILURE);
    }

    for (i = slen-1; i >= 0; i--) {
        result[count++] = string[i];
    }
    result[count] = '\0';

    return result;
}

使用Function反转字符串并将整个字符串返回到main函数

3 个答案: