github上のあるコードについて、それがバグであるかどうかを教えて頂きたいです。

Question

非常に身勝手な質問ですので
お時間のある方のみで結構です。

「30-seconds-of-code」という、ライブラリのバグを発見したかもしれません。その発見が正しいか否か、確かめて頂けますでしょうか。
https://github.com/Chalarangelo/30-seconds-of-code#luhncheck

以下のサイト
http://blog.livedoor.jp/enjoy_math/archives/51792472.html
を参考にした所、'１９５４７'という数字を引数にすると、trueになるはずですが、falseになってしまいます。
そこで、github記載のコードを以下のようにすれば、正しく動作すると思いました。
(val * 2) % 9 || 9の部分を括弧で囲うように修正しました。

const luhnCheck = num => {
  let arr = (num + '')
    .split('')
    .reverse()
    .map(x => parseInt(x));
  let lastDigit = arr.splice(0, 1)[0];
  let sum = arr.reduce((acc, val, i) => (i % 2 !== 0 ? acc + val : acc + ((val * 2) % 9 || 9)), 0);
  sum += lastDigit;
  return sum % 10 === 0;
};

スターを17000ほど集めているライブラリのバグを発見したかもしれない
ということで、一人で舞い上がっておりますので、
是非とも、確かめて頂きたいです。

Comments

[defghi1977]

単なるスニペット集でのバグであればなおさらここで訊くより作者に直接尋ねるべきでしょう

[CHERRY]

https://github.com/Chalarangelo/30-seconds-of-code/issues から登録して、作者に判断してもらうのが良いでしょう。

Answer 1

結局のところ正解は作者が持っているものなので作者に問い合わせてみるべきですね。

Comments

[m.ts10806]

（もしかしたらこういう沢山の場に晒されるのは作者にとってはあまり良い印象を持たないかもしれませんし）

[miyabi-sun]

そうですね、とあるシステムのセキュリティホールを見つけたエンジニアが全世界に公開して大事件になった例もありますからね。

[fjaiofjawiefjaw]

舞い上がった気持ちのまま質問してしまいました。
セキュリティホールの事例があったとのことで、軽はずみでバグ報告は、辞めるように致します。

[m.ts10806]

そういったバグ報告は、あくまで作者に、が原則ですね。
余程プライドの高い人でない限り、作者はユーザーからのフィードバックを喜びます。