変数と配列の要素を比べるループ

質問

すいません。質問タイトルがわかりにくくて、、、
ある数字と、count=5;
順番の不規則で長さも不明な例外番号が入った配列　
ExceptionNum=(20,2,16,4,9,12,13,9,18,5,11)
があり
例外番号と一致しないときそのまま
例外番号と一致する場合はcountより大きく例外番号と一致しない
最小の整数を返す関数を作っています。
例の場合
引数→返り値
5→6
//5が例外のため
3→3
//例外にない
11→14
//11,12,13が例外のため

となってほしいのですが
数が大きくなるとなぜかうまく動きません。
どなたか教えていただけますでしょうか？

function checkDeleteList(count){
var flag=0;
//alert(count);
while (flag == 0) {
flag=1;
for (i = 0; i < ExceptionNum.length; i++) {
if (count == ExceptionNum[i]) {
flag = 0;
}
}
if (flag == 0) {
count++;
}
}
//alert(count);
return count;

}

よろしくお願いいたします。

回答No.1

yambejp · ベストアンサー

>数が大きくなるとなぜかうまく動きません。

どのくらいの数で、どううまく動かないのでしょうか？
たとえばこんな感じでどうでしょうか？

<script>
function check(num){
var ExceptionNum=new Array(20,2,16,4,9,12,13,9,18,5,11);
var temp=new Array();
for(var i=0;i<ExceptionNum.length;i++){
temp[ExceptionNum[i]]=1;
}
while(temp[num]==1){
num++;
}
return num;
}

document.write("5→"+check(5)+"<br>");
document.write("3→"+check(3)+"<br>");
document.write("11→"+check(11)+"<br>");
document.write("10000→"+check(10000)+"<br>");
</script>

お礼

なるほど、代入する用の配列を一回作ればよかったんですね。
whileの使い方を勘違いしていたため失敗していたようです。

ご丁寧にありがとうございました。

非常に参考になりました。