削除ボタン実行で削除 pdo

Question

現在、データベースから取得したデータを表示させ、それを削除ボタンにて削除したいと思っております。
その際に、削除ボタンを押してアラートを出しOKなら削除、キャンセルなら処理取り消しをしたく、jsとphpファイルを分けて記述しました。削除ボタンでアラートは出るもののphpに記述した削除処理を読み込ませるのがうまくできず削除できない状況となっております。

行なったこといたしましては、phpに script処理を無理やりのように埋め込んでボタンが押されたら→削除処理に行くよう記述したのですがどうにも削除ボタンを押したらアラートは出るが削除押さなくてもページをリロードすると消えてしまいます。
アドバイスをいただきたいです。

・削除ボタンでアラート表示OKなら削除

<?php

$sql = 'select id, name, kana, tel, email, body, created_at  from contacts';
$stmt = $db->query( $sql );

echo "<table>\n";
echo "\t<tr><th>id</th><th></th><th>name</th><th></th><th>kana</th><th></th><th>tel</th><th></th><th>email</th><th></th><th>body</th><th></th><th>created_at</th></tr>\n";
while( $result = $stmt->fetch( PDO::FETCH_ASSOC ) ){
    echo "\t<tr>\n";
    echo "\t\t<td>{$result['id']}</td>\n";
    echo "\t\t<td>　</td>\n"; 
    echo "\t\t<td>{$result['name']}</td>\n";
    echo "\t\t<td>　</td>\n"; 
    echo "\t\t<td>{$result['kana']}</td>\n";
    echo "\t\t<td>　</td>\n"; 
    echo "\t\t<td>{$result['tel']}</td>\n";
    echo "\t\t<td>　</td>\n"; 
    echo "\t\t<td>{$result['email']}</td>\n";
    echo "\t\t<td>　</td>\n"; 
    echo "\t\t<td>{$result['body']}</td>\n";
    echo "\t\t<td>　</td>\n"; 
    echo "\t\t<td>{$result['created_at']}</td>\n";
    echo "\t\t<td>　</td>\n"; 
    echo "\t\t<td><a href=edit.php? id=".$result['id'].">編集</a>"."</td>\n";
    echo "\t\t<td>　</td>\n"; 
    ?>
    <td><a href= contact.php class="deleteBtn"  data-id='<?php echo $result['id'] ;?>' onClick="return dspDelete();" >削除</a></td>
    <?php
    echo "\t</tr>\n";
  ?>
  
<script>
    $(".deleteBtn").click(function(){
      
        var btnid = $(this).data("id");
        console.log(btnid);
        deleteData(btnid);
        });

        
  
    function deleteData(btnid){
      if(window.confirm('削除しますか？')){
        $.ajax({
            type: 'POST',
            dataType:'json',
            url:'delete_func.php',
            data:{
                btnid:btnid,
            },
          
              success:function(data) {
                console.log(data);
                  window.location.href = "./";
              },
                  error:function(XMLHttpRequest, textStatus, errorThrown) {
                      alert(errorThrown);
                  
                  }
        
        });
      
    }
    }
</script>

<?php

  }
echo "</table>\n";
  
?>

delete_func.php

 $jsondata = $_POST['btnid'];
	

 $stmt = $db->prepare('DELETE FROM contacts WHERE id = '.$jsondata.' ');
 $stmt->execute(); 
 header('Content-type: application/json');
 echo json_encode( $jsondata );

Answer 1

画面表示したときにはPHPの仕事は全部終わってます。
つまり、表示したときには最後のidが必ず削除され続けているわけです。

表示と処理は完全に別個のリクエストにしてください。

今のコードだとcontact.phpに削除処理を書くことになりますが、現在の書き方だとidは渡らないのでGETリクエストを構成するためにクエリストリングの形式で記載すると良いです（あくまで手法の1つ）

Comments

[m.ts10806]

削除だけならAjaxで処理したほうが良いとは思いますが。

[shos3]

ご回答ありがとうございます。
記述方法がクエリ文字列にしてからわからず、ポイントをお教えしていただきたいです。

[shos3]

Ajaxの方で一度考えてみます。

[m.ts10806]

クエリストリングになってませんよ。
URL?key=value&key=value…
現状だと「編集」のほうには?がありません。「削除」のほうにはvalueしかありません。

クエリストリングでもAjaxでも必要な情報を何かしらの手法でリクエストに含める必要があります。

[shos3]

Ajaxにて行う際にコードを色々確かめたのですが思うようにいかない為、お時間があればお教えしていただきたいです。

[m.ts10806]

あの。「思うようにいかない」「うまくいかない」では赤の他人には何も分からないんですよ（たぶん前も指摘した）
何を参考にどういうコードを書いて何が起きているのか具体的に記載してください。質問は編集できます。

[shos3]

申し訳ございません。
追記させていただきました。
一度調べ、Ajaxを組み込みました。$result['id']にカラムのデータIDが入っている為、ボタンIDを取得しこの$result['id']を削除する様に記述しましたが正常に動かずでした。IDのところは他の値(idなど)で試したところエラーになった為、$result['id']にするとエラーが出ない為、取得はできていると思っております。
記述にてご指摘していただきたいです。

[m.ts10806]

deleteData()
呼び出す方の引数が空ですね。
return dispDelete();
あと、POSTでID送信するならcontact.phpに?を付ける必要がありません。
そもそもcontact.phpのコードはどうなっていますか？
返却値確認せずにlocation.hrefしていたり、無理にやろうとし過ぎてる気がします。

[shos3]

ご回答ありがとうございます。
現在、上記のコードで削除ができる様になりました。一点お聞きしたいことがありまして。。
削除前にアラートを出す様にしたのですが、以下の例えの様に複数回でてしまいます。ここのアラート問題についてon Clickで別ファイルjsでアラート出すようにしたり、 if(window.confirm('削除しますか？')){の位置を変えてみたりしたり、console.log(btnid);で取得したIDを表示させたりしたりして行きましたが、解決できませんでした。もし、お時間がありましたらお願いしたいです。

データベース登録表示
ID　　　名前
１０　　伊藤　　　　　　アラートで呼び出すと3回'削除しますか？が表示されてしまいます。
１１　　佐藤　　　　　　アラートで呼び出すと2回'削除しますか？が表示されてしまします。
１２　　五頭 アラートで呼び出すと1回'削除しますか？が表示されてしまいます。

[m.ts10806]

コードがどんどん意味分からんなってる気が

↓これだけで良かったはずなんですけどね。。
onClick="return deleteData('<?php echo $result['id'] ;?>');" >

全然関係ないと思いますが
ここに何か入ってますね。→>削除</a>

confirmが何回も出るのはループ回数分だけscriptが作られているからでは？whileの中に入ってますし。画面表示後、ブラウザの「ソースを表示」で生成されたHTML確認してみてください（動的Webアプリケーションでは必須です）。