2

大家好，我的代码有问题。使用 __construct() 我收到致命错误：调用非对象上的成员函数 prepare() 但没有它我的代码正在工作。

class Animals{

public $db_fields;

 public function __construct(){
  $this->db_fields = $this->get_fields();

  foreach($this->db_fields as $field){
  $this->$field = "";
 }


 public function get_fields(){
  global $dbh;

  $q = $dbh->prepare("DESCRIBE animals");
  $q->execute();
  $db_fields = $q->fetchAll(PDO::FETCH_COLUMN);

  return $db_fields;
 }
}
$f = new Animals();

/*** mysql hostname ***/
$hostname = 'localhost';
/*** mysql username ***/
$username = 'root';
/*** mysql password ***/
$password = '';
/*** mysql database***/
$dbname = 'animals';

try {
$dbh = new PDO("mysql:host=$hostname;dbname=$dbname", $username, $password);
/*** echo a message saying we have connected ***/
echo 'Connected to database <br />';

$dbh = null;
}
catch(PDOException $e)
{
echo $e->getMessage();
}

我只想让我的字段（animal_id，animal_type，animal_name）与

public $animal_id;
public $animal_type;
public $animal_name;
4

1 回答 1

0

您必须在创建后Animals实例化该对象，否则它未在函数中定义。$dbhget_fields()

$dbh = new PDO("mysql:host=$hostname;dbname=$dbname", $username, $password);
$f = new Animals();

话虽如此，更好的设计是使用依赖注入，并将$dbh对象发送到类，如下所示：

$dbh = new PDO("mysql:host=$hostname;dbname=$dbname", $username, $password);
$f = new Animals( $dbh);

因此，您更新的课程将如下所示：

class Animals{

    private $dbh;

    public function __construct( $dbh){
        $this->dbh = $dbh;
    }

    public function get_fields(){
        $q = $this->dbh->prepare("DESCRIBE animals");
    }
}

好处？你摆脱了global变量。通常，需要一个全局变量是一个糟糕的设计，你想避免它。您可以根据您遇到的问题了解原因 - 它需要设置全局状态才能运行。

于 2012-11-18T04:50:35.780 回答