更新数组中的项目将全部更新

时间:2017-07-01 06:40:14

标签: javascript vue.js vuejs2 vuex

我正在使用带有vuex的vuejs使用项目的应用程序,每个项目都有一个或多个作业。

我可以添加,删除和更新作业。添加和删​​除工作正常,但更新不是。

vuex开发工具中的状态:

projects

我的HTML:

<div class="job-compact row" v-for="(job, index) in project.jobs">
        <div class="col-md-6">
            <div class="form-group" :class="{'has-error' : errors.has('jobs.' + index + '.function')}">
                <input type="text" name="jobs[function][]" class="form-control" v-model="job.function" @change="updateJobValue(index, 'function', $event.target.value)"/>
            </div>
        </div>
        <div class="col-md-4">
            <div class="form-group" :class="{'has-error' : errors.has('jobs.' + index + '.profiles')}">
                <input type="number" name="jobs[profiles][]" class="form-control" v-model="job.profiles" @change="updateJobValue(index, 'profiles', $event.target.value)"/>
            </div>
        </div>
        <div class="col-md-2">
            <button v-if="index == 0" class="btn btn-success btn-sm" @click="addJob"><i class="fa fa-plus"></i></button>
            <button v-if="index > 0" class="btn btn-danger btn-sm" @click="deleteJob(index);"><i class="fa fa-minus"></i></button>
        </div>
    </div>

正如您所看到的,我有一个显示我所有工作的v-for。在我的作业中编辑值时,我使用@change事件来更新我的值。而且,在底部,我有两个按钮来添加和删除作业行。

enter image description here

我的商店分为模块。主要商店看起来像这样:

import Vue from 'vue'
import Vuex from 'vuex'

Vue.use(Vuex);

const state = {};

const getters = {};

const mutations = {};

const actions = {};

//Separate Module States
import jobCreator from './modules/job-creator/store';

export default new Vuex.Store({
    modules: {
        jobCreator: jobCreator
    },
    state,
    actions,
    mutations,
    getters
});

针对此特定问题的模块存储:

import store from './../../store'

const state = {
    project: {
        title: null,
        description: null,
        jobs: []
    },
    defaultJob: {
        function: '',
        title: '',
        description: '',
        profiles: 1,
        location_id: '',
        category_id: '',
        budget: '',
    },
};

const getters = {}

const mutations = {

    addJob(state, job) {
        state.project.jobs.push(job);
    },
    deleteJob(state, index) {
        state.project.jobs.splice(index, 1);
    },
    updateJobValue(state, params) {
        Object.assign(state.project.jobs[params.jobIndex], {
            [params.field]: params.value
        });
    }
};

const actions = {
    addJob: function (context) {
        context.commit('addJob', state.defaultJob);
    },
    deleteJob: function (context, index) {
        context.commit('deleteJob', index);
    },
    updateJobValue: function (context, params) {
        context.commit('updateJobValue', params);
    },
};

const module = {
    state,
    getters,
    mutations,
    actions
};

export default module;

项目状态映射到我的vue实例的计算属性:

computed: {
       ...mapState({
           project: state => state.jobCreator.project,
       }),
}

问题如下:在应用程序的图像中,您可以看到我输入了&#34; vin&#34;在其中一个字段中,但所有字段都在更新。

enter image description here

因此,所有作业的所有function字段都已更新为我的上一个条目,而不是仅我想要的那个。

我做错了什么?

PS:

我还在我的突变功能中尝试了以下内容:

updateJobValue(state, params) {
    var job = state.project.jobs[params.jobIndex];
    job[params.field] = params.value;

    Vue.set(state.project.jobs, params.jobIndex, job);
}

但它给了我相同的结果。

更新:根据要求,我创建了一个jsFiddle来显示我的问题

2 个答案:

答案 0 :(得分:9)

问题出在您的addJob行动中:

addJob: function (context) {
    context.commit('addJob', state.defaultJob);
},

每次添加新作业时都会引用state.defaultJob对象。这意味着state.project.jobs数组中的每个项都引用同一个对象。

您应该在将对象传递给addJob突变时创建对象的副本:

addJob: function (context) {
    context.commit('addJob', Object.assign({}, state.defaultJob));
},

或者,每次只传入一个具有默认属性的新对象:

addJob: function (context) {
    context.commit('addJob', {
        function: '',
        title: '',
        description: '',
        profiles: 1,
        location_id: '',
        category_id: '',
        budget: '',
    });
},

Here's a working fiddle.

这里有一篇文章解释了如何在Javascript中传递变量:Javascript by reference vs. by value

答案 1 :(得分:1)

我会提出以下建议:

使用v-bind:value="job.function代替v-model="job.function",因为您只需要单向绑定。这使您的代码更具可预测性。

v-key="job"元素添加v-for="(job, index) in project.jobs",以确保渲染正常。

应该是前两行,对象仍然是被动的。

var job = state.project.jobs[params.jobIndex];
job[params.field] = params.value;


Vue.set(state.project.jobs, params.jobIndex, job);

PS:在我的小提琴中@change仅在我按下输入或离开输入时才触发。