論理削除の手順について

Question

##解決したいこと
PHPで論理削除を実行したい時、以下のような手順で考え方はあっているか知りたい。

##手順

削除ボタンを押すと、ポップアップで削除するか確認。

OKが押されると、以下のjsが走り、$flag=1を返す。
※.deleteは、削除ボタンのクラスを指してます。

  $(".delete").click(function() {
    if(!confirm("削除を実行しますか？")) {
      return false;
    } else {
      return $flag = 1;
    }
  });

  public function delFlg($id) {
    $sql = 'UPDATE players SET del_flg = :del_flg WHERE id = :id';
    $sth = $this->dbh->prepare($sql);
    $sth->bindParam(':id', $id, PDO::PARAM_INT);
    $sth->bindParam(':del_flg', 1, PDO::PARAM_INT);
    $sth->execute();
    return $result;
  }

$flag=1が返ってきたときに、delFlg関数が実行され、del_flgが1に更新される。
※del_flgはplayersテーブルにある論理削除用のカラムで、デフォルト値は0。0の時にデータは表示、1の時にデータは非表示にするように実装済み。

この関数で、del_flgの値が1に更新されて、データ非表示→論理削除成功！

こんな流れで論理削除を実装したいのですが、jsからdelFlg関数に$flag=1をどうやって渡し、del_flgの値を1に更新すればいいのかでつまずいています。

まとめると、
削除ボタンをおす→削除する実行→$flag=1を返す→delFlg関数を実行→関数内のdel_flgの値を1に更新→
データを非表示

Answer 1

一般的には同期（form送信など）か非同期（Ajaxなど）か　どっちか。
どんな形でも正しくリクエストを送られればいいです。

「やり方」が「何が正しいか」を決めるのは仕様、要件次第。
HowではなくWhyやWhatから考えてください。

固定で1入れるなら別にフロント側から渡す必要ないでしょうね。
削除対象だけ送ってください。
※共通処理にしたいなら渡したうえで渡された情報をセットする

→データを非表示

これはシステム的な表現ではないので設計するときは「del_flgが1以外を取得」としたほうが良いでしょう。

あと、削除とデータ取得は一緒にしてはロジックとして成り立ちません。
今回同じステートメントで取得できるのは「作用した件数」です。
役割分担を考えてください。何でもかんでも詰め込んでしまうとファットコントローラ、ファットモデルになるので。

Answer 2

<?PHP
$del=filter_input(INPUT_GET,"del");
$sql = 'UPDATE players SET del_flg =true WHERE id = ?';
$stmt = $pdo->prepare($sql);
$stmt->execute([$del]);
?>
<script>
document.addEventListener('click', (e)=>{
  if([...document.querySelectorAll('.delete')].includes(e.target)){
    if(!confirm('削除?')) e.preventDefault();
    
    console.log(e.target.href);
  }
});
</script>

<a href="?del=1" class="delete">削除</a>
<a href="?del=2" class="delete">削除</a>
<a href="?del=3" class="delete">削除</a>

updateした場合fetchはできません
deleteするときはセッションなどで削除権限を持っていることをチェックしてください