質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.92%

Todoリスト Vue.js 作業ボタンを完了ボタンに切り替えたい

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 137

Junkichi89

score 6

前提・実現したいこと

<現状>

Vue.jsでtodoリストを作成しています。
現在、完了ボタンと作業中ボタンの設定をしています。

<実現したいこと>
作業中ボタンをクリックすると完了ボタンに切り替わり、
完了ボタンをクリックすると作業中ボタンに戻るようにしたいです。

ここに質問の内容を詳しく書いてください。

現在、作業中ボタンと実行中ボタンをトグルを使用して、trueとfalseで表示が切り替わるようにしているのですが、一つのボタンを押すと、全ての作業中ボタンが完了になってしまいます。
なので、ボタンを押した作業中ボタンだけ、切り替わるようにしたいです。

発生している問題・エラーメッセージ

作業中のボタンが全て同時に切り替わってしまうので、
選択した作業中ボタンだけ、完了に表示を切り替えたい

該当のソースコード

'use strict';

new Vue({
  el: '#app',
  data:{
    isActive: false,
    newTask: '',
    todos: [{
      title: 'task1',
      isActive: false
    },
    {
      title: 'task2',
      isActive: false
    },
    {
      title: 'task3',
      isActive: true
    }
  ]
},
methods: {
  addTask: function() {
    let item = {
      title: this.newTask,
      isActive: false
    }
    this.todos.push(item);
    this.newTask ='';
  },
  statusChange: function(index) {
    this.isActive = !this.isActive
  },
  deleteTask: function(index) {
    if(confirm('are you sure?')) {
      this.todos.splice(index, 1);
    }
  },

}
});
<!DOCTYPE html>
<html lang="ja">

<head>
  <meta charset="UTF-8">
  <title>ToDo List Vue-version</title>

  <link rel="stylesheet" href="css/styles.css">
</head>

<body>
  <h2>ToDoリスト</h2>
  <input type="radio">すべて
  <input type="radio">作業中
  <input type="radio">完了
  <p>ID コメント 状態</p>
  <table></table>
  <h2>新規タスクの追加</h2>
  <div id="app">
    <table>
      <tr v-for="(task, index) in todos">
      <td>{{ index }} {{ task.title }}</td>
        <td><button @click="statusChange()">
          <span v-if="isActive === false">作業中</span>
          <span v-if="isActive === true">完了</span>
        </button></td>
        <td><button @click="deleteTask(index)">削除</button></td>
      </tr>
    </table>
    <form @submit.prevent="addTask">
      <input type="text" v-model="newTask">
      <input type="submit" value="追加" class="button">
    </form>
  </div>
  <script src="https://cdn.jsdelivr.net/npm/vue/dist/vue.js"></script>
  <script src="js/main.js"></script>
</body>

</html>

試したこと

Vue.jsでtoggleを使う方法 
https://designsupply-web.com/media/knowledgeside/4811/
https://into-the-program.com/vue-elem-switch/

上記のサイトを参考にコードを記入しました。

補足情報(FW/ツールのバージョンなど)

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

原因

isActiveがすべて同じ値を指しているので、一個を変更したと思っても、全部連動しています。

せっかく、v-forした時にindexを指定しているのに、イベント内ではindexが仕事をしていません。悲しい。

直接関係はないけど、v-forをするときはv-bind:key="~~"は書きましょう。期待通りの動作をしなくなる場合があります。
公式ドキュメントにも、記載があります。

Vue が各ノードの識別情報を追跡できるヒントを与えるために、また、先ほど説明したような既存の要素の再利用と並び替えができるように、一意な key 属性を全てのアイテムに与える必要があります:

修正箇所

htmlの以下と

          <span v-if="isActive === false">作業中</span>
          <span v-if="isActive === true">完了</span>


js以下

  statusChange: function(index) {
    this.isActive = !this.isActive
  }


isActivetodos[index].isActiveに書き換えるといいかも!

ちなみに私なら

こう書き換えます。

          <span v-if="isActive === false">作業中</span>
          <span v-if="isActive === true">完了</span>

          <--書き換えると-->
          <span v-show="task.isActive == false">作業中</span>
          <span v-show="task.isActive == true">完了</span>


v-ifはDOMから消したり追加したりするので、トグルのような頻繁に変わるものには向きません。
一方で、v-showは条件によってDOM要素をdisplay:none;するだけなので、ロードに時間は多少かかってしまいますが、その後は軽くなります。

これもドキュメントに記載があります。

一般的に、v-if はより高い切り替えコストを持っているのに対して、 v-show はより高い初期描画コストを持っています。 そのため、とても頻繁に何かを切り替える必要があれば v-show を選び、条件が実行時に変更することがほとんどない場合は、v-if を選びます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/20 18:18

    回答ありがとうございました。
    指摘いただいた通り、コードを書き直したところ、上手く作動しました。
    詳しい説明も付けていただき、本当にありがとうございます。
    合わせて、学習させていただきます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る