html フォームの編集 変更 保存 jquery

Question

###前提
マイページにてお知らせメールを受け取る設定の編集画面のhtmlを組みました。
表示はスクリーンショットのようになっています。

###実現したいこと
設定の変更を表示に反映させたいと思っています。
下記流れの２と３のやり方を教えていただけたらと思います。

１「編集」ボタン → 「受け取る」「受け取らない」の選択肢が表示される →
２「受け取らない」に変更 → 保存 → 「キャンペーンやお知らせなどのメールを受け取らない」に表示を変更する
３「受け取らない」に変更 → キャンセル → 編集前の選択肢に戻す

###該当のソースコード

//ボタン表示の切り替え
$(function(){
    $('.edit_off').each(function(){
        $(this).css({display:"none"});
    });
    $(".edit_on").click(function(){
        $(this).css({display:"none"});
        $(this).siblings('.edit_off','.cancel').css({display:"inline"});
    });

    $('.edit_off').click(function(){
        $(this).css({display:"none"});
        $(this).siblings('.edit_off').css({display:"none"});
        $(this).siblings('.edit_on').css({display:"inline"});
    });
    
    $('.cancel').click(function(){
        $(this).css({display:"none"});
        $(this).siblings('.edit_off').css({display:"none"});
        $(this).siblings('.edit_on').css({display:"inline"});
    });
});
//メール受信 ラジオボタン ボックス開閉
$(function(){
    $('#mailMagazine_open').click(function(){
        $('#mailMagazine_box').show();
        //キャンセル
        $(this).siblings(".cancel").click(function(){
            $('#mailMagazine_box').hide();
        });
        //保存
        $(this).siblings(".save").click(function(){
        });
    });
});

                        <!--受信設定-->
                        <tr>
                            <th scope="row" rowspan="2">受信設定</th>
                            <td><span class="checkbox_data">キャンペーンやお知らせなどのメールを受け取る</span>
                                <img src="/images/mypage/btn_edit.png" alt="編集" class="edit edit_on" id="mailMagazine_open">
                                <img src="/images/mypage/btn_save.png" alt="保存" class="save edit_off">
                                <img src="/images/mypage/btn_cancel.png" alt="キャンセル" class="cancel edit_off"></td>
                        </tr>

<!--受信設定 編集-->
                        <tr>
                            <td id="mailMagazine_box">
                                <span class="mwform-radio-field vertical-item"><label for="mailMagazine01" class="checkbox checkbox_left">
                                <input type="radio" name="mailMagazine" value="受け取る" checked="checked" id='mailMagazine01'/>受け取る</label></span>
                                <span class="mwform-radio-field vertical-item"><label for="mailMagazine02" class="checkbox radiobox_right">
                                <input type="radio" name="mailMagazine" value="受け取らない"  id="mailMagazine02" />受け取らない</label></span>
                            </td>
                        </tr>

Answer 1

現状コードに合わせて書いてみました。

//初期値を設定
var checkbox_dataValue_true = "キャンペーンやお知らせなどのメールを受け取る",
	checkbox_dataValue_false = "キャンペーンやお知らせなどのメールを受け取らない",
	inputValue_true = "受け取る",
	inputValue_false = "受け取らない";

$(".save").click(function(){
	//ラジオボタンの値を受け取る
	var inputValue = $("input[name=mailMagazine]:checked").val();
	//初期値と見比べる
	var txt = (inputValue === inputValue_true) ? checkbox_dataValue_true : checkbox_dataValue_false;
	//チェックボックスの値を修正
	$(".checkbox_data").text(txt);
});

失礼ですが今のコードは無駄が多いので色々修正した方がいいと思います。
・DOMのreadyイベントは1つでいいと思います。
・よく使うセレクタはキャッシュする。
・$('.edit_off').eachでdisplay:noneを入れてますが、初期値がそうなら手入力でいれておいた方がいいです。環境やページ量次第でReadyイベントは遅れます。イベントが走るまで開いた状態になる。
・同ページに1度しかでない要素ならセレクタはID指定がいいです。DOM全体を探してしまうので無駄な処理が発生します。

Comments

[may88seiji]

ありがとうございます。
大変さんこうになりました。質問してよかったです。

[IShix]

喜んでいただけてよかったです。せっかくなので修正した方が良さそうな部分を書きますのでご自分で対応してみてください。

・受け取る、受け取らないの文言が一文字でも違うと動作しないので文言を変更しても動作するようにする。
JQueryには独自のタグを使ってデータを受け渡すJQuery.data()という関数があるのでそれを使うといいと思います。

簡単に説明しますとinputにdata-自分の好きな半角英数字を追加します。
<input type="radio" name="mailMagazine" data-mail-magazine-status="YES" (略)
受け取り側で
var inputValue = $("input[name=mailMagazine]:checked").data("mail-magazine-status");
でチェックが入ったラジオボタンのdata-mail-magazine-status=以降のデータが受け取れます。

・キャンセル処理は無視しているのでキャンセルをクリックすると今表示されている文章を元にチェックボックスを入れなおすようにしてもいいかもしれません。
まず下準備で受け取る、受け取らないと各チェックボックスを紐付ける。
ユーザーがクリックした内容を格納する変数を定義して初期値に受け取るを指定する。
編集をクリックと同時に変数の内容を反映させる。
(チェックボックスを入れる $(selector).prop("checked",true); )
あとは、ユーザーが選択したデータを変数に格納する。

Answer 2

//メール受信 ラジオボタン ボックス開閉
$(function(){
    $('#mailMagazine_open').click(function(){
        $('#mailMagazine_box').show();
        //キャンセル
        $(this).siblings(".cancel").click(function(){
            $('#mailMagazine_box').hide();
        });
        //保存
        $(this).siblings(".save").click(function(){
            if ($('input[name=mailMagazine]:eq(1)').prop('checked') == true){
                $(this).siblings('.checkbox_data').html("キャンペーンやお知らせなどのメールを受け取らない");
            }else{
                $(this).siblings('.checkbox_data').html("キャンペーンやお知らせなどのメールを受け取る");
            }
            $('#mailMagazine_box').hide();
        });
    });
});

Comments

[may88seiji]

閲覧いただいた皆様、ありがとうございました。