pdoを外部ファイル化して使ってみたい。

Question

<?php
define('DB_HOST', 'localhost');
define('DB_NAME', 'test_01');
define('DB_USER', 'root');
define('DB_PASSWORD', 'root');

// 文字化け対策
$options = array(PDO::MYSQL_ATTR_INIT_COMMAND=>"SET CHARACTER SET 'utf8'");

// PHPのエラーを表示するように設定
error_reporting(E_ALL & ~E_NOTICE);

// データベースの接続
try {
  $dbh = new PDO('mysql:host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASSWORD, $options);
  $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
} catch (PDOException $e) {
  echo $e->getMessage();
  exit;
}

$sql="SELECT * FROM mondai";
$stmt= $dbh->prepare($sql);
$stmt->execute();

while(1){
  $result = $stmt->fetch(PDO::FETCH_ASSOC);
  if($result==false){
    break;
  }
  print '<div class="card card-body"><a href="1_1.php">'.$result[top].'</a></div>';
}
?>

まずはDBの接続を関数で呼び出せるようにしたいと考え、このように変更しました。
しかし、上のコードで表示されていたものが表示されなくなり、つまずいてしまいました。

<?php
define('DB_HOST', 'localhost');
define('DB_NAME', 'test_01');
define('DB_USER', 'root');
define('DB_PASSWORD', 'root');

// 文字化け対策
$options = array(PDO::MYSQL_ATTR_INIT_COMMAND=>"SET CHARACTER SET 'utf8'");

// PHPのエラーを表示するように設定
error_reporting(E_ALL & ~E_NOTICE);

// データベースの接続
function db() {
  try {
    $dbh = new PDO('mysql:host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASSWORD, $options);
    $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  } catch (PDOException $e) {
    echo $e->getMessage();
    exit;
  }
  return $dbh
}
$hoge=$this->db();
$sql="SELECT * FROM mondai";
$stmt= $hoge->prepare($sql);
$stmt->execute();

while(1){
  $result = $stmt->fetch(PDO::FETCH_ASSOC);
  if($result==false){
    break;
  }
  print '<div class="card card-body"><a href="1_1.php">'.$result[top].'</a></div>';
}
?>

Answer 1

単純に、;がないからsyntax errorが出てるんじゃないでしょうか？

//  return $dbh
↓
  return $dbh;

以下のようにエラーを表示するようにしておきましょう。

<?php
ini_set('display_errors', 1);
ini_set('error_reporting', E_ALL);

Comments

[watta_00]

試してみましたが、return $dbh;にしても動作は変わらずでした・・・。

[Y.H.]

で、以下2行を追加してどういうエラーメッセージが表示されましたか？
ini_set('display_errors', 1);
ini_set('error_reporting', E_ALL);

[watta_00]

はい、
Fatal error
: Uncaught Error: Call to a member function prepare() on null in /Applications/MAMP/htdocs/demo/test/mondai_test.php:28 Stack trace: #0 /Applications/MAMP/htdocs/demo/test/1.php(69): include() #1 {main} thrown in
/Applications/MAMP/htdocs/demo/test/mondai_test.php
on line
28
というエラーが表示されています。

[Y.H.]

その情報が非常に重要なので質問を編集し追記ください

[watta_00]

エラーをみたところ、$dbhがnullだったということがエラーコードの内容でした。
$hoge=db();
$sql="SELECT * FROM mondai";
$stmt= $hoge->prepare($sql);
$stmt->execute();

このように書くことで解決することができました。

PHPを勉強中で
<?php
ini_set('display_errors', 1);
ini_set('error_reporting', E_ALL);
これを書くことによりエラーが記載されることを知らなかったため、勉強になりました。
ありがとうございます。

Answer 2

外部化するならクラスにしてください
表示部分はコンストラクタとは別メソッドにしたほうが使い勝手はよいでしょう

Comments

[watta_00]

クラスにしなければ、この書き方では動作しないということでしょうか？

[yambejp]

動作しないと言うよりはまともな運用ができないと思います

Answer 3

クラスにしてないなら$this不要です。

Comments

[watta_00]

回答ありがとうございます。
$thisをなくし
$hoge=db();としましたが、改善されませんでした。

[m.ts10806]

「うまくいきません」と同じで、結果が何も伝わりません。そこからなにが起きたか具体的に教えてもらわないと次の試行錯誤になりません。
質問者さんと完全に同じ環境で動作確認している人はいませんので、一発で通ると思ってアドバイスはしていませんよ。

[watta_00]

アドバイスありがとうございます。
今後は気をつけたいと思います。

[m.ts10806]

質問に追記してもらえれば良いのですけど

Answer 4

ini_set('display_errors', 1);
ini_set('error_reporting', E_ALL);
をコードに追記し、エラーを確認したところ、

Fatal error
: Uncaught Error: Call to a member function prepare() on null in /Applications/MAMP/htdocs/demo/test/mondai_test.php:28 Stack trace: #0 /Applications/MAMP/htdocs/demo/test/1.php(69): include() #1 {main} thrown in
/Applications/MAMP/htdocs/demo/test/mondai_test.php
on line
28

このようなエラーが出ました。
以下のコード部分を変更することにより解決することができました。

$hoge=$this->db();
$sql="SELECT * FROM mondai";
$stmt= $dbh->prepare($sql);
$stmt->execute();

$hoge=->db();
$sql="SELECT * FROM mondai";
$stmt= $hoge->prepare($sql);
$stmt->execute();

Comments

[m.ts10806]

$hoge=->db();

いや↑これ動かないですよね。

Answer 5

function db(){
}

の中で、

$options

が利用できているかどうか？