Mejor forma pasar parametros a función

[cocodj69]

Tengo un archivo llamado validacion.php donde valido todos los formularios de entrada de datos del usuario. Allí valido todo tipo de datos: enteros, texto, fecha, email, password, etc. Pero ahora quiero validar tambien los campos select. Es decir, que el valor recibido para un determinado select pertenezca a los posibles valores del mismo select, evitando el ataque des de n formulario externo.

La pregunta es si es práctico utilizar variables globales para los selects. Es decir, crear tantos arrays como selects haya y luego en la función en la que lo tengamos que utilizar, accedemos a el a través de la variable global?

Código PHP:

// validacion.php
 
// Variable Global
$select = array('a','b','c');
 
function validaForm1($_POST){
 
  validaSelect($_POST['valor_select'], $select);
}
function validaForm2($_POST){
 
  validaSelect($_POST['valor_select'], $select);
}
function validaForm3($_POST){
 
  validaSelect($_POST['valor_select'], $select);
}
 
validaSelect($valor_select, $select){
  return in_array($valor_select, $select);
}
 

O seria mejor, crear el array sólo en las funciones que lo necesitemos y no como variable global???

Espero me ayuden porque estoy un poco confundido. Grácias

[maycolalvarez]

$_POST no hace falta que la pases porque es una Superglobal, así que esta disponible en todos los ámbitos del script

para hacer referencia a una variable que está fuera de la función, debes llamarla con global dentro de la misma:

Código PHP:

  $variable="hola";
function funct(){
       global $variable;
}  
   

si la variable $select lo vas a usar en muchas funciones, el uso de global es importante, porque si pasas el array como parámetro, éste será tratado por valor y se copiara un array nuevo para ser pasado a la función, consumiendo más recursos, esto no aplica para los objetos que se pasan por referencia

[cocodj69]

Muchas Grácias.

Pero un apunte, puesto que faltaba código ya que esta dentro de una clase:

Código PHP:

class validate {
    
  var $arr_select1; // Desplegable 1
  var $arr_select2; // Desplegable 2
      
  // Code
  function validar(){
 
     $this->validarSelect($_POST['value], $this->arr_select1);
  }
  
}
 

Así no hace falta declarar la variable como global...

Un fallo que cometo es que paso a las funciones la variable superglobal $_POST y las recogo de la siguiente forma...

Código PHP:

function f1($post){
 
   // Trabajo con el $post y no con $_POST
 }
 

Con lo que estoy haciendo una copia no? Tendré que modificarlo

[eulloa]

Iniciado por cocodj69

Muchas Grácias.

Pero un apunte, puesto que faltaba código ya que esta dentro de una clase:

Código PHP:

class validate {
    
  var $arr_select1; // Desplegable 1
  var $arr_select2; // Desplegable 2
      
  // Code
  function validar(){
 
     $this->validarSelect($_POST['value], $this->arr_select1);
  }
  
}
 

Creo q quieres correr antes de gatear. Q vamos hombre, que tienes dudas en funciones y ya te estás utilizando clases.
¿Q PHP usas: PHP4?
Si quieres empezar más o menos serio en PHP POO, debería usar PHP5

Iniciado por cocodj69

Así no hace falta declarar la variable como global...

Un fallo que cometo es que paso a las funciones la variable superglobal $_POST y las recogo de la siguiente forma...

Código PHP:

function f1($post){
 
   // Trabajo con el $post y no con $_POST
 }
 

Con lo que estoy haciendo una copia no? Tendré que modificarlo

¿Cuando dices que las recoges es el cómo la estás pasando a la función? Si pasas de esta forma lo estás pasando por valor, así q lo q hace exactamente es una copia de la variable. ¿En que te afecta eso?

[cocodj69]

Iniciado por eulloa

Creo q quieres correr antes de gatear. Q vamos hombre, que tienes dudas en funciones y ya te estás utilizando clases.
¿Q PHP usas: PHP4?
Si quieres empezar más o menos serio en PHP POO, debería usar PHP5

Utilizo PHP 5. Ya llevo tiempo programando en POO y está claro que estoy haciendo copias con los valores pasados a la función!! Lo modificaré.

Iniciado por eulloa

¿Cuando dices que las recoges es el cómo la estás pasando a la función? Si pasas de esta forma lo estás pasando por valor, así q lo q hace exactamente es una copia de la variable. ¿En que te afecta eso?

Si, la recogo quiero decir pasar a la función. Evidentemente si hago una copia innecesaria ocupa más recuros y tiempo...

Grácias por la ayuda! Múy útil!!

[eulloa]

Iniciado por cocodj69

Si, la recogo quiero decir pasar a la función. Evidentemente si hago una copia innecesaria ocupa más recuros y tiempo...
Grácias por la ayuda! Múy útil!!

Es cierto, pero esto solo se ve en el caso de grandes proyectos, donde habría q entrar a considerar cada cosa. Si tu proyecto no es muy grande pues no veo la causa de preocuparte tanto por esto....... Todo depende de tus necesidades

[cocodj69]

Yaps! Aunque pienso que es mejor acostumbrarse a codificar de la forma más correcta y nítida que se pueda por pequeño que sea el proyecto!! :)

Muchas Grácias por la ayuda!!

[maycolalvarez]

si esta en una clase, simplemente la variable tiene acceso local en todo el ámbito de la clase:

$this->mivariable

para hacerla visible desde afuera solo basta colocar el modificador public:

public $mivariable; //dentro de class

y así: $instanciaclase->mivariable accederias desde otras clases u ámbitos del script