PHP 変数DB一括挿入

Question

前提・実現したいこと

ランダムな数値を作成して
MYSQLへINSERTするという
路頭に迷った結果、変数でDB挿入しようとしています。
やりたいことはランダム作成した数値をDBへ一括挿入したいです。
現在悩んでいることは配列で分割した変数が未定義扱いされていることです。

該当のソースコード

<?php
//function でPDOっが使えないことを調べる・
$db = new PDO('mysql:dbname=c-pro;host=127.0.0.1;charset=utf8', 'root', '' );

$str2 = array();
    for ($i=0; $i<10;$i++) {
        $str3 = substr(str_shuffle("ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnpqrstuvwxyz0123456789"), 0, 15);
        $str2 = array_merge($str2, array($str3));
    }
    extract( $str2, EXTR_PREFIX_INVALID, "pass");

        $db->beginTransaction();
        try {
            $sql = $db->prepare("INSERT INTO enter_pas (password) VALUES ('.$pass_1'),('.$pass'),('.$pass_3'),('.$pass_4'),('.$pass_5'),('.$pass_6'),('.$pass_7'),('.$pass_8'),('.$pass_9'),('.$pass_10')");

        } catch (Exception $e) {
            $db->rollBack();
        }
?>

Answer 1

定義済み変数を確認してみてください。

$str2 = array();
for ($i=0; $i<10;$i++) {
    $str3 = substr(str_shuffle("ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnpqrstuvwxyz0123456789"), 0, 15);
    $str2 = array_merge($str2, array($str3));
}
extract( $str2, EXTR_PREFIX_INVALID, "pass");

var_dump(get_defined_vars());

get_defined_vars()
※スーパーグローバル変数などPHPがデフォルトで定義される変数も含みます

ない変数を利用していますよね。

echo  "INSERT INTO enter_pas (password) VALUES ('.$pass_1'),('.$pass'),('.$pass_3'),('.$;pass_4'),('.$pass_5'),('.$pass_6'),('.$pass_7'),('.$pass_8'),('.$pass_9'),('.$pass_10')";

/**
PHP Warning:  Undefined variable $pass in xx.php on line 10
PHP Warning:  Undefined variable $pass_10 in xx.php on line 10
**/

pass_で変数名つけてるのですからpassはないですし、<10で回しているので10は含まれません。

for ($i=0; $i<10;$i++){
    echo $i;
}

それに、文字列連結をしたいのであれば外がダブルクォーテーションであれば中もダブルクォーテーションです。

$hoge = "e";
echo "t".$hoge."st";

展開可、不可も考慮する必要があります。

いずれにしてもSQLが正しくできているか確認はしてください。
echoで出力された文字列をPHPからではなく直接実行して想定の結果が得られるかどうかの確認です。

---

ただ、変数をそのまま突っ込むのではなく、可能ならプリペアドステートメントからバインドしてください。
PHPでデータベースに接続するときのまとめ#"SELECT * FROM users WHERE id = '$id'" のように変数展開を使ってSQL文を組み立てている

Comments

[GPE46232209]

ありがとうございます。

Answer 2

extractがつける数値は1ではなく0から始まるのでは？
わざわざ変数に落とさなくても配列で渡せばいいと思いますが・・・

sample

$data = [];
for ($i=0; $i<10;$i++) {
  $str3 = substr(str_shuffle("ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnpqrstuvwxyz0123456789"), 0, 15);
  $data = array_merge($data, array($str3));
}
extract( $data, EXTR_PREFIX_INVALID, "pass");
try{
  $pdo = new PDO($dsn, $user,$password);
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
  $sql ="INSERT INTO enter_pas (password) VALUES ";
  $sql.=implode(",",array_fill(0,count($data),"(?)"));
  $stmt = $pdo->prepare($sql);
  $stmt->execute($data);
}catch(PDOException $e){
  die($e->getMessage());
}
print_r($sql);

Comments

[GPE46232209]

恥ずかしながら連想配列での扱いがわからなかったため、変数で用いていましたが、配列での記入、参考にします！！