s[j] = s[i]；分段错误时试图复制字符

Question

我从某个地方学到了这个程序，并试图理解它。

这一行：s[j++] = s[i];是造成崩溃的原因。我的理解是，至少第一次，程序不应该崩溃，因为j稍后会增加。第一次j和i的值是0。

因此，这将是像s= s；

为什么要撞车？

#include <iostream>

using namespace std;


void squeeze(char a[], char c);

int main()
{
    squeeze("qwiert", 'i');

    return 0;
}

void squeeze(char s[], char c)
{
    int i, j;

    for (i = j = 0; s[i] != '\0'; i++)
    {
        if (s[i] != c)
        {
            std::cout << "\ni: " << s[i];
            s[j++] = s[i];
            std::cout << "\nj: " << s[j];

            std::cout << "\nj : " << j;
            exit(0);
        }
    }

    s[j] = '\0';
}

输出：

i: q

在此之后，程序崩溃。

我已经放置了退出语句，以停止程序的指向，以找到分段故障。

Answer 1

将字符串常量"qwiert"传递给squeeze函数。然后，该函数尝试修改该字符串常量，这是非法的。这会导致核心转储。

要使其工作，您需要传入一个数组：

int main()
{
    char str[] = "qwiert";
    squeeze(str, 'i');

    return 0;
}

Answer 2

其他的答案也指出了这个问题以及如何解决这个问题。

我想指出，您可以通过增加编译器上的警告级别来检测此类错误。

在g++ -Wall中，我得到以下警告消息：

socc.cc: In function ‘int main()’:
socc.cc:10:26: warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
     squeeze("qwiert", 'i');