Нужно ли объявлять свойство db в классе?

Question

[Max Ba]

Есть класс базы данных. Статичный синглтон.
Нужно ли при создании новых классов создавать свойство db protected и внутри класса работать с db->, а не в DB::
Если да, то в чем принципиальное отличие?

Answer 1

[Максим Федоров]

Есть класс базы данных. Статичный синглтон.

Синглтон это не очень хорошо ... почему - можно просто погуглить этому посвящено не мало материалов

Нужно ли при создании новых классов создавать свойство db protected и внутри класса работать с db->, а не в DB::

Ну тут все зависит от того как Вы будете передавать объект базы данных в конкретный экземпляр. Если будете например использовать какую-нибудь инъекцию зависимостей, тогда само собой нужно записывать объект базы в свойство класса. Если же Вы планируете в классах делать что-то типа такого $this->db = DB::getInstance() то особого профита это даст (кроме незначительного повышения читабельности кода), а вот количество используемой памяти увеличит (за счет выделения дополнительных ссылок на объект базы данных)

Comments

[Max Ba]

Я могу сделать тремя способами, какой лучше на первый взгляд?

1)
$myclass = new MyClass();
внутри класса делаю protected $db; и работаю с $this->db->
$myclass->db = DB::instance();

2)
Тоже самое через конструктор
$myclass = new MyClass(DB::instance());

3)
$myclass = new MyClass();
//внутри класса сразу использую БД, предварительно подключившись к ней
public function(){
$stmt = DB::query();
.....
}

[Максим Федоров]

Макс Васильев, на первый взгляд вариант 2 лучше

[Станислав Б]

Макс Васильев, $myclass = new MyClass(DB::instance()); лучше
можете попробовать написать пару юнит тестов для MyClass
и вопрос отпадет.

[davidnum95]

Макс Васильев, советую почитать про dependecy injection