Luhn's algorithm in php

gistfile1.aw

<?php
function is_valid_luhn($number) {
  settype($number, 'string');
  $sumTable = array(
    array(0,1,2,3,4,5,6,7,8,9),
    array(0,2,4,6,8,1,3,5,7,9));
  $sum = 0;
  $flip = 0;
  for ($i = strlen($number) - 1; $i >= 0; $i--) {
    $sum += $sumTable[$flip++ & 0x1][$number[$i]];
  }
  return $sum % 10 === 0;
}

@tflori Your updated code doesn't work just FYI. You actually did need the array_sum. This is because of the first step outlined here: https://en.wikipedia.org/wiki/Luhn_algorithm#Description

If the result of this doubling operation is greater than 9 (e.g., 8 × 2 = 16), then add the digits of the result (e.g., 16: 1 + 6 = 7, 18: 1 + 8 = 9)

Your original code achieved this by using a string and summing all the numbers in the string. My code uses the alternate method mentioned:

alternatively, the same final result can be found by subtracting 9 from that result (e.g., 16: 16 − 9 = 7, 18: 18 − 9 = 9)

But your newly edited code does neither, and as such returns false for many valid numbers.

@mogzol ah f**k.. thanks for the info 👍

In case anyone tries using @mogzol solution while working with the Luhn algorithm applied to a base higher than 10, for example to validate an alphanumeric string where each character can go from 0-9 or A-Z, meaning its value ranges from 0-36, you need to be more careful performing the total sum.

In the solution, when we're doing the total sum we are essentially ignoring the fact that Luhn algorithm only wants you to subtract 9 to values that have been doubled. It works for base 10 only because values that have not been doubled can't possibly be greater than 9.

To correctly implement a general any-base Luhn algorithm, you don't have this luxury, and must therefore apply both the doubling of the value and subsequent subtraction only IF the condition is met, in this case ($flag++ & 1), which by the way is quite an unreadable way to flip between true and false.

Here's a general readable example, assuming some valid string $number in base $base

$flip = false;
    $sum = 0;
    for ($i = strlen($number) - 1; $i >= 0; $i--) {
        $value = intval($number[$i], $base);
        if($flip){
            $value = $value * 2;
            $value = $value > 9 ? $value - 9 : $value;
        }
        $sum += $value;
        $flip = !$flip;
    }
    return $sum % 10 === 0;

Hi, this is my implementation: https://github.com/danielebarbaro/laravel-vat-eu-validator/blob/26e65de7df083e729b44e0ece02483dc6076bc71/src/VatValidator.php#L124

it works since 2019 🤓