forの中身をもっと簡潔に書く方法は？

Question

下のコードで同じものを二回書いている部分がちょっと冗長なので、もっと簡潔に書きたいのですが、簡潔になりますでしょうか？

let digit = digitDecide();
console.log(digit);

function digitDecide(){
	let digit = [];
	let indec = 0;
	let digitNumber;
	for(let i = 1;i <= 4;i++){
		digitNumber = 1;
		for(let j = 0;j < i;j++){
			digit[indec++] = digitNumber++;
		}
		digitNumber = 1;
		for(let j = 0;j < i;j++){
			digit[indec++] = digitNumber++;
		}//同じものを二回書いている。
	}
	return digit;
}

Answer 1

コード

解決したようですが、まだ最適化の余地はあるように思います。

function digitDecide2 (length) {
  const digit = [],
        baseDigit = [];

  for (let i = 1, max = length + 1; i < max; ++i) {
    baseDigit.push(i);

    digit.push(...baseDigit, ...baseDigit);
  }

  return digit;
}

console.log(JSON.stringify(digitDecide2(4))); // [1,1,1,2,1,2,1,2,3,1,2,3,1,2,3,4,1,2,3,4]

更新履歴

• 2018/06/16 10:08 Array#slice を削除("..." で展開している為、不要でした)

Re: liflect さん

Comments

[kei344]

主題とは関係ないですが、「Arrat」「スプレッド要素」が気になります。「で展開」につながるので「スプレッド構文」「スプレッド」とかでは？

[think49]

ご指摘ありがとうございます。修正しました。
スプレッド要素は配列の要素限定の用語ですね、調査不足でした。

Answer 2

for(let k=0; k<2; k++){ ～～ }で囲めば良いのでは？

Comments

[liflect]

確かにそれでよかったですね。

Answer 3

とりあえず繰り返し部分をforで回せば記述は1回ですみますね

function digitDecide(){
    let digit = [];
    let indec = 0;
    let digitNumber;
    for(let i = 1;i <= 4;i++){
      for(let j=0;j<=1;j++){
        digitNumber = 1;
        for(let k = 0;k < i;k++){
          digit[indec++] = digitNumber++;
        }
      }
    }
    return digit;
}

Comments

[liflect]

そうですね、たった二回の繰り返しにforを使うことはそんなになかったので、思い付きませんでした。