DBを使用して、ひとこと掲示板を作成する学習中に、名前とコメント、追加日時がinsertできず進みません。

前提・実現したいこと

発生している問題・エラーメッセージ

エラーメッセージは出ていませんが、宣言が出来ていないのか、関数が設定できていないのか、基本的に書き方が違うのかわからない状況です。

該当のソースコード

<?php define('DB_HOST', ''); // データベースのホスト名又はIPアドレス define('DB_USER', ''); // MySQLのユーザ名 define('DB_PASSWD', ''); // MySQLのパスワード define('DB_NAME', ''); // データベース名 define('HTML_CHARACTER_SET', 'UTF-8'); // HTML文字エンコーディング define('DB_CHARACTER_SET', 'UTF8'); // DB文字エンコーディング $error = []; $name = ''; $comment = ''; $log = ''; // DB接続 $link = get_db_connect(); // コメントを挿入 $comment_in = comment_table_in_list($link , $name , $comment , $log); // コメントの一覧を取得 $comment_sel = comment_table_sel_list($link); // DB切断 $closed = close_db_connect($link); // リクエストメソッド取得 $request_method = get_request_method(); // POSTの場合 if ($request_method === 'POST') { // var_dump ($_POST); // POSTデータ取得 $name = get_post_data('name'); var_dump ($name); $comment = get_post_data('comment'); var_dump ($comment); $log = date('Y-m-d H:i:s'); $check = check_name($name); if ($check !== true) { $error[] = $check; } $check = check_comment($comment); if ($check !== true) { $error[] = $check; } if (count($error) === 0) { if ($comment_in !== TRUE) { $error[] = 'コメント登録できませんでした'; } } } // バリデーション処理 // 名前 function check_name($name){ if (mb_strlen($name) !== 0) { if (mb_strlen($name) > 20) { return '名前は20文字以内で入力してください'; } }else if (mb_strlen($name) === 0) { return '名前を入力してください'; }else { return true; } } // バリデーション処理 // コメント function check_comment($comment){ if (mb_strlen($comment) !== 0) { if (mb_strlen($comment) > 100) { return $error[] = 'ひとことは100文字以内で入力してください'; } }else if (mb_strlen($comment) === 0) { return $error[] = 'ひとことを入力してください'; }else { return true; } } /** * DBハンドルを取得 * @return obj $link DBハンドル */ function get_db_connect() { // コネクション取得 if (!$link = mysqli_connect(DB_HOST, DB_USER, DB_PASSWD, DB_NAME)) { die('error: ' . mysqli_connect_error()); } // 文字コードセット mysqli_set_charset($link, DB_CHARACTER_SET); return $link; } //insert時クエリの返り値 function insert_data($link, $sql) { if (mysqli_query($link,$sql) !== true) { return false; } return true; } /** * ひとことを挿入する???????????? * * @param obj $link DBハンドル * @return array ひとこと一覧配列データ追加 */ function comment_table_in_list($link , $name , $comment , $log) { // SQL生成 $sql = 'INSERT INTO comment_table(name, comment, log) VALUES("'.$name.'","'.$comment.'","'.$log.'")'; var_dump ($sql); // クエリ実行 return insert_data($link,$sql); } ### 試したこと var_dump($name);やvar_dump($comment);などでpostデータは取得できていることは確認できましたが、いざinsert部分になると引数で$nameや$commentをいれても、だめでした。var_dump()で確認しても値が来ていません。超初心者で、全然できていませんがご指導宜しくお願い致します。 ### 補足情報（FW/ツールのバージョンなど）ここにより詳細な情報を記載してください。

退会済みユーザー

2020/04/10 09:48 編集

https://teratail.com/help#about-markdown を読んで、コードの箇所を見やすくしていただけたら。

退会済みユーザー

2020/04/10 09:54

mysqli_connect()まわりの、DB接続だけのテストは成功していますか？まずそこからでしょう。mysqlをどこでどう動かしているか、接続の条件も確認する必要があります。

yama_ichi

2020/04/10 10:09

ありがとうございます。読んで勉強してみます。また、ＤＢ接続はできて、phpmyadminには日時のみ追加されて、名前とコメントが空で入ってしまいます。バリデーションはしているのですが

行動規範の内容に同意します

回答1件

ベストアンサー

PHP: mysqli::prepare - Manual
PHP: mysqli_stmt::bind_param - Manual
など駆使して、プリペアドステートメントを活用して書き直してください。

クエリー文字列を、変数の連結で作るのはあまり行儀が良いとは言えません。

php
1$sql = 'INSERT INTO comment_table(name, comment, log) VALUES("'.$name.'","'.$comment.'","'.$log.'")';

これ、なにかしらデータが詰められると、

php
1$sql = 'INSERT INTO comment_table(name, comment, log) VALUES("あああ","いいい","ううう")';

みたいになると思いますが、これは文法エラーです。

php
1$sql = "INSERT INTO `comment_table`(`name`, `comment`, `log`) VALUES('あああ','いいい','ううう')";

とかなら通るのかもしれません。
そんなふうに、引用符の種類で混乱するくらいなら、

php
1$sql = 'INSERT INTO comment_table(name, comment, log) VALUES(?,?,?)';
2$stmt = mysqli_prepare($link,$sql);
3mysqli_stmt_bind_param($stmt, 'sss', $name, $comment, $log);
4mysqli_stmt_execute($stmt);