Luhn算法Ruby无法识别AMEX

Question

所以我在应用 Luhn 算法时遇到了一些问题 以下是一般规则：http://www.codeproject.com/Tips/515367/Validate-credit-card-number-with-Mod-algorithm

这是我的代码

def luhn(credit_card)
  result = 0
    nums = credit_card.split("")
    nums.each_with_index do |item, index|
      if index.even?
        if item.to_i*2>9
            result+= item.to_i*2-9
        else 
            result+= item.to_i*2
        end
      else
        result +=item.to_i
        end
    end
    if (result % 10) == 0
      self.validation = "valid"
    else
      self.validation = "invalid"
    end
end

它适用于大多数卡片

VISA: 4111111111111111       (valid)
VISA: 4111111111111          (invalid)
VISA: 4012888888881881       (valid)
Discover: 6011111111111117   (valid)
MasterCard: 5105105105105100 (valid)
MasterCard: 5105105105105106 (invalid)
Unknown: 9111111111111111    (invalid)

但是说到这个

美国运通：37828224631000（无效）

由于某种原因我的代码说它无效，但它应该根据官方测试卡列表。

我看到了一堆其他的代码，但我想纠正错误并理解我的错误。我会很感激一些解释为什么它会这样工作。

Answer 1

您确定您的美国运通号码应该是有效的吗？

您能否编辑您的问题，向我们展示您从哪里获得测试编号？

这是我在其他 Lunh 测试套件中看到的：

• “378282246310005”应该是真的
• “37828224631000”应该是假的

另外，这里有一些项目供您查看：

• 您的代码以错误的方式在数字中移动：您是从左向右移动，而 Lunh 是从右向左移动。

• 您的代码在校验位上迭代，而 Lunh 不在校验位上迭代。

在循环之前尝试剥去校验位，并反转您的顺序，例如：

def luhn(credit_card)
  (*digits, checksum_digit) = s.split('').map(&:to_i)
  result = 0
  digits.reverse.each_with_index do |item, index|
    …

算完总和后，加上校验和数字，然后比较%10。