consejo con uso de for en funcion

[charlyta]

Hola a todos. Estoy trabajando con una función que limpia las variables post

Código PHP:

  function recoge($var) 
{
    $tmp = (isset($_REQUEST[$var])) ?     strip_tags(trim(htmlspecialchars($_REQUEST[$var]))) : '';
    if (get_magic_quotes_gpc()) {
        $tmp = stripslashes($tmp);
    $tmp = mysql_real_escape_string($tmp);
        $tmp = addslashes($tmp);
        $tmp = strip_tags($tmp);
           }
    return $tmp;
}
$nombre = recoge('variable');
echo $nombre;  
   

Desearía no realizar reiteradamente la llamada a la función para cada variable, por lo que había pensado en utilizar un for: primero declarando los elementos recibidos:

Código PHP:

  $numero = count($_REQUEST);  
   

Y luego realizar el for:

Código PHP:

  for($i=0;$i<$numero;$i++){
//y ahora algo así para llamar a la función
$nombre = recoge('variable');

}  
   

Qué debería poner en recoge('variable') para luego hacer echos? Se os ocurre algo? Gracias

[iukaeru]

Si las variables las estás recogiendo vía POST (por ejemplo), tendrías que cambiar a:

$nombre = recoge($_POST["mi_variable"]);

Pero veo que lo que quieres es tomar todas las variables de un formulario por lo que tendrías que meterlas en un array y allí te queda la tarea para que investigues.

foreach($_POST as $nombre_campo => $valor){
$asignacion = "\$" . $nombre_campo . "='" . $valor . "';";
eval($asignacion);
}

JAJAJAJA

[Nemutagk]

Iniciado por iukaeru

Si las variables las estás recogiendo vía POST (por ejemplo), tendrías que cambiar a:

$nombre = recoge($_POST["mi_variable"]);

Pero veo que lo que quieres es tomar todas las variables de un formulario por lo que tendrías que meterlas en un array y allí te queda la tarea para que investigues.

Código PHP:

  foreach($_POST as $nombre_campo => $valor){ 
   $asignacion = "\$" . $nombre_campo . "='" . $valor . "';"; 
   eval($asignacion); 
}  
   

JAJAJAJA

FAIL, eval JAMAS debe ser utilizado con variables provenientes del navegador/cliente...

Con respecto al problema, lo mejor es que programes una función recursiva....

Código PHP:

function recoge($var) 
{
        if (!is_array($var)) {
                $tmp = (isset($_REQUEST[$var])) ? strip_tags(trim(htmlspecialchars($_REQUEST[$var]))) : '';
                if (get_magic_quotes_gpc()) {
                        $tmp = stripslashes($tmp);
                        $tmp = mysql_real_escape_string($tmp);
                        $tmp = addslashes($tmp);
                        $tmp = strip_tags($tmp);
                }
        }else {
                $tmp = array();
                foreach($var as $val) {
                        $tmp[] = recoger($val);
                }
        }
 
    return $tmp;
}
 
echo '<pre>';
var_dump(recoger($_POST));
echo '</pre>';
 

[charlyta]

Hola! y gracias por responder!

pues el siguiente código no me parece el más adecuado:

Código PHP:

  function recoge($var)  
{ 
        if (!is_array($var)) { 
                $tmp = (isset($_REQUEST[$var])) ? strip_tags(trim(htmlspecialchars($_REQUEST[$var]))) : ''; 
                if (get_magic_quotes_gpc()) { 
                        $tmp = stripslashes($tmp); 
                        $tmp = mysql_real_escape_string($tmp); 
                        $tmp = addslashes($tmp); 
                        $tmp = strip_tags($tmp); 
                } 
        }else { 
                $tmp = array(); 
                foreach($var as $val) { 
                        $tmp[] = recoger($val); 
                } 
        } 
  
    return $tmp; 
} 
  
echo '<pre>'; 
var_dump(recoger($_POST)); 
echo '</pre>';  
   

mi intención era realizar llamar a la función para limpiar y utilizar las variables con un simple:

Código PHP:

  recoge ($_REQUEST[var]); 
echo $nombre; 
echo $apellido;  
   

algo así... es posible??? tu code me daba fallo:

Código:

Undefined variable: tmp in

[GatorV]

Tendrías que hacer algo que no es muy bueno (jugar con globales) pero pues se puede hacer, por ejemplo:

Código PHP:

function recoge($globalVar) {
       // limpias las variables, idealmente usando array_walk
       foreach ($globalVar as $var => $value) {
                global $$var;
                $$var = $value;
       }
}
 
recoge($_POST);
echo $nombre;
 

Saludos.

[charlyta]

Entonces es preferible hacer esto?

Código PHP:

  function recoge() { 
foreach ($_REQUEST as $key => $value) 
{ 
 
$value = mysql_real_escape_string( $value ); 
$value = addslashes($value); 
$value = strip_tags($value); 
if($value != ""){} 
//echo $key. ' - '.$value.'</br>'; 
 
}} 
recoge(); 
echo $_POST['post-title'];  
   

[charlyta]

Pues muchas gracias por la aclaración del concepto!!! Finalmente seré un clásico y seguiré utilizando este método:

Código PHP:

  function clean($str) 
    { 
    $str = @trim($str); 
    if(get_magic_quotes_gpc()) 
    { 
    $str = stripslashes($str); 
    } 
    return mysql_real_escape_string($str); // send back clean 
    } 
function recoger() { 
     
foreach ($_POST as $key => $value) 
{ 
 
$key = clean($key); 
$value = clean($value); 
} 
 
} 
recoger(); 
echo $_POST['post-title'];