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

Question

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

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

<?php
function reg_users($diplay,$grade,$userid,$password,$mail,$img,$level,$notes){

    $hash = password_hash($password,PASSWORD_DEFAULT);

    global $db;
    $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);
    $sql = $db->query($statement);

}
?>

Comments

[takasima20]

どんなエラーでてんの?

[ytnk_731]

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

[takasima20]

ini_set('display_errors', "On"); //を先頭に追記して動かしたら?

[退会済みユーザー]

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

[m.ts10806]

長期間放置されて回答に何も反応のないまま急に解決済みとされましたが、どうなったのでしょう。
フィードバックください。
https://teratail.com/help/question-tips#questionTips42

Answer 1

問題は大きく3点（細かい点は除く）

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

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

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

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

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

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