Jquery でinput のvalue値を取得しようとするとundefinedになってしまう

Question

一覧をチェックして重複登録しないように仕組みを検討してます。

<div id="member">
  <input type="checkbox" class="members" value="200">
  <input type="checkbox" class="members" value="250">
  <input type="checkbox" class="members" value="300">
  <input type="checkbox" class="members" value="350">
  <input type="checkbox" class="members" value="400">
  <input type="checkbox" class="members" value="450">
  <input type="checkbox" class="members" value="500">
</div>

このような感じの項目があります。
他のセクションから項目を選択して上記のvalue値と重複しないようであれば
追加したいと思っています。

//例えば引数で
var val="250";



var row_num	=	$("div#member input").length;
var j		=	0;
for(var i = 1;i <= row_num ;i++){
	val_sb	=	$("div#member input:eq(" + i + ")").val();
	if(val == val_sb){
		j++;
	}
}
if(j>0){
alert("重複しています");
}else{
alert("登録できます")
}

といった感じの仕組みです。
値を取得しようとしましたがundefinedになってしまいます。

count やlengthを使うなどの情報があり試してみましたが、力業でやってみています。
宜しくおねがいいたします。

Comments

[kei344]

まだ質問が「受付中」になっていますが、いったん「解決済」にされてはいかがでしょうか。また、解決されていないなら状況を質問文に追記ください。

[hidepon]

申し訳ございません。ご回答を参考にプログラムをいじっていました。解決済みにします。ご指摘有難うございました。

Answer 1

250は最初からあるので、とりあえず220を追加してみました

$(function(){
  $('input[type=button][value=220追加]').on('click',function(){
    if($('#member .members').filter(function(){return $(this).val()=="220";}).length>0){
      alert("重複しています");
    }else{
      $('#member').append($('<input>').attr({type:'checkbox'}).val('220').addClass('members'));
      alert("追加しました");
    }
  });
});

<div id="member">
  <input type="checkbox" class="members" value="200">
  <input type="checkbox" class="members" value="250">
  <input type="checkbox" class="members" value="300">
  <input type="checkbox" class="members" value="350">
  <input type="checkbox" class="members" value="400">
  <input type="checkbox" class="members" value="450">
  <input type="checkbox" class="members" value="500">
</div>

<input type="button" value="220追加">

Comments

[zohnam]

そこまで書いたなら
if($('#member .members[value=220]').length>0){
でいいですやんｗ

[yambejp]

zohnamさん。ありがとうございます。
たしかに、checkboxのvalueはそれでよさそうですね
type=textだとちょっと事情が違うので汎用的な書き方しちゃいました

[hidepon]

ご返答有難うございます。す、すごい超高レベルの会話・・・ついて行けない・・・orz.

Answer 2

質問内容だけから判断すると、

for(var i = 1;i <= row_num ;i++){

この部分がおかしいですね。

for(var i = 0;i < row_num ;i++){

こうしなければなりません。
最後の1回は存在しないinputのvalueを取得しようとしているため、val_sbがundefinedになっているのでしょう。
とは言え、前回の質問ではこのミスをしていないのですから、事情があって1個ずらしているのだとも思えますが…？

Comments

[hidepon]

ご返答有難うございます。スミマセン。なぜか単純ミスです。お恥ずかしい限りです。こういう時にfindみたいな感じの記述が出来ればカッコいいかなと思って考えてみたんですが、どうも無理っぽくて、力業にしました。本当に有難うございます