How to remove item from array 1 and push item to array 2

Question

What I try is to check the state.newCardArray and remove that item based on index and then push it to state.arrayFoldedCards. This is a card game, so if the newCardArray is empty and there is no winner or loser, then the game stops....

const initialState = {
  newCardArray: ['a', 'b', 'c']; // after removing the array looks like ['a', 'c']
  arrayFoldedCards: [] // pushing to the array resulted in an updated array that looks like ['b']
}

export const game = (state = initialState, action) => {
  switch (action.type) {
    case types.GET_CARD:
      {
        const getRandomNumber = Math.floor(state.newCardArray.length * Math.random());
        console.log(state.newCardArray);
        console.log(state.arrayFoldedCards);
        return {
          ...state,
          randomNumber: getRandomNumber,
          arrayFoldedCards: state.newCardArray[getRandomNumber].push(state.arrayFoldedCards.pop())
        }
      }
  }
}

newCardArray[getRandomNumber] will give you an element from the array calling newCardArray[getRandomNumber].push() means you expect that element to be an array and you want to add an item to the sub-array. Since you have plain strings, that's not going to work. — VLAZ
– VLAZ, Commented Dec 17, 2019 at 13:12
Yes that is why I post my code here...i knew this was not the right way to do that. Please check the out commented text to understand the array. — Fresh developer
– Fresh developer, Commented Dec 17, 2019 at 13:13

Strike Eagle · Accepted Answer · 2019-12-17 13:26:09Z

3

Looks like you want something like splice/concat

var idxToRemove = 1; //Removing 'b'
var arr0 = ['a', 'b', 'c']; 
var arr1 = []; //Array to move to

return arr1.concat(arr0.splice(idxToRemove, 1));

Splice returns an array of removed elements so you can concat them together. Concat merges the two arrays without mutating either.

answered Dec 17, 2019 at 13:26

Strike Eagle

8615 silver badges20 bronze badges

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

1 Comment

Josh Pittman Over a year ago

Splice mutates arr0 in the process, so two birds with one stone. Nice.

Fresh developer · Accepted Answer · 2019-12-17 15:38:29Z

0

case types.GET_CARD: {
    const indexToRemove = Math.floor(state.currentCardArray.length * Math.random());
    const updatedArray = state.arrayFoldedCards.concat(state.currentCardArray.splice(indexToRemove, 1));

    return {
        ...state,
        arrayFoldedCards: updatedArray
    }
}

answered Dec 17, 2019 at 15:38

Fresh developer

811 gold badge1 silver badge10 bronze badges

Collectives™ on Stack Overflow

How to remove item from array 1 and push item to array 2

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related