c - この k&r チャプター 2 の例は間違っていますか?

Question

/* squeeze: delete all c from s */
void squeeze(char s[], int c)
{
  int i, j;
  for (i = j = 0; s[i] != '\0'; i++)
    if (s[i] != c)
      s[j++] = s[i];
  s[j] = '\0';
}

int main(void)
{
  squeeze("squeeze", 'z');
  return 0;
}

gcc でコンパイルして実行したところ、セグメンテーション違反が発生しました。この例で何か問題がありますか?

男性のおかげで、私はいつもの間違いを犯しました。

Answer 1

あなたの例はsqueeze()、文字列リテラル ( "squueze") に適用しようとしていることを示しています。これは正しくありません。文字列リテラルは常に変更可能であるとは限らないため、変更しようとしても無効です。文字配列で呼び出す必要があります。

#include <stdlib.h>

int main(void)
{
   char test[] = "squeeze";

   squeeze(test, 'z');
   return EXIT_SUCCESS;
}