I'm finding some strange behavior with JSON in a Vuex getter: it seems it is contributing to a pass-by-reference type of problem. For context - I'm developing a music app, which will have multiple "scenes", which each include collections of "tracks" (similiar to Ableton Live).

Here's my getter:

  newTrack: state => {
    let newTrack = JSON.parse(JSON.stringify(state.newTrackDefaults))
    return newTrack
  },

Here's the object it refers to:

  newTrackDefaults: {
    tune: [],
    // and other properties
  },

And then it is called by an action:

  setUpNewScene: context => {
    let newScene = JSON.parse(JSON.stringify(context.state.newSceneDefaults))
    let newTrack = context.getters.newTrack  
    console.log(newTrack) // this reveals that the problem is from the getter 
    newScene.tracks.push(newTrack)
    contextmit('addNewScene', newScene)
 }

So the problem with this code is, when I add items (pitch-references) to a track on the first scene, then added a new scene, the new scene automatically receives the same track as the first scene. This is reflected in the Vuex state (according to the DevTool), not just the rendering. Also, when the tracks on the first scene are updated by the user, the tracks on the new scene change accordingly. So instinctively, this feels like a pass-by-reference type of error.

Through various console.log experiments, I found that the getter was returning the "filled" track. It's fixable by skipping the getter and writing the action as:

  setUpNewScene: context => {
    let newScene = JSON.parse(JSON.stringify(context.state.newSceneDefaults))
    let newTrack =  JSON.parse(JSON.stringify(context.state.newTrackDefaults))
    console.log(newTrack) // this works fine
    newScene.tracks.push(newTrack)
    contextmit('addNewScene', newScene)
 }

... so though I have a fix, I'm puzzled as to the original behavior. Would the getter interfere with the JSON functions? Or what else might be going on to cause this?

I'm finding some strange behavior with JSON in a Vuex getter: it seems it is contributing to a pass-by-reference type of problem. For context - I'm developing a music app, which will have multiple "scenes", which each include collections of "tracks" (similiar to Ableton Live).

Here's my getter:

  newTrack: state => {
    let newTrack = JSON.parse(JSON.stringify(state.newTrackDefaults))
    return newTrack
  },

Here's the object it refers to:

  newTrackDefaults: {
    tune: [],
    // and other properties
  },

And then it is called by an action:

  setUpNewScene: context => {
    let newScene = JSON.parse(JSON.stringify(context.state.newSceneDefaults))
    let newTrack = context.getters.newTrack  
    console.log(newTrack) // this reveals that the problem is from the getter 
    newScene.tracks.push(newTrack)
    context.mit('addNewScene', newScene)
 }

So the problem with this code is, when I add items (pitch-references) to a track on the first scene, then added a new scene, the new scene automatically receives the same track as the first scene. This is reflected in the Vuex state (according to the DevTool), not just the rendering. Also, when the tracks on the first scene are updated by the user, the tracks on the new scene change accordingly. So instinctively, this feels like a pass-by-reference type of error.

Through various console.log experiments, I found that the getter was returning the "filled" track. It's fixable by skipping the getter and writing the action as:

  setUpNewScene: context => {
    let newScene = JSON.parse(JSON.stringify(context.state.newSceneDefaults))
    let newTrack =  JSON.parse(JSON.stringify(context.state.newTrackDefaults))
    console.log(newTrack) // this works fine
    newScene.tracks.push(newTrack)
    context.mit('addNewScene', newScene)
 }

... so though I have a fix, I'm puzzled as to the original behavior. Would the getter interfere with the JSON functions? Or what else might be going on to cause this?

• javascript
• vue.js

Share Improve this question Follow asked Apr 27, 2018 at 22:10 drenldrenl 1,35344 gold badges1919 silver badges3232 bronze badges 1

• 1 check this javascript pass reference or value, then another improvement of your codes is using Object.assign replace JSON.parse(JSON.stringify(context.state)) – Sphinx Commented Apr 27, 2018 at 23:11

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 6

Vuex getters are cached.

https://vuex.vuejs/en/getters.html

Like puted properties, a getter's result is cached based on its dependencies, and will only re-evaluate when some of its dependencies have changed.

If their dependencies don't change, getters return their last returned value. Presumably, you aren't mutating newTrackDefaults. This means that the getter newTrack will never update. Which means context.getters.newTrack is returning the same object every time. Therefore all the "new" tracks you think you're creating are actually the same track.

When dealing with default values, I like to create a function which returns the default values. Then I can call the function whenever and wherever I want, no funky caching business involved.

function newTrackDefaults() {
    return {
        tune:[],
        // and other properties
    }
}


setUpNewScene: context => {
    let newScene = JSON.parse(JSON.stringify(context.state.newSceneDefaults))
    let newTrack = newTrackDefaults() 
    newScene.tracks.push(newTrack)
    context.mit('addNewScene', newScene)
 }

Fun fact: Getters can return functions, so you can also fix it like this.

newTrack: state => () => {
    let newTrack = JSON.parse(JSON.stringify(state.newTrackDefaults))
    return newTrack
  },

Now you need to call the getter, because it's a function now.

let newTrack = context.getters.newTrack()