Pagination with PHP and switch statement

[AdRock]

Hello there......I have a huge problem and have no idea how to sort it

I use a switch statement on my index.php page to display each of my pages (only some of them are shown to save space)

PHP: Select all

 <?php
 
switch ($_GET['page'])
{
 
case "contact":
include('contact.php');
break;
case "news":include('news.php');break;
default:
include('home.php');
}
?>

I have a pagination script to display a set number of record per page. The first page displays ok showing http://www.mysite.com/index.php?page=newsitem but when i click on the next link it defaults back to the default page http://www.mysite.com/index.php?page=home (obviuosly because the switch statement isn't set up to handle it)
The link that appears in the browser window is http://www.mysite.com/index.php?page=newsitem?limit=2&page=2 but the default page is displayed.

How do I sort this problem out? Do I need to use nested switch and if so how do I do it.

The pagination script I am using is from http://www.allsyntax.com/tutorials/P...with-PHP/2.php

I am sure if I didn't use the switch statement it would work but it seems very inefficient (been doing visual basic for the last 2 years at college).

I have seen t done in dozens of websites including how google does it searches and need to know how to do it.

Please help as I'm tearing my hair out here

[Gee Bee]

Problem could be in the url, instead of;

http://www.mysite.com/index.php?page=newsitem?limit=2&page=2

try

http://www.mysite.com/index.php?page=newsitem&limit=2&page=2

change that second ? for a &

[AdRock]

I tried that and it seemed to make sense but it still goes back to the default page

I am sure it's to do with the switch statement becuase the link http://www.mysite.com/index.php?page=newsitem&limit=2&page=2
isn't in the case include.

How would I add that to the switch becuase the number of pages can increase?

Thanks for your help

p.s. If you need me to attach my code I will

[Gee Bee]

Hi AdRock,

I've just noticed that i missed something quite obvious last night. The reason that when you use index.php?page=newsitem&limit=2&page=2 it redirects to the default page is that you have page defined twice in the url so your $_GET vars only have two values, limit => 2 and page => 2, as there is no switch check in the switch statement called 2 it loads the default page. I think you need to rename the second page variable to something else and it should solve that problem

[AdRock]

Thanks Gee Bee, I did what you suggested and made a change of my own and now it works.

I do have another question you may be able to help with though.

This is the change I made from $limit = 10 to $limit = 2 becuase my news items can be quite long so I only want 2 per page

PHP: Select all

 //set default if: $limit is empty, non numerical, less than 2, greater than 50 
if((!$limit)  || (is_numeric($limit) == false) || ($limit < 2) || ($limit > 50)) { 
     $limit = 2; //default 

How do I edit these hyperlinks or do i not have to?

PHP: Select all

//Results per page: **EDIT LINK PATH** 
echo("   
<a href=?page=newsitem&limit=10&amp;pagenum=1></a> 
<a href=?page=newsitem&limit=25&amp;pagenum=1></a> 
<a href=?page=newsitem&limit=50&amp;pagenum=1></a>"); 

Thanks for all your help helping me out

[Gee Bee]

your if statement could be written like this;

if(!isset($limit) || empty($limit) || !is_numeric($limit) || ($limit < 2) || ($limit > 50)) {
$limit = 2; //default

Not going to make a difference to how it performs but is more readable... for me anyway!

the <a> tags look fine but you dont need amp; in them so they could just be

<a href=apage.php?page=newsitem&limit=10&pagenum=1></a>

[benbacardi]

you do need &amp; if your going 2 make it validate with xhtml 1.0