Verificar con isset
siempre es recomendable, de lo contrario, tendrás un Undefined index
si intentas a acceder a una clave del $_GET
que no existe.
Por otra parte, no puedes intentar usar variables sin definirlas previamente, por lo que puedes definir los valores de dichas variables en el default
.
Pero, así no queda resuelta la parte del else
(porque el código no entraría en el bloque switch
), debido a que tienes un if
, el cual es innecesario si quieres asignar valores por defecto. El if
tendría sentido si quisieras mostrar otra cosa distinta desde el else
o dirigir el código por otro camino en ese caso o mostrar un mensaje de error.
En cualquier caso, yo lo resolvería usando un array, es más simple y más fácil de mantener.
Veamos todos los casos:
1. Con array, mostrando valores por defecto
/*
Esto reemplaza a los case ...
Si hubiera situaciones nuevas, solamente las agregas aquí
*/
$allData=
array(
'1l'=> array(
'number' => '1 lesson',
'duration' => '1 hour',
'price' => 'usd 6'),
'6l' => array(
'number' => '6 lessons',
'duration' => '1 hour each lesson',
'price' => 'usd 34'),
'12l' => array (
'number' => '12 lessons',
'duration' => '1 hour each lesson',
'price' => 'usd 68'),
'default' => array (
'number' => '',
'duration' => '',
'price' => '')
);
/*
Obtienes la key desde el GET
mediante operador de fusion de null
si tu versión de PHP no lo soporta, usa esto en su lugar
$key=!empty($_GET['id']) ? $_GET['id'] : 'default';
*/
$key=$_GET['id'] ?? 'default';
$html=sprintf(
'<div id="cont_pricing">
<div id="txt_pricing">
<p>
%s
<br/>
<br/>
%s
<br/>
<br/>
%s
</p>
</div>
</div>',$allData[$key]['number'],$allData[$key]['duration'],$allData[$key]['price']);
echo $html;
2. Con array y else (no hay valores por defecto)
/*
Esto reemplaza a los case ...
Si hubiera situaciones nuevas, solamente las agregas aquí
*/
$allData=
array(
'1l'=> array(
'number' => '1 lesson',
'duration' => '1 hour',
'price' => 'usd 6'),
'6l' => array(
'number' => '6 lessons',
'duration' => '1 hour each lesson',
'price' => 'usd 34'),
'12l' => array (
'number' => '12 lessons',
'duration' => '1 hour each lesson',
'price' => 'usd 68')
);
/*
Obtienes la key desde el GET
mediante operador de fusion de null
si tu versión de PHP no lo soporta, usa esto en su lugar
$key=!empty($_GET['id']) ? $_GET['id'] : 'default';
*/
$key=$_GET['id'] ?? null;
if (array_key_exists($key, $allData)) {
$html=sprintf(
'<div id="cont_pricing">
<div id="txt_pricing">
<p>
%s
<br/>
<br/>
%s
<br/>
<br/>
%s
</p>
</div>
</div>',$allData[$key]['number'],$allData[$key]['duration'],$allData[$key]['price']);
} else {
$html="Error. La clave $key no es válida";
}
echo $html;
3. Con switch
(y valores por defecto)
/*
Obtienes la key desde el GET
mediante operador de fusion de null
si tu versión de PHP no lo soporta, usa esto en su lugar
$key=!empty($_GET['id']) ? $_GET['id'] : 'default';
*/
$key=$_GET['id'] ?? 'default';
switch ($key) {
case '1l':
$number = '1 lesson';
$duration = '1 hour';
$price = 'usd 6';
break;
case '6l':
$number = '6 lessons';
$duration = '1 hour each lesson';
$price = 'usd 34';
break;
case '12l':
$number = '12 lessons';
$duration = '1 hour each lesson';
$price = 'usd 68';
break;
default:
$number = '';
$duration = '';
$price = '';
break;
}
$html=sprintf(
'<div id="cont_pricing">
<div id="txt_pricing">
<p>
%s
<br/>
<br/>
%s
<br/>
<br/>
%s
</p>
</div>
</div>',$number, $duration, $price);
echo $html;
4. Con switch
(y else y valores por defecto)
/*
Obtienes la key desde el GET
mediante operador de fusion de null
si tu versión de PHP no lo soporta, usa esto en su lugar
$key=!empty($_GET['id']) ? $_GET['id'] : 'default';
*/
$key=$_GET['id'] ?? null;
if ($key) {
switch ($key) {
case '1l':
$number = '1 lesson';
$duration = '1 hour';
$price = 'usd 6';
break;
case '6l':
$number = '6 lessons';
$duration = '1 hour each lesson';
$price = 'usd 34';
break;
case '12l':
$number = '12 lessons';
$duration = '1 hour each lesson';
$price = 'usd 68';
break;
default:
$number = '';
$duration = '';
$price = '';
break;
}
$html=sprintf(
'<div id="cont_pricing">
<div id="txt_pricing">
<p>
%s
<br/>
<br/>
%s
<br/>
<br/>
%s
</p>
</div>
</div>',$number, $duration, $price);
} else {
$html="Error. No se pasaron datos válidos";
}
echo $html;
De cualquiera de estas formas tu código debería funcionar. Lo fundamental es que:
Deberías primero capturar el valor del dato que viene en el $_GET
, con un ternario o de otro modo.
Si usas switch ... case
deberías prever la situación del default
Si ninguna condición se cumple, debes tener claro qué debe hacer el código en ese caso (por ejemplo, en el caso 4
si el GET tiene una clave id
pero que no vale ninguno de los case, el código tomará los valores por defecto, y sólo entrará en el else
cuando el GET no tenga una clave id
).
$_GET['id']
se salta el bloque if y entonces encuentra las variables sin definir...switch ($_GET['id'])