php - phpmysqlのid=$POST['id']の場所を削除したい

Question

エントリを削除するための私のコードはこれですが、何もしていません

HTML

<form id="form3" name="form3" method="post" onsubmit="return validateForm();" action="">
    Id <input type="text" class="txt" name="id" /><br />
    <input type="submit" id="delete" value="delete"/>
</form>

PHP

global $wpdb;

if ( isset ( $_POST['id'] ) && ! empty ( $_POST['id']  ))
{
    $wpdb->query("DELETE " . PRO_TABLE_PREFIX . "tutorial  WHERE id='{$_POST['id']}'");
}

score 1 · Accepted Answer

それは私が通常それをする方法です：

$wpdb->query("DELETE " . PRO_TABLE_PREFIX . "tutorial  WHERE id='".$_POST['id']."')

score 1 · Accepted Answer

このような混乱を避けるために、私は常にsprintf()文字列を連結する必要がある場所を使用します

変化する：

global $wpdb;
if ( isset ( $_POST['id'] ) && ! empty ( $_POST['id']  )) {
    $wpdb->query("DELETE " . PRO_TABLE_PREFIX . "tutorial  WHERE id='{$_POST['id']}'");
}

に：

global $wpdb;

if ( isset ( $_POST['id'] ) )) {    

   $wpdb->query(sprintf("DELETE %stutorial  WHERE id='%s'", PRO_TABLE_PREFIX, $_POST['id']));
}

注意すべき点がいくつかあります。

1）SQLインジェクションに対して脆弱です
2）isset()のキーが$_POST['id']実際にそうではないかどうかを判断するために使用した後は、そのキーがNULL空であるかどうかを確認する必要はありません。empty()

アップデート

あなたは本当に$_POST['id']それが有効かどうかをテストする必要があります。次のような関数を実装することをお勧めします。is_id_valid()

function is_id_valid(&$id){ //<-- IMPORTANT, Argument should be a reference 

  if ( ! isset($id) ){
     return false;
  }

  if ( empty($id) ){
    return false;
  }

  // add this if you expect $id to be a numeric value
  // otherwise just ignore - do not add
  if ( ! is_numeric($id) ){
    return false;  
  }

  //it's also a good to validate the length 
  if ( strlen($id) > ... && strlen($id) < ... ){
     return false;
  } 

  //seems like all tests passed
  return true;
}

次に、次のように使用します

if ( is_id_valid($_POST['id']) !== false ){
   ....
}

警告：それはまだSQLインジェクションに対して脆弱です

score 0 · Accepted Answer

周りの一重引用符を削除しますpost['id']：

$wpdb->query("DELETE " . PRO_TABLE_PREFIX . "tutorial  WHERE id={$_POST['id']}")

また

echo $query = "DELETE " . PRO_TABLE_PREFIX . " tutorial  WHERE id =".mysql_real_escape_string($_POST['id']);
$wbpd->query($query);

score 0 · Accepted Answer

$wpdb->query("DELETE FROM " . PRO_TABLE_PREFIX . "tutorial  WHERE id='{$_POST['id']}'");

PS：そこに行って、あなたを最も助けた1つの答えを受け入れてください。そしてここにも！：P

score 0 · Accepted Answer

次のコードを試してください：-

global $wpdb;

if ( isset ( $_POST['id'] ) && ! empty ( $_POST['id']  ))
{
$wpdb->query("DELETE FROM " . PRO_TABLE_PREFIX . "tutorial  WHERE id=".$_POST['id']);
}

score 0 · Accepted Answer

give some action path to the form

html

<form id="form3" name="form3" method="post" onsubmit="return validateForm();" action="give some actions">
Id <input type="text" class="txt" name="id" /><br />
<input type="submit" id="delete" value="delete"/>

php

global $wpdb;

if ( isset ( $_POST['id'] ) && ! empty ( $_POST['id']  ))
{
  $id=stripslashes_deep($_POST['id']);
  $wpdb->query("DELETE FROM " . PRO_TABLE_PREFIX . "tutorial  WHERE id=$id");
}

score -1 · Accepted Answer

WHERE句でidをチェックしているので、引用符で囲む必要はなく、deleteステートメントにFROMがないため、標準的な方法は次のようになります。データベースに挿入する前に、次のようにPOSTデータのフィルタリングを実行します。

$id = (int) $_POST['id'];
if( $id > 0 ) {
    $wpdb->query("DELETE FROM " . PRO_TABLE_PREFIX . "tutorial  WHERE id=".$_POST['id']);
}

php - phpmysqlのid=$POST['id']の場所を削除したい

7 に答える 7

Related

Reference