sqlで値が取得できない原因が知りたい

Question

提示コードですが参考画像の通りでPostというテーブルのデータをthread_UUIDと一致していてかつenable=trueのデータを全て取得して
その数をカウントするという処理なのですがなぜかcountの値が０になりvar_dump()をすると何も入っていませんこれは何が原因なのでしょうか？

確認したこと

テーブル名が合っているかどうか確認
thread_UUIDが文字列のためそのテーブルに存在するかどうかを文字列検索で確認
var_dump()で中身がどうなっているかどうか確認

実行画面

mysql(ハイライト部は文字列検索)

    /*#######################　投稿数を取得  #######################*/
    function GetPostNum($uuid)
    {
        try
        {
            echo $uuid."<br>";

            $database = GetDataBase();
            $database->beginTransaction();    
            $sql = 'SELECT * FROM Post WHERE `enable`=true AND `thread_UUID`=:uuid';
            $stmt = $database->prepare($sql);
            $stmt->bindValue(":uuid",$uuid,PDO::PARAM_STR);
            $stmt->execute();
            $database->Commit();    

            if($stmt->fetchAll() == false)
            {
                
                return false;
            }
            else
            {
                //echo $stmt->rowCount();
                var_dump($stmt->fetchAll());
                return count($stmt->fetchAll());
            }
        }
        catch(PDOException $e)
        {
            die($e->getMessage());
        }
    }
    

Comments

[m.ts10806]

SELECTでトランザクションをなぜ入れてるんでしょうか。

[yambejp]

質問の意図がわかりませんが
「var_dump($stmt->fetchAll())」が空なら
「count($stmt->fetchAll())」が0なのは当たり前ですけど・・

[退会済みユーザー]

phpMyAdmin上で正常に結果が得られたSQL文でしょうか？ カラム名やテーブル名が実態と合ってるかどうかは第三者では確認できないので。

Answer 1

仕様通り。

https://www.php.net/manual/ja/pdostatement.fetchall.php
PDOStatement::fetchAll — 結果セットから、残っている行を取得する

未取得が１０件あったら１０件取得されます。
残りは何件でしょう。
あなたはfetchAll()を同じステートメントに対して何回実行しているでしょう？

---

それに「取得できなかったらfalse」という仕様ではないので、
if($stmt->fetchAll() == false)という条件式は不適当。

---

「トランザクション」「コミット」が必要な実装か判断できるくらいには基礎を身につけましょう。

Comments

[退会済みユーザー]

なるほど質問ですがfetchAll()は一回参照されたら使われたら次から結果が変わる関数なのでしょうか？

[m.ts10806]

回答読んでなおその質問がでるのはなぜ？読んでないからか。

[m.ts10806]

あなたは自分で缶ジュースの中身を飲み干した後に「なんでなくなるの？」って他人に聞いてるんだけど。

[m.ts10806]

今回に限らず、全部ね。
一字一句読み込んで全部理解して理解できないところを聞くように。
「なるほど」なんて理解してないのに使っていい語句ではない。

Answer 2

enableがtrueでないか、thread_UUIDが$uuidではないのだと思います

追記

m.ts10806さんのご指摘の通り、fetchAllは複数回実行できないので
一旦変数で受けておいてください

$rows=$stmt->fetchAll(PDO::FETCH_ASSOC);
if( empty($rows)){
  return false;
}else{
  var_dump($rows);
  return count($rows);
}

Comments

[m.ts10806]

例えヒットしてても
if($stmt->fetchAll() == false)
↑ここで全部取り出されているからだと思います。

[yambejp]

ああ、たしかに変数にいれて再利用しないといけないですね・・・
追記しておきます