Можно ли вызвать метод другого класса?

Question

[Оптимус Пьян]

Вопрос по сути о dependency injection описано тут https://github.com/codedokode/pasta/blob/master/ar... но у них валидирующий класс class UserValidator у меня же это всего лишь метод...

//Есть класс работы с БД
class DBworker{
  public function CountUsers(){
    //Сходит в БД и скажет есть ли уже такой логин
  }
}
// Есть класс валидации данных
class Validation{
  public function UniqueUsers(){
    //Должен вернуть true / false
  }
}

Суть в том что, метод UniqueUsers никак не сможет выполнить свою задачу без данных из базы. Он должен сам в неё сходить? Запросить что бы метод из класса работы с БД сходил (это возможно вообще и если да, то как это сделать) или надо делать класс работы с БД абстрактным, класс валидатор его потомков и обращаться что-то типа ->parent что бы получить данные?

Answer 1

[Alexander Pushkarev]

Ну можно где-нибудь хранить ссылку на инстанс класса базы. И к нему уже обращаться.

Можно и такое решение

public function UniqueUsers()
{
   $count = (new DBworker())->CountUsers();
}

Comments

[Оптимус Пьян]

Я правильно понимаю что если мне в класс работы с БД надо конфиг прокинуть для подключения, то тогда нужно через UniqueUsers() его пробросить тоже, например:

public function UniqueUsers($name, $config)
{
   $count = (new DBworker($config))->FindUsers($name);
}

[Alexander Pushkarev]

Оптимус Пьян: Да, как вариант создайте класс для хранения конфигов. Подойдёт Registry pattern

[Оптимус Пьян]

Alexander Pushkarev: Да, я понял как раз по ссылке о недостатках Registry и ServiceLocator, но это я не как упрёк вам, а к тому что как ни крути, глобальные переменные меняются на глобальные классы со статическими методами ))

Answer 2

[Дмитрий Брусенский]

Думаю через DI будет самое то

<?php
class DBworker
{
    public function countUsers()
    {
        // TODO: реализовать проверку в БД
    }
}

class Validation
{
    private $db; 
    public function __construct(DBWorker $db)
    {
        $this->db = $db;
    }
    public function uniqueUsers()
    {
        $this->db->countUsers();
    }
}

Вы сначала объявляете DI зависимости, потом уже он сам внедряет их, определяя что нужно.
P.S.: почитайте про PSR-2, Вам очень нужно.

Comments

[Оптимус Пьян]

Вот не знаю в данном случае правильно ли что Validation будет сам с БД работать? Ведь это нарушает вроде как принцип что у каждого своя маленькая задача. Хотя в данном случае
$this->db->countUsers();
запрос сам будет в countUsers() а это нормально

И ещё вопрос по construct(DBWorker $db) вы аргументом передаёте целый класс? Так можно? Недостаток тут что придётся тогда передавать его при создании
$val = new Validation(DBWorker $db);
так и пойдёт что потом в него надо ещё передать класс логгера ещё что бы залогировать какую-то ситуацию и пошло-поехало ))

Answer 3

[ipokos]

Работу с БД/Моделями можно выносить в singleton