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

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

ただいまの
回答率

89.97%

jQueryでcheckbox:checkedに対して処理を行いたい

解決済

回答 2

投稿 編集

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

thatnaoki

score 26

 前提・実現したいこと

localStorageを使ってメモアプリを作ろうとしています。
jQueryを使って実装していて、作ったメモがテーブル型に並びます。
並んだメモはチェックボックスにチェックを入れてdeleteボタンを押すと、
選択されたメモがlocalStorageからremoveされて、再読込でHTML上でも消えるように実装したいです。

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

しかし、チェックボックスを入れずにdeleteボタンを押しても、localStorageのメモが削除されてしまいます。

 該当のソースコード

<!DOCTYPE html>
<html>
<head>
    <meta charset="UTF-8">
    <title>MemoPad</title>
    <script src="js/jquery-2.1.3.min.js"></script>
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" integrity="sha384-BVYiiSIFeK1dGmJRAkycuHAHRg32OmUcww7on3RYdg4Va+PmSTsz/K68vbdEjh4u" crossorigin="anonymous">
    <link rel="stylesheet" href="css/style.css">
</head>

<body>
    <header>
        <h1>MemoPad</h1>
    </header>

    <div class="container">
        <div class="form-container">
            <input class="form" id="title" type="text" placeholder="タイトル">
            <textarea class="form" id="content" placeholder="メモ内容" cols=30 rows=5></textarea>
            <div class="button-wrapper"></div>
                <button class="btn btn-success btn-block" id="speech">speech</button>
                <button class="btn btn-primary btn-block" id="save">save</button>
                <button class="btn btn-danger btn-block" id="delete">delete</button>
            </div>
        </div>
    </div>
    <div class="container">
        <table class="table">
            <thead>
                <tr>
                    <th>チェック</th>
                    <th>タイトル</th>
                    <th>メモ内容</th>
                </tr>
            </thead>
            <tbody id="memo-list">
                <!-- ここに挿入 -->
            </tbody>
        </table>
    </div>
    <script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/js/bootstrap.min.js"></script>
    <script src="js/script.js"></script>
</body>
</html>
// メモ表示
window.onload = function(){
    for(let i = 0; i < localStorage.length; i++) {
    let key = localStorage.key(i);
    let value = localStorage.getItem(key);
    console.log(value);
    data = JSON.parse(value);
    console.log(data);
    $('#memo-list').append(
        '<tr class="memo" id="memo_' + String(key) + '">' +
        '<td>' + '<input type="checkbox">' +
        '</td>' + '<td>' + data.title + '</td>' +
        '<td>' + data.content + '</td>' + '</tr>'
    );
    }
};
// localstorage用のkeyを作る
const makeKey = function() {
    let keys = [0];
    for(let i = 0; i < localStorage.length; i++) {
        keys.push(localStorage.key(i));
    }
    let maxNum = Math.max.apply(null,keys);
    return 1 + maxNum;
}

// Save
$("#save").on('click', function() {
    let key = makeKey();
    console.log(key);
    // 変数で入力を受け取る
    const title = $("#title").val();
    const content = $("#content").val();
    // 連想配列に入れる
    const data = {title: title, content: content};
    // ストレージへ格納
    localStorage.setItem(key, JSON.stringify(data));
    // 入力を消す
    $("#title").val("");
    $("#content").val("");
    // アラート
    alert("Saved!");
    // リロード
    window.location.reload();
});

// delete
$("#delete").on('click', function() {
    for(let i = 0; i < localStorage.length; i++) {
        let key = localStorage.key(i);
        if ($('#memo_' + key).has("input[type='checkbox']:checked")) {
            localStorage.removeItem(key)
        }
    }
    window.location.reload();
});

 試したこと

input[type='checkbox']:checkedの部分が効いてないのかなと思うのですが、
どう書き換えたらよいのかわからず、、ご教授いただけますでしょうか。

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

Chrome
バージョン: 70.0.3538.67(Official Build) (64 ビット)

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+2

こちらが参考になるかと思います。
https://www.ipentec.com/document/javascript-jquery-get-checkbox-value

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/10/25 15:01

    ありがとうございます!hasをpropに変更したところ動作するようになりました!

    キャンセル

checkベストアンサー

+1

以下でいけそうな気がします。

for(let i = 0; i < localStorage.length; i++) {
    let key = localStorage.key(i);
    // if ($('#memo_' + key).has("input[type='checkbox']:checked")) {  // コメントアウト
    if ($('#memo_' + key).has($("input[type='checkbox']:checked")).length) {
        localStorage.removeItem(key)
    }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/10/25 15:02

    ありがとうございます!lengthを使うと複数のチェックボックスをチェックした時にうまく動作させることが難しく、has()→prop()に変更したところ動作しました。

    キャンセル

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

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