2の要素を1の配列へ前後から順に入れていくのですが、最後だけ理想の結果になりません。

Question

表2の要素を前、後、前、後と順に1に入れていくのですが、最後だけ以下のような結果になりません。
私の結果は、A,I,B,H,C,G,D,E,Fになり、EとFだけ結果のようなになりません。どこがおかしいのでしょうか。
表 1: A
表 2: B,C,D,E,F,G,H,I
結果: A,I,B,H,C,G,D,F,E

for ( var i = 0; i< array2.length; i++){
array1.push (array2.pop());
array1.push(array2.shift());
var hyo = array1.concat(array2);
}
console.log("表: ",hyo);

Comments

[m.ts10806]

array1とarray2の定義を追記願います。
コピペで再現できる情報ですね。

Answer 1

いくつか変更して動かしてみました：

• 恐らく array1 が求める結果であると推測されるので、質問文にあります hyo については省略します。
• for 文の条件がやや複雑なので while 文にしました。while 文は無限ループに陥りやすくなる面もありますが、今回の場合は尺度（array2.length）が明確であるため、このように書きました。
• 実行中の変数の中身を見るため、適度に console.log を補っています。

main.js:

const array1 = ['A'];
const array2 = ['B', 'C', 'D', 'E', 'F', 'G', 'H', 'I'];

while (array2.length > 0) {
  console.log('---');
  console.log(`a1: ${array1}`);
  console.log(`a2: ${array2}`);

  array1.push(array2.pop());
  array1.push(array2.shift());
}

console.log('===');
console.log('表: ', array1);

実行します：

$ node main.js
---
a1: A
a2: B,C,D,E,F,G,H,I
---
a1: A,I,B
a2: C,D,E,F,G,H
---
a1: A,I,B,H,C
a2: D,E,F,G
---
a1: A,I,B,H,C,G,D
a2: E,F
===
表:  [
  'A', 'I', 'B',
  'H', 'C', 'G',
  'D', 'F', 'E'
]

うまく動いているようですね。

ただ、上のコードは array2 の要素数が偶数であることを前提としているように見えます。
array2 の要素数を奇数にしてみましょう：

main.js:

const array1 = ['A'];

const array2 = ['B', 'C', 'D'];
// const array2 = ['B', 'C', 'D', 'E', 'F', 'G', 'H', 'I'];

while (array2.length > 0) {
  console.log('---');
  console.log(`a1: ${array1}`);
  console.log(`a2: ${array2}`);

  array1.push(array2.pop());
  array1.push(array2.shift());
}

console.log('===');
console.log('表: ', array1);

これも実行すると：

$ node main.js 
---
a1: A
a2: B,C,D
---
a1: A,D,B
a2: C
===
表:  [ 'A', 'D', 'B', 'C', undefined ]

末尾に余計な undefined が付いてしまいました。これは最後の繰り返しで、既に空になってしまった array2 を shift した結果を array1 に追加していることによります。
array2 が空であれば shift させたくないとなれば、次のように修正できます。

main.js:

const array1 = ['A'];

// const array2 = ['B', 'C', 'D'];
const array2 = ['B', 'C', 'D', 'E', 'F', 'G', 'H', 'I'];

while (array2.length > 0) {
  console.log('---');
  console.log(`a1: ${array1}`);
  console.log(`a2: ${array2}`);

  array1.push(array2.pop());

  if (array2.length !== 0) {
    array1.push(array2.shift());
  }
}

console.log('===');
console.log('表: ', array1);

実行しましょう：

$ node main.js
---
a1: A
a2: B,C,D
---
a1: A,D,B
a2: C
===
表:  [ 'A', 'D', 'B', 'C' ]

要素数が奇数でも意図通りに動作していますね。

Comments

[Chibichan]

While文にするのは、思いつきませんでした！回数が決まっているとついFor文と思ってしまいますが、While文がいい時もあるんですね！奇数の場合も提示していただいてありがとうございました。勉強になりました。

Answer 2

array2をpopやshiftで破壊的に変更しているためarray2.lengthが減少していき3ループ目で終わっているためです。
最後にarray2には["E", "F"]が残りますが、その時点でi === 2,array2.length === 2となるためループを抜けてしまうためです。

let array1 = ["A"];
let array2 = ['B','C', 'D', 'E', 'F', 'G', 'H', 'I'];
for ( var i = 0; i< array2.length; i++){
array1.push (array2.pop());
array1.push(array2.shift());
var hyo = array1.concat(array2);
console.log("array2.length --> ", array2.length)
console.log("array1 --> ", array1);
console.log("array2 --> ", array2);
console.log("hyo --> ", hyo)
}
console.log("表: ",hyo);

出力結果

array2.length -->  6
array1 -->  (3) ["A", "I", "B"]
array2 -->  (6) ["C", "D", "E", "F", "G", "H"]
hyo -->  (9) ["A", "I", "B", "C", "D", "E", "F", "G", "H"]

array2.length -->  4
array1 -->  (5) ["A", "I", "B", "H", "C"]
array2 -->  (4) ["D", "E", "F", "G"]
hyo -->  (9) ["A", "I", "B", "H", "C", "D", "E", "F", "G"]

array2.length -->  2
array1 -->  (7) ["A", "I", "B", "H", "C", "G", "D"]
array2 -->  (2) ["E", "F"]
hyo -->  (9) ["A", "I", "B", "H", "C", "G", "D", "E", "F"]

表:  (9) ["A", "I", "B", "H", "C", "G", "D", "E", "F"]

Comments

[Chibichan]

ありがとうございます。その点が理解できなかったところです。ようやく理解できました。

Answer 3

var a=["A"];
var b=["B","C","D","E","F","G","H","I"];
var flg=true;
while(b.length>0){
  a.push(flg?b.pop():b.shift());
  flg=!flg;
}
console.log(a);