计算GS1 / SSCC / UPC校验位数

Question

我试图编写一个函数，根据GS1 http://www.gs1.org/how-calculate-check-digit-manually提供的算法计算运输标签的校验位数。

第一版:我的第一次尝试

public static int GetGTINCheckDigitV1(string code)
{
    int sum = 0;
    for (int i = 0; i < code.Length; i++)
    {
        int n = int.Parse(code.Substring(code.Length - 1 - i, 1));
        sum += i % 2 == 0 ? n * 3 : n;
    }

    return sum % 10 == 0 ? 0 : 10 - sum % 10;
}

版本2:使用Reverse()

public static int GetGTINCheckDigitV2(string code)
{
    int sum = 0;
    var list = code.Reverse().ToArray();

    for (int i = 0; i < list.Length; i++)
    {
        int n = (int)Char.GetNumericValue(list[i]);
        sum += i % 2 == 0 ? n * 3 : n;
    }

    return sum % 10 == 0 ? 0 : 10 - sum % 10;
}

版本3:使用lambda

public static int GetGTINCheckDigitV3(string code)
{
    var sum = code.Reverse().Select((c, i) => (int)char.GetNumericValue(c) * (i % 2 == 0 ? 3 : 1)).Sum();
    return (10 - sum % 10) % 10;
}

我有什么可以改善这个功能的吗?或者有什么值得关注的？

Answer 1

看上去不错！我只有一些小小的建议。您可能不希望LINQ表达式全部放在一行上。现在我必须滚动才能看到这一切。

另一种方法是使用Enumerable.Range()创建一系列索引：

public static int GetGTINCheckDigitUsingRange(string code)
{
    var reversed = code.Reverse().ToArray();
    var sum = Enumerable
        .Range(0, reversed.Count())
        .Sum(i => (int)char.GetNumericValue(reversed[i]) * (i % 2 == 0 ? 3 : 1));        
    return (10 - sum % 10) % 10;
}

另一种选择是使用查询语法：

public static int GetGTINCheckDigitUsingQuery(string code)
{
    var reversed = code.Reverse().ToArray();
    var sum = 
       (from i in Enumerable.Range(0, reversed.Count())
        let digit = (int)char.GetNumericValue(reversed[i])
        select digit * (i % 2 == 0 ? 3 : 1)).Sum();
    return (10 - sum % 10) % 10;
}

我认为你的第三版仍然比我的好，但你可能更喜欢其中之一。Enumerable.Range()用于替换for循环，查询语法用于声明中间值(如digit)和替换嵌套循环。但是，将包含索引的lambda作为参数的Select()方法非常适合这种情况。有了足够的实践，您就可以立即达到代码看起来像第三版的地步。我很少再写循环了。我发现他们更难写，因为他们让你更多地关注“如何”而不是“什么”。

Answer 2

在没有参数验证的情况下，这样的公共方法是红色的。永远不要相信公共方法的任何给定输入。将null传递给该方法将抛出一个ArgumentNullException，这将是正确的异常，但提供的详细信息将具有误导性，并将向用户公开实现细节。该方法的用户不需要知道您使用的是System.Linq.Enumerable.Reverse。

您没有考虑到传递的值所包含的最大位数也不应该超过17位数(SSCC)。