2

所以我在应用 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(无效)

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

我已经看到了许多其他代码正在运行,但我想纠正错误并理解我的错误。我会很感激一些解释为什么它会这样工作。

4

1 回答 1

2

你确定你的美国运通号码应该是有效的吗?

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

这是我在其他 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。

于 2015-01-13T21:34:28.417 回答