PHPからSQLのクエリが実行できない

Question

実現したいこと
SQL接続のオブジェクトからMySQLに接続し、クエリ(select文)を実行する

以下のようなコードがあります。

<?php

class DbObject
{
    const DSN = "mysql:dbname=sampledb;host=localhost;";
　　// const DSN = "mysql:dbname=sampledb;host=localhost;port:3307";
    const USER = "root";
    const PASSWORD = "root";

    public function __construct()
    {
        $dbh = new PDO(self::DSN, self::USER, self::PASSWORD);
        return $dbh;
    }
}

<?php
//Controller
require_once(dirname(__DIR__, 1) . "\Models\DbObject.php");
require_once(dirname(__DIR__, 1) . "\Models\User.php");

$dbh = new DbObject();
$sql = "select * from users";

foreach ($dbh->query($sql) as $user) {
    // 処理
}

状況
Controllerの部分で以下のエラーが出ます。

Fatal error: Uncaught Error: Call to undefined method DbObject::query() in C:\xampp\htdocs\Sample\Controllers\IndexController.php:8 Stack trace: #0 {main} thrown in C:\xampp\htdocs\Sample\Controllers\IndexController.php on line 9

なぜ上記のような現象が発生するのでしょうか。
またMySQLのポートは「3307」でデフォルトから変更しているのにこれでMySQL自体に接続成功しているのでしょうか？
※定数のDSNにポート番号を追加しても同じエラーが返されます。

環境
Webサーバー Apache(8080ポート)
DBサーバー MySQL(3307ポート)

Answer 1

おす！　これ回答じゃねぇぞ。
m.ts10806が変なこと言ってっからm.ts10806に説明してやろうと思っただけだかんな。

こういう事、普通にやるだろ。
質問者がやりたかったの、こういう事じゃねぇのか？

<?php

class DbObject
{
    const DSN       = "mysql:dbname=sampledb;host=localhost;";
    const USER      = "root";
    const PASSWORD  = "root";
    
    private $dbh = null;

    public function __construct()
    {
        $this->dbh = new PDO(self::DSN, self::USER, self::PASSWORD);
        $this->dbh->setAttribute(\PDO::ATTR_ERRMODE, \PDO::ERRMODE_EXCEPTION);
        $this->dbh->setAttribute(\PDO::ATTR_DEFAULT_FETCH_MODE, \PDO::FETCH_CLASS);
    }
    
    public function query($query) : array
    {
        $stmt   = $this->dbh->query($query);
        
        return $stmt->fetchAll();
    }
    
    public function select(string $table, array $params) : array
    {
        $where_array  = [];
        foreach($params as $key => $val)
        {
            $where_array[]  = "{$key} = " . $this->dbh->quote($val);
        }
        $where  = (count($where_array) !== 0)
                        ? " WHERE " . implode(" AND ", $where_array)
                        : ""
                        ;
        
        $qry    = "SELECT * FROM {$table}{$where}";
        $stmt   = $this->dbh->query($query);
        
        return $stmt->fetchAll();
    }
}


////////////////////////////////////////////////////////////////////////////////


try
{
    $dbh = new DbObject();

    $sql = "select * from users";
    foreach ($dbh->query($sql) as $user)
    {
        // 処理
    }

    $params = ["col1" => "test1", "col2" => "test2"];
    foreach ($dbh->select("users", $params) as $row)
    {
        // 処理
    }
}
catch(\Exception $ex)
{
    echo "DB access error. {$ex->getMessage()}";
    die;
}

Comments

[退会済みユーザー]

これは良いお手本です。個人的には「ぐうの音も出ない」です。勉強させていただきました。

[退会済みユーザー]

おす！
別に回答したかったわけじゃねぇんだけどな。
m.ts10806が変なこと言ってっからちょっと遊んでみただけだ。bindもしてねぇしな。
まぁ、参考になったんなら、そりゃ、おらも良かったぞ。

[退会済みユーザー]

こんなコード書いたらレビューとおらんわｗ

[退会済みユーザー]

> こういう事、普通にやるだろ。

レビューとおらん = 普通やらんって言ってるんだけど。
日本語が通じないと辛い。。。

[退会済みユーザー]

te2jiさ、おめぇ、理解力が低すぎることいい加減に理解してくんねぇかな？
普通にやるのは、『インスタンスをクラス内のprivateメンバで保持して使い回す事』だ。

[退会済みユーザー]

te2ji、質問内容も、他の回答者が指摘してる焦点も全くわかってねぇだろ。

[退会済みユーザー]

> 『インスタンスをクラス内のprivateメンバで保持して使い回す事』だ。

別にそこは否定した覚えないけど。

ひどいのは
・PDO インスタンスを __construct で作ってること
・query/select メソッドがゴミ

回答するなら PDO インスタンスを作るスタティックメソッドでも書いときゃよかったのに。

Answer 2

PHPマニュアル参照のこと。
コンストラクタとデストラクタ

__construct ([ mixed $args = "" [, $... ]] ) : void

返却値 voidですのでreturn書いたところでその情報は返しません。
newの結果、つまりDbObjectオブジェクトです。

コンストラクタでDB接続して保持したいのでしたらオブジェクトのメンバー変数にPDOをnewした結果を保持させておく必要があります。

その使い方だとDbObjectというクラスの存在はほぼ無意味です。
PDO::queryはPDOが持っているので。
それだけラッパーしたとしても記述が冗長になるだけでなく、newが二度発生するのでむしろ余計なコストが発生します。
そのコストを補ってあまりある恩恵を受けられる設計を考えられるのであれば是非やってみてください。
きちんとやればできますし、ちょっとしたフレームワークくらいのボリュームにはなります。
ただ現状では無駄なコストだけが悪目立ちする言わば「使えないクソコード」です。
なにも考えてないというのが手に取るようにわかる実装です。
再利用性など加味されているか？使いやすいか？汎用性はあるか？
など、オブジェクト指向の利点がなにも活かされていません。
これならPDOそのままと全くかわりません。

オブジェクト指向をもっときちんと学んで使いこなさないことにはあなたが作ったであろうDbObjectというクラスはただのお荷物のままになるでしょう。

ちなみに気になったのでコメント拾います

業務で完成されたものばかり触っていたので、忘れてしまいました。

忘れたとかそういうレベルのコード書いてません。
基本です。その程度分からないのに元々知っていたかのような口ぶりは笑止千万。
PHPマニュアルすら確認しないで何を言ってるんでしょうか。

Comments

[退会済みユーザー]

おす。

>PDO::queryはPDOが持っているので。
>それだけラッパーしたとしても記述が冗長になるだけでなく、newが二度発生するのでむしろ余計なコストが発生します。

おめぇ、マジで言ってんのか？
ライブラリ設計とかした事ねぇだろ、おめぇ。

[退会済みユーザー]

> m.ts10806

簡単なライブラリ設計の仕方、回答で例示しといてやったぞ。
おめぇ、もうちょっと勉強しろ。

[m.ts10806]

あなたはもうちょっと別の勉強が必要なようです(ずいぶん前から)。
この質問者を最後まで面倒見る気があるなら幾らでもなんでも言ったら良いです。私はそんな気はさらさらありません。
では。

[maisumakun]

> 今回の件については明らかにおめぇの言ってる事がおかしいんだから素直に認めとけよ。

自分の方についたコメントも兼ね合わせて見てみると、もとの質問者は「PDOオブジェクトをそのまま使うつもりだった」ということで間違いなさそうです。

[退会済みユーザー]

というわけで、『間違っている回答』として低評価してやったぞ。
もうちょっと勉強しろ、おめぇ。

[退会済みユーザー]

>もとの質問者は「PDOオブジェクトをそのまま使うつもりだった」ということで間違いなさそうです。

いやぁ、そんな事すら分かってねぇから自分で何やってるかわかってなかっただけだと、オラ思うぞ。

Answer 3

__constructはコンストラクタという特殊な関数なので、returnしても無意味です。

new DbObjectの返り値はコンストラクタの返り値とは関係なく、DbObject型のオブジェクトになります。そして、このclass DbObjectにqueryメソッドはありませんので、ご提示のようなエラーとなります。

Comments

[Chandler_Bing]

ありがとうございます。
業務で完成されたものばかり触っていたので、忘れてしまいました。

DbObjectの実体はPDOかと思います。インターネットを調べるかぎりそのインスタンスが
queryというメソッドを持っていそうな感じだったのですが、違うのでしょうか。

もう少し書き方を訂正し、ダメなら追記します。よろしくお願いします。

[maisumakun]

> DbObjectの実体はPDOかと思います。

違います。あなたが「class DbObject」として作ったクラスです。

[maisumakun]

進むべき道は「DbObjectを諦めてPDOをそのまま使う」もしくは「（goku59さんの回答のように）DbObjectをきちんと実装して使い物になるようにする」の2つに1つです。

[m.ts10806]

「PDOをちゃんと使えるようになる」が先と思います。

[退会済みユーザー]

m.ts10806はすぐに頭に血が上らなくなるようにするのが先みてぇだな。

[m.ts10806]

？この質問者がちゃんとPDO使えてるんですか？
どこをどう見て頭に血が上ってると思い込んだのか知りませんが、それをわざわざ書いて何を得たいのか何のアドバイスになると思ってるのか分からないのでこれにて終。

[退会済みユーザー]

>？この質問者がちゃんとPDO使えてるんですか？

いや、使えてねぇぞ。だから何だ？
それと、おめぇがしなきゃならねぇ事は全く別の事だぞ。

>どこをどう見て頭に血が上ってると思い込んだのか知りませんが

おめぇの回答の編集履歴を見て思ったに決まってんだろ。
１分で書き直さなきゃなんねぇ程頭に血が上ってるじゃねぇか。
いい大人なんだから冷静に読み直せねぇほどカッカすんな。

Answer 4

なんかひどいコードが BA になってるので参考まで。

PHP7 で堅牢なコードを書く - 例外処理、表明プログラミング、契約による設計 / PHP Conference 2016

質問内容からは大きくハズレてますけどねｗ

Comments

[退会済みユーザー]

また、古臭い設計のスライド持ってきたな。DAO型じゃねぇか。
これでPHP7向けに作ったスライドなんか。
こんなコード書いてたら工数いくらあっても足りねぇぞ。

Answer 5

ご自身で新たにクラスを作っていて、そこにqueryを用意していないのに呼び出しているからエラーになっているのでは。

Comments

[Takumiboo]

あとポート番号は指定しないとデフォルトで接続しに行くと思うので、当然指定するべきです。