array.push() method is not adding anything to array and working weirdly

Question

I am new to programming and also stackoverflow. My problem is related to array.push() method in JavaScript. See the code first

var buttonColors = ["red", "blue", "green", "yellow"];
var gamePattern = [];
gamePattern = gamePattern.push(nextSequence());

function nextSequence(){
        var randomNumber = Math.floor( Math.random() * 4);
        var randomChosenColor = buttonColors[randomNumber];
        return randomChosenColor;
}

Kindly check this image too... This is chrome console output

The problem is that the randomNumber is being generated properly and randomChosenColor is also getting the color properly but it is not being pushed in gamePattern array at line number 3. Also help me if there is some alternative to this method.

push returns the number inserted, not the new array. The push happens in-place. — Mark
– Mark, Commented May 31, 2020 at 4:36
The code you posted does not create the output in the screenshot. If you still need help after debugging your code, please post a minimal reproducible example. — Code-Apprentice
– Code-Apprentice, Commented May 31, 2020 at 4:42

Kartikeya Sharma · Accepted Answer · 2020-05-31 04:39:40Z

1

Push changes the original state of the array. So you don't need to re-initialize the array.

 var buttonColors = ["red", "blue", "green", "yellow"];
    let temp=[];
    temp.push(seq());
    console.log(temp);
    function seq(){
        var randomNumber = Math.floor( Math.random() * 4);
        var randomChosenColor = buttonColors[randomNumber];
        return randomChosenColor;
    }

answered May 31, 2020 at 4:39

Kartikeya Sharma

5531 gold badge5 silver badges22 bronze badges

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

2 Comments

Ahmad Hassan Over a year ago

Thank You Brother, you have solved my problem. Please tell me what is this "let" keyword that you have added

Kartikeya Sharma Over a year ago

"let" keyword is used for variable declaration it is block scoped . To read about it more . You can refer this document developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/…

Camilo · Accepted Answer · 2020-05-31 04:36:42Z

0

push() mutates the original array (and returns the new length), so you don't need to assign a new value to gamePattern. Try something like this:

var buttonColors = ["red", "blue", "green", "yellow"];
var gamePattern = [];
gamePattern.push(nextSequence());

function nextSequence(){
  var randomNumber = Math.floor(Math.random() * 4);
  var randomChosenColor = buttonColors[randomNumber];
  return randomChosenColor;
}

answered May 31, 2020 at 4:36

Camilo

7,3065 gold badges45 silver badges66 bronze badges

Collectives™ on Stack Overflow

array.push() method is not adding anything to array and working weirdly

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related