phpで配列を返したいのですが良い方法はございませんでしょうか？

Question

お世話になっております。

グーグルさんで調査しても
https://teratail.com/questions/10692
や
https://hydrocul.github.io/wiki/programming_languages_diff/tuple/return-tuple.html
などと無理らしいのですが足掻きたいので質問失礼します。

宣言

    function getDB () {
		$idpw = '../../../../../idpw.php';//この中に$dsnで連想配列で入っている
		require("$idpw");
		$con = mysql_connect($dsn['host'],$dsn['user'],$dsn['pass']);
		if (!$con) {
		  echo('データベースに接続できませんでした。');
		}
		$db = mysql_select_db($dsn['dbname'], $con);
		if (!$db) {
		  echo('データベースを選択できませんでした。');
		}
		$db = mysql_query('SET NAMES utf8', $con);
		if (!$db) {
		  echo('文字コードを指定できませんでした。');
		}
		return $db;
    }

使用方法

		$list = 'SELECT * FROM HOGETABLE';
		$db=getDB();
		$list = $db->Execute($list_query);

のようにgetDB()関数を宣言したいのですが$dbを渡す良い方法がないようで
困っております。

どのようにコーディングするのが良いか伺ってよろしいでしょうか？

お忙しいところ大変申し訳ないのですが
上記お問い合わせいたします。

Answer 1

まあmysql_*系の関数を使っている時点で自己責任なので
なんとか自分で解決すべきですが、おかしいとこだけ

getDB()の戻りは$dbをやめて$conにしたほうがよいでしょう。
そのコネクションを利用してデータを抽出します。
あとは以下のような書き方をします。

$sql = 'SELECT * FROM HOGETABLE';
$con=getDB();
$res=mysql_query($sql,$con)
while($row = mysql_fetch_array($res,MYSQL_ASSOC)){
 $rows[]=$row;
}

Comments

[moitaro]

ありがとうございます。
古いソース手直しだったため失礼しました。

mysql_*系を止めて
$con = new PDO('mysql:host='.$dsn['host'].';dbname='.$dbname.';charset=utf8', 'username', 'password');
として　$con　を返し

使用側で
$con->query($list_query)と実装してみます。

厳しいお言葉ありがとうございました。

Answer 2

なぜExecuteが使えないかというと、getDBで返却しているのが、mysql_queryの戻り値だからです。
mysql_queryの戻り値は、resourceまたはtrue/falseです。
http://php.net/manual/ja/function.mysql-query.php
今回使っているクエリはSETなので、戻り値はtrue/falseでしょうか。
で、あるから当然bool値にExecuteなんてメソッドは持っていません。
なので配列が返せないとかそういう話ではなく、関数の使い方の理解不足です。
で、解決策はというとyambejpさんの方法になるかと思います。

Answer 3

素直に、MySQLiのオブジェクト指向型を使いましょう。

$db = new mysqli('host', 'pass');