Как грамотно реализовать пагинацию через класс?

Question

[VladyaKun1210]

Требуется создать пагинацию на сайте. Как грамотно можно реализовать пагинацию через класс?

class Paginate extends Connection{
                            public function paginateionSelect($page, $total_page){
                                $sql = "SELECT * FROM `clients`";
                                $result = mysqli_query($this->con, $sql);
                                $total_records = mysqli_num_rows($result);
                                $records_per_page = 5;
                                $total_page = ceil($total_records / $records_per_page);

                                if(!isset($_GET['page'])){
                                    $page = 1;
                                }
                                else{
                                    $page = $_GET['page'];
                                }
                                return $total_page;
                            }
                            
                            public function paginationLimit($result1){
                                $start = ($page - 1) * $records_per_page;
                                $sql1 = "SELECT * FROM `clients` LIMIT $start, $records_per_page";
                                $result1 = mysqli_query($this->con, $sql1);
                                return $result1;
                            }
                        }
                        
                        
                        $pagin = new Paginate();



<?php 
          for($page = 1; $page <= $total_page; $page++){
               ?>
          <li class="page-item"><a lass="page-link" href="clients.php?page=<?php echo $page ?>"><?php echo $page ?></a></li>
 <?php
 }
?>

Comments

[tukreb]

Просто возьмите готовое решение :)
https://github.com/KnpLabs/knp-components/blob/mas...

[Дмитрий]

По хорошему пагинатор должен знать номер страницы, количество элементов на страницу и общее количество элементов
https://github.com/Compolomus/Pagination

Answer 1

[ThunderCat]

Еще раз: Тут нет никакого ооп, у вас получается какой-то процедурный код, засунутый внутрь класса. Это к ооп не имеет никакого отношения. Не пытайтесь сделать деревянную печку, ничего хорошего не получится.
У вас:
Код реализует (такой себе) пэйджинг для одного конкретного запроса (нахрена такой пейджинг засовывать в класс?)
Код ничего не инкапсулирует, тупо выполняется как функция. По сути если мы уберем class Paginate extends Connection{ то ничего не поменяется.
Внутри методов используются глобальные переменные (GET), что само по себе уже лютый говнокод.
Много слов про солид и прочие вещи из ооп не пишу, в виду полного их отсутствия в коде, да и смысл?

По этому решения для вашего случая в контексте ооп просто нет, так как:
Пэйджинг работает с моделями на основе ОРМ, в пэйджинг в качестве аргумента передается модель, и далее уже пэйджинг работает с моделью, используя ее методы работы с хранилищем, а не просто тупо фигачит запросы. У вас моделей просто нет.
Модели подразумевают конкретный интерфейс, который позволяет работать с пэйджингом адекватно. Ну и моделей нет ))
Точкой взаимодействия между моделью и пэйджингом будет контроллер, про который тут вообще у вас никто не в курсе.
Вывод (view) никогда не пересекается с кодом логики, в него просто передаются готовые данные, после чего они вставляются в нужные точки шаблона.

Comments

[Дмитрий]

По хорошему S из солид, пагинация ни чего не должна знать. Внутренняя логика должна проверить юзера на мудака, посчитать все и отдать цифры для вью и лимиты для выборок.

[ThunderCat]

Григорий Васильков,

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

Отлично помню, только когда я увидел слово класс, я понял что просто так привязать к моему коду не получится. По этому через 5 проектов на процедурщине, нормально почитал по теме и уже тогда написал что-то на ооп. Да, шедевра не случилось, но какие-то азы я уже знал.

И вы ему говорите про SOLID.

Нет, не говорю, как раз про это и написал.

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

Или утонет (значит не тем занимается), или станет нормальным прогером, бо понятно что докручивать можно до бесконечности, но надо понимать где остановиться. Но хотябы будет понятно зачем вообще ооп и что это такое, в настоящий момент у ТС есть понятие что "ООП это классы", то есть добавить 2 строчки в процедурную хрень и думать что это уже ооп.

[ThunderCat]

Дмитрий,

По хорошему S из солид, пагинация ни чего не должна знать.

Сингл респонсибилити не говорит о том что пэйджинг не должен чего-то знать. Для S он должен решать только одну задачу (пагинацию), а не "ничего не знать". Про "не знать" можно за уши притянуть Лисков или инверсию зависимостей, но никак не сингл респонсибилити. И то, там про то что мы работаем с абстрактным интерфейсом, а не с конкретной моделью, а уж про интерфейс мы должны знать.

[Дмитрий]

Григорий Васильков, тут это про ответ. Это как в ларе пагинация на моделе, и это очень не правильно. Я же про то, что разбить по страницам можно что угодно, например файл по строкам или ещё чего, и не привязывать её к базам

[Дмитрий]

Григорий Васильков, свой путь помню, когда я написал свою первую навигацию мне сразу показали как из неё получается xss и sql inj

[ThunderCat]

Григорий Васильков,

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

Не совсем так, хотя может и звучит жестковато в моей интерпретации. Скорее я о том что нормально писать нельзя научиться просто тыкая палкой в разные лужи, если чел не будет сначала понимать как это работает, а только потом писать код, то в программировании он никуда не поедет... Просто тупо не возьмут никуда. По этому - читать, потом писать, а не как известный анекдот про северный народ...