尝试反转字符串时发生分段错误[关闭]

Question

所以我有这个函数应该反转一个字符串然后返回它

char *my_revstr(char *str)
{
    int i = 0;
    int j = 0;
    int k = 0;

    while(str[j] != '\0') {
        j++;
    }
    j--;
    while(i < j) {
        k = str[i];
        str[i] = str[j];
        str[j] = k;
        i++;
        j--;
    }
    return (str);
}

但是每当我尝试运行它时，我都会遇到分段错误，老实说，我不太清楚为什么。我希望有人能帮我解决这个问题^^。

Answer 1

我确信你有一个分段错误，因为向函数传递了类似的字符串文字

my_revstr( "Hello" );

您不能更改字符串文字。任何更改字符串文字的尝试都会导致未定义的行为。

你应该这样写

char s[] = "Hello";
my_revstr( s );

注意变量 i 和 j 应该是 size_t 类型，因为 int 类型可能不足以存储字符串的大小。

函数可以定义如下，如下面的演示程序所示，

#include <stdio.h>

char * my_revstr( char *s )
{
    size_t n = 0;

    while ( s[n] != '\0' ) ++n;

    if ( n != 0 )
    {
        for ( size_t i = 0; i < --n; ++i )
        {
            char c = s[n];
            s[n] = s[i];
            s[i] = c;
        }
    }

    return s;
}

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

    return 0;
}

程序输出是

Hello
olleH

函数也可以定义如下方式

char * my_revstr( char *s )
{
    size_t n = 0;

    while ( s[n] != '\0' ) ++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;
}

或者您可以使用指针来实现该功能。例如

char * my_revstr( char *s )
{
    char *first = s, *last = s;

    while ( *last ) ++last;

    if ( first != last )
    {
        for ( ; first < --last; ++first )
        {
            char c = *first;
            *first = *last;
            *last = c;
        }
    }

    return s;
}