Este é o jeito certo pra abrir páginas em PHP?

Galera, como alguns sabem, eu estou mechendo com PHP, por enquanto estou parado, mas escrevi este código em um papel e passei pro celular pra postar aqui (estou sem internet no pc).

Esta certo?

Código:

---

$page = "pages/home";
                                if (isset($_GET['pg'])) {
                                    $page = $_GET['pg'];
                                }

                                switch ($page) {
                                 
                                    case 'home':
                                    include "pages/{$page}.php";
                                    break;
                                    default:
                                    include "pages/404.html"; // página de erro "404"
                                    break;
                                }

---

Existe alguma outra forma? Amanhã vou no pc da minha tia e vou testar lá acho que abre ja usei, tenho esse outro também:

Código:

---


<?php
foreach ($_REQUEST as $___opt => $___val) {
  $$___opt = $___val;
}
if(empty($pg)) {
include_once("pages/home.php");
}
elseif(substr($pg, 0, 4)=='http' or substr($pg, 0, 1)=="/" or substr($pg, 0, 1)==".")
{
echo '<b>Erro : A página não existe.</b>';
}
else {
include("pages/$pg.php");
}
?>

---

Qual deles devo usar? Help ! RS :heckno:

Citação:

---

Postado originalmente por gokuka [Only registered and activated users can see links. Click Here To Register...]

Galera, como alguns sabem, eu estou mechendo com PHP, por enquanto estou parado, mas escrevi este código em um papel e passei pro celular pra postar aqui (estou sem internet no pc).

Esta certo?

Código:

---

$page = "pages/home";
                                if (isset($_GET['pg'])) {
                                    $page = $_GET['pg'];
                                }

                                switch ($page) {
                                 
                                    case 'home':
                                    include "pages/{$page}.php";
                                    break;
                                    default:
                                    include "pages/404.html"; // página de erro "404"
                                    break;
                                }

---

Existe alguma outra forma? Amanhã vou no pc da minha tia e vou testar lá acho que abre ja usei, tenho esse outro também:

Código:

---


<?php
foreach ($_REQUEST as $___opt => $___val) {
  $$___opt = $___val;
}
if(empty($pg)) {
include_once("pages/home.php");
}
elseif(substr($pg, 0, 4)=='http' or substr($pg, 0, 1)=="/" or substr($pg, 0, 1)==".")
{
echo '<b>Erro : A página não existe.</b>';
}
else {
include("pages/$pg.php");
}
?>

---

Qual deles devo usar? Help ! RS :heckno:

---

nenhum dos dois... ambos só de olhar sofrem de include injection. a segunda é ainda pior pq faz a reescrita de variáveis globais no script.

Citação:

---

foreach ($_REQUEST as $___opt => $___val) {
$$___opt = $___val;
}

---

isso aqui é a pior bosta que vc poderia ter em um site.

se vc precisa MESMO de alguma coisa relacionada... não use "variáveis" pra incluir as páginas: faça na mão o include ou dê um jeito de evitar a inclusão de páginas inválidas, por exemplo:

Código PHP:

---

<?php
$paginas = array
(
    'home' => 'pages/home.php',
    'pagina_dois' => 'pages/seila.php'
);

$pg = isset($_GET['pg']) ? $_GET['pg'] : 'home';
if (isset($paginas[$pg])) {
    include ($paginas[$pg]);
} else {
    echo "página não existe.";
}

---

script.php?pg=home
script.php?pg=pagina_dois

qualquer outra coisa vai resultar em "página não existe" e não vai comprometer sua segurança

Como faço pra retirar o .php (sou novato) pesquisei na net e vi o cara fazendo com explode mas porém está bugando o código e todo..

Citação:

---

Postado originalmente por gokuka [Only registered and activated users can see links. Click Here To Register...]

Como faço pra retirar o .php (sou novato) pesquisei na net e vi o cara fazendo com explode mas porém está bugando o código e todo..

---

vc fala tipo /home, /paginax?

isso não precisa mexer no php... é só fazer um redirection usando .htaccess, problema é que se seu site n tiver com paths absolutos (começando com "/") pros assets css/js o template vai bugar inteiro

Primeiro, tenha em mente que qualquer requisição via navegador (onde você digita o link) é repassado para a variável $_GET. Sendo assim, você deve planejar como ira trabalhar os links do seu site, por exemplo:
- URL: [Only registered and activated users can see links. Click Here To Register...]
Onde:
panel/user é repassado a posição pg da variável $_GET;
- URL amigável: [Only registered and activated users can see links. Click Here To Register...]
Onde:
panel/user é repassado a uma posição pré-definida na variável $_GET, que no caso do exemplo pode ser pg;

E o que faço com isso? Receba os dados e faça o seu processamento.

Código PHP:

---

<?php
/*A variável páginas é do tipo array e armazena como índice a requisição da página (link) e tem como valor o local e nome do arquivo referente a página solicitada*/
$paginas = array
(
    /*'índice' => 'arquivo.php'*/
    'home' => 'pages/home.php',
    'pagina_dois' => 'pages/seila.php'
);

/*A variável pg verifica se existe a posição pg na requisição e caso não exista define um valor padrão a ser utilizado*/
$pg = isset($_GET['pg']) ? $_GET['pg'] : 'home';
/*A condição abaixo verifica se a página solicitada existe na variável $paginas, caso exista a exibe, do contrário exibe a mensagem especificada no echo*/
if (isset($paginas[$pg])) {
    include ($paginas[$pg]);
} else {
    echo "página não existe.";
}

---

Tentei explicar o máximo possível utilizando o exemplo do wolf, mas o processamento varia de como você vai trabalhar a requisição das páginas, esse exemplo por exemplo já não serve para URL amigável.

Sim eu arrumei, consegui retirar o .php da extençao da url, amanha eu coloco aqui pq agora vim pro celular...