JQuery でli の要素の中身をループで調べて内容を取得して削除したいが変数に格納されない

Question

<ul id="member">
  <li>山田 孝雄</li>
  <li>川田 優実</li>
  <li>高橋 健太</li>
  <li>長谷川 仁</li>
  <li>横山 肇</li>
  <li>黒田 ひとみ</li>
  <li>山内 久美</li>
</ul>

というリストがあります。
この内容をループで調べて元の内容と一致したら削除したいのですが、
上手くいきません。

//例えば引数で
var str="横山 肇";

//対象者一覧からの削除
var count_li	=	$("ul#member li").length;
//対象のラベルの場所を探る
for(var j = 0;j < count_li ;j++){
	var name_li =	$("ul#member li:eq(j)").text();
	if(str == name_li){
		var pos	=	j;
	}
}
$("li:eq(pos)").remove();

横山 肇さんだけを削除するようなことをしたいと思っています。
ところが、
name_liに変数が入ってきません。
どこがおかしいでしょうか？
宜しくお願いいたします。

Answer 1

ループする意義がよくわかりませんが

$(function(){
  var str="横山 肇";
  $('#member li').filter(function(){return $(this).text()==str;}).remove();
});

Comments

[hidepon]

有難うございます。たしかにこちらの方法であればループに意味がないですね。いつもありがとうございます！！

Answer 2

 var name_li =    $("ul#member li:eq(j)").text();

$("li:eq(pos)").remove();

この行らがおかしいですね。変数j、変数posの値を利用したいのですから、それぞれ

 var name_li =    $("ul#member li:eq(" + j + ")").text();

$("li:eq(" + pos + ")").remove();

このようにする必要があります。
他の人の回答のように、.eqメソッドで外に出すと、よりよいコードになりますね。

で、それはそれとして、こういう処理をしたいときは、

var str="横山 肇";

$("#member li").filter(function(){
  return $(this).text() == str;
}).remove();

これだけでできますよ。
削除対数が複数あった場合でも適切に動作してくれます。

Comments

[hidepon]

ご返答有難うございます。すごすぎます・・・一発で削除できました。有難うございました
。

Answer 3

$("ul#member li:eq(j)")ではなく、$("ul#member li").eq(j)ではないでしょうか？

Comments

[hidepon]

有難うございます。なるほど・・・。書き方がまちがっていたんですね・・・勉強不足です。。。

Answer 4

セレクタに"ul#member li:eq(j)"と書いているので、変数ではなく本当にjが渡されてしまいます。

$("ul#member li").eq(j)と書けば、目的のことをできると思います。

Comments

[hidepon]

いつも有難うございます。こちらの記述方式で上手くいきました。