Codeigniterで$ _POSTを使用してデータベースレコードを削除する

Question

私は間違ったことをしてきましたが、私はURIセグメントの値をとっていました。だから私は私のアプローチを変え、$ _POSTですべてを手に入れました。私はこれを正しくしているかどうか分からないが、誰かが光を当てることができるだろうか？私のビューには、DBから取り出された表データ項目が含まれています。各項目には、「表示」と「削除」という2つのリンクがあります。コードはうまくいくように見えますが、コード化できるかどうかは不思議でした。私はフォーム名が一意ではないことを忘れていたので、レコードを削除しようとすると、常に最新のレコードが削除されます（最後の隠しフィールドが設定されました）。

myview.php URI IDを経由して削除すると、完全に罰金です（スニペット）

<?php foreach($records as $record): ?> 
    <form method="POST" name="myform<?php echo $location->id;?>" action="/location/delete"> 
    <a href="/location/view/<?php echo $location->id;?>">View</a> <a href="#" onclick="document.myform<?php echo $location->id;?>.submit();">Delete</a> 
    <br /> 
    <input type="hidden" name="location_id" value="<?php echo $location->id;?>"> 
    </form> 
<?php endforeach ?> 

Answer 1

私はこの問題をどのように解決したのですか。

私のコントローラでは、フォームフィールドが$_POSTを介して送信されたかどうかを確認するdeleteというメソッドがあります。変数がない場合は、エラーメッセージを使ってどこかにリダイレクトしてください。フィールドが渡された場合は、通常のチェックを行ってレコードを削除できることを確認します。

if(!isset($_POST['item_id'])) 
{ 
    $this->session->set_flashdata('message', 'item cannot be removed!'); 
    redirect("/item"); 
} 


if($this->input->post('item_id')) {   
    ... code .... 
    ... code .... 
} 

Answer 2

の表示/、私は$_POSTを使用することは間違っていると言うことは、ベンチャーが、すべてのための新しいユニークなフォームを作成していないだろうdelete要素はひどく乱雑で、あなたが得ているものと比べて重視されています（露出IDはありませんか？）、私は削除関数にuriを使用する方が正しいと考えています。

特定のレコードのみを削除できるようにするには、削除機能自体でプログラムで処理しますが、その要求は$_POSTによってのみ送信されるということには依存しません。これは信頼できるものではなく、誰でも投稿リクエストを生成することができます。

Answer 3

あなたの構文エラーは、この行を次のとおりです。

<?php foreach($records as $record): ?> 
     <form method="POST" name="myform<?php echo $location->id;?>"  action="/location/delete"> 
     <a href="/location/view/<?php echo $location->id;?>">View</a> <a href="#" onclick="document.myform<?php echo 
$location->id;?>.submit();">Delete</a> 
     <br /> 
      <input type="hidden" name="location_id" value="<?php echo $location->id;?>"> 
     </form> 
     <?php endforeach ?> 

あなたは、フォームのためにループすることができません。代わりに、次のコードを使用してください：

<form method="POST" name="myform<?php echo $location->id;?>" action="/location/delete"> 
<?php foreach($records as $record): ?> 
    a href="/location/view/<?php echo $location->id;?>">View</a> <a href="#" onclick="document.myform<?php echo $location->id;?>.submit();">Delete</a> 
     <br /> 
     <input type="hidden" name="location_id" value="<?php echo $location->id;?>"> 
    <?php endforeach ?> 
     </form>