関数を別のファイルにまとめたら元ファイルで関数を呼び出しても実行できません。

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）PHPで掲示板を作っています。
データベースへの接続処理を、別ファイルから関数で呼び出そうとしたら以下のエラーメッセージが発生しました。
関数を使わずに全て同じファイルに書いた場合は正常に動作しました。

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

Notice: Undefined variable: db in /opt/lampp/htdocs/keijiban1.copy/model.php on line 14

Fatal error: Uncaught Error: Call to a member function prepare() on null in /opt/lampp/htdocs/keijiban1.copy/model.php:14 Stack trace: #0 /opt/lampp/htdocs/keijiban1.copy/comment.php(20): outputdb() #1 {main} thrown in /opt/lampp/htdocs/keijiban1.copy/model.php on line 14

該当のソースコード

try{
    $db=new PDO('mysql:dbname=keijiban1;host=127.0.0.1;charset=utf8','root','');

    function inputdb(){
      //コメントをデータベースに登録
      $statement=$db->prepare('INSERT INTO comments SET comment=?, created_at=NOW()');
      $statement->execute(array($_POST['comment']));
    }

    function outputdb(){
      //コメントをデータベースから取り出し
      $comments=$db->prepare('SELECT * FROM comments ORDER BY id DESC LIMIT ?,5');
      $comments->bindParam(1,$start,PDO::PARAM_INT);
      $comments->execute();
    }
}catch(PDOException $e){
    echo 'DB接続エラー：'.$e->getMessage();
}

試したこと

try{}の中に書かれているデータベース登録、取出しの処理をtry{}catch{}の下に書いてみた。
データベース登録、取り出しの処理を元のファイルの関数呼び出しの部分に書いていた時は正常に動作していた。

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

元のファイルでの関数の呼び出しは、
inputdb();
と
outputdb();
で書いています。

Answer 1

素直に

inputdb($db);
//とか
outputdb($db);

みたいな感じにする方がなにかと分かりやすい気がします。

Answer 2

これらの関数は、グローバル変数である$dbを参照しているので、関数内の先頭にglobal $db;と書く必要があります。

また、tryはスコープを作らないので、関数宣言はtryの外に出しても大丈夫だと思います。

Comments

[imawork]

回答ありがとうございます。
global $db;　を関数の先頭に入力したら下記のエラーに変わりました。
dbについてのエラーは解決されたと思うのですが、、、
関数宣言はtryの外に出しました。
Notice: Undefined variable: comments in /opt/lampp/htdocs/keijiban1.copy/comment.php on line 24

Fatal error: Uncaught Error: Call to a member function fetch() on null in /opt/lampp/htdocs/keijiban1.copy/comment.php:24 Stack trace: #0 {main} thrown in /opt/lampp/htdocs/keijiban1.copy/comment.php on line 24

[magf]

`$comment->fetch`メソッドが呼び出せていないというエラーなので、`$comment`について何か似たようなことが起きているのではないでしょうか
詳細な回答はcomment.phpの該当部分のソースがないと難しいです

[magf]

おそらくoutputdp関数の先頭にglobal $comments;と書くととりあえず解決するのではないでしょうか

[imawork]

ありがとうございます。
global $comments;と書いたらエラー表示がなくなりました！
データベースにも無事登録されています。
しかし、データベースからの取り出しができず、記事が表示されません。
以下がcomment.phpです。
<?php
require('model.php');
require('template.html');

if(isset($_POST['comment'])){
print('投稿しました<br>');
inputdb();

}else{

}

if(isset($_REQUEST['page']) && is_numeric($_REQUEST['page'])){
$page=$_REQUEST['page'];
}else{
$page=1;
}
$start=5*($page-1);
outputdb();

//コメント、時間を繰り返し表示
while($comment=$comments->fetch()){
print('<p>');
print($comment['comment']);
print('</p>');

print('<time>');
print($comment['created_at']);
print('</time>');
print('<hr>');
}

////２ページ目以降なら前のページを表示する（１ページ目は表示しない）
if($page>=2){

print('<a href="comment.php?page='.($page-1).'">前へ</a >');
}

//最終ページは表示コメント数／5の繰り上げで表示
$counts=$db->query('SELECT COUNT(*) AS cnt FROM comments');
$count=$counts->fetch();
$max_page=ceil($count['cnt']/5);
//現在のページ＜最終ページなら次ページを表示する
if($page<$max_page){
print('<a href="comment.php?page='.($page+1).'">次へ</a>');
}

?>

[imawork]

outputdp関数の先頭にglobal $db、global $comments、global $startを追加したら全て解決しました！！スコープの内容についても知ることができたのですごく勉強になりました。ありがとうございました！！

[imawork]

ちなみに、globalを使わないためのアイデアもありましたら教えて頂きたいです。

[magf]

クラスを作って$dbなどをprivateなプロパティにするのが昨今では自然かなと思います
https://www.php.net/manual/ja/language.oop5.visibility.php

[imawork]

たびたびありがとうございます。クラスの勉強を進めていきたいと思います。助かりました。