phpでデータベースからの情報取得を関数化

Question

前提・実現したいこと

phpとmySQLでデータベースに接続し、情報の取得・更新を行えるシステムを作っています。

情報を取得する際のコードを関数にして共通化したいと思いチャレンジしてみたのですが、
エラーコードが出てしまいます。

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

Fatal error: Uncaught Error: Call to a member function prepare() on null in C:\xampp\htdocs\www***\post.php:6 Stack trace: #0 C:\xampp\htdocs\www***\post.php(23): getCode() #1 {main} thrown in C:\xampp\htdocs\www***\post.php on line 6

該当のソースコード

function getCode() {
	$stmt = $dbh->prepare('
		SELECT
			`id`,
			`code`,
			`value`
		FROM
			`code_area`
		WHERE
			`del` = 0
	');

	$stmt->execute();
	$result = $stmt->fetchAll();

	return $result;
}

getCode();

var_dump($result);

試したこと

関数にせず直接このように記載したら取得できます。

	$stmt = $dbh->prepare('
		SELECT
			`id`,
			`code`,
			`value`
		FROM
			`code_area`
		WHERE
			`del` = 0
	');

	$stmt->execute();
	$result = $stmt->fetchAll();


var_dump($result);

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

XAMPP

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[m.ts10806]

>補足情報
ここには挨拶ではなく（）にある通りバージョンや環境情報を記載します。
基本、要件を書くことに終始すべきで挨拶はむしろノイズにさえなり得ます。
「読まなくても問題解決できる部分」だからです。
「分かりづらい」と自ら書くのではなく、納得いくまで推敲されたら良いかなと。
質問は何度でも編集できます。

[kakedashi_php]

ご指摘ありがとうございます。
以後質問の際気を付けたいと思います。

[m.ts10806]

解決後なので「次から」でいいですが、
「XAMPP」はバージョン1つではないですし、
時間がたてば今ご自身が使っているものが今は最新でもそのうち最新ではなくなります。
バージョンをきちんと書きましょう。
OS,PHPのバージョン、DBの種類・バージョン

Answer 1

変数のスコープに関する理解が足りていない様に見受けられますので、まずは参照先のサンプルコードをよく読む&実際に試して把握されることを強くお勧めします。

その後に
関数の引数
についても同様に学習してください。

---

Fatal error: Uncaught Error: Call to a member function prepare() on null in C:\xampp\htdocs\www*\post.php:6 Stack trace: #0 C:\xampp\htdocs\www*\post.php(23): getCode() #1 {main} thrown in C:\xampp\htdocs\www***\post.php on line 6

このエラーは、何もないところからprepare()を使おうとしているけどそれはアウト　というエラーです。
具体的には

function getCode() {
$stmt = $dbh->prepare('
SELECT
id,
code,
value
FROM
code_area
WHERE
del = 0
');

で、存在しない$dbhを使おうとしています。
引数で$dbhを渡すなどして$dbhを用意する必要があります。

前後のコードが無いので勘になりますが

function getCode($dbh) {
$stmt = $dbh->prepare('
SELECT
id,
code,
value
FROM
code_area
WHERE
del = 0
');

$stmt->execute();
$result = $stmt->fetchAll();

return $result;
}

$result = getCode($dbh);

var_dump($result);

と言う感じにすれば想定している挙動をすると思います。
（修正部分の解説を全てするのは無理なので、まずは前述のリンクのサンプルを理解してみてから修正箇所を把握してみてください。）

Comments

[kakedashi_php]

ご回答ありがとうございます。
変数・関数に関しての基本的な知識がありませんでした。
大変勉強になりました。

頂いたご指摘で関数ができました。
本当にありがとうございます。

Answer 2

直接の回答は既に出ているので、今後に向けた提案をアドバイスとして置いておきます。

情報を取得する際のコードを関数にして共通化したいと思い

でしたら、一部ではなく「DB接続に関する一連の流れ」を全て関数化・・・
というより、クラス化したほうが良いように思います。

例えばコンストラクタで接続確立して、各メソッドで登録、参照、更新、削除を行う（各英語の頭文字を取り”CRUD”と言います）
テーブルが複数あるのでしたら継承なり何なりで共通部分は親側に任せてテーブル個別の処理を書く、とか（もしくはテーブル名とカラム名の宣言だけとか）。
続けていったらフレームワークっぽいものが出来上がります。

関数化はその1歩目と思います。考え方は「役割分担」
「まとめられそうかな」と思ったら切り出すのは良い傾向と思います。

Comments

[kakedashi_php]

ご回答いただきありがとうございます。
クラス化に関しても知識がありませんでした。
目指しているところはご提案頂いている内容のものになります。
勉強して挑戦してみようと思います。

ありがとうございます。

[m.ts10806]

既存のフレームワークのコードを読んでみるのが一番勉強になると思います。