strcmp 返回错误值

Question

有人能解释一下为什么strcmp 会返回相同的值，即使密码正确/不正确？我在包含部分下方定义了有效密码，并在程序结束时输入了一个来检查它。

这是我的代码：

#include <stdio.h>
#include <signal.h>
#include <string.h>
#include <unistd.h>
#include <stdlib.h>
#include <termios.h>

#define TIME 10
#define MAXPASSWORD 12
#define PASSWORD "pass123"

void sigalrm_handler() {
    printf("\nERR: Time is up...\n");
}

int getch() {
    struct termios oldtc, newtc;
    int ch;
    tcgetattr(STDIN_FILENO, &oldtc);
    newtc = oldtc;
    newtc.c_lflag &= ~(ICANON | ECHO);
    tcsetattr(STDIN_FILENO, TCSANOW, &newtc);
    ch=getchar();
    tcsetattr(STDIN_FILENO, TCSANOW, &oldtc);
    return ch;
}

int main(int argc, char * argv[]) {
    char password[MAXPASSWORD] = {0};
    printf("Enter correct password. You have %d seconds: ", TIME);
    signal(SIGALRM, sigalrm_handler);
    alarm(TIME);
    fflush(stdout);

    for(int i=0; i<MAXPASSWORD; i++)
    {
      password[i] = getch();
      if (password[i] == '\n')
        break;
      printf("*");
    }

    if (strcmp(password, PASSWORD) == 0) {
        printf("\nValid password\n");
    } else {
        printf("\nInvalid password\n");
    }
}

Answer 1

问题是您将\n 存储到输入缓冲区中，因此pass123 和pass123\n 将不匹配。

因此，如果您发现\n，则null 终止输入，如下所示。

  if (password[i] == '\n')
  {
     password[i] = '\0';
     break;
  }

Answer 2

您忘记将 \n 替换为 NUL

...
for(int i=0; i<MAXPASSWORD; i++)
{
  password[i] = getch();
  if (password[i] == '\n')
  {
    password[i] = 0;   // <<<< add this line
    break;
  }
  printf("*");
}
...

还有另一个问题：考虑一下如果用户在点击 Enter 之前输入了超过 11 个字符会发生什么。我让你发现自己作为一个练习。

Answer 3

您已经说过strcmp() 表示两个字符串 相等，但当您知道它们不相等时...

其他答案中提到的\n 的问题没有得到解决，如果您确实看到strcmp() 返回错误的指示，那么问题就变成了为什么？

在 C 中，string 定义为以null 结尾的字符数组。因此，例如，如果您有以下情况：

char password[MAXPASSWORD] = {0};//where MAXPASSWORD == 12

|p|a|s|s|1|2|3|\0|\0|\0|\0|\0| // `PASSWORD ` - legal string
|s|o|m|e|p|a|s|s|1|2|3|4|\n|  // 'password' 

即使替换了\n 字符，这个数组还是太长了一个字符：

|s|o|m|e|p|a|s|s|1|2|3|4|\0|  // 'password' - too long by 1 character
                        ^     // end of legal definition of `password`

如果password 数组的字符太多，即使在这种情况下，在将最后一个char \n 替换为字符串合法定义的位置中的NULL 之后，代码成为受制于undefined behavior。

字符串函数专门用于strings。当出现非空终止字符数组时，函数（在本例中为 strcmp()），因为它正在寻找 nul 终止符以知道字符串的结尾在哪里，因此不能期望其行为具有可预测性。 （在这种情况下，nul 字符的位置将是未定义行为的原因。）

为防止这种情况发生，即使用户能够在password 中输入太多字符，也始终以如下语句终止：

password[MAXPASSWORD-1] = 0;  //for a properly initialized array, (as your code indicates)
                              //this guarantees termination occurs 
                              //within legal memory area of defined variable. 

这样，就不会出现未定义的行为，如果字符串不同，strcmp() 会指出这一点。