Проверяйте инициализирована ли переменная page перед проверкой ее на корректность.
Я так понимаю, что ссылка index.php == index.php?page=1, тогда

if (!isset($_GET['page'])){
   $page = 1;
}

И не надо никакого перенаправления.
Скрипты есть и получше. Ваш скрипт сложно расширять, то есть если вам нужно будет вывести по 10 страниц с обеих сторон, то нужно будет дописывать еще строки. Используйте циклы, код получится несколько короче и более гибким.

;;Как сделать, так что бы человек зайдя на index.php перенаправлялся на index.php?&page=1
я бы на вашем месте курил в сторону .htaccess (; ну или как T1grOK посоветовал (;

;;Error_Reporting(E_ALL & ~E_NOTICE);
а вот тут лучше бы перенаправлять ошибки в лог-файл, не стоит юзверю показывать подноготную (;

T1grOK, спасибо сделал проверку, как ты посоветовал:

if (!isset($_GET['page']))
{
$page = 1;
}

/* Проверяем, является ли переменная числом */
if (!preg_match("|^[\d]+$|", $page)) 
{
exit ("<p>Неверный формат запроса! Проверьте URL!");
}

Хотел ещё спросить стоит ли делать ссылки вида: index.php?page=1 вместо index.php?&page=1 и что это даст?

;;А если это вообще убрать
логи - они нужны (; покури вот эту статью - ИМХО оч. пользительно:
http://i-novice.net/skryvaem-vyvod-oshibok-v-php/

xJlaIIax
Знаешь репутация Попова? Они ниже плинтуса, этот скрипт Попова =)
1) irbis-team.com // постраничная навигация
2) на форуме в конкурсах были попытки сделать качественные постраничные навигации, на их примере + комментариях экспертов можно и самому написать все качественно.
3) Тигрок ответил правильно, если переменной нету, значит будет ставиться 1. Так же в этой навигации нету даже простой защиты от инъекций! Надо написать (int) при получении номера страницы.

Посмотрел, там 8 работ.
Какую Вы бы посоветовали и почему?
Так же нашел статейку: http://pyha.ru/articles/php/paginator/
Что Вы думаете об ней?

xJlaIIax

twin, подредактировал скрипт:
MySQL таблица:

CREATE TABLE `options`
(
`id` INT(2) NOT NULL AUTO_INCREMENT,
`nums` INT(2) NOT NULL,
`neighbours` INT(2) NOT NULL,
PRIMARY KEY (`id`) 
);

Скрипт:

<?php
/* Проверяем является ли переменная page числом */
$page = (int)$_GET['page'];
if (!isset($_GET['page']))
{
$page = 1;
}
/* Количество записей выводимых на страницу */
$result1 = mysql_query ("SELECT nums FROM options", $db);
$myrow1 = mysql_fetch_assoc($result1);
$nums = $myrow1['nums'];
/* Подсчитываем количество строк */
$result2 = mysql_query ("SELECT COUNT(*) AS id FROM data", $db);
$myrow2 = mysql_fetch_assoc($result2);
$elements = $myrow2['id'];
/* Выясняем количество страниц */
$pages = ceil($elements/$nums);
/* Теперь проверяем чтобы переменная page не была меньше 1 и не больше последней страницы */
if ($page < 1)
$page = 1;
elseif ($page > $pages)
$page = $pages;
/* Для лимита в нашем запросе, подсчитываем первый оператор, который показывает с какой записи мы начинаем делать выборку */
$start = ($page-1)*$nums;
/* В конце нашего запроса добавляем LIMIT $start, $nums */
$result = mysql_query ("SELECT name,description,author,date FROM data LIMIT $start, $nums", $db);
?>
<!-- Стиль для листинга -->
<div class="navigation">
<?php
/* Количество выводимых страниц, выводим только соседние страницы по отношению к текущей */
$result3 = mysql_query ("SELECT neighbours FROM options", $db);
$myrow3 = mysql_fetch_assoc($result3);
$neighbours = $myrow3['neighbours'];
$left_neighbour = $page - $neighbours;
if ($left_neighbour < 1) $left_neighbour = 1;
/* Выводим листинг */
$right_neighbour = $page + $neighbours;
if ($right_neighbour > $pages) $right_neighbour = $pages;

if ($page > 1)
{
echo ' <a href="?page=1">начало</a> ... <a href="?page='. ($page-1) .'">←сюда</a> ';
}

for ($i=$left_neighbour; $i<=$right_neighbour; $i++)
{
if ($i != $page)
{
echo ' <a href="?page='. $i .'">'. $i .'</a> ';
}
else
{
echo ' <b>'. $i .'</b> ';
}
}

if ($page < $pages)
{
echo ' <a href="?page='. ($page+1) .'">туда→</a> ... <a href="?page='. $pages .'">конец</a> ';
}
?>
<!-- Закрываем стиль для листинга -->
</div>

Стиль для листинга:

.navigation 
{
    font-family: Tahoma; /* Шрифта для текстов и номеров */
    text-align:center; /* Выравниваем листинг по центру */
}

.navigation a:link,
.navigation a:visited,
.navigation a:hover,
.navigation a:active 
{
    text-decoration:none; /* Отмена подчёркиваний у ссылок */
    color: #000; /* Цвета ссылок */
    background-color: #fff; /* Цвета блоков в случае, если картинка не найдена */
    background-image: url(nav.gif); /* Фоновое изображение */
    padding:3px 6px; /* Отступы для формирования блоков */
    font-size:12px; /* Размер шрифта ссылок */
    font-weight:bold; /* Полужирный шрифт ссылок */
}

/* Тоже самое, только после наведения курсора мыши */
.navigation a:hover 
{
    color:#000;
    background-color: #fff;
    background-image: url(fon.gif);
}

.navigation span 
{
    text-decoration:none;
    color:#000;
    background:#fff;
    padding:3px 6px;
    font-size:12px;
}

Хочу услышать Ваше мнение, какие есть недостатки, достойная ли защита?

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

/* Проверяем является ли переменная page числом */
if (!preg_match("|^[\d]+$|", $page)) 
{
exit ('<p>Неверный формат запроса! Проверьте URL!</p>');
}

?

Нет, я же не могу залесть и посмотреть php код на других сайтах, как там проверяется переменная на число.
Или ты про то что выводится если переменная не является числом?
twin, если я сделал не правильно объясни, как надо.

Регулярка - это тяжелый процес, лучше просто: $page = (int)$page; и нет проблем, пройдут только цифры.

twin, если я правильно тебя понял, то не хорошо пользователю показывать белый экран с этой надписью, надо это оформить под сайт?

xJlaIIax
Конечно, в основном это редирект на 404, или специальные предупреждения, что не правильно сделали то и то.

В постраничке обычно на первую. Можно прям на этом форуме посмотреть.

twin, а если так:

if (!preg_match("|^[\d]+$|", $page)) 
{
exit ('<meta http-equiv="refresh" content="0; url=index.php">');
}

Я думаю простой пользователь не будет в переменную вместо числа ставить символы.

Еще хуже. Если не число, сделай числом. Единицей. и не пудри людям моск всякими перенаправлениями и смешными надписями с восклицательными знаками. Откуда мода такая... Подозреваю Мэтра.

twin, ну так я ещё в этом деле не опытен, вот и делаю такие ошибки.
Так делать надо:

if (!preg_match("|^[\d]+$|", $page)) 
{
$page = 1;
}

xJlaIIax
thispage.php?page=9999999 - какая выведется?
И зачем, зачем регулярка? Можешь объяснить?

Хотел ещё спросить, как лучше переменную проверять на число?
Так:

if (!is_numeric($page)) 
{
$page = 1;
}

Или как было:

if (!preg_match("|^[\d]+$|", $page)) 
{
$page = 1;
}

Не нужна проверка вообще.

$page = (int)$page;
if(($page < 1) || ($page > $pages)
{
    $page = 1;
}

inpost, хотя я сейчас подумал там косяка с page нет.
То есть, если page менешь 1, то делаем page=1, если page больше чем количество страниц, то делаем page=номер последней страницы.

xJlaIIax
Прочитай мой код, у меня отсутствует регулярка!

inpost, то есть регулярка это elseif?
А чем плоха регулярка?

int - переводит любые данные в число! Если там страшный скрипт, то будет ответ "0", если там числа - вернёт числа. Выполнит эту работу в 10 раз быстрее.
Регулярка лишь проверит строку, и если вдруг там будут не только числа - выйдет false, и тогда получится page=1. Разница в скорости выполнения.
Смотри, в адресную строку я введу: index.php?page=5`'|;+" - при твоей регулярке откроется страница 1, при обычном int возьмется число до первого знака, значит выйдет ответ 5, и откроется 5-ая страница, как бы вырежет остальное.
Это такой, детский-легкий пример, чтобы ты понимал, как устроено приблизительно. Регулярка - зло в ПХП, просто в некоторых моментах без него нельзя! Если есть любой другой способ без них сделать тоже самое, то лучше обойтись без них.

inpost, если я буду использовать int, то в начале скрипта нужно?

if (!isset($_GET['page']))
{
$page = 1;
}

Ну эта надпись нужна независимо от того, используешь int или нет! Я говорю, что вместо регулярки надо int.

То есть, мы убираем из скрипта код:

if (!preg_match("|^[\d]+$|", $page)) 
{
exit ('<p>Неверный формат запроса! Проверьте URL!</p>');
}

И меняем

if ($page < 1)
{
$page = 1;
}
elseif ($page > $pages)
{
$page = $pages;
}

На

$page = (int)$page;
if(($page < 1) || ($page > $pages)
{
    $page = 1;
}

и всё, больше ничего менять не надо, я правильно понял?

Можно и твоим способом:

$page = (int)$_GET['page']; // В самом начале, далее обращаться непосредственно к $page!
// тут куча текста
if ($page < 1)
    $page = 1;
elseif ($page > $pages)
    $page = $pages;

UPDATE: Я подправил, чтобы не портить твой скрипт

Я подредактировал скрипт, посмотрите.

Где смотреть?

Вот это

$result00 = mysql_query("SELECT COUNT(*) FROM data WHERE cat='$cat'");
$temp = mysql_fetch_array($result00);
$posts = $temp[0];

лучше так

$result00 = mysql_query("SELECT COUNT(*) AS cnt FROM data WHERE cat='$cat'");
$posts = mysql_result($result00, 0);

А в целом пойдет.

twin, не то
12 пост в этой теме, http://phpforum.ru/index.php?act=Print&cli...imin=11&limit=1

А тоже самое. Это место поправь и пойдет.

Заменил:

/* Подсчитываем количество строк */
$result2 = mysql_query ("SELECT COUNT(*) AS id FROM data", $db);
$myrow2 = mysql_fetch_assoc($result2);
$elements = $myrow2['id'];
/* Выясняем количество страниц */
$pages = ceil($elements/$nums);

На:

/* Подсчитываем количество строк */
$result2 = mysql_query ("SELECT COUNT(*) AS cnt FROM data", $db);
$myrow2 = mysql_result ($result2, 0);
/* Выясняем количество страниц */
$pages = ceil($myrow2/$nums);

Если больше нет проблем к безопасности скрипта и к коду, можно создать тему с этим скриптом и прицепить её, много новичков ищут хорошую и не сложную постраничную навигацию.

P.S. twin, у тебя опечатка, не

$posts = myscl_result($result00, 0);

а

$posts = mysql_result($result00, 0);

<?php
/*
* Количество записей выводимых на страницу 
*/
    $num   = 10;
    
/* 
* Количество выводимых страниц, выводим только соседние страницы по отношению к текущей 
*/    
   $height = 10;

/*     
* Проверяем является ли переменная page числом 
*/
    $page = !empty($_GET['page']) ? (int)$_GET['page'] : 1;    

/*    
* Подсчитываем количество строк 
*/
    $result   = mysql_query("SELECT COUNT(*) AS `cnt` FROM `table` ", $db);
    $elements = mysql_result($result, 0);
    
/*
* Выясняем количество страниц 
*/
    $total = ceil($elements / $num);
    
/*
* Теперь проверяем чтобы переменная page не была меньше 1 и не больше последней страницы 
*/
    if ($page < 1)
        $page = 1;
    elseif ($page > $total)
        $page = $total;
    
/* 
* Для лимита в нашем запросе, подсчитываем первый оператор, который
* показывает с какой записи мы начинаем делать выборку 
*/
    $start = ($page - 1) * $num;
    
/* 
* В конце нашего запроса добавляем LIMIT $start, $nums 
*/
    $result = mysql_query ("SELECT * FROM `table` LIMIT ". $start .", ". $num, $db);
	
?>
<!-- Стиль для листинга -->
<div class="navigation">
<?php
    
    $left = $page - $height / 2;
    
    if ($left < 1) 
        $left = 1;
    
    $right = $page + $height / 2;
    
    if ($right > $total) 
        $right = $total;

/*
* Выводим листинг 
*/    
    if ($page > 1)
        echo ' <a href="?page=1">начало</a> ... <a href="?page='. ($page - 1) .'">←сюда</a> ';

    
    for ($i = $left; $i <= $right; ++$i)
    {
        if ($i != $page)
            echo ' <a href="?page='. $i .'">'. $i .'</a> ';
        else
            echo ' <b>'. $i .'</b> ';
    }
    
    if ($page < $total)
        echo ' <a href="?page='. ($page + 1) .'">туда→</a> ... <a href="?page='. $total .'">конец</a> ';

?>
<!-- Закрываем стиль для листинга -->
</div>

Проверяй и пиши описание пошагово, как тут, закрепим. Отчего же нет.

twin, хорошо сегодня вечером сделаем. Описание писать в этой теме или создавать новую?
Ты думаешь, значение переменным лучше писать так: $num = 10 $height = 10?
Просто, зачем я сделал выборку значений из базы, потому что если человек в пять файлов вставил скрипт, то ему будет проще подредактировать значение в БД, чем открывать 5 этих файлов и править в них.

Создай новую, её и закрепим.

twin, я не много не пойму, ты написал

    $left = $page - $height / 2;
    
    if ($left < 1) 
        $left = 1;
    
    $right = $page + $height / 2;
    
    if ($right > $total) 
        $right = $total;

Зачем делить на 2, если можно сделать просто без деления

    $left = $page - $height;
    
    if ($left < 1) 
        $left = 1;
    
    $right = $page + $height;
    
    if ($right > $total) 
        $right = $total;

Можно. Просто хочется задать 10 и чтобы 10 ссылок и было. А не 20...
Мож я не прав, обоснуй.

А зачем нужны лишние телодвижения?
Ты не много не так понял скрипт, в height пишем сколько мы хотим выводить страниц, не считая начало <сюда текущей странице сюда> конец.
На пример, если делить на 2, то нам надо писать в height 8, если не делить на 2, то пишем 4.
Например, мы находимся на 2 страницы: начало...<сюда 1 2 3 4 5 сюда>...конец

Ну или если взять при делении height 10, то без деления надо писать 5
Получится:
начало...<сюда 1 2 3 4 5 6 сюда>...конец

Ну, как убираем деление?

Ты попробуй, установи скрипт на тестовой сайт, как ты делал в конкурсах и сразу всё поймёшь.