variables globales dentro de funciones..

[chsc]

que es lo que está "mejor visto" ??

1) Pasarle variables globales a una función enviándola como parámetro.
2) Usarla dentro de la función, avisando con global $var, evitando así el uso de los parámetros

??

gracias

[pateketrueke]

1) Buena practica, mis respetos
2) Pésima practica, ni lo pienses

Lee lo siguiente: http://phptherightway.com/

No se trata de como "vemos" las cosas, sino de hacer bien tu trabajo.

[carlos_belisario]

3 evitar en lo posible el uso de variables globales =).

Normalmente si una función necesita un valor externo lo paso como parámetro

[chsc]

genial graciasss

[pateketrueke]

Iniciado por carlos_belisario

3 evitar en lo posible el uso de variables globales =).

Normalmente si una función necesita un valor externo lo paso como parámetro

Tienes toda la razón, no alcancé a leer bien el primer punto.

Definitivamente es mala idea usar variables globales en todos los contextos posibles, es decir, que se debe evitar su uso a desmedida.

1) Buena practica, mis respetos
2) Pésima practica, ni lo pienses

[metacortex]

Una pregunta: tengo esta función

Código PHP:

function guardar() {
    global $post, $datos_editor, $datos_editor_listas;
    ...
 

¿Debo hacerlo así?

Código PHP:

function guardar($post , $datos_editor , $datos_editor_listas) {
    ...
 

[pateketrueke]

Sí, además cuando invoques la función también deberás pasar los argumentos en dicho orden.

[Italico76]

@metacortex: hola... lo que dices esta bien pero ten en cuenta que las variables globales convierten a la funcion en un procedimiento o sea que... mas bien tu funcion si la traduces "tal cual" seria asi:

Código PHP:

function guardar(&$post , &$datos_editor ,&$datos_editor_listas) {
    ...
 

Y eso respetando el orden de los parámetros...... [ ahi me gano Pate je ] .. lo que te digo es que esos parametros deben ser input-output para que puedas plantear equivalencia excepto que los "output" los saques con return ... aunque en una funcion que solo "guarda" datos seria un boolean indicando exito / fracaso:

Código PHP:

function guardar($post ,$datos_editor ,$datos_editor_listas) {
    ...
return $ok;
 

[metacortex]

Gracias por la explicación amigos. Esta noticia está a punto de descomponerme el estómago porque casi estoy que termino un trabajo largo cuyos archivos están plagados de variables globales. Otra pregunta consecuente y por favor disculpen la joda: ¿Es muy crítica esa vulnerabilidad, así como para tomar acciones inmediatas? Es decir ¿Sería válido o no dejar el asunto así por unos meses, al menos hasta que reuna X conjunto de actualizaciones de la aplicación?.

[Italico76]

En mi opinion hay cuestiones mas criticas pero esto te puede joder en el sentido de que podria haber colisiones entre distintas librerias / archivos

Si te llegara a pasar... que empiezan a colisionar tus "paquetes", podrias usar Namespace que pues restringiria un poco el alcance de las variables globales. Ya no serian "globales"

Ejemplo: supongamos tienes una libreria... llamada hello_lib.php y tiene variables globales pero tu quieres restringir el impacto a los archivos que sabes la usan porque casualmente tu tienes una funcion que se llama igual en otra libreria u archivo, digamos es la funcion saludar()

// Archivo # 1 que incluye la funcion saludar()

Código PHP:

<?php
Namespace Cortesia;  // Incluyes esta linea
 
function saludar(){
    global $saludo;
    $saludo = 'Hola Mundo';
}
 

Y... preciso tienes otro archivo... que tiene la misma funcion... digamos que este segundo archivo INCLUYE al primero, pero ambos podrian ser incluidos por un tercero... etc

// Archivo # 2 que estamos de malas e incluye la misma funcion saludar() ... observa que fue necesario definir varios "espacios" de trabajo para evitar las colisiones

Código PHP:

<?php 
Namespace Cortesia {
    Include "hello_lib.php";
}
 
// Espacio de trabajo "Global" ... es como decir que no pertenece a nigun Namespace
Namespace {
    // Otra funcion que se llama saludar y colisionaria
    function saludar($nombre){    
        return "Hola $nombre";
    }
}
 
Namespace Cortesia {
    saludar();  
    echo $saludo;
}
 

Veamos:

1) Las funciones (podrian ser clases) se pueden llamar igual.. pero la implementacion distinta o no tener nada que ver pero tener el mismo nombre

2) Tu podrias querer "renombrar" la funcion y tener en cuenta todas sus dependientes (donde la llamas) pero... si la o las librerias son "comerciales" ? renombrando estaras en mas problemas porque tu codigo se volvera mucho mas incompatible al momento de actualizarlas.

3) Podras decir...y por que colocas tantos espacios de bloque Namespace ? sera que puedo quitarle el Namespace al include ? RTA: nope. Ese no porque los archivos que tengan Namespace deben comenzar como primera clausula con un Namespace justamente!

Curiosidad... el Namespace "global" equivale a no tenerlo... excepto que aplica la regla de arriba... entonces el codigo que teniamos antes equivale a:

Código PHP:

<?php 
Namespace Cortesia {
    Include "hello_lib.php";
}
 
// ACA estaba el NAMESPACE GLOBAL  (pero lo quite)
 
    // Otra funcion que se llama saludar y colisionaria
    function saludar($nombre){    
        return "Hola $nombre";
    }
 
 
Namespace Cortesia {
    saludar();  
    echo $saludo;
}
 

En fin.... no soy un experto... a penas las estoy usando... ojala te solucionen algo

Mas

[metacortex]

Qué interesante. Ni idea que existía. También creo que parecen clases, pues encapsulan las funciones. Gracias por compartir la información.

Por otra parte te comento que estoy resolviendo el asunto de los parámetros con un pequeño método que compartí en el foro hace tiempo, el cual permite colocarlos en cualquier orden -y hasta de manera más amigable- al invocar la función: http://www.forosdelweb.com/f18/aport...dpress-906484/

Saludos.

[Italico76]

Iniciado por metacortex

... estoy resolviendo el asunto de los parámetros con un pequeño método que compartí en el foro hace tiempo, el cual permite colocarlos en cualquier orden -y hasta de manera más amigable- al invocar la función: http://www.forosdelweb.com/f18/aport...dpress-906484/

Me parece bien, esa solucion que encontrastes para vos tiene una Gran ventaja y es que puedes hacer un Search&Replace en algun editor que soporte expresiones regulares como el Notepad++ para las llamadas a esas funciones y solucionar facil tu problema en 5 min a nivel de todo tu codigo. EXCELENTE SOLUCION !!!

Otra forma (la mas usual) de no liarse con el orden de parametros es enviando un array asociativo de los mismos

Supon tenias esto:

Código PHP:

function hablar()
{    
    global $sujeto, $accion, $articulo, $objeto;
 
    $articulo = isset($articulo) ? $articulo : null;
 
        return "$sujeto $accion $articulo $objeto";
}
 
$sujeto = 'I';
$accion = 'will eat';
$articulo = 'a';
$objeto = 'cookie';
 
echo hablar();
 

Fijate lo facil que la adapto:

Código PHP:

<?php
 
function hablar ($p)
{    
    if  (!is_array($p)  || (count($p)<3) || (count($p)>4) )
        Throw new InvalidArgumentException("Parametros estan mal en ".__FUNCTION__);
 
    extract($p);  // convierto array en algo retro-compatible
 
    // DE AQUI para abajo tu funcion no cambia en nada ***
    $articulo = isset($articulo) ? $articulo : null;
 
        return "$sujeto $accion $articulo $objeto";
}
 
echo hablar (array('articulo'=>'a','objeto'=>'cookie','sujeto'=>'I','accion'=>'will eat'));
 

Al mini-chequeo no le des importancia.... solo verifico que sea un array() y que tenga entre 3 y 4 "parametros" puesto que uno de ellos es opcional (articulo)

*** Tu funcion puede tener como invariante "todo" excepto las primeras lineas o primera linea,.. el extract() aunque si tu necesitas devolver "algo" haras un return