sirve - Compruebe si existen var antes de desarmar en PHP?

php tutorial

Question

Con el informe de errores, o incluso las mejores prácticas, al desarmar una variable en PHP, ¿debería verificar si existe primero (en este caso, no siempre existe) y desarmarla, o simplemente desarmarla?

<?PHP
if (isset($_SESSION[''signup_errors''])){
    unset($_SESSION[''signup_errors'']);
}

// OR

unset($_SESSION[''signup_errors'']);
?>

Answer 1

El uso de unset en una variable indefinida no causará ningún error (a menos que la variable sea el índice de una matriz (u objeto) que no existe).

Por lo tanto, lo único que debe considerar es qué es lo más eficiente. Es más eficiente no probar con ''isset'', como lo mostrará mi prueba.

Prueba:

function A()
{
    for ($i = 0; $i < 10000000; $i++)
    {
        $defined = 1;
        unset($defined);
    }
}

function B()
{
    for ($i = 0; $i < 10000000; $i++)
    {
        $defined = 1;
        unset($undefined);
    }
}

function C()
{
    for ($i = 0; $i < 10000000; $i++)
    {
        $defined = 1;
        if (isset($defined))
            unset($defined);
    }
}

function D()
{
    for ($i = 0; $i < 10000000; $i++)
    {
        $defined = 1;
        if (isset($undefined))
            unset($undefined);
    }
}

$time_pre = microtime(true);
A();
$time_post = microtime(true);
$exec_time = $time_post - $time_pre;
echo "Function A time = $exec_time ";

$time_pre = microtime(true);
B();
$time_post = microtime(true);
$exec_time = $time_post - $time_pre;
echo "Function B time = $exec_time ";

$time_pre = microtime(true);
C();
$time_post = microtime(true);
$exec_time = $time_post - $time_pre;
echo "Function C time = $exec_time ";

$time_pre = microtime(true);
D();
$time_post = microtime(true);
$exec_time = $time_post - $time_pre;
echo "Function D time = $exec_time";
exit();

Resultados:

1. Function A time = 1.0307259559631
   • Definido sin isset
2. Function B time = 0.72514510154724
   • Indefinido sin isset
3. Function C time = 1.3804969787598
   • Definido usando isset
4. Function D time = 0.86475610733032
   • Indefinido usando isset

Conclusión:

Siempre es menos eficiente usar isset , sin mencionar la pequeña cantidad de tiempo extra que lleva escribir. Es más rápido intentar unset una variable indefinida que verificar si se puede unset .

Answer 2

Si quieres desarmar una variable, entonces puedes usar unset

unset($any_variable); // bool, object, int, string etc

Verificar su existencia no tiene ningún beneficio cuando se trata de desarmar una variable.

Si la variable es una matriz y desea desarmar un elemento, debe asegurarse de que el padre exista primero, esto también se aplica a las propiedades del objeto.

unset($undefined_array[''undefined_element_key'']); // error - Undefined variable: undefined_array

unset($undefined_object->undefined_prop_name); // error - Undefined variable: undefined_object

Esto se resuelve fácilmente envolviendo el unset en un if(isset($var)){ ... } .

if(isset($undefined_array)){
    unset($undefined_array[''undefined_element_key'']); 
}

if(isset($undefined_object)){
    unset($undefined_object->undefined_prop_name); 
}

La razón por la que solo verificamos la variable ( principal ) es simplemente porque no necesitamos verificar la propiedad / elemento y hacerlo sería mucho más lento de escribir y computar, ya que agregaría una verificación adicional.

if(isset($array)){
...
}

if(isset($object)){
...
}

.vs

$object->prop_name = null;
$array[''element_key''] = null;

// This way elements/properties with the value of `null` can still be unset.

if(isset($array) && array_key_exists(''element_key'', $array)){
...
}

if(isset($object) && property_exists($object, ''prop_name'')){
...
}

// or 

// This way elements/properties with `null` values wont be unset.

if(isset($array) && $array[''element_key''])){
...
}

if(isset($object) && $object->prop_name)){
...
}

Esto es evidente, pero también es crucial que conozca el type de variable al obtener, configurar y deshacer un elemento o propiedad; usar la sintaxis incorrecta arrojará un error.

Es lo mismo cuando intentas deshacer el valor de una matriz u objeto multidimensional. Debe asegurarse de que exista la clave / nombre principal.

if(isset($variable[''undefined_key''])){
    unset($variable[''undefined_key''][''another_undefined_key'']);
}

if(isset($variable->undefined_prop)){
    unset($variable->undefined_prop->another_undefined_prop);
}

Cuando se trata de objetos hay otra cosa en que pensar, y esa es la visibilidad.

El hecho de que exista no significa que tenga permiso para modificarlo.

Answer 3

Simplemente desarmalo, si no existe, no se hará nada.

Answer 4

Del manual de PHP :

Con respecto a cierta confusión anterior en estas notas acerca de qué causa unset () para desencadenar avisos al desarmar variables que no existen ....

Desarreglando variables que no existen, como en

<?php
unset($undefinedVariable);
?>

no activa un aviso de "variable no definida". Pero

<?php
unset($undefinedArray[$undefinedKey]);
?>

desencadena dos avisos, porque este código es para desarmar un elemento de una matriz; ni $ undefinedArray ni $ undefinedKey están siendo desarmados, simplemente están siendo utilizados para localizar lo que debería estar desarmado. Después de todo, si existieran, igual esperarías que ambos estuvieran presentes después. ¡NO querrías que tu matriz completa desapareciera solo porque desarmaste () uno de sus elementos!