Solaris 编译器让我的 C 代码变得很奇怪

Question

我正在尝试完成 C 编程科目的大学作业。

我已经在 MacBook 上完成了我的作业，并且我的代码按照预期完美运行。然而，我们的讲师告诉我们，我们所有的代码都将使用大学的 Solaris 10 Server 进行编译，并且我们被告知，如果您的代码不能在其上编译和运行，则会被扣分。

我们还被告知，我们的应用程序将由具有以下语法的标记进行编译：

gcc -ansi -Wall -pedantic assign1.c -lm -o assign1

并使用以下命令执行：

./assign1

我的代码当前编译时没有错误或警告，并且可以正确执行。然而，我的一个功能在这个 Solaris 机器上无法按预期工作。该函数应该是我们自己的基本字符串排序实现 - 用户输入一个最长 40 个字符的字符串，它应该转换为小写字母并按字母顺序排序，并删除所有非字母字符。

在我的 Mac 上，它输出：

Enter a string (1-40 characters): asdfghjkl
Output: adfghjkls

在学院的 Solaris 机器上，它输出：

Enter a string (1-40 characters): asdfghjkl
Output: aa#?dfghjkls

我执行（大部分）输出的函数是：

void sortLine(int *optionStats, char* source)
{
   char tempsort[MAX_SORT_LENGTH + 1];
   char comp_c;
   int i,j,k;
   char c = source[i++]; 
   i = j = k = 0;
   optionStats[2]++;
   while (c != '\n' && c != '\0' && c != EOF) {
      /* convert uppercase characters to lowercase */
      if ((int)c >= (int)'A' && (int)c <= (int)'Z') 
         c = c + ((int)'a' - (int)'A');
      /* if the character is alphabeic then sort it else skip it */
      if ((int)c <= (int)'z' && (int)c >= (int)'a') {
         for (j = 0; j <= MAX_SORT_LENGTH + 1; j++) {
            comp_c = tempsort[j];
            if (comp_c == '\n' || comp_c == '\0' || comp_c == EOF) {
               tempsort[j] = c;
               break;
            }
            if ((int)c <= (int)comp_c) {
               for (k = MAX_SORT_LENGTH + 1; k > j; k--) {
                  tempsort[k] = tempsort[k - 1];
               }
               tempsort[j] = c;
               break;
            }
         }
      }
      c = source[i++];
   }
   /* copy the sorted temporary array into the source array */
   for (i = 0; i <= MAX_SORT_LENGTH + 1; i++) {
      source[i] = tempsort[i];
   }
}

但是有几个注释：

• 函数定义本身（ 签名）由提供 讲师，所以名称，返回类型， 参数等无法更改。 我们必须按原样使用它（但是我们 可以在里面做任何我们想做的事情）。

• 代码必须符合 ANSI/C90 标准 （grr！）

任何人都可以帮我找出是什么导致这些奇怪的额外字符被从这个函数中吐出来 - 它让我头疼？

Answer 1

int i,j,k;
char c = source[i++]; 
i = j = k = 0;

您在为 i 赋值之前就使用了它。您不能假设自动变量将被初始化为0。

while (c != '\n' && c != '\0' && c != EOF) {

因为 c 是 char，所以它永远不能等于 EOF。不相关，但有一天你会看到这个错误:)你必须使用 int 来实现这个习惯用法：

int c;

while((c=getchar()) != EOF) {
    /* process c */
}

你也在初始化之前使用 tempsort[] ：

        comp_c = tempsort[j];

您正在将大量数据写回到source[]数组中：

for (i = 0; i <= MAX_SORT_LENGTH + 1; i++) {
   source[i] = tempsort[i];
}

我希望source[]数组得到保证 足够大以容纳 MAX_SORT_LENGTH 数据，而不是仅仅大到足以容纳标准 C 字符串。由于这可能是一位友好的教授，所以这可能没问题，但这不是我会轻易做出的假设。

最后一点提示，每一个 (int) 转换都是无用的；编译器知道如何对 char 变量进行比较和算术运算。 :)

Answer 2

不确定这是否是原因，但我注意到您在正确初始化之前使用了 i：

char c = source[i++]; 
i = j = k = 0;

您需要翻转这两行。

Answer 3

也许错误在于没有将本地变量 i 初始化为零。

Answer 4

一些提示：

• 如果您的程序在不同的系统上显示不同的结果，这通常表明它正在使用未初始化的数据，
• 您将“-Wall”（打印所有警告）标志传递给 gcc，这很好，但是您可能想知道，如果您还使用“-O”（优化）标志，则 gcc 可以更好地检测（并警告）未初始化的变量。

---

此外，请仔细查看循环上的边界条件。您确定没有访问超出数组末尾的内存吗？请记住，如果您有一个像 char x[10] 这样的数组，则有效值为 x[0] 到 x[9]； x[10] 超出范围。