Верно ли реализован метод фильтрации?

Question

[Max Ba]

Мужики, правильно ли я сделал реализацию данного класса. В частности метода filter(). Если метод не вызван, то выводим все строки.

$invoice = new Invoice(DB::init());
$invoice->setUser(1);

$invoice->filter('company_name', 'ромашка');
$invoice->filter('company_inn', '123451234512');
$invoice->filter('email', '[e-mail]');

$rows = $invoice->selectAll();
...

echo '<br>Всего найдено фирм: '.$invoice->count_rows();

company_name, company_inn, email и тд - заранее прописанные переменные внутри класса

Answer 1

[Александр Аксентьев]

Нет в коде понятия правильно или неправильно самописном, если это не совсем трешак или лапша.
Работает или нет - главный критерий.
Нравится или не нравится(самому) - второй по важности критерий.

Comments

[Max Ba]

Спасибо за мнение. не режет глаза код?)

[Александр Аксентьев]

Макс Васильев, режет глаза работа с базой через магию, почему бы просто ORM не использовать с понятными названиями методов where() и т.д.

Ну и передача DB::init() пиздос конечно таким образом. Это глобально должно быть, в каждую функцию свой init что ли передавать так.

[Леонид Князев]

Макс Васильев, Т.е. вы пишите код, что бы нравилось нам на Тостере?

[Max Ba]

Александр Аксентьев, почему глобально-nj? Раз передали объект БД в класс и работай сколько влезет. Вы про зависимости не читали?
Пздц, одни уверяют что ЛУЧШЕЕ решение это как раз передать объект бд через конструктор. Другие - можно присваиванием $invoice->db = DB::instance();
Но зато ацки гнобят, когда внутри класса есть что-то nbgf DB::...

[Max Ba]

Леонид Князев, мне важно понять, говнокод это или обычный приемлемый код.

[Александр Аксентьев]

Пздц, одни уверяют что ЛУЧШЕЕ решение это как раз передать объект бд через конструктор.

Это и есть лучшее решение, но с помощью DI, а не передачей в момент создания каждого класса в каждом месте кода.

почему глобально-nj? Раз передали объект БД в класс и работай сколько влезет.

Потому что каждый раз передавать в каждый класс это как минимум неудобно.

Уже ниже написали что смотрите реальные проекты и как там работа идёт.
Модель изначально должна наследоваться от общей модели которая так или иначе связана с базой любым способом.

Вы как разработчик не должны каждый раз когда используете модель думать о том что надо туда базу прокинуть или другие зависимости. Задача в момент написания код с моделью получить данные, а не подключить класс базы, соединиться с базой, передать инстанс в модель и только потом получить данные.

[Александр Аксентьев]

Макс Васильев, так же в реальности надо вытаскивать данные больше чем из одной модели за раз, или еще лучше вытаскивать связанные модели.

Ваш код будет выглядеть как-то так?

$invoice = new Invoice(DB::init());
$user = new User(DB::init());
$service = new Service(DB::init());

Сами оцените нормальность этого куска кода теперь, как вам?

[Max Ba]

Ну а как иначе? Работать сразу внутри класса с БД? ну делал так раньше, пока профи не увидели код и не написали пару комментов :)

$db = DB::init();
$invoice = new Invoice($db);
$user = new User($db);
$service = new Service($db);

[Max Ba]

Александр Аксентьев, не увидел коммент. Спасибо, понятней стало.

[Александр Аксентьев]

Макс Васильев,

$db = DB::init();
$invoice = new Invoice($db);
$user = new User($db);
$service = new Service($db);

Те же яйца только в профиль.
ссылка выше "как надо": www.phptherightway.com/#dependency_injection

Лучше бы внутренности выложили, а то обсуждаем насколько красиво это выглядит снаружи, а не то какой трешак там может быть спрятан под капотом.

[Max Ba]

Александр Аксентьев, не, на отдельную тему потянет. Там трешак нормальный такой.
Спасибо за поддержку. Создам класс App, в нем буду соединять все классы и БД.

Answer 2

[Максим Тимофеев]

А почему бы не подсмотреть у популярных решений?
https://www.yiiframework.com/doc/guide/2.0/ru/db-q...

Я так понимаю Ваш ->filter('email', '[e-mail]'); может строить только 1 тип запроса. А если надо like, <> и т.д.? Будете другой метод создавать?

И согласен с Александр Аксентьев, new Invoice(DB::init()) это очень странно, согласитесь более красиво выглядит Invoice::find()

Comments

[Max Ba]

Ну а почему не создать? Я изначально продумываю то, как будет класс работать. Внутри класса при получении переменных через filter как раз модифицируется запрос к БД посредством =, like%, IN.

[Максим Тимофеев]

Макс Васильев, Я же не сказал "скопировать", я сказал "подсмотреть". Делайте свое - это отлично. Но подсмотреть реализации стоит. Понимая как это сделали 2-3 frameork-а, Вы свое сделаете лучше.