関数化の過程でFatal error:

Question

• 現状

下のfetch部分の関数化しようと試みたところ、下のようなエラーが吐かれた。

function getName(){
  $result = $stmt->fetchAll(PDO::FETCH_ASSOC);
  print_r($result);
}

$getName = getName();
echo $getName;
echo "<br>";

Fatal error:  Uncaught Error: Call to a member function fetchAll() on null in /var/www/html/index.php:50
Stack trace:
#0 /var/www/html/index.php(60): getName()
#1 {main}
  thrown in /var/www/html/index.php on line 50

エラー文にある「line 50」はfunction内の「$result = $stmt->fetchAll(PDO::FETCH_ASSOC);」に、
「line 60」は「$getName = getName();」にあたります。

• 考えられること/試してみたこと

エラー文を紐解くと、feachAllが正常に動作しないという事が分かり、
「$result = $stmt->fetchAll(PDO::FETCH_ASSOC);」の「PDO::FETCH_ASSOC」部分に原因があるとまでは考えられました。
しかし、具体的にその先をどのようにするかの筋道が見えません。

どなたか解決策の教授をしてもらえないでしょうか？

• 追記

コメントいただいた方から、「$stmtの定義がない」という指摘をいただきましたが、
すいません、書いてなかっただけでfunction部分の前の部分であります。
そのため、

  $sql = "SELECT * FROM `users`,`user_id` LIMIT 20";
  $stmt = $dbh->prepare($sql);//←ここが$stmtの定義
  $stmt->execute();
  
function getName(){
  $result = $stmt->fetchAll(PDO::FETCH_ASSOC);
  print_r($result);
}

$getName = getName();
echo $getName;
echo "<br>";

が、現状のコードです。
DBからusers,user_idを２０件取り出し、ブラウザ上に表示させる過程で関数化をしようとしています。
改めて、解決方法をどなたかご教授おねがいします。

Comments

[m.ts10806]

回答に変更はないです。
変数のスコープの問題。

Answer 1

そもそも$stmtという変数が定義なしにいきなり使われています。
未定義変数は扱いとしてはnullなので、「nullにはそんな関数はいない（意訳。正確には”nullから関数が呼び出された”）」　とエラーに書かれています。

すいません、書いてなかっただけでfunction部分の前の部分であります。

スコープが違います。

こうやってるのと同じ。

$a = "test";
function b(){
   echo $a;
}
b();

globalするか（個人的には非推奨）
引数で渡すか（簡単にはできるけど、全体の作りとかにもよる）
クラスにしてプロパティとして参照させるか（個人的には推奨）

して、確実に必要な情報を参照させてください。

Answer 2

とりあえず引数で渡せば解決します。

function getName(&$stmt){
  $result = $stmt->fetchAll(PDO::FETCH_ASSOC);
  print_r($result);
}

$getName = getName($stmt);
echo $getName;
echo "<br>";

以下の点気になりますが設計の問題ですね。

• 引数のPDOStatementはexecuteされていることを前提としている
• getXxxxというメソッドは何かしら返り値を期待する
• というか返り値を期待した使われ方をしているが実際なにもreturnしてない

m.ts10806さんが指摘されているスコープについても重要な概念なので、
今後もプログラムを書くご予定があるならこの機会に調べてみたほうがいいかと思います。
理解していないとこれからもハマり続けると思います。

Comments

[m.ts10806]

あぁ質問者さんこちらのコードそのままでしたね。
参照渡しにする意味と返却してないのに返却値を使おうとする意図は何でしょうか

[KAOsaka]

本スレッドは`Uncaught Error: Call to a member function fetchAll() on null`に対する解決策を質問しているスレッドです。まずその点ご理解ください。返却してないのに返却値を使おうとしている部分は質問者さんのコードの原文ママです。意図は分かりかねます。また、参照渡しにしている意味ですが、`$stmt`が破壊的メソッドを含むためです。

Answer 3

解決しました！

KAOsakaさんの説明の通り、
下のように「&$stmt」を引数で渡すかたちでエラーが解消されました。

function getName(&$stmt){
  $result = $stmt->fetchAll(PDO::FETCH_ASSOC);
  print_r($result);
}

$getName = getName($stmt);
echo $getName;

返信された方のおっしゃる通り、スコープについての理解がまだ曖昧なので、一度調べてみようと思います。
ありがとうございました。

Comments

[m.ts10806]

参照渡しにする必要ないのでは。
あと、returnしてないので想定のとおりには動かないと思います。
print_rで出力はしてるでしょうけど、返却してないので空なはず。