1

我需要我的程序提示用户输入并在输入不遵循信用卡格式（例如：负数或字母等）的情况下重新提示，然后应用算法查看数字是否为有效的信用卡号码，如果是，是 Visa、MasterCard 还是 AmEx。

我知道这个问题已经在这个网站上用不同的代码回答了，我发誓我阅读了我可能找到的所有东西（在这个网站和网络上的其他地方），但我很难理解 C 语法，我想尝试自己想出一些东西，而不是从其他答案中复制一些我不理解的代码。如果有人可以帮助我并看看我到目前为止所做的事情并告诉我我做错了什么，我将非常感激。此外，任何可以帮助我更好地理解 C 语法逻辑的提示将不胜感激。

我的程序正在编译，但是当我运行它时，它以一种非常奇怪的方式运行：当我输入一个输入时，有时它会说它是无效的（即使它是一个有效的数字），有时它在我按下后它不会返回任何东西不管按回车键多少次都不会停止运行。

到目前为止，这是我的代码：


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

int main(void)
{
    printf("Please give me your credit card number:\n") ;

    long long card_num ;

    do
    {
        card_num = GetLongLong() ;
    }
    while (card_num < 1 || card_num > 9999999999999999) ;

    // Make a copy of the card number to be used and modified throughout the process.

    long long temp_num = card_num ;
    int digit = 0 ;
    int count = 0 ;
    int sum_a = 0 ;
    int sum_b = 0 ;

    // Isolate every digit from the credit card number using a loop and the variable 'digit'.
    // Keep track of the amount and position of each digit using variable 'count'.

    while (card_num >= 0)
    {
        digit = card_num % 10 ;
        count++ ;
        temp_num = (card_num - digit) / 10 ;
        break ;


     // Apply Luhn's algorithm using two different 'for' loops depending on the position of each digit.
        
        for (count = 0 ; count % 2 == 0 ; count++)
        {
            sum_a = sum_a + ((card_num % 10) * 2) ;
        
            while ((card_num % 10) * 2 >= 10)
            {
                sum_a = (sum_a % 10) + 1 ;
            } 
        }
        
        for (count = 0 ; count % 2 != 0 ; count++)
        {
            sum_b = sum_b + digit ;
        }
        
        return sum_a ;
        return sum_b ;
        return count ;
    }

    // Checking the validity of the number according to Luhn's algorithm

    int total_sum = sum_a + sum_b ;

    if (total_sum % 10 != 0)
    {
       printf("This is an invalid number.\n") ; 
    }

// If the number entered doesn't have the right amount of digits according
// to variable 'count', declare the number as invalid.

    if (count != 13 || count != 15 || count != 16)
    {
        printf("This is an invalid number.\n") ;
    }

// Reset value of variable 'temp_num' and apply calculations that will isolate the first two digits. 
// Store the results in a variable 'company_id'.

    temp_num = card_num ;
    int company_id ;

    while (temp_num > 100)
    {
        temp_num = card_num - (card_num % 10) ;
        company_id = temp_num / 10 ;
    }

    return company_id ;

// Print the type of credit card depending on the company ID and amount of digits.

    if (company_id > 50 && company_id < 56 && count == 16)
    {
        printf("MASTERCARD\n") ;
    }
    else if ((company_id == 4) && (count == 13 || count == 16))
    {
        printf("VISA\n") ;
    }
    else if ((company_id == 34 || company_id == 37) && (count == 15)) 
    {
        printf("AMEX\n") ;
    }
    else
    {
        printf("This is an invalid number.\n") ;
    }

    return 0 ;

}
4

2 回答 2

2

您的答案是乱序的拼贴，其中的部分在逻辑上与前一个不符。

具体问题：

这个逻辑：

if (count != 13 || count != 15 || count != 16)

使每张卡无效，or s (||) 应该是和s (&&) 才能工作。

这个循环没有意义：

while (card_num >= 0)
{
    digit = card_num % 10 ;
    count++ ;
    temp_num = (card_num - digit) / 10 ;
    break ;
    ...
}

是无条件的break，因此它退出循环并忽略接下来的二十行。

return当您调用五次时，您似乎已经从其他地方拼接了子例程，只有最后一个是有效的：

return sum_a ;
return sum_b ;
return count ;
return company_id ;
return 0 ;

card_num在你应该使用时使用的几个地方temp_num。

一旦您知道该卡无效，您就无法退出程序——而是继续测试。您无法确认卡何时有效。

您计算卡号中的位数，但要等到运行其他检查之后再测试该位数是否有效。

以下是我对您的代码的修改，以解决上述问题和一些样式问题：

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

int main(void)
{
    printf("Please give me your credit card number: ") ;

    long long card_num = 0LL;

    while (card_num < 1LL || card_num > 9999999999999999LL)
    {
        card_num = GetLongLong();
    }

    // Make a copy of the card number to be used and modified throughout the process.

    long long temp_num = card_num;

    // Isolate every digit from the credit card number using a loop and the variable 'digit'.
    // Keep track of the amount and position of each digit using variable 'count'.

    int count = 0;

    while (temp_num > 0LL)
    {
        temp_num = temp_num / 10LL;
        count++;
    }

    // If the number entered doesn't have the right amount of digits according
    // to variable 'count', declare the number as invalid.

    if (count != 13 && count != 15 && count != 16)
    {
        printf("This is an invalid number (# of digits).\n");
        return 1;
    }

    // Reset value of variable 'temp_num' and apply calculations that will isolate the first two digits. 
    // Store the results in a variable 'company_id'.

    temp_num = card_num;

    while (temp_num > 100LL)
    {
        temp_num = temp_num / 10LL;
    }

    int company_id = temp_num;

    // Print the type of credit card depending on the company ID and amount of digits.

    if (company_id > 50 && company_id < 56 && count == 16)
    {
        printf("MASTERCARD\n") ;
    }
    else if ((company_id == 34 || company_id == 37) && (count == 15)) 
    {
        printf("AMEX\n") ;
    }
    else if ((company_id / 10 == 4) && (count == 13 || count == 16 || count == 19))
    {
        printf("VISA\n") ;
    }
    else
    {
        printf("This card was issued by an unknown company.\n");
    }

    // Apply Luhn's algorithm.

    int sum = 0;

    temp_num = card_num;

    for (int i = 1; i <= count; i++)
    {
        int digit = temp_num % 10LL;

        if (i % 2 == 0)
        {
            digit *= 2;

            if (digit > 9)
            {
                digit -= 9;
            }
        }

        sum += digit;

        temp_num /= 10LL;
    }

    // Checking the validity of the number according to Luhn's algorithm

    if (sum % 10 != 0)
    {
        printf("This is an invalid number (Luhn's algorithm).\n");
        return 1; 
    }

    printf("This is a valid number.\n");

    return 0;
}

这不是一个完成的程序——需要进行错误检查和其他细节。当双倍卡号大于 9 时，我没有将数字相加，而是使用了减 9 的更简单方法。

于 2016-10-12T21:55:32.120 回答
2

这里是我的解决方法，这个方法收到的是var long信用卡号，希望对你有帮助。

void check_card(long n)

{
    long temp_n = n;
    int count = 2;
    while(temp_n > 100)
    {
        temp_n = temp_n / 10;
        count ++;
    }

    long temp_n2 = n;
    int sum = 0;

    for (int i = 1; i <= count; i++)
    {
        int digit = temp_n2 % 10;
        if (i%2 == 0)
        {
            if (digit * 2 > 9)
            {
                sum += (digit * 2) - 9;
            }
            else
            {
                sum += digit * 2;
            }
        }
        else
        {
             sum += digit;
        }  
        temp_n2 /= 10;
    }

    bool flag = (sum % 10 == 0) ? true : false;

    if (count == 15 && (temp_n == 34 || temp_n == 37) && flag)
    {
        printf("AMEX\n");
    }
    else if(count == 16 && (temp_n > 50 && temp_n < 56) && flag)
    {
        printf("MASTERCARD\n");
    }
    else if((count == 13 || count == 16) && (temp_n / 10 ==4) && flag)
    {
        printf("VISA\n");
    }
    else
    {
        printf("INVALID\n");
    }
于 2019-09-25T05:47:30.017 回答