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

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

ただいまの
回答率

90.23%

PDOでのMysql接続関連

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 260

NOBUTUNA

score 9

前提・実現したいこと

以下のclass Sql 別々のファイルで2箇所でインスタンス化しfunction ProductDataSelect と WpIdCheck をそれぞれ使用しているのですが、WpIdCheckの方が途中で処理が止まってしまいます。
調べたところWpIdCheck の方は $dbh が null となっているため、正しくDBとアクセスできていないようです。
どのようにしたらそれぞれの WpIdCheck でもMysqlに接続させることができますでしょうか。

class Sql
{
    private function DBconnect() {
        // DB 接続処理
        try {
            $dsn = "mysql:dbname=".DBNAME.";host=".HOST.";";
            $dbh = new PDO($dsn, USERNAME, PASSWORD);
            $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_WARNING);

        } catch (Exception $e) {
            echo "DB アクセス失敗";
            echo $e ;
        }
        return $dbh;
    }

    public function ProductDataSelect($shop_name) {
        $dbh    = $this->DBconnect();
        $sql    = "SELECT * FROM research_data WHERE shop = :shop";
        $stmt   = $dbh->prepare($sql);
        $params = [':shop' => $shop_name];

        $stmt->execute($params);
        $research_url = $stmt->fetchAll(PDO::FETCH_ASSOC);

        return $research_url;
     }

     public function WpIdCheck () {
        try {
            $dbh = $this->DBconnect();
            $sql = "SELECT * FROM research_data WHERE shop = :shop";
error_log(var_export('ここのエラーログは正しく表示されている', true));
       // ここで処理が止まっている、エラーろぐも発生していない.
            $stmt = $dbh->prepare($sql);
error_log(var_export('ここの文字列はエラーログに出力されない。。。', true));

            $param = [':shop' => 'xxxxxxxx'];
            $stmt->execute($param);
            $result = $stmt->fetchAll(PDO::FETCH_ASSOC);
        } catch (PDOException $e) {
            print $e->getMessage();
            print "WpIdCheck エラーテスト用";
            die();
        }
    }
}

他ファイルでの呼び出し例
A file

$sql_obj      = new Sql();
$test         = $sql_obj->WpIdCheck();


B file

$db_obj = new Sql();
$url_list = $db_obj->ProductDataSelect($shop_name);

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

なし
エラーは発生しておらず catch の方にも処理は流れてきませんでした。

試したこと

function DBconnect を当初 __construct で処理していましたが、WpIdCheck でアクセスできなくてから共通ではダメなのかと考え各function で一番初めに以下処理を追加しました。
結局解決できませんでしたが。。。

$dbh    = $this->DBconnect();


非常に単純な内容の可能性もあり申し訳ありませんが、各所にエラーログを仕込んでも解決に至りませんでした。

補足情報(FW/ツールのバージョンなど)

ローカル MAMP環境
php7.1.23
MySQL 5.6.38

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2019/03/05 09:17

    PHPのバージョンは7.xとマイナーバージョンまで正しく記載してください。
    ただ、そもそもSqlクラス 閉じ}が足りてないのでシンタックスエラーでそもそも動かないような。

    キャンセル

  • m.ts10806

    2019/03/05 09:19

    あと大量に全角空白があるのでこのコード、正しく動かないですよ。

    キャンセル

  • NOBUTUNA

    2019/03/05 09:49

    返答ありがとうございます。
    全角は確認しましたが実ソースコード上では一つもありませんでした。
    クラスの閉じ } 追加いたします。こちらも実コード上では付いておりました

    キャンセル

  • m.ts10806

    2019/03/05 09:51

    現在のコードが正しく提示されないことには本来必要のないやり取りが発生しますので、そこは工夫していただければと。
    コードブロック内は原則、現在のコードのみで補足コメントは外だしされたほうが良いですね。

    キャンセル

回答 3

+1

本件、解決に至りましたので内容を記載いたします。
Sqlクラスをインスタンス化し使用している A file なのですが、このファイルは他ファイルから以下のような形式で呼び出されておりました。

exec("php A_file.php");

上記のような呼ばれ方をしている処理を他file同様 require_once関数でA_fileを読み込み、クラスをインスタンス化して使用することで$dbhの中がnullではなく実行できるようになりました。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/05 11:00

    それはまあ、exec()はコマンド実行するだけですからね・・
    さすがに今回の質問内容だけでは分からないので、ファイルが複数あって読み込ませているのでしたらそこまで書くようにしてください。
    何もないとincludeなりrequireで読み込ませている前提と考えますので。

    キャンセル

checkベストアンサー

0

直接の回答ではないですけど

function DBconnect を当初 __construct で処理していましたが、WpIdCheck でアクセスできなくてから共通ではダメなのかと考え

いえ、コンストラクタで接続した結果をプロパティに持てば良いです。

クラス

class Sql
{
    private $dbh = null;
    private $dbname = "xxx";
    private $host = "xxx";
    private $user = "xxx";
    private $pass = "xxx";

    function __construct(){
        // DB 接続処理
        try {
            $dsn = "mysql:dbname=".$this->dbname.";host=".$this->host.";";
            $this->dbh = new PDO($dsn, $this->user, $this->pass);
            $this->dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_WARNING);

        } catch (PDOException $e) {
            echo "DB アクセス失敗";
            var_dump($e);
            die();
        }
    }

    public function getTest() {
        try {
            $sql = "SELECT * FROM test WHERE name like :name";
            $stmt = $this->dbh->prepare($sql);
            $param = [':name' => '%test%'];
            $stmt->execute($param);
            return $stmt->fetchAll(PDO::FETCH_ASSOC);
        } catch (PDOException $e) {
            echo "getTest失敗";
            var_dump($e);
            die();
        }
    }
}


利用する側

$dbsql = new Sql();
var_dump($dbsql->getTest());

newしたときに実行されるコンストラクタでnullだった$dbhに接続情報が入るので、生成されたインスタンスを使えば$dbhは有効です。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

本件、解決に至りましたので内容を記載いたします。
Sqlクラスをインスタンス化し使用している A file なのですが、このファイルは他ファイルから以下のような形式で呼び出されておりました。

exec("php A_file.php");

上記のような呼ばれ方をしている処理を他file同様 require_once関数でA_fileを読み込み、クラスをインスタンス化して使用することで$dbhの中がnullではなく実行できるようになりました。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 90.23%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る