0

我目前正在使用 VS2010 中的 Windows 窗体,该窗体应该为新部件号创建 UPC-A 并将它们存储在数据库中。到目前为止,我仍在尝试将这些部分组合在一起,并且在计算校验位时遇到了问题。我想出了以下两种情况:

场景 1:按照我们目前在 Excel 中执行的方式进行计算:

string value = uPCDataSet.UPC.Rows[uPCDataSet.UPC.Rows.Count - 1]["UPCNumber"].ToString();
string manCode = value.Substring(1, 5);
string prodCode = value.Substring(7, 5);
int first = Math.Abs(Convert.ToInt32(value));

while (first >= 10)
first /= 10;

int chkDigitSubtotal;

string UPCNumber = first + manCode + prodCode;

chkDigitSubtotal = Convert.ToInt32((UPCNumber.Substring(1, 1)) + (UPCNumber.Substring(3, 1)) + 
(UPCNumber.Substring(5, 1)) + (UPCNumber.Substring(7, 1)) + (UPCNumber.Substring(9, 1)) + 
(UPCNumber.Substring(11, 1)));

chkDigitSubtotal = (3 * chkDigitSubtotal) + Convert.ToInt32((UPCNumber.Substring(2, 1)) + 
(UPCNumber.Substring(4, 1)) + (UPCNumber.Substring(6, 1)) + (UPCNumber.Substring(8, 1)) + 
(UPCNumber.Substring(10, 1)));

 string chkDigit = ((300 - chkDigitSubtotal).ToString()).Substring(12, 1);

运行此代码时,我收到错误消息“对于 Int32,值太大或太小”。我不明白为什么因为“值”是varchar12 个字符(它们都是数字)。

场景2:我在网上找到了一段代码。此代码有效,但给了我错误的校验位。我正在使用我们使用您的 Excel 工作表计算的 UPC:

string value = "606891001678"; //valid UPC with check digit
string UPCBody = value.Substring(0, 11);
int sum = 0;
bool odd = true;

for (int i = value.Length - 1; i >= 0; i--)
{
    if (odd == true)
    {
        int tSum = Convert.ToInt32(value[i].ToString()) * 2;

        if (tSum >= 10)
        {
            string tData = tSum.ToString();
            tSum = Convert.ToInt32(tData[0].ToString()) + Convert.ToInt32(tData[1].ToString());
        }
        sum += tSum;
    }
    else
        sum += Convert.ToInt32(value[i].ToString());

    odd = !odd;
}

int result = (((sum / 10)+1) *10) - sum;
result = result % 10;
MessageBox.Show(UPCBody + " & " + result);

在这个例子中,校验位显然应该是 8,但它返回 2。我在这上面花了几乎一整天的时间,可以使用一些建议。为什么我在第一种情况下会收到错误,为什么在第二种情况下我会收到与应有的不同的校验位?

在此先感谢您的帮助!

克里斯

编辑:

我删除了之前的两次编辑,因为代码存在缺陷,从某种意义上说我没有得到正确的结果。我发现我们使用 Azalea 条形码,他们有自己的计算校验位的方法(参见 ChkDigitSubtotal)。所以下面的代码有效,并为我的场景提供了正确的校验位。我认为对于所有 UPC-A 计算,此代码将适用于 chkDigitSubtotal。感谢哈里森的出色指导和足智多谋的回答!

string value = uPCDataSet.UPC.Rows[uPCDataSet.UPC.Rows.Count - 1]
["UPCNumber"].ToString();
long chkDigitOdd;
long chkDigitEven;
long chkDigitSubtotal;
string UPCNumber = value.Substring(0, 11);
chkDigitOdd = Convert.ToInt64(UPCNumber.Substring(0, 1)) +
Convert.ToInt64(UPCNumber.Substring(2, 1)) + Convert.ToInt64(UPCNumber.Substring(4, 1)) 
+ Convert.ToInt64(UPCNumber.Substring(6, 1)) + Convert.ToInt64(UPCNumber.Substring(8, 
1)) + Convert.ToInt64(UPCNumber.Substring(10, 1));
chkDigitOdd = (3 * chkDigitOdd);
chkDigitEven = Convert.ToInt64(UPCNumber.Substring(1, 1)) + 
Convert.ToInt64(UPCNumber.Substring(3, 1)) + Convert.ToInt64(UPCNumber.Substring(5, 1)) 
+ Convert.ToInt64(UPCNumber.Substring(7, 1)) + Convert.ToInt64(UPCNumber.Substring(9, 
1));
chkDigitSubtotal = (300-(chkDigitEven + chkDigitOdd));
string chkDigit = chkDigitSubtotal.ToString();
chkDigit = chkDigit.Substring(chkDigit.Length - 1, 1);
4

2 回答 2

1

根据我在维基百科上阅读的内容,您在第一个示例中的计算不正确。

  • String.Substring+String.Subtring或“1”+“1”=“11”。您正在寻找 1+1=2。您需要在添加之前将每个子字符串转换为 int。

+ 运算符(C# 参考)

二进制 + 运算符是为数字和字符串类型预定义的。对于数字类型,+ 计算其两个操作数的和。当一个或两个操作数是字符串类型时,+ 连接操作数的字符串表示形式。

  • String.Substring是基于 0 的,所以你缺少 String.Substring(0,1)偶数的

String.Substring 方法

startIndex:此实例中子字符串的从零开始的起始字符位置。

  • 您的校验位字符串尝试获取小于 12 位的字符串的第 12 位。此外,这不是链接中看到的计算。它正在寻找该数字的 10 - mod 10。这也发生在您更新的编辑

string chkDigit = ((300 - chkDigitSubtotal).ToString()).Substring(12, 1);

您正在尝试Substring从第 12 个基于 0 的字符或第 13 个字符中获取长度为 1 的字符。但是,chkDigitSubtotal 的值不会那么大/长。

至于你的错误,正如金龙在他的回答中所说,你正在将一个 12 个字符的字符串转换为一个数字,它不适合一个Int32,你应该使用一个 long。

int first = Math.Abs(Convert.ToInt32(value));

long first = Math.Abs(Convert.ToInt64(value));

我会首先检查您的计算在前面的步骤中是否在逻辑上正常运行。

更新

在您的编辑 2中,您的最后一行似乎有% operator相反的变量。您想将 109 的余数除以 10(即 9),而您的代码将 10 的余数除以 109(即 0) 所以改变。

chkDigitSubtotal = 10 - (10 % (chkDigitEven + chkDigitOdd));

chkDigitSubtotal = 10 - ((chkDigitEven + chkDigitOdd) % 10);

现在 10 - 9 = 1 这是你的 checkDigit。

于 2013-10-24T20:45:20.940 回答
0

如果value是 12 位长的字符串,那么它太大而无法放入int. 改用longa

于 2013-10-24T20:07:47.937 回答