動的に生成した要素への指定（jQuery）

Question

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

・動的に生成した”削除ボタン”に対する下記コードが動作しない
//$(this).prevtAll('input[type="file"]').val(null);
//$(this).prevAll(".filename").html("選択されていません");

・削除ボタンクリック後に生成したinput="file"ボタンが動作しない（ボタンの切り替えは成功しているもののファイル読み込みができない。）


エラーメッセージは出ておらず、迷宮入りしてしまいました。
ご教示頂けないでしょうか。よろしくお願いいたします。

該当のソースコード

< -- html -- >

<div id="input-group"> <input type="file" id="01" name="01" class="none"> <label for="01" id="input-label">ファイルを選択</label> <span class="filename">選択されていません</span> </div>

    $("input[type='file']").on('change',function(){
         var file = $(this).prop('files')[0];
         $(this).nextAll('#input-label').replaceWith($('<button class="file_clear">削除</button>').clone(true));
         $(this).nextAll('.filename').html(file.name);
    });

    $(document).on('click', '.file_clear',function(){
         $(this).replaceWith($('<label id="input-label">ファイルを選択</label>').clone(true));
         $(this).prevtAll('input[type="file"]').val(null);
        $(this).prevAll(".filename").html("選択されていません");
    });

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

動作の流れ
”ファイルを選択ボタン”
↓クリック
”削除ボタン”に切り替え（生成）”、同時に.filenameを変更
↓クリック
”ファイルを選択ボタン”に切り替え（生成）、同時に.filenameを変更

Answer 1

いろいろ突っ込みどころがありますが

• まず、「prevtAll」→「prevAll」
• nextAll('#input-label')・・・idで指定しているのでnextで位置関係を指定する必要ある？
• nextAllやprevAllなどすべての要素を対象とする意味ある？
• 自分を書き換えた後自分との相対位置を探すのはNG
• 削除ボタンの後ろの.filenameを操作しているのに削除ボタンでは前の.filenameを掴むの？

sample

総じてこんな感じ
想定と違う構成でしたらHTMLも提示した上で挙動を再度説明してください

<script>
$(function(){
  $("input[type='file']").on('change',function(){
    var file = $(this).prop('files')[0];
    $(this).nextAll('#input-label').replaceWith($('<button class="file_clear">削除</button>').clone(true));
    $(this).nextAll('.filename').html(file.name);
  });
  $(document).on('click', '.file_clear',function(){
    $(this).prevAll('input[type="file"]').val(null);
    $(this).nextAll(".filename").html("選択されていません");
    $(this).replaceWith($('<label id="input-label">ファイルを選択</label>').clone(true));
  });
});
</script>
<input type="file">
<div id="input-label"></div>
<div class="filename"></div>
<div class="filename"></div>
<div class="filename"></div>

追記

今回の課題はlabelのforが消えてしまうため、
削除ボタンを押した後ファイル選択が効かないことです
削除せずに非表示にするとかで対応してもよいかもしれません

<script>
$(function(){
  　$(":file").on('change',function(){
    var file = $(this).prop('files')[0];
    $(this).siblings('.input-label').hide().after($('<button class="file_clear">削除</button>'));
    $(this).siblings('.filename').text(file.name);
  });
  
  $(document).on('click', '.file_clear',function(){
    $(this).siblings(':file').val(null);
    $(this).siblings('.filename').text("選択されていません");
    $(this).siblings('.input-label').show()
    $(this).remove();
  });
});
</script>
<div>
　<input type="file" id="01" name="01" class="none">
　<label for="01" class="input-label">ファイルを選択</label>
　<span class="filename">選択されていません</span>
</div>
<div>
　<input type="file" id="02" name="02" class="none">
　<label for="02" class="input-label">ファイルを選択</label>
　<span class="filename">選択されていません</span>
</div>

Comments

[退会済みユーザー]

yambejp様

ご指摘ありがとうございます。回答を参考にhtml，jsを書き換え問題点を１つ解決できました。

現状、発生している問題点は削除ボタンクリック後の
// $(document).on('click', '.file_clear',function(){　
以下で生成したファイル選択ボタンで
ファイルを読み込むことができないことです。

記載漏れしておりましたが下記htmlのように
<div class="input-group">
　<input type="file" id="01" name="01" class="none">
　<label for="01" class="input-label">ファイルを選択</label>
　<span class="filename">選択されていません</span>
</div>
<div class="input-group">
　<input type="file" id="02" name="02" class="none">
　<label for="02" class="input-label">ファイルを選択</label>
　<span class="filename">選択されていません</span>
</div>
.
.
.
と複数、同様の要素が存在する形です。

＿＿＿＿＿
　$("input[type='file']").on('change',function(){
var file = $(this).prop('files')[0];
$(this).nextAll('.input-label').replaceWith($('<button class="file_clear">削除</button>').clone(true));
$(this).nextAll('.filename').html(file.name);
});

$(document).on('click', '.file_clear',function(){
$(this).prevAll('input[type="file"]').val(null);
$(this).nextAll('.filename').html("選択されていません");
$(this).replaceWith($('<label class="input-label">ファイルを選択</label>').clone(true));
});
＿＿＿＿＿
動的に生成した要素箇所の兄弟要素への正しい指定が分からず..
ご教示頂けないでしょうか。よろしくお願いいたします。

[yambejp]

id="input-group"がダブっています。idは同じページに同じ名前で存在できません

[退会済みユーザー]

yambejp様

失礼しました。
こちらclassで記述していても生成したファイル読み込みボタンでは
読み込めないようです..。

[yambejp]

とりあえずHTMLを元に調整版をあげておきました

[退会済みユーザー]

@yamabejp

解決できました！また、説明が疎かのなか細かくご指摘頂きありがとうございました。
大変助かりました。

Answer 2

処理がわかりやすくなるよう、まるっと書き換えました。シンプルだと思います。

ただ、ファイル入力を隠したので、それの見た目がブラウザ依存にはなりませんが…。

また、これはやっていませんが、文言を統一的に管理するために、htmlに直書きせず変数で持っておき、初期化か何かでセットするようにしたほうがよいかもしれません。

コード

<html>

<head>
    <style>
        .invisible { display: none; }
    </style>
</head>

<body>
    <div id="input-group">
        <input type="file" id="iptFile" class="invisible" onchange="onChangeIptFile(event)">
        <input type="button" id="selectBtn" value="ファイルを選択" onclick="onClickBtn(event)">
        <input type="button" id="removeBtn" value="削除" class="invisible" onclick="onClickBtn(event)">
        <label for="btn">ファイルが選択されていません</label>
    </div>
    <script src="https://code.jquery.com/jquery-3.3.1.min.js"></script>
    <script>
        function onClickBtn(event) {
            var iptFile = $('#iptFile');

            // ファイル未選択であれば、ファイル入力要素のクリックイベントを発火させます
            if (iptFile.val() == '') {
                iptFile.click();
            }
            // ファイル選択済みであれば、選択を解除します
            else {
                // ファイル選択解除はイベントが発火しないので、こちらで発火させる
                iptFile.val('');
                onChangeIptFile(null);
            }
        }
        function onChangeIptFile(event) {
            var fileName = $('#iptFile').val();
            var label = $('#input-group > label');
            var selectBtn = $('#selectBtn');
            var removeBtn = $('#removeBtn');
            var invisibleClass = 'invisible';

            // ファイルが選択済みになった場合
            if (fileName != '') {
                label.text(fileName);
                selectBtn.addClass(invisibleClass);
                removeBtn.removeClass(invisibleClass);
            }
            // ファイルが未選択になった場合
            else {
                label.text('ファイルが選択されていません');
                selectBtn.removeClass(invisibleClass);
                removeBtn.addClass(invisibleClass);
            }
        }
    </script>
</body>

</html>

Comments

[退会済みユーザー]

@ppn様
ご教示頂きましてありがとうございました。
無事解決に至りました。