Как я могу улучшить этот алгоритм разбиения на страницы PHP? - PullRequest
Новая
       1

Как я могу улучшить этот алгоритм разбиения на страницы PHP?

3 голосов
/ 19 сентября 2010

Я работаю над алгоритмом разбиения на страницы в PHP. Я могу догадаться, что ему нужно пространство для улучшения, поэтому я хотел бы подумать о том, как его улучшить, будь то очистка самого кода с точки зрения UI / UX или всего, что вы можете придумать.

Алгоритм должен выводить нумерацию страниц следующим образом: 

1 2 3 ... 6 7 8 ... 97 98 99

или это: 

1 2 3 4 5 ... 6 7 8 9 10

или это: 

1 2 3

Вот мой код: 

<?php

if(!is_null($_GET['page'])) {
  $currentPage = $_GET['page'];
} else {
  $currentPage = 1;
}

if($pages != null) {
  echo 'Page: ';
}

// Less than 10 pages
if($pages <= 10) {
  for($page = 1; $page <= $pages; $page++) {
    echo '<a href="?page=' . $page . '">' . $page . '</a> ';
  }

// Greater than 10 pages, and we're somewhere in the middle of them
} elseif(($pages > 10) && ($currentPage > 4) && ($currentPage < $pages - 3)) {
  for($page = 1; $page <= 3; $page++) {
    echo '<a href="?page=' . $page . '">' . $page . '</a> ';
  }
  echo '... ';
  for($page = $currentPage - 1; $page <= $currentPage + 1; $page++) {
    echo '<a href="?page=' . $page . '">' . $page . '</a> ';
  }
  echo '... ';
  for($page = $pages - 2; $page <= $pages; $page++) {
    echo '<a href="?page=' . $page . '">' . $page . '</a> ';
  }

// Greater than 10 pages, and we're towards the end of the pages
} else {
  for($page = 1; $page <= 5; $page++) {
    echo '<a href="?page=' . $page . '">' . $page . '</a> ';
  }
  echo '... ';
  for($page = $pages - 5; $page <= $pages; $page++) {
    echo '<a href="?page=' . $page . '">' . $page . '</a> ';
  }
}

Ответы [ 3 ]

2 голосов
/ 19 сентября 2010

еще одна хорошая функция нумерации страниц в этой статье

http://www.frobie.com/pagination-with-php/2008-03-18/

1 голос
/ 19 сентября 2010

Я не уверен, что мой код лучше вашего, но вот как я решил похожую проблему.

Он принимает параметр для количества страниц, генерируемых и создает div с помощьюкласс pages, содержащий якоря, текущая страница имеет класс current.Это решение кажется немного чище, потому что меньше повторений.

function generate_pages($total,$current)
{ //Will generate pages and link to ?page=x when passed total pages to output
    if($total > 1)
    {
        $total=intval($total);

        $output='<div class="pages">';
        $current_page= (false == isset($current)) ? 1 : $current;
        for($page=1;$page<$total+1;$page++)
        {
            $lower=$current_page-3;
            $upper=$current_page+3;
            $special = ($page==$current_page) ? " class=\"current\"" : "";
            if(($page > $lower && $page < $upper) || $page < 2 || $page > ($total-1))
            {
                if($last_done_page+1 != $page) $output.= '... ';
                $output.='<a'.$special.' href="?page='.$page.'">'.$page.'</a>';
                $last_done_page=$page;
            }
        }
        $output.='</div>';
        return $output;
    }
}
0 голосов
/ 19 сентября 2010
...