MVCモデルでデータが更新できない

Question

前提・実現したいこと

MVCモデルでデータを更新したい。

PHPのMVCモデルを勉強中。MVCモデルでデータの更新を実装しています。
データを更新する機能を実装中に、データが更新できないことで困っています。

##手順の流れ
一覧ページから編集ページへ移動

編集ページで、体重を100キロに変更

更新ページに遷移

一覧で体重が100キロに変更できたか確認してみるが、データを更新できていない

ビューページ

edit.php(編集ページ)で以下のようにupdate.phpにデータを渡す

    if(isset($_GET['id']) && $_GET['id']) {
      if(empty($errors)) {
        $_SESSION['id'] = $id;
        $_SESSION['uniform_num'] = $uniform_num;
        $_SESSION['position'] = $position;
        $_SESSION['name'] = $name;
        $_SESSION['club'] = $club;
        $_SESSION['birth'] = $birth;
        $_SESSION['height'] = $height;
        $_SESSION['weight'] = $weight;
        $_SESSION['country'] = $country;
        header('Location: ./update.php');
        exit();
      }

update.php(更新ページ)で以下のようにデータを受け取る。var_dumpでデータが全て受け取れていることは確認済み。

<?php
  session_start();
  ini_set('display_errors', "On");

  $id = $_SESSION['id'];
  $uniform_num = $_SESSION['uniform_num'];
  $position = $_SESSION['position'];
  $name = $_SESSION['name'];
  $club = $_SESSION['club'];
  $birth = $_SESSION['birth'];
  $height = $_SESSION['height'];
  $weight = $_SESSION['weight'];
  $country = $_SESSION['country'];

  require_once(ROOT_PATH .'Controllers/PlayerController.php');
  $player = new PlayerController();
  $player->update($id,$uniform_num,$position,$name,$club,$birth,$height,$weight,$country);

  // var_dump($id);
  // var_dump($uniform_num);
  // var_dump($position);
  // var_dump($name);
  // var_dump($club);
  // var_dump($birth);
  // var_dump($height);
  // var_dump($weight);
  // var_dump($country);

?>

<br>更新しました。
<a href="index.php">選手一覧へ</a>

モデル

<?php
require_once(ROOT_PATH .'/Models/Db.php');

class Player extends Db {
  public function __construct($dbh = null) {
    parent::__construct($dbh);
  }

  // 選手編集
  public function updatePlayer($id,$uniform_num,$position,$name,$club,$birth,$height,$weight,$country) {
    $sql = 'UPDATE players
            SET uniform_num = :uniform_num,position = :position, name = :name, club = :club, birht = :birth, height = :height, weight = :weight, country_id = :country_id
            WHERE id = :id';
    $sth = $this->dbh->prepare($sql);
    $sth->bindValue(':id', $id, PDO::PARAM_INT);
    $sth->bindValue(':uniform_num', $uniform_num, PDO::PARAM_INT);
    $sth->bindValue(':position', $position, PDO::PARAM_INT);
    $sth->bindValue(':name', $name, PDO::PARAM_STR);
    $sth->bindValue(':club', $club, PDO::PARAM_STR);
    $sth->bindValue(':birth', $birth, PDO::PARAM_INT);
    $sth->bindValue(':height', $height, PDO::PARAM_INT);
    $sth->bindValue(':weight', $weight, PDO::PARAM_INT);
    $sth->bindValue(':country_id', $country, PDO::PARAM_INT);
    $sth->execute();
  }

##コントローラー

<?php
require_once(ROOT_PATH .'/Models/Player.php');
require_once(ROOT_PATH .'/Models/Goal.php');

class PlayerController {
  private $request;
  private $player;
  private $goal;

  public function __construct() {
    $this->request['get'] = $_GET;
    $this->request['post'] = $_POST;
    $this->request['session'] = $_SESSION;
    $this->Player = new Player();
    $this->Goal = new Goal();
  }

  // 選手編集
  public function update($id,$uniform_num,$position,$name,$club,$birth,$height,$weight,$country) {
    if(empty($this->request['session']['id'])) {
      echo '指定のパラメーターが不正です。このページを表示できません。';
      exit;
    }

    $this->Player->updatePlayer($id,$uniform_num,$position,$name,$club,$birth,$height,$weight,$country);
  }

試したこと

・データベースの接続は一覧ページや詳細ページなどで確認。
・編集ページや更新ページでも簡単にテストし、接続はできている。

●を▶︎に変更すると、Fatal error: Uncaught ArgumentCountError: Too few arguments to function Player::updatePlayer()と出るので、全て変数を入れている。

  public function update($id,$uniform_num,$position,$name,$club,$birth,$height,$weight,$country) {
    if(empty($this->request['session']['id'])) {
      echo '指定のパラメーターが不正です。このページを表示できません。';
      exit;
    }

    ●$this->Player->updatePlayer($id,$uniform_num,$position,$name,$club,$birth,$height,$weight,$country);
  }

▶︎$this->Player->updatePlayer($this->request['session']['id']);

どのようにすれば、データを更新できるようになるでしょうか？
もし、足りない情報がございましたらご指摘ください。

追記
Db.php(データベースの接続)

<?php
  require_once(ROOT_PATH .'/database.php');

  class Db {
    protected $dbh;

    public function __construct($dbh = null) {
      if(!$dbh) {
        try {
          $this->dbh = new PDO(
            'mysql:dbname='.DB_NAME.
            ';host='.DB_HOST, DB_USER, DB_PASSWD
          );
        } catch (PDOException $e) {
          echo "接続失敗:".$e->getMessage()."\n";
          exit();
        }
      } else {
        $this->dbh = $dbh;
      }
    }
    public function get_db_handler(){
      return $this->dbh;
    }
  }

Answer 1

正直これだけの情報だと他人からだとはっきり原因をつかむのは難しいと思います。フォームのページのHTMLとかも記載されていませんし、Db.phpの正体がどうなっているかが謎ですから。

とりあえず手っ取り早く調査するなら$sqlの値が正しいSQL文になっているか？ とか $sth->execute() の返り値がどうなっているかとか、IDE使ってないならprintデバッグでもして中身を確認してみるとかですかね。ちゃんとSQLが実行されるかどうかが肝だとおもうので、その処理の手前から遡っていって、一つ一つ変数の中身がどうなっているかや処理がちゃんと来ているかをprintデバッグなどで確認していくしかないと思います。

PhpStormやEclipseなどの統合開発環境を用いれば、こういう不具合は簡単に発見できると思うので、開発環境の整備も検討すると良いと思います。

Comments

[退会済みユーザー]

ご指摘ありがとうございます。整備も検討してみます。
コードの字数の関係上、Db.phpだけ追記させていただきました。

[退会済みユーザー]

ちなみに、$sth->execute() の返り値の調べ方ってどうやればいいんですかね？
var_dump($sth);?
それか、
if($sth->execute() == true)みたいに書くのでしょうか？

Answer 2

最終追記です

この回答のコメント欄にて顛末が載っていますので、気になる方はそちらをご覧いただければと思います。
要素の値の取得漏れがあったらしく提示されている情報からは絶対にわかりません。
「これだけじゃわからないよ」が正解でした！申し訳ございませんっ

まぁ、タイポもエラー吐いたと思うし無駄ではなかったと思うのですが…
圧倒的敗北感でございます。いったい何と戦ってるんでしょうかね？

以下直接的な原因ではなかったと思われる要素（カラム無いからたぶんエラーは吐く）

club = :club, birht = :birth,
SQL文の中にこれがあったので上に載せておきます。
birhtのカラムが見つからなくて登録弾かれていた可能性あり？
※タイポしてるの※

###　超恥ずかしいけど考え方の参考になると思って消さずに載せておくよっ!（DB情報載る前）

あまり得意ではないのだけども、
updateがされないということはupdateに関連する記述に問題があるのだと予想できます。

$sth = $this->dbh->prepare($sql);

ここが気になるかなぁ。
僕がPHPで組んだデータを更新できるシステムだと、

$sth = $dbh->prepare("SQL文ですよっと".$id);

という感じになっているので、
$thisで何をdbhに渡しているのか分からないけど…
$thisが要らない可能性があるかな？
あとは$dbhにすると動くようになるかなぁ？と。

$dbh = new PDO(PDO_DSN,DB_USERNAME,DB_PASSWORD);

添付のコードには定義が載っていないけれど、※←探す人が出ないようにここ足しました※
dbhってたぶんこれの事ですよね？変数で扱ってるはずなので、$が必要になるかな。

以下追記です。
$sth = $this->dbh->prepare($sql);
この一文をググってみてもヒットしてこないので、
DBで処理するときにこの記述を使うことは無いんじゃないかなと思います。

$this（updatePlayerを指してる？）の中にDB接続情報は入っていないはずなので、
$sth = $dbh->prepare($sql);こうすれば動くかもです。

以下蛇足です。
難しいと言われてしまった…まぁ直らなかったらごめんだけども。

考え方を記しておきますね。
データが登録できているということは登録時のDBへの接続は問題無い。
ただし、updateが出来ない。ということはupdate時の接続に問題があるかも？
※この時点でupdateに関する記述以外を追う必要がほぼない。DBの定義も問題なし。

update関数の中を追って、
updatePlayerで処理してるっぽいのでそこを見て、
ミスが起きやすいSQLとかのチェックをして、
変数名のチェックをして…DBも変数だよね？ここじゃね？
これthisいるのかな？DBにSQL文渡すだけだから要らん気がするけども…

みたいな感じです。これで直らなかったらめちゃくちゃ恥ずかしいやつですわっ
ちょっと隠れる準備しておきますね。

Comments

[退会済みユーザー]

ご回答ありがとうございます。データベースの接続について追記いたしました。
$this->dbhは、Db.phpで定義しており、記述としては問題ないと思っています。
理由は、データの詳細で$this->dbh->prepare()と記述し、詳細のデータは取得できているからです。

[退会済みユーザー]

追記ありがとうございます。
問題なさそうですか。失礼いたしました。
恥ずかしいやつ確定です。まいった。
もうちょっと考えてみますね～。

[退会済みユーザー]

birht ←　これのせいだったりしますかね？
updateのSQL文にタイポがあったようです。
birhtカラムは無いと思うので、
SQL文自体が通らず登録弾かれていたというオチのような気がするんですが…
これでもダメならもう一回ご説明の追記をお願いいたしますっ

以下蛇足です。
言い訳しとくよっ、目が疲れるからリラックスメガネしてるんだよっ
いや…気づけないって、まぢでっ。タイポ直して通ると良いなぁ。

[退会済みユーザー]

ご指摘ありがとうございます！タイポ確認し、すぐに修正したのですが、更新できず....
いよいよ沼沼沼になってきて、なんで？って感じです。

[退会済みユーザー]

あれ、タイポじゃなかったですか！？
一応カラム無いと更新はできないはずなのでタイポは修正して正解なんですが、
これは本当にきつくなってきましたね…
もう少し見てみますねっ

[退会済みユーザー]

ありがとうございます！自分も格闘します。

[退会済みユーザー]

print_r($sth -> errorInfo());
これ試して頂いても良いですか？
エラーの確認ができるようです。

[退会済みユーザー]

エラー確認したところ、positionのカラムがunknown カラムって表示されたので、原因調べました。
positionは、セレクトボックスを用いて実装しており、positionのデータを見つけられなかったために更新できなかったようです。ひとまずpositionを消して実行したところ、データは無事に更新することができました。お力添えいただきましてありがとうございました。

[退会済みユーザー]

なるほどっ、そりゃわからんですｗ
僕の回答に謝罪文追加しておかないとっ
でも解決されたようでよかったです。
ありがとうございました！