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

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

ただいまの
回答率

87.37%

todoリストのタスクの状態を変更させたい

解決済

回答 1

投稿

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

score 10

実装したい機能:todoリストのタスクの状態を作業中、完了と変更させたい
わからない部分:todoリストのタスクの状態を作業中から完了にする事はできたのですが、かんりょうから作業中にすることができないのと、一枚目の写真の写真の状態で、ID3の削除ボタンを押すと、二枚目の写真のようにID0の部分に作業中がくるという現象が起きてしまっています。
現在のコードは

'use strict';

const addbtn = document.getElementById('add_btn');
const addvalue = document.getElementById('add_value');
const text = document.getElementById('add_text');
const todos = [];
const displayTodos = (selecttodos) => {
  addvalue.textContent = '';
  selecttodos.forEach((todo) => {
    const createTr = document.createElement('tr'); //trを作成
    const createId = document.createElement('td'); //tdを作成
    createId.textContent = todo.taskid; //idに要素数をカウント
    const createComment = document.createElement('td');
    createComment.innerHTML = todo.comment;
    const status_td = document.createElement('td');
    createTr.appendChild(createId); //idをtrの子要素として追加
    createTr.appendChild(createComment); //commentをtrの子要素として追加
    createTr.appendChild(status_td); //status_tdをtrの子要素として追加
    status_td.append(createstatusbutton(todo));
    status_td.append(createremoveButton());
    addvalue.appendChild(createTr);
    text.value = '';
  });
};

function createTodo() {
  const taskid = todos.length;
  const comment = text.value;
  const status = '作業中';
  const todo = {
    taskid,
    comment,
    status,
  };
  return todo;
}

addbtn.addEventListener('click', () => {
  const todo = createTodo();
  todos.push(todo);
  radioChange();
});

const createstatusbutton = (todo) => {
  const createstatusBtn = document.createElement('button');
  createstatusBtn.textContent = todo.status;
  createstatusBtn.addEventListener('click', () => {
    if (todos.status = '作業中') {
      console.log(todo);
      todo.status = '完了';
    } else if(todos.status = '完了') {
      todo.status = '作業中';
      console.log(todo)
    }
    radioChange();
  });
  return createstatusBtn;
};

const createremoveButton = (index) => {
  const todo = createTodo();
  const createremoveBtn = document.createElement('button');
  createremoveBtn.textContent = '削除';
  createremoveBtn.addEventListener('click', () => {
    todos.splice(index, 1);
    radioChange();
    todos.reduce((Idnum, todo) => (todo.taskid = Idnum + 1), -1);
    radioChange();
    console.log(todos)
  });
  return createremoveBtn;
};

function radioChange() {
  const todo = createTodo();
  const radio1_1 = document.getElementById('radio1_1');
  const radio1_2 = document.getElementById('radio1_2');
  const radio1_3 = document.getElementById('radio1_3');

  if (radio1_1.checked) {
    todos.slice();
    return displayTodos(todos);
  } else if (radio1_2.checked) {
    let filterdoing = todos.filter((todo) => {
      return todo.status === '作業中';
    });
    return displayTodos(filterdoing);
  } else if (radio1_3.checked) {
    let filterdone = todos.filter((todo) => {
      return todo.status === '完了';
    });
    return displayTodos(filterdone);
  }
}
<!DOCTYPE html>
<html lang="ja">

<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <link rel="stylesheet" href="css/todo.css">
  <title>Document</title>
</head>

<body>
  <div class="radio_button">
    <input type="radio" name="syurui" checked="checked" id="radio1_1" onchange="radioChange()">すべて
    <input type="radio" name="syurui" id="radio1_2" onchange="radioChange()">作業中
    <input type="radio" name="syurui" id="radio1_3" onchange="radioChange()">完了 </div>
  <div class="task">
    <table border="0">
      <thead>
        <tr>
          <td>ID</td>
          <td>コメント</td>
          <td>状態</td>
        </tr>
      </thead>
      <tbody id="add_value">

      </tbody>
    </table>
  </div>
  <header class="header">
    <h1 class="title">新規タスクの追加</h1>
  </header>
  <div class="textbox">
    <input type="text" id="add_text" value="">
    <input type="button" value="追加" class="add_task_button" id="add_btn">
  </div>

  <script src="js/todo.js"></script>

</body>

</html>


です。
該当部分のコードは、

const createstatusbutton = (todo) => {
  const createstatusBtn = document.createElement('button');
  createstatusBtn.textContent = todo.status;
  createstatusBtn.addEventListener('click', () => {
    if (todos.status = '作業中') {
      console.log(todo);
      todo.status = '完了';
    } else if(todos.status = '完了') {
      todo.status = '作業中';
      console.log(todo)
    }
    radioChange();
  });
  return createstatusBtn;
};


です
何が悪いのかわからないです。
ご教授よろしくお願いします。
![イメージ説明
イメージ説明

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

1つ目の問題は
= と == の違いです。 
あと人のことを言えた義理じゃないんですが、
全体的に見辛いコードだなぁと感じました。
命名だったりコードの書き方だったり
ですので最初から書き直すのをおすすめします。

コメントアウトをもっと利用するとかJSDocを書くとかおすすめです。

もう一つの問題は他の人に任せます。

        const createstatusbutton = (todo) => {
            const createstatusBtn = document.createElement('button');
            createstatusBtn.textContent = todo.status;
            createstatusBtn.addEventListener('click', () => {
                if (todo.status == '作業中') {
                    todo.status = '完了';
                } else {
                    todo.status = '作業中';
                    console.log(todo)
                }
                radioChange();
            });
            return createstatusBtn;
        };

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/12/18 08:21

    できました
    ありがとうございました

    キャンセル

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

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

関連した質問

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