Знакомлюсь с Vue. Этот метод для работы с data сделан корректно?

Question

[bormor]

Знакомлюсь с Vue.
Пишу для ToDo метод toggleTask (т.е. отмечает сделано / не сделано).

Это решение корректно? Есть какие-то Vue-way способы его улучшить?

methods:{
    toggleTask(id){
      let taskIndex;
      this.tasks.forEach( (item, index) => {
        if ( item.id === id){
          taskIndex = index
        }
      });

      this.tasks[taskIndex].completed = !this.tasks[taskIndex].completed;
    }
  },

Как я понимаю - если в react предпочтительнее иммутабельные операции с данными, то тут - мутабельные.

Answer 1

[Koteezy]

Можно просто передать весь task, и все сведется к:
task.completed = !task.completed

Comments

[Иван]

И я бы сказал это таки правильный метод!

[Koteezy]

Иван, Можно еще вынести task в отдельный компонент, и проводить такие манипуляции не передавая task аргументом, а просто:
this.task.completed = !this.task.completed;