9

我正在尝试反转一个字符串。

这是我试过的代码:

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

int main(){
    char *c="I am a good boy";
    printf("\n The input string is : %s\n",c);
    printf("\n The length of the string is : %d\n",strlen(c));
    int i,j;
    char temp;
    int len=strlen(c);
    for(i=0,j=len-1;i<=j;i++,j--)
    {
            temp=c[i];
            c[i]=c[j];
            c[j]=temp;
    //printf("%c\t%c\n",*(c+i),*(c+(len-i-1)));
    }
    printf("\n reversed string is : %s\n\n",c);
}

代码输出一个Bus error : 10.

但是如果我重写相同的代码:

int main(void)
{
    char *str;
    str="I am a good boy";
    int i,j;
    char temp;
    int len=strlen(str);
    char *ptr=NULL;
    ptr=malloc(sizeof(char)*(len));
    ptr=strcpy(ptr,str);
    for (i=0, j=len-1; i<=j; i++, j--)
    {
        temp=ptr[i];
        ptr[i]=ptr[j];
        ptr[j]=temp;
    }
    printf("The reverse of the string is : %s\n",ptr);
}

它工作得很好。

为什么第一个代码会抛出总线错误或分段错误?

4

5 回答 5

16

发生总线错误是因为在许多(如果不是大多数或所有现代)C 编译器中,字符串文字被分配在只读内存中。

您正在将字符串反转到位。在您的第一个代码片段中,您尝试写入字符串文字。不是一个好主意。

在第二种情况下,您 malloc 了一个将其放在堆上的字符串。现在可以安全地将该字符串反转到位。

附录

对于询问段错误与总线错误的评论者,这是一个很好的问题。我都见过。这是mac上的总线错误:

$ cat bus.c 
char* s = "abc"; int main() {s[0]='d'; return 0;}

$ gcc --version bus.c && ./a.out
i686-apple-darwin10-gcc-4.2.1 (GCC) 4.2.1 (Apple Inc. build 5659)
Copyright (C) 2007 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Bus error

在其他操作系统/编译器上,您确实可能会遇到段错误。

于 2011-09-18T01:05:54.170 回答
6

将其复制到堆中是一种选择。但是,如果您只想分配一个本地(堆栈)数组,您可以这样做:

char str[] = "I am a good boy";

然后,常量字符串将被复制到堆栈中。

于 2011-09-18T01:15:51.173 回答
1

以 形式指定的字符数组"I am a good boy"通常是常量 - 您不能修改它们。这就是您的第一个变体崩溃的原因。第二个没有,因为您制作了数据副本然后对其进行修改。

于 2011-09-18T01:07:33.893 回答
0

char *str = "我是个好孩子"; 视为文字并尝试对其进行修改将导致总线错误。相当于 const char *str = "I am a good boy",即指向常量字符串的指针,不允许尝试修改常量字符串。

编辑:当您使用 malloc() 并复制时,您正在使用原始字符串的副本,而 ptr 不是“const char *”类型,而是“char *ptr”并且不会抱怨。

于 2011-09-18T15:16:45.330 回答
0

使用 c++ (g++) 编译显示了将字符串文字分配给非常量 char* 的弃用,该非const char* 旨在防止此错误:

ed@bad-horse:~/udlit_debug$ g++ ../buserr.cpp 
../buserr.cpp: In function ‘int main()’:
../buserr.cpp:5:13: warning: deprecated conversion from string constant to ‘char*’
../buserr.cpp:7:61: warning: format ‘%d’ expects type ‘int’, but argument 2 has type ‘size_t’

相关警告位于第 5 行。

如所示将声明更改为 const char * 可防止分配到文字字符串中。

这也是为什么你不应该忽略警告的教训。

于 2011-09-24T02:37:35.070 回答