我正在尝试用C语言实现Luhn算法来检查信用卡的有效性，对于那些不知道的人... 就是这个：
从数字的倒数第二位开始，每隔一个数字乘以2，然后将这些乘积的数字相加。
将上述求和结果加上所有未被乘以2的数字的总和。
如果总和的个位数为0（或更正式地说，如果总和模10等于0），则该数字是有效的！
为了实现这一点，我循环遍历整个数字，如果我所处的数字位置具有模2等于0，则将其乘以2并加到名为totalEven的变量中。
如果不是这种情况，我会将我所在的数字添加到totalOdd中，而不进行乘法运算。
然后我会递增位置一，并检查其他数字，直到达到16（卡片的最大数字）。
稍后我会添加两个变量并检查总数模10是否等于0。 如果等于0，则表示信用卡号码正确，否则是错误的。
以下是代码：
#include <stdio.h>
#include <cs50.h>
//list of variables
//is the card valid
bool isValid = true;
// the creditcard number
long input;
//mod stands for modules, and is used to single out each number as seen later
int mod = 10;
//the location at which number I am checking
int place = 1;
//num is the number I am checking that has been singled out
int num = 0;
//total of numbers * 2 located at locations numbered with even numbers
int totalEven = 0;
//total of numbers located at locations numbered with odd numbers
int totalOdd = 0;
//gets input and stores it in well.. input
input = get_long("Number: ");
// a formula to single out a number, starting with the ones and then as you can see, mod is muliplied by 10 to go over the second number.
num = ((input % mod) - (input % (mod /10))) / (mod/10);
//loops 16 times
for(int i = 0; i < 16; i++)
{
// if the place is even execute below
if(place % 2 == 0)
{
totalEven = totalEven + num * 2;
}
//else do this
else if (place % 2 != 0)
{
totalOdd = totalOdd + num;
}
//moves to the next number
mod = mod * 10;
place++;
}
//fufils the last step of the algorithm
if((totalEven + totalOdd) % 10 == 0 )
{
isValid = true;
}
else
{
isValid = false;
}
问题是即使信用卡号码正确，这个代码块也会给我无效或!isValid的错误提示，而且我已经检查了我的“公式”，它仍然能够很好地工作...
我完全不知道该怎么办...我只是一个谦卑的业余爱好者，请不要因为上面的畸形代码而嘲笑我。
这里是完整版本的代码
#include <stdio.h>
#include <cs50.h>
long power();
int main(void)
{
//AMERX 15 STRT 34 OR 37
//MC 16 STRT 51, 52, 53, 54, 55
//VZA 13 OR 16 STRT 4
long input;
bool isValid = true;
string type;
int mod = 10;
int place = 1;
int num = 0;
int totalEven = 0;
int totalOdd = 0;
do
{
input = get_long("Number: ");
}
while(input < 0);
for(int i = 0; i < 16; i++)
{
num = ((input % mod) - (input % (mod /10))) / (mod/10);
if(place % 2 == 0)
{
totalEven = totalEven + num * 2;
}
else
{
totalOdd = totalOdd + num;
}
mod = mod * 10;
place++;
}
if((totalEven + totalOdd) % 10 == 0 )
{
isValid = true;
}
else
{
isValid = false;
printf("%i , %i", totalEven, totalOdd);
}
if (isValid == true){
if((input < (38 * power(10, 13)) && input >=(37 * power(10, 13))) || (input < (35 * power(10,13)) && input >= (34 * power(10, 13))))
{
type = "AMEX\n";
}
else if(input >= (51 * power(10, 14)) && input < (56 * power(10, 14)))
{
type = "MASTERCARD\n";
}
else if((input < (5 * power(10, 12)) && input >= (4 * power(10, 12))) || (input < (5 * power(10, 15)) && input >= (4 * power(10, 15))))
{
type = "VISA\n";
}
else{
type = "error\n";
}
}
else
{
type = "INVALID\n";
}
if((totalEven + totalOdd) % 10 == 0 )
{
isValid = true;
}
else
{
isValid = false;
}
printf("%s", type);
}
long power(int n, int p)
{
long result = 1;
for(int i = 0; i<p; i++)
{
result = result * n;
}
return result;
[cs50] luhn'。 - Jonathan Lefflermod/10就是 10/10，也就是 1，所以我怀疑num = ((input % mod) - (input % (mod /10))) / (mod/10);中有错误。虽然速度较慢，但我同意字符串操作会更容易理解。 - erik258long类型是否足够大，能够容纳16位数字？尝试使用#include <limits.h>和printf("%ld\n", LONG_MAX);。 - pmg