-1

我正在尝试根据一组规则制作一个程序来确定密码的有效性。

这是我所拥有的:

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#include <math.h>

int main()
{
  int uppercase = 0;
  int length = 0;
  int numbers = 0;
  int others = 0;
  char password[13];
  char yesOrNo;
  printf("Your password must be 8-12 characters long.\n"
         "It must contain at least one symbol, uppercase letter, and number.\n\n");
 COMEBACK:
  printf("Please enter your password:");
  scanf(" %s", &password);
  while (password != 'NULL') { // Tried 0 here, tried '\0', but to no avail.
    if (isalpha(password)) {
      length += 1;
      if (isupper(password)) {
        uppercase += 1;
      }
    }
    else if (isdigit(password)) {
      numbers += 1;
      length += 1;
    }
    else {
      length += 1;
    }
    // This is just a test, to see if it was working.
    printf("%d - %d - %d - %d --- %s",
           uppercase, length, numbers, others, password);
  }
  if ((uppercase > 0) && (numbers > 0)
      && (length >= 8) && (length <= 12) && (others > 0)) {
    printf("Good job, you've done your password correctly.");
  } else {
    printf("%d - %d - %d - %d --- %s \t Incorrect..",
           uppercase, length, numbers, others, password); // Same thing here.
    scanf("%s", &yesOrNo);
    switch (yesOrNo) {
    case 'y':
      goto COMEBACK;
      break;
    case 'n':
      printf("Sorry you're dumb man..");
      break;
    default:
      printf("Please enter a valid password.");
    }
  }
  return 0;
}

我遇到的问题是,while 循环永远不会结束,因为它似乎无法为我的密码数组找到终止符。我输入了“\0”以及“0”。但我还是想不通。任何帮助表示赞赏。谢谢。

4

2 回答 2

5

这段代码:

while (password != 'NULL') { 

应该生成大量编译器警告。多字符文字是不可移植的,不应与指针进行比较。

您可能需要:

char *ptr = password;
while (*ptr != '\0') {
    ...
    ptr++;
}

或(C99 或更高版本):

for (char *ptr = password; *ptr != '\0'; ptr++)
{
    ...
}

并用于*ptr识别字符(或者,通常,(unsigned char)*ptr因为 plainchar通常是有符号的,并且isalpha()et al 需要一个正值或 EOF 作为输入值)。如果你没有C99,你可以char *ptr;在循环外声明并移除char *循环内控制。

你有:

if (isalpha(password)) {

由于password是一个数组,因此您将一个固定指针传递给一个需要非指针 ( int) 值的函数。我可能会在循环中添加:

{
    unsigned char uc = *ptr;
    if (isalpha(uc))
        ...

请注意,length++;对于所有情况,您可能只需要一个。

另请注意,任何密码都不会满足“至少一个符号”标准,因为您永远不会增加others.

并且goto可以用一个while()也可以检测 EOF 的循环替换:

while (scanf("%12s", password) == 1)
{
    length = others = uppercase = numbers = 0;  // Ignore previous attempts
    for (char *ptr = password; *ptr != '\0'; ptr++)
    {
        unsigned char uc = *ptr;
        length++;
        ...character testing code...
    }
    ...validity checking code...
}

在学习 C 时,假设编译器警告是可靠的错误。它比你在这个阶段更了解 C。

于 2015-01-22T23:15:04.210 回答
-1

password是一个数组。它永远不会NULL。但它可以包含一个空终止符,这可能是你所追求的。只需检查是否password[0] == '\0'.

于 2015-01-22T23:15:05.687 回答