使用指针修改字符串时发生分段错误？

Question

上下文

我正在学习 C，并且我正在尝试使用指针反转字符串。 （我知道您可以使用数组；这更多的是关于了解指针。）

问题

当尝试运行下面的代码时，我不断遇到分段错误。 GCC 似乎不喜欢 *end = *begin; 行。 这是为什么？

特别是因为我的代码几乎与非邪恶的代码相同C 函数已在另一个问题中讨论过

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

void my_strrev(char* begin){
    char temp;
    char* end;
    end = begin + strlen(begin) - 1;

    while(end>begin){
        temp = *end;
        *end = *begin;
        *begin = temp;
        end--;
        begin++;
    }
}

main(){
    char *string = "foobar";
    my_strrev(string);
    printf("%s", string);
}

Answer 1

一个问题在于传递给函数的参数：

char *string = "foobar";

这是分配在只读部分的静态字符串。当你尝试用它覆盖它时，

*end = *begin;

你会得到段错误。

尝试一下

char string[] = "foobar";

，您应该会注意到差异。

关键点是，在第一种情况下，字符串存在于只读段中，仅使用指向它的指针，而在第二种情况下，在堆栈和静态字符串上保留具有适当大小的字符数组（始终存在）被复制到其中。之后您就可以随意修改数组的内容。

Answer 2

您还可以利用字符串末尾的空字符来交换字符串中的字符，从而避免使用任何额外的空格。这是代码：

#include <stdio.h>

void reverse(char *str){    
    int length=0,i=0;

    while(str[i++]!='\0')
        length++;

    for(i=0;i<length/2;i++){
        str[length]=str[i];
        str[i]=str[length-i-1];
        str[length-i-1]=str[length];
    }

    str[length]='\0';
}

int main(int argc, char *argv[]){

    reverse(argv[1]);

    return 0;
}

Answer 3

在您的代码中，您有以下内容：

*end--;
*begin++;

这只是纯粹的运气，这做了正确的事情（实际上，原因是运算符优先级）。看来您想让代码实际执行

(*end)--;
(*begin)++;

此操作，这是完全错误的。按照您的方式，操作发生时

• 递减 end，然后取消引用它
• 递增 begin，然后取消引用它

在这两种情况下，取消引用都是多余的，应该删除。您可能希望这种行为是

end--;
begin++;

这些让开发人员抓狂的事情，因为它们很难追踪。

Answer 4

这将就位并使用指针

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

 void reve(char *s)
 {
    for(char *end = s + (strlen(s) - 1); end > s ; --end, ++s)
    {
        (*s) ^= (*end);
        (*end) ^= (*s);
        (*s) ^= (*end);
    }
 }

int main(void)
{
    char *c = malloc(sizeof(char *) * 250);
    scanf("%s", c);
    reve(c);
    printf("\nReverse String %s", c);
}

Answer 5

将 char *string = "foobar"; 更改为 char string[] = "foobar";。问题是 char * 指向只读内存，然后您尝试修改该内存，从而导致分段错误。

Answer 6

这构成了一个小型递归函数，其工作原理是在堆栈中存储值，并在返回（返回）时将指针递增到字符串的开头（*s）。

代码看起来很聪明，但在堆栈使用方面却很糟糕。

#include <stdio.h>

char *reverse_r(char val, char *s, char *n)
{
    if (*n)
        s = reverse_r(*n, s, n+1);
   *s = val;
   return s+1;
}

int main(int argc, char *argv[])
{
    char *aString;

    if (argc < 2)
    {
        printf("Usage: RSIP <string>\n");
        return 0;
    }

    aString = argv[1];
    printf("String to reverse: %s\n", aString );

    reverse_r(*aString, aString, aString+1); 
    printf("Reversed String:   %s\n", aString );

    return 0;
}

Answer 7

这是我的就地 C 字符串反转版本。

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

int main (int argc, const char * argv[])
{
    char str[] = "foobar";
    printf("String:%s\n", str);
    int len = (int)strlen(str);
    printf("Lenth of str: %d\n" , len);
    int i = 0, j = len - 1;
    while(i < j){
        char temp = str[i];
        str[i] = str[j];
        str[j] = temp;
        i++;
        j--;
    }

    printf("Reverse of String:%s\n", str);
    return 0;
}

Answer 8

下面，您可以看到我针对此问题的代码：

#include <string>
#include <iostream>

char* strRev(char* str)
{
    char *first,*last;

    if (!str || !*str)
        return str;

    size_t len = strlen(str);
    for (first = str, last = &str[len] - 1; first < last ; first++, last--)
    {
        str[len] = *first;
        *first = *last;
        *last = str[len];
    }
    str[len] = '\0';
    return str;
}

int main()
{
    char test[13] = "A new string";
    std::cout << strRev(test) << std::endl;
    return 0;
}