質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

MariaDB

MariaDBは、MySQL派生のオープンソースなリレーショナルデータベースシステムです。 また、MySQLとほぼ同じデータベースエンジンに対応しています。

Q&A

解決済

1回答

435閲覧

PHPでデータベースへの登録を関数化したい

ytnk_731

総合スコア3

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

MariaDB

MariaDBは、MySQL派生のオープンソースなリレーショナルデータベースシステムです。 また、MySQLとほぼ同じデータベースエンジンに対応しています。

0グッド

0クリップ

投稿2022/02/27 13:14

下記のような関数を作成していますが、うまく動きません。
関数にせずに動かすと動いています。何が原因でしょうか?
どなたかご教示ください。

なお、global $dbでデータベースへの接続は完了しています。

PHP

1<?php 2function reg_users($diplay,$grade,$userid,$password,$mail,$img,$level,$notes){ 3 4 $hash = password_hash($password,PASSWORD_DEFAULT); 5 6 global $db; 7 $statement = sprintf("insert into users ('display_name','user_name','password','mail','user_img','user_grade','user_level','notes') values (%s,%s,%s,%s,%s,%d,%d,%s)",$diplay,$userid,$hash,$mail,$img,$grade,$level,$notes); 8 $sql = $db->query($statement); 9 10} 11?>

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

takasima20

2022/02/27 13:39

どんなエラーでてんの?
ytnk_731

2022/02/27 13:40

何もエラーはでてないんですが、データベースに登録ができていません涙
takasima20

2022/02/27 13:44

ini_set('display_errors', "On"); //を先頭に追記して動かしたら?
退会済みユーザー

退会済みユーザー

2022/02/28 00:20

webサーバープログラムのエラーログを確認してください。どういう環境で実行しているかわかりませんが、もしもXAMPPだったら C:\xampp\apache\logs に error.log として出力されています。
guest

回答1

0

ベストアンサー

問題は大きく3点(細かい点は除く)

  1. SQLとして正しい形ではない =>PHPから直に実行するのではなくDBに対して直に実行して想定の結果を得られるか先に確認すべき
  2. プリペアドステートメントとして正しい形ではない=>PHPマニュアルで基本を理解すべき
  3. エラーハンドリングされていない

php

1$diplay = "hogehoge"; 2$statement = sprintf("insert into users ('display_name') values (%s)",$diplay); 3echo $statement; 4//insert into users ('display_name') values (hogehoge)

$statementって変数名にしてるけど、保持されてるのはinsertから始まる単なる文字列です。
値がシングルクォート(MySQLはダブルでも良いようだが)で囲われていないとSQLではエラーです。
更に、提示の書き方ではエスケープされないのでSQLインジェクション可能です。
また、カラムがシングルクォートで囲われてると「文字列」として扱われるので、それはそれでマズいのでは(全角カラム名とかならいけたかな?使うべきではないが)

プリペアドステートメントにするなら[?]なり[:hogehoge]なりでパラメータつけてバインドする必要があります。

1つ前の質問では対象は「カラム」で間違えていたけど、[?]パラメータを使われてたように思います。(理解してたわけではない?)

いずれにしてもエラーハンドリングされたほうが良いでしょう。
DBはMySQL外での出来事なので例外処理でないとDB側で起きているエラーは捕捉できません。
下記のような記事を読んで「取るべき流れ」をおさえてください。
PHPでデータベースに接続するときのまとめ

投稿2022/02/27 20:31

編集2022/02/27 20:34
m.ts10806

総合スコア80765

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問