最新消息:雨落星辰是一个专注网站SEO优化、网站SEO诊断、搜索引擎研究、网络营销推广、网站策划运营及站长类的自媒体原创博客

JavaScript Arrays .push() not working as Expected - Stack Overflow

programmeradmin3浏览0评论

I'm writing this JavaScript code expecting the output as

[["o","o","x","o","o"],["o","x","x","x","o"]]

but instead it gives

<script>
function createArray(x) {
  var array=[];
  var finalArray=[];
  
  for(var i=1;i<=x;i++){
    array[i-1]='O';
  }

  var midIndex=Math.round(x/2)-1;
  
  array[midIndex]='X';


  finalArray.push(array);

  var num1=midIndex-1;
  var num2=midIndex+1;
 
 
  
    array[num1]="X";
    array[num2]="X";
    
    finalArray.push(array);

  console.log(finalArray) ;
  
}

createArray(5);


</script>

please some one show the reason for this.

I'm writing this JavaScript code expecting the output as

[["o","o","x","o","o"],["o","x","x","x","o"]]

but instead it gives

<script>
function createArray(x) {
  var array=[];
  var finalArray=[];
  
  for(var i=1;i<=x;i++){
    array[i-1]='O';
  }

  var midIndex=Math.round(x/2)-1;
  
  array[midIndex]='X';


  finalArray.push(array);

  var num1=midIndex-1;
  var num2=midIndex+1;
 
 
  
    array[num1]="X";
    array[num2]="X";
    
    finalArray.push(array);

  console.log(finalArray) ;
  
}

createArray(5);


</script>

please some one show the reason for this.

Share Improve this question asked Jul 28, 2020 at 6:00 BenSVBenSV 1691 gold badge4 silver badges15 bronze badges 3
  • 1 Try finalArray.push(array.slice()); the first time. – Unmitigated Commented Jul 28, 2020 at 6:03
  • 1 .push() doesn't clone the array you're pushing into finalArray. You push the "same array" (a reference to the same array) twice. – Andreas Commented Jul 28, 2020 at 6:03
  • 2 Is JavaScript a pass-by-reference or pass-by-value language? – Andreas Commented Jul 28, 2020 at 6:06
Add a ment  | 

3 Answers 3

Reset to default 3

In Javascript, objects(in your case array) are passed by reference. You push the array into finalArray and then update the array. But you have updated the data from same reference. So you could use spread operator which would effectively create a new array(with new reference) and you can push this array to finalArray

finalArray.push([...array]);

That's because Javascript objects are passed by reference. You should create a copy of the array and push to the final one

function createArray(x) {
  var arr=[];
  var finalArray=[];
  
  for(var i=1;i<=x;i++){
    arr[i-1]='O';
  }

  var midIndex=Math.round(x/2)-1;
  
  arr[midIndex]='X';


  finalArray.push(arr.slice());

  var num1=midIndex-1;
  var num2=midIndex+1;
 
 
  
    arr[num1]="X";
    arr[num2]="X";
    
    finalArray.push(arr.slice());

  console.log(finalArray) ;
  
}

createArray(5);

It's for JS pointer When you Push array into final u push pointer of array into an index of final and when change array the final changed too.

for resolve it u can use easy Code:

finalArray.push([...array]);

GoodLuck

发布评论

评论列表(0)

  1. 暂无评论