ループで処理

Question

//「年」でEnterキーが押された時
$('input[name=txtYear1]').keypress( function (e) {
if ( e.which == 13 ) {
// 「月」にフォーカスを当てる
$('input[name=txtMonth1]').focus();
return false;
}
});

//「月」でEnterキーが押された時
$('input[name=txtMonth1]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「日」にフォーカスを当てる
            $('input[name=txtDay1]').focus();
	return false;
}
});

//「日」でEnterキーが押された時
$('input[name=txtDay1]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「」にフォーカスを当てる
            $('input[name=txtYear2]').focus();
	return false;
}
});

//「年」でEnterキーが押された時
$('input[name=txtYear2]').keypress( function (e) {
if ( e.which == 13 ) {
// 「月」にフォーカスを当てる
$('input[name=txtMonth2]').focus();
return false;
}
});

//「月」でEnterキーが押された時
$('input[name=txtMonth2]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「日」にフォーカスを当てる
            $('input[name=txtDay2]').focus();
	return false;
}
});

//「日」でEnterキーが押された時
$('input[name=txtDay2]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「」にフォーカスを当てる
            $('input[name=txtYear3]').focus();
	return false;
}
});

//「年」でEnterキーが押された時
$('input[name=txtYear3]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「月」にフォーカスを当てる
            $('input[name=txtMonth3]').focus();
	return false;
}
});

//「月」でEnterキーが押された時
$('input[name=txtMonth3]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「日」にフォーカスを当てる
            $('input[name=txtDay3]').focus();
	return false;
}
});

//「日」でEnterキーが押された時
$('input[name=txtDay3]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「」にフォーカスを当てる
            $('input[name=txtYear1]').focus();
	return false;
}
});

上記の処理をループ文を使って下記のコードを書いてみました

for (var i = 1; i < 3; i++) {
//「年」でEnterキーが押された時
$('input[name=txtYear + i]').keypress( function (e) {
if ( e.which == 13 ) {
// 「月」にフォーカスを当てる
$('input[name=txtMonth + i]').focus();
return false;
}
});

//「月」でEnterキーが押された時
$('input[name=txtMonth + i]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「日」にフォーカスを当てる
            $('input[name=txtDay + i]').focus();
	return false;
}
});

//「日」でEnterキーが押された時
$('input[name=txtDay + i]').keypress( function (e) {
if ( e.which == 13 ) {
            // 「」にフォーカスを当てる
            $('input[name=txtYear + i + 1]').focus();
	return false;
}
});

}

実行すると
Error: Syntax error, unrecognized expression: input[name=txtYear + i]

と表示されます

'input[name=txtYear + i]' この部分の文法(書き方が悪い)のは解るのですが、対応方法が解らず投稿しました

よろしくお願いします

Comments

[kei344]

コードはコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

[aglkjggg]

msz006氏はマルチポストや自分で調べもせず何度も質問を行っていますね。

Answer 1

この手の処理はnameで判断するのではなくtabindexで処理したほうが良いのでは？

$(function(){
  $('input[type=text]').on('keydown',function(e){
    if(e.keyCode==13){
      var ti=parseInt($(this).prop('tabindex')); 
      if(ti<=0) return false;
      var flg=(typeof $('input[tabindex='+(ti+1)+']').get(0)!=="undefined");
      $('input[tabindex='+(flg?ti+1:1)+']').focus();
    }
  });
});

<form>
年1:<input type="text" name="txtYear1" tabindex="1"><br>
月1:<input type="text" name="txtMonth1" tabindex="2"><br>
日1:<input type="text" name="txtDay1" tabindex="3"><br>
年2:<input type="text" name="txtYear2" tabindex="4"><br>
月2:<input type="text" name="txtMonth2" tabindex="5"><br>
日2:<input type="text" name="txtDay2" tabindex="6"><br>
年3:<input type="text" name="txtYear3" tabindex="7"><br>
月3:<input type="text" name="txtMonth3" tabindex="8"><br>
日3:<input type="text" name="txtDay3" tabindex="9">
<input type="submit" name="go"><br>
dummy1:<input type="text" name="dummny1"><br>
dummy2:<input type="text" name="dummny2"><br>
dummy3:<input type="text" name="dummny3"><br>
</form>

Answer 2

ループでやるのも良いと思いますが、
上記のサンプルを実現するのであれば、DOM構造も含めて整理すれば、以下のコードで済みませんか？

//「日」でEnterキーが押された時
$('input.keychk').keypress( function (e) {
if ( e.which == 13 ) {
// 「」にフォーカスを当てる -> 次のコントロールにフォーカスを移動すると解釈、DOM構造もそのように設定する
$(this).next().focus();
return false;
}
});