phpでPEAR::DBを使っているのですがPEAR::DB関連の処理を関数化して使い易くしようと思

ったのですが以下のように関数化するとエラーになってPHPが動きません
どうしたらえらーがなくなりますか？

$sql="delete from hy_kgrs";
$res=& $db->query($sql);
if(PEAR::isError($res))
{
die($res->getMessage());
}

↓

dtbs("delete from hy_kgrs");

function dtbs($sql)
{
$res=& $db->query($sql);
if(PEAR::isError($res))
{
die($res->getMessage());
}
return $res;
}

質問者からの補足コメント

• 回答ありがとうございます。下記で動きました。
  
  dtbs("delete from hy_kgrs",$db);
  
  function dtbs($sql,$db)
  {
  $res=& $db->query($sql);
  if(PEAR::isError($res))
  {
  die($res->getMessage());
  }
  return $res;
  }

    補足日時：2024/04/21 17:40

• &を付ける必要は無い様です

  No.1の回答に寄せられた補足コメントです。 補足日時：2024/04/22 10:35

No.1 ベストアンサー 回答者： Ogre7077 回答日時： 2024/04/21 17:01

エラーが出たらエラー文言を読みましょう。

エラーを無くすために必要な情報が書いてあります。

ご提示の内容だけでは的確に回答できないので、
質問する際はエラー文言も併せて投稿するとよいでしょう。
例)
エラー文言が「変数 $db が定義されていない云々」ならば、
関数 dtbs の引数に $db を追加しましょうと回答

この回答へのお礼

select文に対応して関数側に参照渡しを付けた方がいいかもしれませんね

function dtbs($sql,&$db)
{
$res=& $db->query($sql);
if(PEAR::isError($res))
{
die($res->getMessage());
}
return $res;
}

お礼日時：2024/04/21 17:47