(Resolvido) Probleminha com paginação

[#Tigre]

Olá pessoal,

estou usando uma função de paginação que encontrei e está funcionando corretamente, mas na primeira página aparecem dois probleminhas que não consigo identificar a causa.

A função está neste link:

http://forum.wmonline.com.br/index.php?showtopic=129828

E os problemas são essas duas mensagens de erro:

Notice: Undefined index: pagina in c:\classificados\admanuncios_ativos_usuarios.php on line 110

Notice: Undefined variable: html in c:\classificados\admanuncios_ativos_usuarios.php on line 73

A linha 110 é essa em azul:

$lpp = "10";

$pagina = $_REQUEST['pagina'];

if ($pagina == "") {$pagina = 1;}

$inicio = ($pagina * $lpp) - $lpp;

E a linha 73 essa:

while ($primeira <= $ultima) {

if ($primeira == $pagina) {

$html .= $primeira;

}

As mensagens só aparecem na primeira página.

Se alguém souber o porque das mensagens e puder passar a dica, agradeço.

[Beraldo]

O índice "pagina" não existe no array $_REQUEST. Use isset() para verificar a existência de uma variável.

Inicie $html com o valor nulo antes de concatenar um valor a ela.

[#Tigre]

Olá Beraldo, vou testar as suas dicas. Agradeço pela atenção.

[Henderson]

só completando o que o beraldo disse, pra pegar o valor dá página sem dar aquele erro, faça assim

$pagina = (isset($_GET['pagina'])) ? $_GET['pagina'] : 1;

;)

[Beraldo]

Exatamente, Henderson. Essa sintaxe nos poupa linhas de código. Essa simples linha resume isto:

if (isset($_GET['pagina'] ))
{
    $pagina = (int)$_GET['pagina'];
}
else
{
    $pagina = 1;
}

Coloquei o (int) para ter certeza de que o retorno será um inteiro. ;)

Como nem todos conhecem o Operador Ternário, eis o link explicativo:

http://www.php.net/manual/pt_BR/language.o....comparison.php

[#Tigre]

Fazendo assim também funcionou, mas será que tem lógica?

a primeira linha eu eliminei.

//$pagina = $_REQUEST['pagina']; (linha eliminada)
if (isset($pagina)){
$pagina="";
}
if ($pagina == "") {$pagina = 1;}
$inicio = ($pagina * $lpp) - $lpp;

Corrigiu o problema, mas o esquema que vocês passaram parece ser mais confiável.

Editado Janeiro 14, 2008 por #Tigre

[Beraldo]

$pagina só existirá automaticamente se register_globals estiver ativado, o que não é padrão desde o PHP 4.2.0.

[#Tigre]

  Beraldo disse:

$pagina só existirá automaticamente se register_globals estiver ativado, o que não é padrão desde o PHP 4.2.0.

Beraldo, isto quer dizer o que? Que o modo que passei depende do register_globals estar ou não ativo?

[Beraldo]

você comentou aquela primeira linha. logo, seu código só funcionará com register_globals = on.

Opte por usar a dica do Henderson. ;)

[#Tigre]

Dica usada e aprovada.

Valeu.