-2

我目前正在尝试解决 K+R 书中的练习 2.4，但遇到了一个我无法在其他地方真正重现的奇怪错误。我在用着：

Apple LLVM version 8.0.0 (clang-800.0.42.1)
Target: x86_64-apple-darwin16.4.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

代码是：

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

/*
 * Write an alternate version of `squeeze(s1, s2)' that deletes each 
character
 * in s1 that matches any character in the string s2.
 */

void squeeze(char *s1, const char *s2);

int main(int argc, char **argv) {
  char *tests[] = {"hello", "world", "these", "are", "some", "tests"};
  for (unsigned int i = 0; i < sizeof(tests) / sizeof(tests[0]); i++) {
    printf("'%s' = ", tests[i]);
    squeeze(tests[i], "aeiou");
    printf("'%s'\n", tests[i]);
  }
  return 0;
}

void squeeze(char *s1, const char *s2) {
  const size_t s2len = strlen(s2);
  s1[0] = s1[0];
  unsigned int j = 0;
  for (unsigned int i = 0; s1[i] != '\0'; i++) {
    unsigned int k;
    for (k = 0; k < s2len; k++)
      if (s1[i] == s2[k]) break;
    if (k == s2len)  // we checked every character once, didn't find a bad char
      s1[j++] = s1[i];
  }
  s1[j] = '\0';
}

GDB 说：

Thread 2 received signal SIGBUS, Bus error.
0x0000000100000e57 in squeeze (s1=0x100000f78 "hello", s2=0x100000fa1 
"aeiou")
at exercise2-4.c:23
23    s1[0] = s1[0];

该错误最初发生在s1[j++] = s1[i]，但我插入s1[0] = s1[0]以独立于变量进行测试，并且它也发生在那里。显然，我在这里遗漏了一些东西。

clang -O0 -g -Weverything exercise2-4.c -o exercise2-4如果这有任何相关性，我正在编译。

非常感谢您抽出宝贵的时间，如果这个问题之前已经回答过，我很抱歉，我还没有发现任何问题发生在这样一个奇怪的地方。

4

1 回答 1

1

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

来自 C 标准（6.4.5 字符串文字）

7 如果这些数组的元素具有适当的值，则未指定这些数组是否不同。如果程序尝试修改这样的数组，则行为未定义。

而不是指向字符串文字的指针数组

char *tests[] = {"hello", "world", "these", "are", "some", "tests"};

你应该声明一个二维字符数组。例如

char tests[][6] = {"hello", "world", "these", "are", "some", "tests"};

此外，如果要向其附加新字符，则必须为数组的每个元素保留足够的空间。

于 2017-12-20T15:41:04.463 回答