Actualmente estoy trabajando en el 'Comprobador de tarjetas de crédito' de codeacademy usando el algoritmo de Luhn, sin embargo, mi código devuelve falso en matrices válidas (que deberían devolverse). ¿Podría ayudarme a resolver el problema?
// All valid credit card numbers const valid1 = [4, 5, 3, 9, 6, 7, 7, 9, 0, 8, 0, 1, 6, 8, 0, 8]; const valid2 = [5, 5, 3, 5, 7, 6, 6, 7, 6, 8, 7, 5, 1, 4, 3, 9]; const valid3 = [3, 7, 1, 6, 1, 2, 0, 1, 9, 9, 8, 5, 2, 3, 6]; const valid4 = [6, 0, 1, 1, 1, 4, 4, 3, 4, 0, 6, 8, 2, 9, 0, 5]; const valid5 = [4, 5, 3, 9, 4, 0, 4, 9, 6, 7, 8, 6, 9, 6, 6, 6]; // All invalid credit card numbers const invalid1 = [4, 5, 3, 2, 7, 7, 8, 7, 7, 1, 0, 9, 1, 7, 9, 5]; const invalid2 = [5, 7, 9, 5, 5, 9, 3, 3, 9, 2, 1, 3, 4, 6, 4, 3]; const invalid3 = [3, 7, 5, 7, 9, 6, 0, 8, 4, 4, 5, 9, 9, 1, 4]; const invalid4 = [6, 0, 1, 1, 1, 2, 7, 9, 6, 1, 7, 7, 7, 9, 3, 5]; const invalid5 = [5, 3, 8, 2, 0, 1, 9, 7, 7, 2, 8, 8, 3, 8, 5, 4]; // Can be either valid or invalid const mystery1 = [3, 4, 4, 8, 0, 1, 9, 6, 8, 3, 0, 5, 4, 1, 4]; const mystery2 = [5, 4, 6, 6, 1, 0, 0, 8, 6, 1, 6, 2, 0, 2, 3, 9]; const mystery3 = [6, 0, 1, 1, 3, 7, 7, 0, 2, 0, 9, 6, 2, 6, 5, 6, 2, 0, 3]; const mystery4 = [4, 9, 2, 9, 8, 7, 7, 1, 6, 9, 2, 1, 7, 0, 9, 3]; const mystery5 = [4, 9, 1, 3, 5, 4, 0, 4, 6, 3, 0, 7, 2, 5, 2, 3]; // An array of all the arrays above const batch = [valid1, valid2, valid3, valid4, valid5, invalid1, invalid2, invalid3, invalid4, invalid5, mystery1, mystery2, mystery3, mystery4, mystery5]; // Add your functions below: const validateCred = arr => { let totalSum = 0; let revList = arr.reverse(); for (let i = 0; i < revList.length; i++) { let calcAmount = revList[i]; if (i !== 0 && i%2 === 0) { calcAmount = revList[i] * 2; if (calcAmount > 9 ) { calcAmount -= 9; totalSum += calcAmount; } else { totalSum += calcAmount; } } else { totalSum += revList[i]; } } return (totalSum%10 === 0 ? true : false); }; console.log(validateCred(valid3)) console.log(validateCred(valid4)) console.log(validateCred(valid5))Reescribir y arreglar.
// All valid credit card numbers const valid1 = [4, 5, 3, 9, 6, 7, 7, 9, 0, 8, 0, 1, 6, 8, 0, 8]; const valid2 = [5, 5, 3, 5, 7, 6, 6, 7, 6, 8, 7, 5, 1, 4, 3, 9]; const valid3 = [3, 7, 1, 6, 1, 2, 0, 1, 9, 9, 8, 5, 2, 3, 6]; const valid4 = [6, 0, 1, 1, 1, 4, 4, 3, 4, 0, 6, 8, 2, 9, 0, 5]; const valid5 = [4, 5, 3, 9, 4, 0, 4, 9, 6, 7, 8, 6, 9, 6, 6, 6]; // All invalid credit card numbers const invalid1 = [4, 5, 3, 2, 7, 7, 8, 7, 7, 1, 0, 9, 1, 7, 9, 5]; const invalid2 = [5, 7, 9, 5, 5, 9, 3, 3, 9, 2, 1, 3, 4, 6, 4, 3]; const invalid3 = [3, 7, 5, 7, 9, 6, 0, 8, 4, 4, 5, 9, 9, 1, 4]; const invalid4 = [6, 0, 1, 1, 1, 2, 7, 9, 6, 1, 7, 7, 7, 9, 3, 5]; const invalid5 = [5, 3, 8, 2, 0, 1, 9, 7, 7, 2, 8, 8, 3, 8, 5, 4]; // Can be either valid or invalid const mystery1 = [3, 4, 4, 8, 0, 1, 9, 6, 8, 3, 0, 5, 4, 1, 4]; const mystery2 = [5, 4, 6, 6, 1, 0, 0, 8, 6, 1, 6, 2, 0, 2, 3, 9]; const mystery3 = [6, 0, 1, 1, 3, 7, 7, 0, 2, 0, 9, 6, 2, 6, 5, 6, 2, 0, 3]; const mystery4 = [4, 9, 2, 9, 8, 7, 7, 1, 6, 9, 2, 1, 7, 0, 9, 3]; const mystery5 = [4, 9, 1, 3, 5, 4, 0, 4, 6, 3, 0, 7, 2, 5, 2, 3]; // An array of all the arrays above const batch = [valid1, valid2, valid3, valid4, valid5, invalid1, invalid2, invalid3, invalid4, invalid5, mystery1, mystery2, mystery3, mystery4, mystery5]; function luhnCheck(num) { let digit, j, len, odd = true, sum = 0 const digits = (num + '').split('').reverse() for (j = 0, len = digits.length; j < len; j++) { digit = parseInt(digits[j], 10) if ((odd = !odd)) digit *= 2 if (digit > 9) digit -= 9 sum += digit } return sum % 10 === 0 } batch.forEach((item) => { let number = item.join('') console.log(number, luhnCheck(number)) })Idealmente, debe pasarlo como una cadena y dejar que la función realice la división (separación de preocupaciones) como en una aplicación real, desea hacer varias cosas en el número ingresado antes de ejecutar luhn como ... cardTypeFromNumber (visa, maestro, dankort , mastercard, amex, dinersclub, discover, chinaunionpay (no usa luhn), jcb en su mayoría tiene diferentes longitudes que no desearía ejecutar luhn hasta que valide la longitud y el formato), formatCardNumber, validCardExpiry, validateCardCVC, por nombrar algunas .
Un simple error, comprueba if (i%2 === 1) { , el algoritmo necesita multiplicar 2 por cada segundo elemento
arr.reverse() también es una mala idea, se cambia la matriz original.
// All valid credit card numbers const valid1 = [4, 5, 3, 9, 6, 7, 7, 9, 0, 8, 0, 1, 6, 8, 0, 8]; const valid2 = [5, 5, 3, 5, 7, 6, 6, 7, 6, 8, 7, 5, 1, 4, 3, 9]; const valid3 = [3, 7, 1, 6, 1, 2, 0, 1, 9, 9, 8, 5, 2, 3, 6]; const valid4 = [6, 0, 1, 1, 1, 4, 4, 3, 4, 0, 6, 8, 2, 9, 0, 5]; const valid5 = [4, 5, 3, 9, 4, 0, 4, 9, 6, 7, 8, 6, 9, 6, 6, 6]; // All invalid credit card numbers const invalid1 = [4, 5, 3, 2, 7, 7, 8, 7, 7, 1, 0, 9, 1, 7, 9, 5]; const invalid2 = [5, 7, 9, 5, 5, 9, 3, 3, 9, 2, 1, 3, 4, 6, 4, 3]; const invalid3 = [3, 7, 5, 7, 9, 6, 0, 8, 4, 4, 5, 9, 9, 1, 4]; const invalid4 = [6, 0, 1, 1, 1, 2, 7, 9, 6, 1, 7, 7, 7, 9, 3, 5]; const invalid5 = [5, 3, 8, 2, 0, 1, 9, 7, 7, 2, 8, 8, 3, 8, 5, 4]; // Can be either valid or invalid const mystery1 = [3, 4, 4, 8, 0, 1, 9, 6, 8, 3, 0, 5, 4, 1, 4]; const mystery2 = [5, 4, 6, 6, 1, 0, 0, 8, 6, 1, 6, 2, 0, 2, 3, 9]; const mystery3 = [6, 0, 1, 1, 3, 7, 7, 0, 2, 0, 9, 6, 2, 6, 5, 6, 2, 0, 3]; const mystery4 = [4, 9, 2, 9, 8, 7, 7, 1, 6, 9, 2, 1, 7, 0, 9, 3]; const mystery5 = [4, 9, 1, 3, 5, 4, 0, 4, 6, 3, 0, 7, 2, 5, 2, 3]; // An array of all the arrays above const batch = [valid1, valid2, valid3, valid4, valid5, invalid1, invalid2, invalid3, invalid4, invalid5, mystery1, mystery2, mystery3, mystery4, mystery5]; // Add your functions below: const validateCred = arr => { let totalSum = 0; let revList = arr.reverse(); for (let i = 0; i < revList.length; i++) { let calcAmount = revList[i]; if (i%2 === 1) { calcAmount = revList[i] * 2; if (calcAmount > 9 ) { calcAmount -= 9; totalSum += calcAmount; } else { totalSum += calcAmount; } } else { totalSum += revList[i]; } } return(totalSum%10 === 0 ? true : false); }; batch.forEach(c => console.log(validateCred(c)));El algoritmo de Luhn se explica en Wikipedia. Otros han respondido la pregunta, esta es solo una implementación diferente.
function luhnCheck(cardNumber) { let nums = cardNumber.split('').reverse(); let checkValue = nums.shift(); let luhnSum = nums.reduce((sum, n, i) => { let val = n*(i % 2? 1 : 2); sum += val > 9? val - 9 : val; return sum; }, 0); return checkValue == 10 - (luhnSum % 10); } ['4539677908016808', // valid1 '5535766768751439', // valid2 '4532778771091795', // invalid1 '5795593392134643', // invalid2 '344801968305414', // mystery1 '5466100861620239', // mystery2 ].forEach(cardNum => console.log(luhnCheck(cardNum)) );