function 总是返回真

Question

我在编写一个 function 时遇到了麻烦，它使用 Luhn 的算法检查信用卡是否有效（从倒数第二个开始，每隔一个数字乘以 2；加上未乘以 2 的数字之和；如果总数的最后一位为 0，则该数字有效）。

当我开始编写代码时，我检查了这些行：

long number2 = ((card % x) / y)\*2;
long number1 = ((card % (x/10)) / (y/10));

他们似乎工作正常。 当我现在再次尝试检查时，它会忽略以下几行：

printf("%li", number1);

printf("%i", sum)

function 始终返回 true。 另外，我可以两次声明可变digits吗？

#include <stdio.h>
#include <cs50.h>

bool luhns_algorithm (long, int);
int number_of_digits(long);

int main (void)
{
  int digits = 0;//if i remove this then on line 15 appears mistakes
  long card = get_long("credit card: ");

  number_of_digits(card);

  luhns_algorithm(card, digits);//use of undeclared identifier 'digits'; variable 'digits' is uninitilazed when used here

}


int number_of_digits(long card)
{
  int digits = 0;//cant be used in do while loop if undeclared and uninitialized so I need to write it twice
  do
  {
    digits++;
    card = card/10;
  } while(card != 0);

  printf("%i\n", digits);
  return digits;
}


bool luhns_algorithm (long card, int digits)//always valid for some reason
{
   int sum = 0;
   long x = 100;
   long y = 10;
   for (int i = 0; i < digits/2; i++)
   {
     long number2 = (card % x)/y;
     long number1 = (card % (x/10))/(y/10);

     printf("%li\n", number1);//doesn't work
     printf("%li\n", number2);//doesn't work

     sum = sum + number2 + number1;

     printf("%i\n", sum);//doesn't work

     x = x*100;
     y= y*100;
  }

  if((sum/10) == 0)
  {
    printf("VAlID\n");
    return true;
  }
  printf("INVALID\n");
  return false;
}

Answer 1

function number_of_digits的结果被丢弃。 这意味着main()中的digits始终为零。 您需要将结果存储为digits ：

#include <stdio.h>
#include <cs50.h>

bool luhns_algorithm (long, int);
int number_of_digits(long);

int main (void)
{
  int digits;
  long card = get_long("credit card: ");

  digits = number_of_digits(card); //FIXED

  luhns_algorithm(card, digits);

}

正如您所注意到的，如果digits被单元化，则会出现错误。 在 function 中声明的单元化变量具有未知值，如果您在赋值之前尝试使用它们，编译器将显示警告。

---

另外，我可以两次声明变量digit吗？

在 C 中，与大多数语言一样，变量具有作用域。 每个变量仅在其自身的 scope 及以下可见。 因为您的两位digits变量在不同的范围内，所以它们在其他 scope 中不可见。您的main() digits完全独立于您的number_of_digits() digits 。 有关更多信息，请参阅有关变量 scope 的解释。

另请注意，您实际上声明了 3 次。 function 参数的行为完全像一个变量，唯一的区别是它是用调用 function 时传递给参数的值初始化的。