PHPによるDB書き換えについて

Question

前提・実現したいこと

前回と同様に、DBの書き換えです。
'money'の欄を書き換えたいです。

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

実行　を押してもDBに反映されない。

該当のソースコード

<?php
session_start();
include_once 'dbconnect.php';
if(!isset($_SESSION['user'])) {
	header("Location: index.php");
}

// ユーザーIDからユーザー名を取り出す
$query = "SELECT * FROM users WHERE user_id=".$_SESSION['user']."";
$result = $mysqli->query($query);

$result = $mysqli->query($query);
if (!$result) {
	print('クエリーが失敗しました。' . $mysqli->error);
	$mysqli->close();
	exit();
}

// ユーザー情報の取り出し
while ($row = $result->fetch_assoc()) {
	$username = $row['username'];
	$email = $row['email'];
        $money = $row['money'];
        $user_id = $row['user_id'];
}

// データベースの切断
$result->close();

// データベースに接続するために必要なデータソースを変数に格納
  // mysql:host=ホスト名;dbname=データベース名;charset=文字エンコード
$dsn = 'mysql:host=mysql1.php.xdomain.ne.jp;dbname=rara22_001;charset=utf8';
 
  // データベースのユーザー名
$user = 'rara22_onoue';
 
  // データベースのパスワード
$password = 'onoue622';
 
// tryにPDOの処理を記述
try {
 
  // PDOインスタンスを生成
  $dbh = new PDO($dsn, $user, $password);
 
// エラー（例外）が発生した時の処理を記述
} catch (PDOException $e) {
 
  // エラーメッセージを表示させる
  echo 'データベースにアクセスできません！' . $e->getMessage();
 
  // 強制終了
  exit;
 
}

$genzan = $money;
$genzan = $genzan - 100;

echo "$money";
echo "$genzan";
echo "$user_id";


if($money < 100){
echo "残額が足りません。";
} else {

// UPDATE文を変数に格納
$sql = "UPDATE users SET money = :money WHERE user_id = :id";
 
// 更新する値と該当のIDは空のまま、SQL実行の準備をする
$stmt = $dbh->prepare($sql);
 
// 更新する値と該当のIDを配列に格納する
$params = array(':money' => '$genzan', ':id' => '$user_id');
 
// 更新する値と該当のIDが入った変数をexecuteにセットしてSQLを実行
$stmt->execute($params);
 
// 更新完了のメッセージ
echo '更新完了しました';
}
?>

↓DBの情報
https://teratail-v2.storage.googleapis.com/uploads/contributed_images/115aab51d93f0a065ed8ffdf0f53eabc.jpeg

できていること

DBへの接続、$genzan　の計算

おそらく、update　のどこかがおかしいのだと思いますが、エラーは特に出ていないため、わかりません。

Comments

[退会済みユーザー]

シンタックスエラーになるか　０と一致比較してるんだろうなきっと

[daichi-onoue]

どこがどのようにエラーになっているのか教えていただけますか？

[退会済みユーザー]

とりあえず、変数展開とかPHPの基本を覚えてから問題にとりかかってください

[退会済みユーザー]

とりま https://teratail.com/questions/234518　と同じ理由です

[daichi-onoue]

本当に同じ理由なのでしょうか？
ソース自体も変えてますし、DB接続が前回はできていませんでしたが今回はできています。わからないからと言って、適当な回答をしないでください。

[FKM]

php.iniでエラー出るように設定してます？何も出ないはずないです。
目隠しで車を運転するようなものです。

[daichi-onoue]

設定しましたが、何も出ませんでした。。。

[FKM]

まずPHPで構築するときは、php.iniでエラー表示するようにしてください。準備運動無しで海に行っても溺れるだけです。
display_errors = On　にした後、必ずApacheやNginxなどを再起動すること。
そして未定義の関数やわざと間違えて、ちゃんとエラーメッセージがブラウザから表示されるか確かめてから取り掛かってください。

Answer 1

回答ではないです

原因追求以前にツギハギでコードを作成するのをやめたほうが良いです。
最低限、mysqli なのか pdo なのかの統一をしてください。
try - catch も意味がわかっていないのでは？

参考：
PHP で MySQL 接続時に必要な知識(最小限版)

Answer 2

$params = array(":money" => "$genzan", ":user_id" => "$user_id");

上記のようにしたら、正常に機能しました。

Comments

[退会済みユーザー]

これ、回答として不完全です。
変数展開とかPHPの基本を学習した上で修正してください。