pythonによるJANコードのチェックディジット計算コードについて

Question

13桁のJANコードを生成するプログラムを作っているのですが、
13桁目のチェックディジットの計算が上手く動いていないようで。
13桁の数字が出力された場合は、計算が上手くいってるのですが、
たまに14ケタのコードが生成されてしまい原因がわかりません。

お知恵をお借りできれば幸いです。

Answer 1

たとえば、product_id = 123456789005の時、途中のretの値が100になります。この結果、10という2桁の数字を返す不具合が発生しているのだと思います。

欲しい値は「10からretの下1桁の値を引いたもの、ただしretの下1桁が0の場合は0」だと思うので、以下のようにダイレクトに書いてしまった方が簡単だと思います。

def get_jan_checkdigit(product_id):
    # 前半は省略
    ret = odd_digits_vals + (even_digits_vals * 3)
    return (0, 9, 8, 7, 6, 5, 4, 3, 2, 1)[ret % 10]

---

以下はコメントを受けての追記です。

pythonではリストやタプルのようなシーケンス型のオブジェクトの後ろに[i]のように大括弧を書くことで(最初の要素を0番目として)i番目の要素を指すことができます。よって、以下のように書くことでretの下1桁の値に応じて狙った値をreturnすることができます。

retの下1桁の値	returnされる値
0	0
1	9
2	8
...	...
9	1

    table = (0, 9, 8, 7, 6, 5, 4, 3, 2, 1)
    return table[ret % 10]

上の2行をさらにまとめると、以下の1行になります。

    return (0, 9, 8, 7, 6, 5, 4, 3, 2, 1)[ret % 10]

Comments

[Meteor]

ありがとうございます
ご指摘の通りでした。

お聞きしたいのですが、
return (0, 9, 8, 7, 6, 5, 4, 3, 2, 1)[ret % 10]　これは何を表しているのか理解ができませんでした。

ご教授いただけたら幸いです

[Meteor]

ご丁寧に解説ありがとうござます
理解できました。

Answer 2

    ret = ret % 10
    ret = 10 - ret
    ret = ret % 10
    return ret 

Comments

[Meteor]

解決できました！ありがとうございます

Answer 3

所々にprint文を入れて、変数の内容を表示させてみて、計算があってるかみてみればどうでしょうか

＃　この関数の戻り値が１０以上になってないですか？

Comments

[Meteor]

ご回答感謝します。
私自身はほぼ素人で、このコードは依頼して作っていただきました。
よって、内容を詳しく理解仕切れていません。
戻り値は１０以上になることがあります。
そのため、１４桁が発生してしまうのかと