Vue.js:配列を「v-for」で回してテキスト内容とボタンを動的に作成する処理で2回目から内容とボタンが動的に作成されない。

Question

前提・実現したいこと

いつもお世話になっております。
下記質問1の解決方法についてご教授いただきたいです。
※可能であれば質問2にもご教授いただきたいです。

質問1
Vue.jsで下記機能実装時に1回目は作成されるのですが、2回目以降処理が走っていないのか動的に作成されません。

・機能：テキストボックスに入力された内容を配列に入れて、チェックボックスとボタンを動的に作成する。

実現したいこと：登録ボタン押下時にテキストボックスの内容とチェックボックスと削除ボタンが動的に作成されること

どなたか解決方法のご教授お願い致します。

質問2
「v-for」の認識は下記のようなものだと思っているのですが、認識として合っていますのでしょうか？
公式サイトの「Vue.js」も確認はしたのですが、認識があっているか不安なので質問しました。
・<タグ名 v-for=""></タグ名> ←左記のタグ名の間をループ処理する。
※今回のソースだと「App.vue」の下記「ループ部分」が登録ボタンを押下するたびに処理される。
■ループ部分
<input type="checkbox" id="check" v-model="todolist">
<label for="check">{{todolists}}</label>
<input type="button" v-on:click="deleatebutton" value="削除">

該当のソースコード

<template>
  <div id="app">
      <h1>ToDoManagement</h1>
      <input type="text" v-model="textinput">
      <input type="button" v-on:click="insertbutton" value="登録">
      <p><span>{{textinput.length}}</span>文字</p>
      <div v-for="todolists in todolist" v-bind:key="todolists">
        <input type="checkbox" id="check" v-model="todolist">
        <label for="check">{{todolists}}</label>
        <input type="button" v-on:click="deleatebutton" value="削除">
      </div>
  </div>
</template>

<script>
export default {
  data() {
    return {
      textinput: '',
      todolist: [],
    };
  },
  methods: {
    insertbutton() {
      if (this.textinput === '') return;
      this.todolist.push(this.textinput);
    },
  },
};

</script>

<style lang="scss">
#app {
  font-family: 'Avenir', Helvetica, Arial, sans-serif;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
  text-align: center;
  color: #2c3e50;
}
#nav {
  padding: 30px;
  a {
    font-weight: bold;
    color: #2c3e50;
    &.router-link-exact-active {
      color: #42b983;
    }
  }
}
</style>

Answer 1

質問の事象の回答ではないのですが、一応気になったので。

<div v-for="todolists in todolist" v-bind:key="todolists">
  <input type="checkbox" id="check" v-model="todolist">

ここ、少しおかしいです。

inputのv-modelに設定されているのがtodolistですが、これ配列なのでinputのvalueに設定してもまともな値は入らないです。
たぶんtodolistsの方を入れたいのかな？と思いますが...

そもそもv-forに利用する変数名の付け方がイマイチです。

v-for="配列から取り出した要素 in ループしたい配列"なので、「取り出した要素」にsを付けて複数形にしてしまうと混乱の原因になります。
この場合、v-for="todo in todolist"の方が適切かなと思います。

Comments

[pokemn]

アドバイスありがとうございます。
確かに下記だと私の付けている変数名はおかしいですね。
「v-for="配列から取り出した要素 in ループしたい配列"」
ご指摘ありがとうございます。

Answer 2

動かしてみたところ
deleatebuttonというメソッドが存在しないのに指定されていることに対してエラーが出ています。
そこを削除するか、メソッドを実装すれば動くと思います。

v-forに関してはタグ名の間だけでなく、タグ自体もループ処理されます。
今回だとdivも含まれます。

Comments

[pokemn]

質問1~2の回答ありがとうございます。

・質問1
初回は動的に作成されていたので、メソッドを実装していないことが原因
だとは考えませんでした。

・質問2
v-forはタグ自体もループをするのですね。勉強になりました。
ありがとうございます。