PHPからPDOを利用してSQLを操作する時に、SQL文の作成実行結果取得を関数にまとめたい。

前提・実現したいこと

$user = "testuser";
$password = "testuser";
$dbname = "testdb";
$host = "localhost";
$dsn = "mysql:host={$host};dbname={$dbname};charset=utf8";

try{
$pdo = new PDO($dsn,$user,$password);
$pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
$pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

//ここのsql文を増やすときに４行を度々書くのは大変なので、まとめたい。
$sql = "SELECT * FROM member";
$stm = $pdo->prepare($sql);
$stm->execute();
$results = $stm->fetchAll(PDO::FETCH_ASSOC);

}catch(exception $e){
echo $e->getMessage();
exit();
}

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

SQL文を関数にまとめるのが上手くいきません。
どのように改善すれば良いでしょうか。お願いします。

該当のソースコード

PHP
1function sql($sql,$code,$stm,$results){
2    $$sql = $code;
3    $$stm = $pdo->prepare($sql);
4    $stm->execute();
5    $$results = $stm->fetchAll(PDO::FETCH_ASSOC);
6        
7        return $sql,$code,$stm,$results;
8    }
9
10エラーが発生せずに実行できたとして、この関数を2つ実行する時に
11sql("sql","SELECT * FROM member","stm","results")
12sql("sql2","SELECT * FROM home","stm2","results2");
13
14関数名を変更しないと$resultsの値を取得するときにエラーになるのかなと思いました。

試したこと

//関数の実行
sql("test","SELECT * FROM member","stm","results");

//エラー
Parse error: syntax error, unexpected ',', expecting ';' in

sazi

2018/12/30 03:48

「//ここのsql文を増やすときに４行を度々書くのは大変なので、まとめたい。」何のために４行記述するのですか？

m.ts10806

2018/12/30 04:03

SQL文を増やすとは？その先の大目的なども記載された方が良いですね

toto654

2018/12/30 04:44

条件に合ったデータを取得するためです。例えば $sql = "SELECT * FROM member"; $sql2 = "SELECT * FROM home"; を指定したい時に、それぞれのコードを $sql = "SELECT * FROM member"; $stm = $pdo->prepare($sql); $stm->execute(); $results = $stm->fetchAll(PDO::FETCH_ASSOC); このように記述するのはコードが多くなってしまいますし、まとめたほうがいいのかと思いました。これは、全体を何度も書くよりもほかの方法があるのでしょうか

m.ts10806

2018/12/30 04:46

質問は編集できるので適宜追記してください

m.ts10806

2018/12/30 04:47

そういえば変数名冒頭$$となっているのは？ $resultだけ返せば良い気がします

行動規範の内容に同意します

回答2件

回答じゃないですが、私の辿った道を書いときますｗ

ほぼ、同じ考え方で、sql を組み立て結果を取り出す仕組みが作れないか考えました。
結果は sazi さんが紹介してくれているサイト+ α を見て、挫折しましたｗ

で、とりあえず放置して、別の学習を始めます。

オブジェクトを使用したプログラミングを学ぶと、sql を組み立てたり、取り出すのは、クラスに閉じ込めるのであまり効率化が出来ないことに気が付きます。
ここで、sql を組み立てて結果を返す仕組みは、効率化にあまり寄与しないと気が付きました。

なので、「再放置決定」ですｗ

そのうち、フレームワークを使用するようになります。

フレームワークには大抵の場合、クエリを作る機構が組み込まれています。
興味を持って中身を見ると、結構複雑なことをしていることに気が付きます。
ただ、これは、複雑な検索条件をより簡易に安全に記述することを目的としていて、当初考えていたような単純な「一つのテーブルから結果を取り出す」等を処理するのに効率化が進むものでは無いことを確認しました。

つまり、一度すっ飛ばして学習を進めると、次の躓きまで問題ありませんでした！ので、放置して、学習を進めることをオススメします。
当面、PDO が使いこなせれば十分です。

投稿2018/12/30 05:25