PHPでデータベースのSQLをやっているのですが・・・

Question

前提・実現したいこと

PHPで作成しているフォームをデータベースに送るSQLがうまく機能しない

該当のソースコード

try{
$d = new PDO("mysql:host=localhost;dbname=db","root","1234");
print "接続成功<br>";
} catch (PDOException $e) {
exit('データベース接続失敗。'.$e->getMessage());
}

$karams = 'namae,nen,sex,jusyo,tel,mail,index,toi';

$value = "'".$namae."','".$nen."','".$sex."','".$jusyo."',
'".$denwa."','".$mail."','".$index."','".$naiyou."'";

$d->query("INSERT INTO form1 ($karams) values ('$value')");

試したこと

$d->query("INSERT INTO
form1 (namae,nen,sex,jusyo,tel,mail,index,toi)
values ('$namae','$nen','$sex','$jusyo',
'$denwa','$mail','$index','$naiyou')");
↑これでもデータベースに記入できませんでした。
一応接続成功は出ているのでmysqlには入れているとは思います。

Answer 1

まず、$value を表示してみてください。
意図しているものと違うことが分かると思います。

次に、マニュアルを見てください。
文字列
引用符と二重引用符で実行結果が違うことが分かると思います。

で、少し難しくなりますが、以下を参照すると良いです。
PHPでデータベースに接続するときのまとめ

DB 接続は、基本ですが結構やっかいです。
試行錯誤しながら、セキュアな使い方を覚えてください。

Comments

[退会済みユーザー]

ほんと、この記事有能すぎ

[退会済みユーザー]

前半完全に勘違いしていた回答にもかかわらず、リンク貼っただけで高評価が入る mpyw さんの記事すげぇｗ

最近になってやっと書いてあること全部理解できた気がする。
何度も読み返すべき良い記事だと思います。

せっかくナンで、こっちも読むと良いです。
https://qiita.com/mpyw/items/2f9955db1c02eeef43ea

[m.ts10806]

私も月に1回は回答に貼ってる気がします。（そういえば今日も）

Answer 2

prepare()→bindValue()→execute()の繰り返しをして、
最後にcommit()しよう。
PHP: PDOStatement::bindValue - Manual

try{
    $d = new PDO("mysql:host=localhost;dbname=db;charset=utf8mb4","root","1234",[
        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
        PDO::ATTR_EMULATE_PREPARES => false,
    ]);
    print "接続成功<br>";
    $d->beginTransaction();  // トランザクション開始

    $s = $d->prepare("INSERT INTO form1 (`namae`,`nen`,`sex`,`jusyo`,`tel`,`mail`,`index`,`toi`) values (:namae,:nen,:sex,:jusyo,:tel,:mail,:index,:toi)");   // SQLインジェクション対策にはprepare
    $s->bindValue(':namae', $namae, PDO::PARAM_STR);
    $s->bindValue(':nen', $nen, PDO::PARAM_STR);
    $s->bindValue(':sex', $sex, PDO::PARAM_STR);
    $s->bindValue(':jusyo', $jusyo, PDO::PARAM_STR);
    $s->bindValue(':tel', $tel, PDO::PARAM_STR);
    $s->bindValue(':mail', $mail, PDO::PARAM_STR);
    $s->bindValue(':index', $index, PDO::PARAM_STR);
    $s->bindValue(':toi', $toi, PDO::PARAM_STR);
    $s->execute();

    $d->commit();  // コミットも忘れずに

} catch (PDOException $e) {
    exit('データベース接続失敗。'.$e->getMessage());
}

みたいな書き方になるかと。
接続時に例外をスローするようにしたのと、utf8接続指定したのと。
カラム名をバッククォートで囲むと、予約語とかぶっても安心。
もしも数値型のカラムの場合は、PDO::PARAM_STRじゃなくPDO::PARAM_INTを使います。

Comments

[退会済みユーザー]

この記事の受け売りっす→ https://qiita.com/mpyw/items/b00b72c5c95aac573b71

[kondokaisya]

助かりました、何とかなりそうです＾＾

Answer 3

質問のプログラムコードは対応しているMarkdownの機能 の[コードを入力]を使いましょう。

PHPで作成しているフォームをデータベースに送るSQLがうまく機能しない

差支えない限り、エラーメッセージは表示された内容をそのまま載せてください。CREATE TABLE文も載せましょう。

コピペで簡単に済む業界ではありませんから、まずは、PHPからデータベースに接続 辺りをじっくり勉強されては？

Answer 4

とりあえずindexは予約語なのでカラム名につかっているなら
バッククォートで囲んで下さい
また、insertするデータはprepareで処理して下さい

Answer 5

rootでテストされていますよね。

query("INSERT INTO form1 ($karams) values ('$value')");

このあとにcommit文を実施されていますか。

Comments

[m.ts10806]

mysqlは明示的に設定いじらない限りオートコミットだったと思います。