２回クリックでアラート文を表示

Question

以下のプログラムで各マスにおいて一度クリックしたマスをもう一度クリックするとアラート文が表示したいのですがどのようにプログラムを組めばいいんでしょう？
今のプログラムだと全体で２回以上クリックするとアラート文が表示されてしまう状態です。

<script>
try{
  document.addEventListener ('click',function(e){myfunc(e)},true);
}catch(e){
  document.attachEvent('onclick',function(e){myfunc(e)});
}
var cnt=0;
function myfunc(e){
  var t = (e.srcElement || e.target);
  if(t.nodeName=="INPUT" && t.type=="button"){
    if(t.value==""){
      cnt++;
      t.value=cnt%2==1?"○":"×";
      if(cnt>=9){
        alert("GAMEOVER!(OKでリセットします)");
        resetFunc(t);
        cnt=0;
      }else if(cnt >=2){
 alert("no!");
 }
 }
  }

  
  if(t.nodeName=="INPUT" && t.type=="reset"){
    cnt=0;
    resetFunc(t);
  }
}
function resetFunc(t){
    var f=t.form;
    for(var i=0;i<f.length;i++){
      if(f[i].type=="button") f[i].value="";
    }
}
</script>
</head>
  <form name="tick">
  <table>
    <tr>
        <td><input type="button"></td>
        <td><input type="button"></td>
        <td><input type="button"></td>
    </tr>
    <tr>
        <td><input type="button"></td>
        <td><input type="button"></td>
        <td><input type="button"></td>
    </tr>
    <tr>
        <td><input type="button"></td>
        <td><input type="button"></td>
        <td><input type="button"></td>
    </tr>
  </table>

  <input type="reset" value="リセット">
  </p>
  </form>
  </body>

よろしくお願いします。

Answer 1

if(cnt >=2)は必要ないです。

if(t.value==""){
//省略
}else{
  alert("no!");
}

みたいな感じでいいと思います。たぶん。



インデントはしっかりしないとバグの元ですぜ。

Comments

[Alpha]

これに修正するとクリックした瞬間にアラート文が出てしまいます。
アラート文を出したいのは「一度クリックしてしまったマス(既に○or×になっているマス)をもう一度クリックしてしまった場合」です。

[Zuishin]

一度クリックしたマスというのは t.value != "" のマスですから Lhankor_Mhy さんの方法で正しいのではないですか？

[Alpha]

<Zuishinさん>Lhankor_mhyさんの方法だとどこのマスをクリックしても一回目で「no!」のアラート文が表示されてしまいます。

[Lhankor_Mhy]

たぶんですね、else文を書く場所を間違えてるんじゃないかと思いますよ。かっこの対応をチェックしてみてください。インデントはしっかり書いた方がいいですよ。

現在、if(cnt >=2){... が書いてある場所は間違っています。

[Alpha]

ちなみにもう一度クリックというのはダブルクリック時ではありません。

[Lhankor_Mhy]

うんそうですね。
もう一度かっこの対応をチェックしてみてください。

[Alpha]

見直したらif(cnt>=9){にelse ifがかかっていてif(t.value==""){にかかるように修正したらできました。ありがとうございます！！

Answer 2

cnt が一つしか用意されていないので当然ですね。
マスの数だけ用意すべきです。

Comments

[Alpha]

となると<input type="button">にそれぞれ属性などを置いてからcntの処理をさせるということでしょうか？