比较字符串

Question

谢谢！现在工作完美。 Java 让我变得愚蠢:(

我在比较 C 中的字符串时遇到了一些困难。当我不使用 isMorse 函数时，我得到正确的输出，但是当我使用它时，输出变得不准确并显示随机字符。据我所知告诉我，当调用 strcmp 时，变量“morse”实际上发生了变化，我认为这与“morse”不是常量有关，但我不确定如何补救，

谢谢！

char *EnglishToMorse(char english)
{
   static char *morse;

   int i;
   for (i = 0; i < LOOKUP_SIZE; i++)
   {
      if (lookup[i].character == english)
      {
         morse = lookup[i].morse;
         return morse;
      }
   }

   morse = &english;  // Problem was here!!!
   return morse;
}

Answer 1

我有一点猜测。函数EnglishToMorse()可能会从堆栈返回一个指向内存的指针。如果是这样，在 EnglishToMorse() 之后运行另一个函数将改变该内存。这可能是由于 EnglishToMorse() 中的错误所致——声明了 char 的本地数组并返回了指向它的指针。

如果没有看到EnglishToMorse()的代码，这只是在黑暗中刺伤。您可以向我们提供更多代码供我们查看并获胜。

Answer 2

您在 EnglishToMorse 中有一个静态变量，但它是错误的。莫尔斯电码不需要是静态的——您只需将其返回即可。但你确实需要 english 是静态的——而不是在堆栈上——因为你返回了它的地址。此外，它必须是一个以 NUL 结尾的字符串。执行类似

char *EnglishToMorse(char english)
{
   static char save_english[2]; /* initialized to 0's */ 

   int i;
   for (i = 0; i < LOOKUP_SIZE; i++)
      if (lookup[i].character == english)
         return lookup[i].morse;

   save_english[0] = english;
   return save_english;
}

“注意”的操作，但是，EnglishToMorse 的调用者必须使用结果或在再次调用 EnglishToMorse 之前保存它，因为第二次调用可能会覆盖 static_english。

Answer 3

morse 变量出现变化的原因是它指向堆栈上的一个区域。它指向堆栈上的一个区域的原因是因为您为其分配了参数english的地址，该地址在您调用函数时被压入堆栈，然后在函数完成后从堆栈中弹出。

现在，您的莫尔斯变量将指向堆栈上相同位置的任何内存，该位置将在程序的整个生命周期中不断变化。

在我看来，解决此问题的最佳方法是如果字符不是 AZ，则从 EnglishToMorse 返回 NULL 指针...然后检查 isMorse 中的 NULL 指针代码>函数。毕竟，检查代码中的 NULL 指针是一个很好的做法。

char* EnglishToMorse(char english)
{
    int i;

    english = toupper(english);
    for (i = 0; i < LOOKUP_SIZE; i++)
    {
        if (lookup[i].character == english)
            return lookup[i].morse;
    }

    return NULL;
}

int isMorse(char* morse)
{
    int i;

    /* Check for NULL, so strcmp doesn't fail. */
    if (morse == NULL) return 0;

    for (i = 0; i < LOOKUP_SIZE; i++)
    {
        if(strcmp(morse, lookup[i].morse) == 0) 
            return 1;
    }

    return 0;
}

Answer 4

看起来问题可能出在这个函数中：

char *EnglishToMorse(char english) {
    static char *morse;
    // ...
    morse = &english;
    return morse;
}

您正在返回传递到函数中的参数 (english) 的地址。该参数在函数返回后（并且在调用者有机会实际看到该值之前）不再存在。看起来好像您试图通过将 morse 变量声明为静态来解决此问题，但这只会使 morse 变量本身静态，而不是无论它指向什么。

此外，C 中的字符串必须以 NUL 字符终止。通过返回指向单个字符（如english）的指针，无法保证内存中的下一个字节是或不是NUL字符。因此，期望看到以 NUL 结尾的字符串的调用者可能会得到比他们预想的更多的结果。

Answer 5

char *EnglishToMorse(char english)
和
莫尔斯电码 = &english;
是问题所在。

永远不应该返回指向局部变量或函数参数的指针。