Как правильно реализовать метод вывода списка?

Question

[Max Ba]

Привет мужики. Создаю новый класс. В нем есть метод вывода массива данных из БД. Иногда надо вывести одну строку, иногда все, а иногда и какие-то параметры указать особые (заранее прописанные). Я делаю так:
public function getAll($options = []){}
Пока что массив принимает только: $options['id']

Вопрос:
1) Верно ли я построил логику вывода данных в одном методе, то есть либо можем указать ID, либо нет и тогда выведутся все записи, или надо было создать отдельный метод, выводящий только одну запись?
2. Правильно ли я назвал метод getAll. ведь переводится как "получить всё". А есть сокращу до get, по моему не тот смысл уже. Как такие методы вообще принято называть по PSR?

Answer 1

[Lander]

А чего бы не сделать так?:

...
    public function select($conditions = [])
    {
        ...
    }

    public function selectAll()
    {
        return $this->select([]);
    }

    public function selectOne($id)
    {
        $result = $this->select(['id' => $id]);
        return count($result)
            ? current($result)
            : null;
    }
    ...

Answer 2

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

1. Для поиска одной записи по ID было бы логичным и удобным реализовать отдельный метод, например какой нибудь findOne($id). Что касается метода getAll - что содержит массив Options? Набор параметров по которым необходимо выбирать записи? если да тогда передавать это массивом, не очень хорошая практика, лучше сделать какой-нибудь объект критерию. Да и само название $options не особо удачно...
2. Лучше называть подобные методы например find()

Comments

[Max Ba]

Получится дублирование кода? Пока только мысль, для findOne($id) и findAll() вывести запрос в свойства, а внутри метода видоизменять запрос, к примеру дописав WHERE id = ?

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

Макс Васильев, в подобной задачи легко избавиться от дублирования, как это сделать - зависит от реализации

Answer 3

[Denis]

ну разные есть методики некоторые используют название get как взять т.е. либо данные либо ошибка + методы с префиксом find где либо данные либо false.
А в вашем случае можно изощерятся как угодно типа findAll, findById, findByConditions и т. д.
PS прочитайте про ORM

Answer 4

[Andreo]

Не используйте в одном методе кучу функционала. Каждый метод должен делать строго определенную функцию и возвращать значение строго определенного типа иначе будут всплывать неожиданности при обработке таких "универсальных функций".
Для вывода одной записи и списка используйте отдельные методы.
Для именования методов поглядывайте на стандарты, но не заморачивайтесь - без фанатизма. Главное в наименовании переменных и функций было понятно что это такое, желательно соблюдать синтаксис английской мовы, и обязательно следовали единому правилу именования внутри проекта.

Comments

[Max Ba]

Метод ВСЕГДА строго возвращает массив. Получится дублирование кода? Или как? Я просто не могу найти как это реализовать? Запрос к БД прописывать в общих переменных а внутри метода видоизменять запрос?

[Andreo]

Если всегда возвращает массив, значит скорее всего выполняет одну функцию. То есть используя ваш $options вы задаете параметры для последующего запроса, ответ на который будет всегда список. То есть все, вроде, правильно. Единственное, чтоб я порекомендовал - это изменить название с getAll() на какой нибудь get().

[Andreo]

Методы, что рекомендуют выше, они достаточно полезны и визуально могут помогать при чтении такого кода. Но при их добавлении в ваш класс для работы с БД, конечно, старайтесь избегать дублирования кода.