OOP PHP load не возвращает правильный объект

Question

Проблема: Когда я загружаю модель с объектом load, она НЕ вернет вторую модель.

Посмотрите на класс Test, где я загружаю модель с объектом load. Код $this->two вернет объект One (должен загрузить объект Two).

Вопрос: Как решить эту проблему? Я открыть ваши предложения/идеи/код

Текущий результат:

one is working 
object(One)#3 (1) { ["error":"Model":private]=> NULL } 
one is working 
object(One)#4 (1) { ["error":"Model":private]=> NULL } 

Правильный результат:

one is working 
object(One)#3 (1) { ["error":"Model":private]=> NULL } 
two is working 
object(Two)#4 (1) { ["error":"Model":private]=> NULL } 

PHP:

one_model.php

<?php 

class One extends Model { 

    public function test() { 
     echo '<p>one is working</p>'; 
    } 

} 

two_model.php

<?php 

class Two extends Model { 

    public function test() { 
     echo '<p>two is working</p>'; 
    } 

} 

index.php

<?php 

class Controller { 

    public $load; 

    public function __construct() { 
     $this->load = new Load(); 
    } 

} 

class Load { 

    public function model($name) { 
     if (!class_exists($name)) { 
      require(strtolower($name) . '_model.php'); 
     } 
     $model = new $name; 
     return $model; 
    } 

} 

class Model extends PDO { 

    private $error; 

    public function __construct() { 

     $options = array(
      PDO::ATTR_PERSISTENT => true, 
      PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION 
     ); 

     try { 
      $dsn = "mysql:dbname=test;host=localhost;charset=utf8"; 
      parent::__construct($dsn, 'root', '', $options); 
     } 
     catch (PDOException $e) { 
      $this->error = $e->getMessage(); 
     } 
    } 

} 

class Test extends Controller { 

    public $one; 
    public $two; 

    public function __construct() { 
     parent::__construct(); 

     $this->one = $this->load->model('one'); 
     $this->two = $this->load->model('two'); 
    } 

    public function testing() { 
     $this->one->test(); 
     var_dump($this->one); 
     $this->two->test(); 
     var_dump($this->two); 
    } 

} 

// Usage 
$test = new Test(); 
$test->testing(); 

Answer 1

Я переместил оболочку PDO из модели для разделения класса и создал объект для подключения к базе данных.

Это хороший подход? Есть еще значений "waste of resources"?

class Database extends PDO { 

    private $error; 

    public function __construct() { 

     $options = array(
      PDO::ATTR_PERSISTENT => true, 
      PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION 
     ); 

     try { 
      $dsn = "mysql:dbname=test;host=localhost;charset=utf8"; 
      parent::__construct($dsn, 'root', '', $options); 
     } 
     catch (PDOException $e) { 
      $this->error = $e->getMessage(); 
     } 
    } 

} 


class Model { 

    public $db; 

    public function __construct() { 
     $this->db = new Database(); 
    } 

} 

Answer 2

Проблема есть, кажется, что вы передаете PDO::ATTR_PERSISTENT => true конструктору PDO. Это укажет PDO на кэширование и повторное использование соединений - как вы, несомненно, намереваетесь, но также, по-видимому, вызывает PDO для создания клонов первого экземпляра PDO, сгенерированного в рамках одного сценария, включая дочерний класс, используемый для создания этого первого экземпляра , (По-видимому, это не так много документации.)

Если вам нужны постоянные соединения, моя рекомендация заключалась бы в том, чтобы определить модели таким образом, чтобы они не расширялись от PDO. В любом случае это действительно не обязательно.

Например, если ваш код использует модель CRUD, абстрагируя ее от модели и помещая код базы данных в базу , то вместо реальной модели может решить эту проблему. Например, учитывая их в качестве основы:

class DatabaseSingleton { 
    private static $instance; 

    public static function get() { 
     if (!self::$instance) { 
      $options = array(
       PDO::ATTR_PERSISTENT => true, 
       PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION 
      ); 
      $dsn = "mysql:dbname=test;host=localhost;charset=utf8"; 
      self::$instance = new PDO($dsn, 'root', '', $options); 
     } 
     return self::$instance; 
    } 
} 

abstract class Crud { 
    private $pdo; 

    public function __construct() { 
     $this->pdo = DatabaseSingleton::get(); 
    } 

    public function create(...) { 
     $this->pdo->... 
    } 
    public function retrieve(...) { 
     $this->pdo->... 
    } 
    public function update(...) { 
     $this->pdo->... 
    } 
    public function delete(...) { 
     $this->pdo->... 
    } 
} 

Тогда ваши модели могут быть определены просто как дети:

abstract class Model extends Crud { 

    public function __construct() { 
     parent::__construct(); 
     // Etc... 
    } 
} 

и функциональность CRUD, определенной в Crud будут доступны для них, используя один экземпляр PDO, предоставляемый синглтоном.