crear fucnion a partir de un codigo

[saraivaruas]

Hola A todos tengo el siguiente codigo y necesito crear una funcion que me devuelva el valor de transporte por region. como lo quiero llamar la funcion en otras paginas como lo puedo hacer?

micodigo es el siguiente:

Código PHP:

$region = strip_tags ($_POST['region']);
$lugar = "$region";
if($lugar = $region) {
if (($region) <= 4){
$ValorRegion = 100;
}
}
if(($region) == 5){
$ValorRegion = 40;
}
if($lugar = $region) {
if(($region) >= 6){
$ValorRegion = 130;
}
}
if(($region) == 9) {
$ValorRegion = 150;
}
if(($region) >= 10) {
$ValorRegion = 170;
}
if(($region) == 13) {
$ValorRegion = 0;
}
 

pense en poner el siguiente:

Código PHP:

function transporte($region){
 
mi codigo;
 
return $transporte;
}
 

seria correcto?

[alvaro_trewhela]

No entiendo esto:

$region = strip_tags ($_POST['region']);
$lugar = "$region";
if($lugar = $region) {
}


Para que el strip? si $region es un entero?
Para que $lugar = $region //que es lo mismo que $lugar = "$region" (si es un string)

Cuando haces esto if($lugar = $region) estas asignando y no comparando:
if($lugar == $region)

Pero no entiendo, $lugar siempre es igual a $region....

Que pasaria si pongo en el parametro otra region que no esta en los ifs?

Yo haría algo como esto:

Código PHP:

function transporte($region,$default){ 
$valores = array(100,100,100,100,900,1100,1300,1500,1700,1900,2100,2300,2700); //arreglo de valores por regiones...
 
if($region == null || $region < 0 || $region >= sizeof($valores)){
$valor = $default;
}
else{
$valor = $valores[$region-1];
}
 
$transporte = ; //aca tu formula, ya tienes $valor =)
 
return $transporte;
}
 

Entonces lo que hice fue, pasar por parametro dos cosas, la region y un valor default, explicacion:

Ya que como tu deseas una funcion con parametro region, y en tus ifs solo atiendes solo unas opciones, supongo que debe haber un valor por defecto. y ese sería el parametro default, ahora para las regiones que no etiendes, y no quieres atenderlas, simplemente en el parametro region, pone null

Saludos