javascript replaceが上手く置き換わらないのかforループがおかしいのか

Question

前提・実現したいこと

javascriptで数独を作ろうと思っております。
以下のコードで実行してみるとエラーはでないのですが、

試しに一番上の行を
０，１，２，３，４，５，６，７，８
と選択してみると

０のところが9になって、出力されてほしいのですが、上手くいきません。

0のところが、123456789なってそのまま出力されます。

123456789はkouhoという変数でおいているので、replaceがおかしいのか、もしくはどこかで上書きされて123456789に戻ってしまっているのかもしれません。

コードを書いていただかなくても構いません。
「ここのせいで、上手くいかないんだよ！」など、些細なアドバイスでも構いませんので、よろしくお願い致します。

該当のソースコード

 <div id="box"></div>
        <button type=“button” onclick="buttonClick()">決定</button>
        <button type=“button” onclick="judge()">計算</button>
        <div id="ans"></div>

var values=[] 
function buttonClick(){
  for(let i=1 ; i<=9 ; i++){
    values[i-1] = [];
    for(let j=1 ; j<=9 ; j++){
      values[i-1][j-1] = document.getElementById("num"+ i + "_" + j).value;
    }
  }
}


//表示させる関数
function output(){
  let str="";
  for(let i=1 ; i<=9 ; i++){
    for(let j=1 ; j<=9 ; j++){
      str = str + " " + values[i-1][j-1];
    }
    str = str + "<br>";
  }
  document.getElementById("ans").innerHTML = str;
}

//i行目で被ったら候補から消す
function judge_x(i){
    let kouho = "123456789";
    for(let j=1 ; j<=9 ; j++){
        let igyou = values[i-1][j-1];
            for(let k=1 ; k<=9 ; k++){
                if(igyou ==="k"){
                    kouho = kouho.replace("k",""); 
                }
            } 
     }
  return kouho;
}

//全ての判断
function judge(){
    for(let i=1 ; i<=9 ; i++){
        for(let j=1 ; j<=9 ; j++){
            if(values[i-1][j-1]==="0") {
                // values[i-1][j-1]="×";
                values[i-1][j-1] =judge_x(i);
            }
       } 
    }
        output();
 }
 


//9×9を作る
function appendBox(i) {
  let box = document.createElement('div');
  box.id = "num" + i;
  document.getElementById("box").appendChild(box);
  return box;   
}
function appendSel(i, box) {
  let sel = document.createElement('select');
  sel.name = box.id + "_" + i;
  sel.id = box.id + "_"  +i;
  box.appendChild(sel);
  return sel;
}
function appendOpt(i,sel){
  let option = document.createElement('option');
  option.value = i;
  option.text = i ;
  sel.appendChild(option);
}
for(let i =1 ; i<=9 ; i++){
  let box1 = appendBox(i);
  for(let i =1 ; i<=9 ; i++){
    let sel1 = appendSel(i, box1);
    for(let i =0 ; i<=9 ; i++){
      appendOpt(i, sel1);       
    }
  }
}

Answer 1

for(let k=1 ; k<=9 ; k++){
    if(igyou ==="k"){
        kouho = kouho.replace("k",""); 
    }
} 

このif文だと「igyouに入っている文字列が"k"のとき」しか置換できません。
おそらく意図としてはループカウンタのkの値を文字列にして比較したかったのだと思いますが、"k"だと変数kを文字列にしたことにはなりません。

for(let k=1 ; k<=9 ; k++){
    if(igyou === String(k)){
        kouho = kouho.replace(igyou,""); 
    }
} 

でおそらく質問者さんの意図通りの動作だと思います。

Comments

[shidada]

なるほど！！
string型に直さないといけないのですね。
読んでいただきさらに考えていただき、ありがとうございます。
無事、望んでいる動作になりました。