javascript 宝くじの問題

Question

宝くじの当選番号を求める問題です
100000から199999の当選番号が渡されます。

1等：当選番号と一致する番号
前後賞：当選番号の ±1 の番号　当選番号100000の場合100001のみ、199999の場合199998のみになります。前後いずれかになります。
2等：当選番号と下 4 桁が一致する番号
3等：当選番号と下 3 桁が一致する番号

let lines = [ 当選番号, 宝くじ券の番号_1, 宝くじ券の番号_2, 宝くじ券の番号_n ]
というような形で配列に値が渡されます。

コード
  let lines = [ '111111', '111110', '121111', '122111', '111111',];
  let Numbers = lines.slice(1);

  let SumUp = Number(lines[0]) + 1;
  let SumDown = Number(lines[0]) - 1;

  let First = lines[0];
  let Seconds = lines[0].slice(-4);
  let Third = lines[0].slice(-3);
  
  for (let i = 0; i < Numbers.length; i++) {
    if (First === Numbers[i]) {
      console.log('1等：' + Numbers[i]);
    } else if (SumUp === Number(Numbers[i]) || SumDown === Number(Numbers[i])) {
      console.log('前後賞：' + Numbers[i]);
    } else if (Seconds === Numbers[i].slice(-4)) {
      console.log('2等：' + Numbers[i]);
    } else if(Third === Numbers[i].slice(-3)) {
      console.log('3等：' + Numbers[i]);
    } else {
      console.log('外れ：' + Numbers[i]);
    }
  }

このコードで問題はなかったのですが、ここはこう書くべき、私はこう書く、現場ではこう書く等アドバイス頂きたいです。宜しくお願い致します。

Comments

[yambejp]

前後賞の考え方は大丈夫でしょうか？
https://all-guide.com/nn0040/

[cal_1]

ほぼ同じですが、2点違います。
組は渡されません。
100000の前後賞は100001のみ、199999は当たりになりません。
199999の前後賞も199998のみになります。

Answer 1

当選判定は関数に分離したほうがいいですかね。

let lines = [ '111111', '111110', '121111', '122111', '111111',];
let winningNumber = lines[0];
let numbers = lines.slice(1);

for (let i = 0; i < numbers.length; i++) {
  const winningOrder = GetWinningOrder(winningNumber, numbers[i]);
  console.log(winningOrder + '：' + numbers[i]);
}

function GetWinningOrder(winningNumber, lotteryNumber) {
  if (winningNumber === lotteryNumber) {
    return '1等';
  } else if (Number(winningNumber) + 1 === Number(lotteryNumber) || Number(winningNumber) - 1 === Number(lotteryNumber)) {
    return '前後賞';
  } else if (winningNumber.slice(-4) === lotteryNumber.slice(-4)) {
    return '2等';
  } else if (winningNumber.slice(-3) === lotteryNumber.slice(-3)) {
    return '3等';
  } else {
    return '外れ';
  }
}

Answer 2

let Numbers = lines.slice(1);では？

また、一等、二等、三等が文字列なので、前後賞も文字列で保持した方が良いでしょう。

あとは、すでに指摘のあることくらいです。

Answer 3

コードのロジックはとくに問題ないと思いますが、コード可読性の観点でいつくか。

let lines = [ 当選番号, 宝くじ券の番号_1, 宝くじ券の番号_2, 宝くじ券の番号_n ]

この仕様がわかりにくいので、let firstPrize = lines[0]; など説明変数を使うのがよいと思います。

JavaScriptでは、変数名を大文字で始めるのは一般的ではありません。

for (let i = 0; i < Numbers.length; i++) {

for (let number of Numbers) { とすれば、何度も Numbers[i] と書かずに number で済みます。

Comments

[cal_1]

変数名、for ofに関してご教示頂きありがとう御座います。参考にさせていただきます。