duda con switch

[tokiodata]

hola buen dia tengo una duda con el switch

<?php
$id=$_GET['id'];
switch ($id) {
case 1:
include("1.html");
break;
case 2:
include("2.html");
break;
default:
include("inicio.html");
}
?>

si estoy en el index me muestra la pag de inicio.html, pero si por algun motivo pongo por decir index.php?id=455, me muestra = la paguina de inicio.html por tenerlo como default.
mi pregunta es como poder evitar esto?
que al no existir el parametro me mande a una web de error y no ala de inicio

[SurKaiser]

facil
$id = $_GET['id'];
if(empty($id))
{
echo "Error pagina no encontrada";
}
else {
if($id == 1)
{
echo "Hola estas en la pagina 1";
}
}
Saludos!

[andresdzphp]

Usa la función file_exists para averiguar si existe el archivo o no

http://docs.php.net/manual/es/function.file-exists.php

Saludos.

[tokiodata]

no pero yo me refiero al parametro.

aun que una buena opcion es usar el if, pero en cuanto al switch como queda ? tendria que meter un if ?

[SurKaiser]

A con el switch seria asi pero es mejor con if en este caso xD

$id = $_GET['id'];

switch($id)
{
case '1' :
echo "Hola estas en la pagina 1";
break;
default :
echo "Error pagina no encontrada";
break;
}
Saludos!

[malakian]

Hola, que tal esto, puede ser una opcion

Código PHP:

<?php
if (isset($_GET['id'])){
$id=$_GET['id'];
}else{
$id=0;
}
switch ($id) {
case 0:
include("inicio.html");
break;
case 1:
include("1.html");
break;
case 2:
include("2.html");
break;
default:
include("error.html");
}
?>
 
 
No lo he probado, pero lo puedes adaptar... :-)
 

[GatorV]

El problema es que tu etiqueta default va a cachar cuando no exista el parámetro o no este dentro del rango que quieres, es por eso que debes de usar un if() para verificar si el número esta fuera del rango de paginas que tienes y ya en base a eso mostrar el error.

Saludos.

[SurKaiser]

Basicamente ya tienes la solucion amigo saludos!

[andresdzphp]

Siguiendo la idea de malakian lo puedes dejar así:

Código PHP:

<?php
$id = isset($_GET['id']) ? $_GET['id'] : '';
switch ($id) {
        case '':
            include "inicio.html";
            break;
        case 1:
            include "1.html";
            break;
        case 2:
            include "2.html";
            break;
        default:
            echo 'Error';
}
 

[tokiodata]

y una pregunta, por seguriad que es mejor usar switch o if ? o es igual ?

[malakian]

Bueno pareciera que cada vez quieren reducir mas el codigo jajaja, bueno ahi va el mio :)

Código PHP:

<?php
  $id = isset($_GET['id']) ? $_GET['id'] : 0;
  if (!@include(($id==0)?"inicio.html":$id.".html")){
    echo "error!!";
  }
?>
 

respondiendo tu pregunta, pienso que es igual, tiene mas que ver con la logica del programa

[tokiodata]

mas que nada solo quiero incluir las demas hojas y ya, ha y poner el error por si no existe el parametro

[malakian]

Iniciado por tokiodata

mas que nada solo quiero incluir las demas hojas y ya, ha y poner el error por si no existe el parametro

Si, el ultimo codigo que coloque funciona para eso, pero solo con la condicion que los demas archivos se llamen 1.html, 2.html, 3.html etc etc

[tokiodata]

en mi codigo la forma de incluir es la correcta ?
tambien mas que nada quiero filtrar bien para evitar rfi

[andresdzphp]

También puedes comparar si el archivo con ese id existe o no:

Código PHP:

<?php
$id = isset($_GET['id']) ? $_GET['id'] : '';
 
if (file_exists("$id.html")) {
    include "$id.html";
} else if ($id === '') {
    include "inicio.html";
} else {
    echo 'Error';
}