DB返り値の指定方法誤っていますでしょうか？

Question

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

データベースの接続しその値を返答するような関数を作っております。

func.php
<?php
    /**
     * DBに接続し $dbを取得する。
     * @return db を返す
     */

    function getDB () {
		//../../../../../idpw.php
		$idpw = '../../../../../idpw.php';
		require("$idpw");
		$con = mysql_connect($dsn['host'],$dsn['user'],$dsn['pass']);
		if (!$con) {
		  echo('データベースに接続できませんでした。');
		}
		$db = mysql_select_db('xs345', $con);
		if (!$db) {
		  echo('データベースを選択できませんでした。');
		}

		$db = mysql_query('SET NAMES utf8', $con);
		if (!$db) {
		  echo('文字コードを指定できませんでした。');
		}
		echo "↑で宣言終わりここまで";
		return $db;
	}

?>

<?php
require('func.php');
?>

<?php
//使う時
		$db=getDB();
		$list = $db->Execute($list_query);
?>

とやっているのですが落ちてしまいます……
上手く読み込めません。

何が誤っていてどのようにすれば良いか伺ってよろしいでしょうか？

Comments

[s.t.]

どのような落ち方をしているのでしょうか？

Answer 1

他の回答にあるようにPHPのバージョンによっては使えないので、それが一番の原因かと思います。
ですが、それ以前に使い方自体にも問題があるように思えます。

・mysql_select_dbとmysql_queryの戻り値の受取先が同じ変数
・mysql_queryの戻り値を返して戻り先でExecuteをしているが、mysql_queryの戻り値はリソースなのでExecuteのようなメソッドは含んでいない
・確認はしていませんが、関数内はローカル変数なので、接続情報など失われていないか心配

Answer 2

mysql_ 系の API は、PHP 7.0 以降で削除された 古い API ですので、使わないようにしましょう。

たぶん古い文献なり Webページの記述を参考にされているのだと思います。

Comments

[moitaro]

ありがとうございます。
古い文献を参考にしていたため
mysql_ 系は古いのですね……

http://freefielder.jp/blog/2014/05/php-mysql-mysqli.html
を参考に書き換えてみてみようかと思います。

他にも違う点がある気がしてますけど……

Answer 3

PHPのバージョンはいくつでしょうか？
最近のPHPはmysql_*系の古い接続方法はサポートしていませんよ

Answer 4

「落ちてしまいます」とか「上手く読み込めません」では意味がわからないって。

落ちるって具体的にどういう状態ですか？
500エラーですか？

読み込めないというのは具体的にどういう状態ですか？
画面が真っ白になっているのですか？
それとも何かエラー文が表示されているのですか？

Comments

[KiyoshiMotoki]

回答ではありません。
「質問への追記・修正の依頼」欄に記載すべき内容です。

[zico_teratail]

いいえ違います。
これはれっきとした回答です。

このような質問にはこのような回答しかできないという注意喚起も込めています。こういう具体性に欠ける「できません」のみの質問があまりにも多いので。
「質問への追記・修正の依頼」では目立たないので注意喚起になりません。

[zico_teratail]

ほらね、s.t.さんの修正依頼を無視して先にベストアンサーを決めちゃったでしょう？
そういう質問者なんですよ。

[moitaro]

ご迷惑をおかけし申し訳ございません。

$list = $db->Execute($list_query);
の所で返って来なくなるのです……