JSPのコードで冗長な箇所をすっきりさせたい

// 1の時
if ($trs.find("[id$='changeId']").val() === "1"){
if($trs.find("[id$='kbnChange']").val() != "9"){
$trs.find("[id$='kbnChange']").val("1");
$trs.find("[id$='kbnChange_hidden']").val("1");
}

//2の場合
}else if ($trs.find("[id$='changeId']").val() === "2"){
$trs.find("[id$='kbnChange']").val("1");
$trs.find("[id$='kbnChange_hidden']").val("1");

//3の場合
}else if ($trs.find("[id$='changeId']").val() === "3"){
var kbn = ['3','4','5','7','8','9','10'];
if(!($trs.find("[id$='kbnChange']").val() === $trs.find("[id$='kbnChange']").val("3")
|| $trs.find("[id$='kbnChange']").val("4")
|| $trs.find("[id$='kbnChange']").val("5")
|| $trs.find("[id$='kbnChange']").val("7")
|| $trs.find("[id$='kbnChange']").val("8")
|| $trs.find("[id$='kbnChange']").val("9")
|| $trs.find("[id$='kbnChange']").val("10")
{
$trs.find("[id$='kbnChange']").val("3");
$trs.find("[id$='kbnChange_hidden']").val("3");
}

//4のの場合
}else if ($trs.find("[id$='changeId']").val() === "4"){
$trs.find("[id$='kbnChange']").val("2");
$trs.find("[id$='kbnChange_hidden']").val("2");


//5のの場合
}else if ($trs.find("[id$='changeId']").val() === "5"){
$trs.find("[id$='kbnChange']").val("x");
$trs.find("[id$='kbnChange_hidden']").val("x");

}
}

JSP初心者なのですが、やりたいことは
①条件が「1または2または4または５以外の場合3の設定を行うです。
②また、３の場合は||で繋いでいるのでjavaのcontainメソッドの様な感じで
実装の行いたいのですが、うまくいかずどなたかご教授ください。

自分でやったことは
①3の条件式を最後に持ってきました
//　３の場合
var Id= ['1','2','3','4','5'];
}else {
var kbn = ['3','4','5','7','8','9','10'];
if(!($trs.find("[id$='kbnChange']").val() === kbn)){
　　　　$trs.find("[id$='kbnChange']").val("3");
$trs.find("[id$='kbnChange_hidden']").val("3");
}

No.1 回答者： Ogre7077 回答日時： 2018/12/04 15:05

これは JSP ファイル内に埋め込まれた JavaScript のプログラムと見受けられます。

JSP や Java とは何ら関係ない言語なので、混同しないようにしましょう。

同じ値を複数個所で使うなら変数にまとめましょう。すっきりします。
例)
var id = $trs.find("[id$='changeId']").val();
if (id == "1") { ... }
例)
var kc = $trs.find("[id$='kbnChange']");
var hi = $trs.find("[id$='kbnChange_hidden']");
kc.val("1");
hi.val("1");

else if が連続するならば switch の導入を検討ください。
「1または2または4または５以外の場合」が見通しやすく書けます。

「containメソッドの様な感じで実装の行いたい」ならば、
配列から値を検索する JavaScript のメソッドを利用しましょう。
https://developer.mozilla.org/ja/docs/Web/JavaSc …
https://developer.mozilla.org/ja/docs/Web/JavaSc …