指针字符串未返回预期输出

Question

 #include <stdio.h>
    
    void revstr(char str[])
    {
        char temp;
        int size = 0;
        while(*str != '\0'){
            size++;
            str++;
        }
        for (int i = 0; i < size/2; i++)        
        {
            temp = str[i];               
            str[i] = str[size-1-i];
            str[size-1-i] = temp;
        }
        for(int i = 0; i < size; i++)
        {
            printf("%c\n", *(str+i));
        }
    }
    int main()
    {
        char str[20];
        printf("enter a string: \n");
        scanf("%s", &str);
        revstr(str);
        return 0;
    }

为什么我的 rev 字符串没有打印出它正在打印出一些垃圾值的字符串的反向。 你能指出原因吗？

Answer 1

在您的代码中，您推进指针 str 以计算其长度，但之后您忘记将其重置为开头。此外，您对 scanf 的调用应该传递 str，而不是其地址。此外，如果用户输入的字符串长度超过 19 个字符，scanf 将超出 str 的末尾。试试这个：

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

void revstr(char str[])
{
    char temp;
    int size = strlen(str);
    for (int i = 0; i < size / 2; i++) {
        temp = str[i];
        str[i] = str[size - 1 - i];
        str[size - 1 - i] = temp;
    }
}

int main()
{
    char str[20];
    printf("enter a string: \n");
    fgets(str, sizeof(str), stdin);
    str[strcspn(str, "\n")] = '\0';
    revstr(str);
    printf("%s\n", str);
    return 0;
}

Answer 2

我会以另一种方式实现它：

char *reverse(char *str)
{
    char *wrk = str, *end = str;

    if(str && *str)
    {
        while(*(end + 1)) end++;
        while(end > wrk)
        {
            char tmp = *end;
            *end-- = *wrk;
            *wrk++ = tmp;
        }   
    }
    return str;
}

1. 最好检查一下参数是否正确。
2. 我会返回值。它允许您在表达式中使用反转字符串并作为函数参数。示例如下：

int main(void)
{
    char s[] = "Hello world";

    printf("%s\n", reverse(s));
}

https://godbolt.org/z/fbo4nsn38

Answer 3

在这个while循环之后

    while(*str != '\0'){
        size++;
        str++;
    }

指针str不指向字符串的开头。

相反，你可以写例如

    while( str[size] != '\0'){
        size++;
    }

尽管如此，这样的函数应该只做一件事：反转字符串。是函数的调用者决定是否输出反转的字符串。

所以函数看起来像

char * revstr( char s[] )
{
    size_t n = 0;

    while ( s[n] ) ++n;

    for ( size_t i = 0; i < n / 2; i++ )
    {
        char c = s[i];
        s[i] = s[n - i - 1];
        s[n - i - 1] = c;
    }

    return s; 
}

主要你可以写

puts( revstr( str ) );

这是一个演示程序。

#include <stdio.h>

char * revstr( char s[] )
{
    size_t n = 0;

    while ( s[n] ) ++n;

    for ( size_t i = 0; i < n / 2; i++ )
    {
        char c = s[i];
        s[i] = s[n - i - 1];
        s[n - i - 1] = c;
    }

    return s; 
}

int main(void) 
{
    char s[] = "Hello";
    
    puts( s );
    puts( revstr( s ) );
    
    return 0;
}

程序输出是

Hello
olleH

当然，您可以使用标准字符串函数 strlen 来代替 while 循环。

size_t n = strlen( s );

在scanf的调用中注意

scanf("%s", &str);

第二个参数应该是表达式str

scanf("%s", str);