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

Question

[Max Ba]

Мужики, в классе "Фирма", есть метод вывода всех фирм в массив. public function selectAll(){}
Внутри создается запрос к БД и возвращается массив. Иногда надо вывести весь массив фирм (около 100, а иногда с разделением на страницы LIMIT 30, 30).
Правильно ли я понял, что всё это добро делается в одном методе и сам запрос модифицируется через параметр передаваемый в selectAll(). То есть selectAll() выведет весь массив. selectAll(30,30) выведет с ограничением.
PS. К (30,30) не цепляйтесь. Там будет ассоциативный массив, просто для наглядности.

Comments

[alex-1917]

охренеть...

[Max Ba]

alex-1917, чё, охренеть?

[alex-1917]

Макс Васильев, если вы считаете, что дали достаточно информации для ответа, то я продолжаю охреневать

[Max Ba]

Да я б еще дал, если б знал что)
Ниже я написал то, что придумал. Хз как еще делать пагинацию. По логике вещей selectAllLimit() нормально справится. Но нет общего мнения на этот счет, в интернет все по разному реализуют. Я вот и думал, может есть какая-то более менее внятная схема (идея)

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

Правильно ли я понял, что всё это добро делается в одном методе и сам запрос модифицируется через параметр передаваемый в selectAll(). То есть selectAll() выведет весь массив. selectAll(30,30) выведет с ограничением.

select all, но не всегда, иногда не all, а только 30 :)

[Max Ba]

не знаю правильно или нет, создал приватный метод select.
А далее создал несколько методов.
selectAll()
selectOne() //передаем ID
selectAllLimit() //передаем 30, 30
А они уже сами передадут то, что нужно приватному методу select

Answer 1

[Evgeny Maltsev]

Если правильно понял, то лучше сделать два метода. Один на весь список, второй на пагинацию. Но внутри них часть кода будет одинаковая, её можно вынести в отдельный метод (если она большая).

Comments

[Max Ba]

Ну да, я так и сделал. Метод select приватный выводит все абсолютно, получая лишь ассоциативный массив как параметр. Смотрит что из этого массива isset и видоизменят запрос :)
А методы All, One, AllLimit и тд просто делают этот массив и вставляют в select.

Answer 2

[Сергей]

Танцуй от DI
Т.е. нужно решить какие методы нужны от класса.
Если выборка всех фирм, то что-нибудь типа getAll
Если выборка по ид, то getById(id)
и т.д.
Исходить надо из интерфейса который требует клиент класса. А сам класс реализует его.
Так ты сделаешь скрытым то, что не требуется клиенту и дашь себе возможность менять реализацию класса, без изменения его клиента.
Реализацию класса старайся делать без повторений используя скрытые методы.
Если в классе есть места которые будут часто изменяться, то выноси их в другие классы и делегируй им выполнение. SRP
Ну и прочитай про остальные принципы SOLID. Что бы использовать их. Без фанатизма. Это очень поможет тебе делать понятный для себя и других код. Ну и тестируемый.