1

In my project there is a monochromatic screen you can write to and clear. Within the class I have these two lines of code for clearing the screen. the first pushes the current pixel values from the screen onto a stack so the clearscreen can be undone later. The second line clears the screen. the problem is that the undo stack is getting a reference to this.pixels rather than getting the value of it at the time.

this.pushUndo(this.pixels); //this.pixels is an array of ints
this.updateScreen(new Array(64*32)); //this.pixels changes at the end of this line but the undo stack shouldn't change its value
anthius balaraw Xanthius
  • 143
  • 1
  • 2
  • 8

2 Answers2

4

You can use slice to create a shallow copy:

const pixels0 = [0, 1, 2, 3, 4, 5];
const pixels1 = pixels0.slice(0);

// modify only index 0 of copy.
pixels1[0] = 1;

console.log(pixels0);
// expected output: Array [0, 1, 2, 3, 4, 5]

console.log(pixels1);
// expected output: Array [1, 1, 2, 3, 4, 5]

If you need a deep copy, you can check What is the most efficient way to deep clone an object in JavaScript?

celicoo
  • 1,100
  • 8
  • 18
3

I usually do Object.assign when I need to clone values. Example:

const clone = Object.assign([], yourArray);

Can you show what is inside the updateScreen method in your question?

Alex Pappas
  • 2,377
  • 3
  • 24
  • 48