会員情報をUPDATEで非表示

Question

前提・実現したいこと

（初心者なのであほなことを言ってるかもしれません）
名前をデータベースに登録して、削除を押すとデータベースのdel_flgとidをGETで送ってdel_flgを１にして
del_flgが０のものだけを表示するというものなんですがうまく動きません

GETで送っているつもりなんですが１しかおくられてきません

発生している問題・エラーメッセージ

該当のソースコード

function del_player($host,$db_user,$db_pass,$db,$id){
    $sql = "UPDATE player SET del_flg=1 WHERE id=".$id.";";

    $cn=mysqli_connect($host,$db_user,$db_pass,$db);
    mysqli_set_charset($cn, 'utf8');
    mysqli_query($cn, $sql);

    mysqli_close($cn);
}

<?php foreach($player_list as $player): ?>
  <tr>
  <td><?php echo $player['name']; ?></td>
  
  <form action="" method="GET" enctype="multipart/form-data">
  <td><button type="samit" name="touroku">削除</button></td>
  </form>
  </tr>
<?php endforeach; ?>


if(isset($_GET['id'])){
  var_dump($_GET);
  del_player('localhost','root','','bowling',$_GET['id']);

  exit;
}

Comments

[Orlofsky]

[MySQL]タグを追加しては？

[Orlofsky]

タイトルは「会員情報をDELETE」としては？

[popopop]

ありがとうございます編集しました！

[Orlofsky]

DELETEのスペルが間違っています。
きちんと仕事ができる人に助っ人を頼みましょう。

会社なら即クビになっても当然です。

[退会済みユーザー]

button type="samit" って？submitのミスかな？

[Orlofsky]

summit なら 山頂や頂上なんだが。

[m.ts10806]

commitだとあながち間違いではないかもしれない・・（ここで大喜利しても・・）

[m.ts10806]

>（初心者なのであほなことを言ってるかもしれません）
とか余計なことは書かないで欲しいなぁ。初心者アイコンだけでいいし、要件だけ書いてほしい。「あほなことと思えるなら自分で解決できるんじゃない？」って感じる。

[kei344]

まだ質問が「受付中」になっていますが、「ベストアンサー」を選び「解決済」にされてはいかがでしょうか。

[m.ts10806]

結局どのように解決されたのでしょうか。
https://teratail.com/help/question-tips#questionTips4-2

kei344さんも急かしているわけではなく、質問者さんの「できました」を以てコメントされているだけなので、本当に解決されたならきちんと「どう解決したか」はコメントしてください。
「言われたからとりあえず解決済みにする」のが一番よくありません（回答のコメントにつけた通り）

[popopop]

遅れてすいません
idがちゃんとおくれていなかったのが原因でした

Answer 1

おかしいところが散見されます。

• formタグはtdタグなどの外ではなく中に入れましょう。
• samitではなくsubmitでは。デフォルト値がsubmitなのでたまたま送信されていますが、記述は間違いです。
• formタグにある入力コントロールはあくまでbuttonだけなので何も送信していません。var_dump($_GET);してますが、idは送信されてますか？
• 「del_flgが０のものだけを表示するというものなんですが」とありますがどこにもそのコードはないようです。ただ、「更新処理をしてからその後取得」をしないと「削除フラグが0のものを表示」はできません。
• del_player()とネーミングするのでしたらDB接続情報は不要でしょう。別途コネクションを確立するメソッドを作るべきです。
• $_GETはスーパーグローバル変数です。もし$_GET参照がマストなら引数で渡す必要はありません。
• 入力情報そのまんまSQLに投入しているのでSQLインジェクションの脆弱性があります。プリペアドステートメントを利用しましょう。というか、特別な理由がないのであればmysqli*系でDB関係を対応するのではなくPDOで対応してください

コードも結構つまんで提供されているように見受けられるので、現状から言えるのはこれくらいで（適宜追記してください）

Comments

[popopop]

なんとかできました
助かりました

[m.ts10806]

回答をどのように理解したいのかそれでは分かりません。
正しく理解できてないと同じようなところでつまずき、同じ質問をすることになりますし、それでは回答した人の厚意が無駄になります。
https://teratail.com/help/question-tips#questionTips4-2

Answer 2

まずはGET方式とPOST方式の違いを理解したほうが良いかと思います。

GETとPOSTの違いについて

Comments

[popopop]

なんとかできました
助かりました

Answer 3

下記部分でidを渡していません。

  <form action="" method="GET" enctype="multipart/form-data">
  <td><button type="samit" name="touroku">削除</button></td>
  </form>

Comments

[popopop]

なんとかできました
助かりました

Answer 4

function del_player($host,$db_user,$db_pass,$db,$id){
$sql = "UPDATE player SET del_flg=1 WHERE id=".$id.";";

function 名が del_player ですけど、SQLの命令が UPDATEです。

DELETE FROM player WHERE id = ...

MySQLのマニュアルでDELETE命令を確認してください。

なお、間違ってデータを削除してどうしようもなくなる前に経験者の支援を受けた方が良いです。

MySQL SQL 入門

でググって勉強もしてください。

Comments

[JinwonKim]

論理削除を行う意味ではupdate文が合ってると思います。

[Orlofsky]

この質問者は見捨てます。

[popopop]

なんとかできました
助かりました