js クリックした要素のidを取得したい

Question

<ul class="list">の<li>タグが押された時に ```ここに言語を入力 <a href="#"><img lang="jp" id="nyanco" src="画像のURL";></a> ``` の中のlang と　idを取りたいのですが今のソースコードではうまく動いてくれません

lang と　idを取得するにはどうしたら良いのでしょうか？

現在のコード

<ul class="list">
  <li class="item" data-category="still"
    data-date="1992-01-20"
    data-score="11111">
    <a href="#"><img lang="jp" id="nyanco" src="画像のURL"></a>
　</li>
</ul>
  <li class="item" data-category="still"
    data-date="1992-01-20"
    data-score="22222"
    <a href="#"><img lang="jp" id="wanko" src="画像のURL"></a>
</li>

$(document).ready(function () {　

    function test(e) {
        e.preventDefault();
        e.stopPropagation();
        var t=e.target;
        var id = t.id;
        var lang = t.lang;
        alert(lang);
        alert(id);
    }
$(document).on('click', '.list li', test);
}

Comments

[m.ts10806]

html がおかしいところが散見されますが大丈夫でしょうか。li閉じてないのにaが始まったり、imgタグ最後の「;」だったり。jQueryも一部変ですね。　(document).ready　→　$(document).ready エラーなく動作するソースコードをご提示ください。

[amaguri]

imgタグがphpの変数のものをそのままコピペしたので;が残ってしましましたすみません。

[m.ts10806]

回答つきましたが、たぶんまだjavascriptのエラー出ると思いますよ。htmlがおかしいところも修正しておいてくださいね。

[amaguri]

liタグの中にimgタグ入れるのは何か問題ありますか？調べたのですが可能と記載されていましたが...

[amaguri]

http://webdesign-css-html-koyaman.hatenablog.com/entry/2016/10/01/025155

[amaguri]

liタグの中にaタグでしたね、特に禁止されてはいないようなのですが。。。

[m.ts10806]

いえ、<li><a><img /></a></li>とするのは全く問題ないです。<li <a>・・・ となってるところが問題です。

[amaguri]

すみませんコピペミスになります修正します

Answer 1

$(function(){
  $(document).on('click', '.list li', function(){
    console.log($(this).find('img').attr('id'));
    console.log($(this).find('img').attr('lang'));
  });
});

Comments

[amaguri]

回答ありがとうございます！試して見たいと思います！