php - 準備されたステートメントを実装せずに mysqli を使用する方法は?

Question

<?php

...........

$connection = mysqli_connect('......com', 'login', 'password', 

'dbname'); 
if (!$connection) { 
    die('Connect Error (' . mysqli_connect_errno() . ') '
            . mysqli_connect_error());
}

echo "Connection Successful";



// creating a query 
mysqli_query($connection, "INSERT INTO `users` (`userid`,`firstname`, `lastname`, `username`, `email`, 

`password`) VALUES (NULL, $firstname, $lastname, $username,$email,$password);");
mysqli_close($connection);



// printing to the screen
echo "Welcome $lastname . You have successfully logged in!";

?>

接続はOKですが、値が挿入されません。コードを改善するにはどうすればよいですか? 値がデータベースに挿入されない理由がわかりません。変数を確認しましたが、問題ありません。

Answer 1

防御インジェクションとプレースホルダーの使用にラッパー mysqli::query() を使用します

<? class db{
private static $_instance;
private $db;
public function q(){return $this->db->query($this->make(func_get_args()));}
public function __get($property){return $this->db->$property;}
public function __call($method,$args){return call_user_func_array(array($this-   >db,$method),$args);}

public function qr(){
  $q = $this->make(func_get_args());
  echo $q;
  return $this->db->query($q);
}

public static function getInstance($conf){
  if(null===self::$_instance){
    self::$_instance = new self($conf);
  }
  return self::$_instance;
}

private function make($args){
  $tmpl =& $args[0];
  $tmpl = str_replace("%","%%",$tmpl);
  $tmpl = str_replace("?","%s",$tmpl);

  foreach($args as $i=>$v){
    if($i && !is_int($v)){
      $args[$i] = "'".$this->db->real_escape_string($v)."'";
    }
  }

  for($i=$c=count($args)-1; $i<$c+20; $i++)
    $args[$i+1] = "UNKNOWN_PLACEHOLDER_$i";
  return call_user_func_array("sprintf", $args);
}

private function __construct($conf){
  $this->db = new  mysqli($conf['mysql_host'],$conf['mysql_user'],$conf['mysql_pass'],$conf['mysql_db']);
  }

private function __clone(){
}
}

$mysql = db::getInstance([
'mysql_host' => '...',
'mysql_user' => '...',
'mysql_pass' => '...',
'mysql_db' => '...'
]);


$mysql->q('INSERT INTO `user` SET `first_name`=?,`last_name`=?',$f_name,$l_name);