Splice value from an array and push spliced value into another array

Question

Hello and here is my problem.I have a task,i need to remove elements from an array and push them into another array and then return the array with removed elements. For example, in this case i have to remove all 0 values, and push them into 'a' array and then return 'a' array with there 0 values. I removed 0 values from the array by using splice method and loop, but i don't realy know how to push removed elemts into 'a' array, i've tried to use push method but i does not work for me. Hope you'll help me.Thank you everyone.

function moveZeros(array) {
  var a = [];
  for (var i = array.length - 1; i--;) {
    if (array[i] == "0") {
      array.splice(i, 1);
    }

  }
  
  return a;
}

moveZeros([1, 2, 0, 1, 0, 1, 0, 3, 0, 1]);

please add the wanted result. do you like to get all zeroes back and array has no zeros? — Nina Scholz
– Nina Scholz, Commented Mar 30, 2018 at 20:39
According to OP i don't realy know how to push removed elemts into 'a' array. — idream1nC0de
– idream1nC0de, Commented Mar 30, 2018 at 20:43
I wanna remove all zeroes from an array an push them into a new array and return the new array. — Олег Войтинський
– Олег Войтинський, Commented Mar 30, 2018 at 20:48

Ori Drori · Accepted Answer · 2018-03-30 20:59:06Z

2

Array.splice() returns an array of removed elements, and you can use Array.concat() to add them to the a array.

Notes:

Init i to array.length without the -1, since the condition i-- is checked before the execution of the loop's body.
Since the the array is iterated from end to starts, concat a to the removed elements to maintain the original order.

function moveZeros(array) {
  var a = [];
  for (var i = array.length; i--;) {
    if (array[i] === 0) {
      a = array.splice(i, 1).concat(a);
    }

  }
  
  return a;
}

var result = moveZeros([1, 2, 0, 1, 0, 1, 0, 3, 0, 1, 0]); // I've added a last 0 to the array to show that all items were removed
console.log(result);

edited Mar 30, 2018 at 20:59

answered Mar 30, 2018 at 20:37

Ori Drori

196k32 gold badges243 silver badges233 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

CRice · Accepted Answer · 2018-03-30 21:33:18Z

2

Using push should work. The .splice method will return an array with the removed elements, and you can use the spread operator ... to pass it as a set of arguments to push:

function moveZeros(array) {
  var a = [];
  for (var i = array.length - 1; i >= 0; i--) {
    if (array[i] == "0") {
      a.push(...array.splice(i, 1));
    }

  }
  
  return a;
}
  
const array = [0, 1, 2, 0, 1, 0, 1, 0, 3, 0, 1, 0, 0];
console.log(moveZeros(array));
console.log(array)

Finally, you should put the i-- as the final part of the loop so that it only executes when each iteration finishes (instead of as they begin). Then, change your condition to be i >= 0 so that you don't miss a zero at the front of the array.

edited Mar 30, 2018 at 21:33

answered Mar 30, 2018 at 20:37

CRice

32.6k5 gold badges63 silver badges75 bronze badges

Comments

Nina Scholz · Accepted Answer · 2018-03-30 20:46:06Z

1

You could iterate from the end and splice the array if necessary. This solution mutates the array.

function moveZeros(array) {
    var i = array.length,
        a = [];

    while (i--) {
        if (array[i] === 0) {
            a.push(array.splice(i, 1)[0]);
        }
    }
    return a;
}

var array = [1, 2, 0, 1, 0, 1, 0, 3, 0, 1];

console.log(moveZeros(array));
console.log(array);

.as-console-wrapper { max-height: 100% !important; top: 0; }

edited Mar 30, 2018 at 20:46

answered Mar 30, 2018 at 20:36

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

Collectives™ on Stack Overflow

Splice value from an array and push spliced value into another array

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related