Question

过去36个小时，我一直在研究cs50 pset1积分。我的代码没有产生任何错误，但是当我输入卡的详细信息时，它显示无效，这可能是问题所在。我是菜鸟，请帮忙。

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

 int main (void)
{
    long long ccnumber;

     do 
     {
         //Prompting the user for his creditcard number
         ccnumber=get_long_long("What is your creditcard number\n");
     }
     while (ccnumber<= 0);

     //validating the credit card number to ascertain if its upto 13. 14 or 16
    long long numbers = ccnumber;
     int totaldigits = 0; 

     while (numbers > 0)
     {
         numbers = numbers/10;
         totaldigits++;
     }
     if(totaldigits != 13 || totaldigits !=14 || totaldigits !=16)
     {
         printf("Invalid\n");
     }

     //Starting from the second to last digit
     //And multiplying through by 2
     int number1 = ((ccnumber/10) % 10) *2;
     int number2 = ((ccnumber/1000) % 10) *2;
     int number3 = ((ccnumber/100000) % 10) *2;
     int number4 = ((ccnumber/10000000) % 10) *2;
     int number5 = ((ccnumber/1000000000) % 10) *2;
     int number6 = ((ccnumber/100000000000) % 10) *2;
     int number7 = ((ccnumber/10000000000000) % 10) *2;
     int number8 = ((ccnumber/10000000000000000) % 10) *2;

     //Getting the Sum of the first number series
     int sum1;
         sum1 = (number1 + number2 + number3 + number4 + number5 + number6 + number7 + number8);

     //Populating the second number series
     int number9 = ccnumber % 10;
     int number10 = ((ccnumber/100) % 10);
     int number11 = ((ccnumber/10000) % 10);
     int number12 = ((ccnumber/1000000) % 10);
     int number13 = ((ccnumber/100000000) % 10);
     int number14 = ((ccnumber/10000000000) % 10);
     int number15 = ((ccnumber/1000000000000) % 10);
     int number16 = ((ccnumber/100000000000000) % 10);

    int sum2;
        sum2 =(number9 + number10 + number11 + number12 + number13 + number14 + number15 + number16);

     //Adding sum1 and sum2
     int checksum;
         checksum = sum1 + sum2;
     if (checksum % 10 != 0)
     {
         printf("invalid/n");
      }

     //Verifying for visacard(1)
     if(totaldigits == 13)
     {
         printf("Visa\n");
     }

     // Verifying for visa(2)
     if(totaldigits == 16)
     {
         long long visa;
         visa = ccnumber/1000000000000000;
       if(visa!=4)
       {
           printf("Invalid\n");
       } 
         else 
         {
             printf("VISA\n");
         }
         //Verifying for MasterCard
         long long master;
         master = ccnumber/100000000000000;
         if(master!=51 || master!=52 ||master!=53 ||master!=54 ||master!=55)
         {
             printf("Invalid\n");
         }
         else
         {
             printf("MASTER\n");
         }
     }

       //Verifying for Amex
     }

/计数器您的信用卡号是多少 2221000000000009 无效无效/无效无效

Answer 1

首先，您的某些if条件是错误的。例如，

if(totaldigits != 13 || totaldigits != 14 || totaldigits != 16)

您要检查总数是否为13或14或16，但是在这里检查的是如果总数不是13，则打印无效，或者如果总数不是14，打印无效，依此类推。因此，它将始终打印无效。您需要使用&& (AND)而不是|| (OR)。

此行中也存在相同的问题。

if(master!=51 || master!=52 ||master!=53 ||master!=54 ||master!=55)

另一个问题是，您在此行中将1个不必要的0分配给了除法，因此它总是产生0。

int number8 = ((ccnumber/10000000000000000) % 10) *2; // 1 zero unnecessary

如注释中所建议，您可能需要考虑创建一个用于存储单个数字的数组，而不是使用16个不同的整数。当您有一个数组时，可以在for循环中进行除法，以免混淆很多零。例如，您可以参考此链接 A solution for cs50's credit card validation problem 。

为什么我总是收到无效的反馈？

1 个答案: