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

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

ただいまの
回答率

87.78%

createElementで生成した要素を他の関数で使用する方法を知りたい。

解決済

回答 2

投稿 編集

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

score 24

addTodoで生成したliを他の関数で使いたい。

svgアイコンにクリックイベントを付与してクリックされたアイテム、その親を含めて丸ごと削除する為に用いる。

発生している問題

renderTodo()の中身にあるliを他の関数に渡せない問題。

→createElementで生成した要素にクリックイベントを追加する為にグローバル関数でsvgボタンを定義。
const icon = document.querySelector('.todo__item').children;

//エラー
Uncaught TypeError: Cannot read property 'children' of null

該当のソースコード

<div class="todo">
                <h2 class="heading-2 todo__title">To Do List</h2>
                <form action="" class="todo__form" id="todoForm">
                    <input class="todo__input" type="text" id="todoInput">
                    <button class="todo__button" id="todoButton">add</button>
                </form>
                <ul class="todo__list" id="todoList">
                    <!-- <li class="todo__item">
                        <svg class="todo__icon">
                            <use xlink:href="img/sprite.svg#icon-cross"></use>
                        </svg>
                    </li>
                    <li class="todo__item">
                        <svg class="todo__icon">
                            <use xlink:href="img/sprite.svg#icon-cross"></use>
                        </svg>
                    </li> -->
                    <!-- <li class="todo__item">
                        <svg class="todo__icon">
                            <use xlink:href="img/sprite.svg#icon-cross"></use>
                        </svg>
                    </li>
                    <li class="todo__item">
                        <svg class="todo__icon">
                            <use xlink:href="img/sprite.svg#icon-cross"></use>
                        </svg>
                    </li>
                    <li class="todo__item">
                        <svg class="todo__icon">
                            <use xlink:href="img/sprite.svg#icon-cross"></use>
                        </svg>
                    </li> -->
                </ul>
            </div>
let todoList = [];

const todoForm = document.getElementById('todoForm');
todoForm.addEventListener('submit', event => {
    event.preventDefault();
})

const todoButton = document.getElementById('todoButton');
todoButton.addEventListener('click', () => {
    const input = document.getElementById('todoInput');
    const text = input.value.trim();
    if (text !== '') {
        addTodo(text);
        input.value = ''
        input.focus();
    }
})

function addTodo(text) {
    const todo = {
        text,
        checked: false,
        id: Date.now()
    };
    todoList.push(todo);
    console.log(todoList);
    renderTodo(todo);
}

function renderTodo(todo) {
    let list = document.getElementById('todoList');
    let li = document.createElement('li');
    li.setAttribute('class', 'todo__item');
    li.setAttribute('check', todo.checked);
    li.setAttribute('id', todo.id);
    li.innerHTML += `
    ${todo.text}
    <svg class="todo__icon">
    <use xlink:href="img/sprite.svg#icon-cross"></use>
    </svg>
    `;
    list.appendChild(li);
    return li;
}

const icon = document.querySelector('.todo__item').children;
console.log(icon);

試したこと

How to pass created element to another function

//onclickが動かない問題

function renderTodo(todo) {
    let list = document.getElementById('todoList');
    let li = document.createElement('li');
    li.setAttribute('class', 'todo__item');
    li.setAttribute('check', todo.checked);
    li.setAttribute('id', todo.id);
    li.innerHTML += `
    ${todo.text}
    <svg class="todo__icon">
    <use xlink:href="img/sprite.svg#icon-cross"></use>
    </svg>
    `;

     document.querySelector('.todo__icon').onclick = function (e) {
         const parentLi = this.parentElement;
         parentLi.style.border = 'solid red';
     }

    list.appendChild(li);
}

//addEventListnerが動かない問題

function renderTodo(todo) {
    let list = document.getElementById('todoList');
    let li = document.createElement('li');
    li.setAttribute('class', 'todo__item');
    li.setAttribute('check', todo.checked);
    li.setAttribute('id', todo.id);
    li.innerHTML += `
    ${todo.text}
    <svg class="todo__icon">
    <use xlink:href="img/sprite.svg#icon-cross"></use>
    </svg>
    `;

    document.querySelector('.todo__item svg').addEventListener('click',()=>{
        const parentLi = this.parentElement;
        parentLi.style.border = 'solid red';
    })

    list.appendChild(li);
}

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

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+1

まず一つ目はエラーが出る問題だと思いますが

const icon = document.querySelector('.todo__item').children;

renderTodo 処理が終わる前にこの行が呼ばれると存在しないのでエラーが起こってしまいます。

const iconElement = document.querySelector('.todo__item')
if (iconElement) {
  const icon = iconElement.children
}

のようにするか、クリックイベントの部分に書くか、renderTodo 後に呼び出す位置に移動するなどで問題ないと思います。

本題の他の要素に渡せない(識別できない)問題について

li.setAttribute('id', todo.id); ここで適切に id が振られているのであれば

document.querySelector('.todo_item svg').onclick = function (e) { // this を取るので (e) => は使えない
  const parentLi = this.parentElement
  parentLi.style.border = "solid red"
}

というかたちでクリック時に親要素を取り出せます。

あと id はドキュメント内で一意になるように気をつけてください。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2021/07/22 16:55

    回答ありがとうございます。
    ””本の他の要素に渡せない(識別できない)問題について”

    こちらのコードを書いてみたところエラーが表示されてしまいました。
    onclickが使用できない問題です。

    キャンセル

check解決した方法

0

function renderTodo(todo) {
    let list = document.getElementById('todoList');
    let li = document.createElement('li');
    li.setAttribute('class', 'todo__item');
    li.setAttribute('check', todo.checked);
    li.setAttribute('id', todo.id);
    li.innerHTML += `
    ${todo.text}
    <svg class="todo__icon">
    <use xlink:href="img/sprite.svg#icon-cross"></use>
    </svg>
    `;

    const icon = li.children[0];
    icon.addEventListener('click', (event) => {
        const parentElement = icon.parentElement;
        parentElement.remove();
    })

    list.appendChild(li);
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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