Skip to main content
Ampliada explicación y agregado operador ternario
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

El problema que tienes es que cuando no existe el parámetro GET CodUsua en la URL intentas acceder a él igualmente. PHP lanza una advertencia cuando tratassiempre que trates de acceder al índice de una matriz que no existe para advertirte que ésto podría llevarte a un error no controlado en tu aplicación.

Para detectar cuándo un índice de una matriz existe o no antes de acceder a su valor debes hacer uso de isset() de la siguiente manera:

<?php if (isset($_GET['CodUsua']) && ($_GET['CodUsua'] != $_SESSION['CodUsua'])): ?>

Donde la evaluación fallará si no existe el índice antes de acceder a él. Esto es así porque en una operación booleana && (Y lógico) se detiene la evaluación del resto de operandos (se evalúan de derecha a izquierda) en cuanto hay uno que vale false.

Otra zona donde no compruebas la validez del parámetro es ésta:

<li><a class="icon-users" class="amigos" href="amigos.php?CodUsua=<?=
  htmlspecialchars($_SESSION['CodUsua']isset($_GET['CodUsua']) ?$_GET['CodUsua']:'')
?>">Ver amigos <?php 
  if(isset($_GET['CodUsua'])) && !empty(amigos::cantidad_amigos($_GET['CodUsua']))) {
    echo amigos::cantidad_amigos($_GET['CodUsua'])[0][0];
  } else {
    echo 0;
  }
?> </a></li>

Entiendo que quieres acceder a $_SESSION['CodUsua'] en vez de a $_GET['CodUsua'] para acceder a los amigos de tu código de usuario.

Además, he agregado htmlspecialchars() para recordarte que toda salida al navegador debería pasar por esta función. El contenido de la variable súper global $_GET no la controlas, por lo que podrían hacerte XSS fácilmente.

He hecho uso del operador ternario ?: para que en caso de que el índice no exista devuelva una cadena vacía, pero queda en tu mano adoptar otra solución en caso de que ésta no exista.

El problema que tienes es que cuando no existe el parámetro GET CodUsua en la URL intentas acceder a él igualmente. PHP lanza una advertencia cuando tratas de acceder al índice de una matriz que no existe.

Para detectar cuándo un índice de una matriz existe o no antes de acceder a su valor debes hacer uso de isset() de la siguiente manera:

<?php if (isset($_GET['CodUsua']) && ($_GET['CodUsua'] != $_SESSION['CodUsua'])): ?>

Donde la evaluación fallará si no existe el índice antes de acceder a él. Esto es así porque en una operación booleana && (Y lógico) se detiene la evaluación del resto de operandos (se evalúan de derecha a izquierda) en cuanto hay uno que vale false.

Otra zona donde no compruebas la validez del parámetro es ésta:

<li><a class="icon-users" class="amigos" href="amigos.php?CodUsua=<?= htmlspecialchars($_SESSION['CodUsua']) ?>">Ver amigos <?php 
if(isset($_GET['CodUsua'])) && !empty(amigos::cantidad_amigos($_GET['CodUsua']))) {
  echo amigos::cantidad_amigos($_GET['CodUsua'])[0][0];
} else {
  echo 0;
}
?> </a></li>

El problema que tienes es que cuando no existe el parámetro GET CodUsua en la URL intentas acceder a él igualmente. PHP lanza una advertencia siempre que trates de acceder al índice de una matriz que no existe para advertirte que ésto podría llevarte a un error no controlado en tu aplicación.

Para detectar cuándo un índice de una matriz existe o no antes de acceder a su valor debes hacer uso de isset() de la siguiente manera:

<?php if (isset($_GET['CodUsua']) && ($_GET['CodUsua'] != $_SESSION['CodUsua'])): ?>

Donde la evaluación fallará si no existe el índice antes de acceder a él. Esto es así porque en una operación booleana && (Y lógico) se detiene la evaluación del resto de operandos (se evalúan de derecha a izquierda) en cuanto hay uno que vale false.

Otra zona donde no compruebas la validez del parámetro es ésta:

<li><a class="icon-users" class="amigos" href="amigos.php?CodUsua=<?=
  htmlspecialchars(isset($_GET['CodUsua'])?$_GET['CodUsua']:'')
?>">Ver amigos <?php 
  if(isset($_GET['CodUsua'])) && !empty(amigos::cantidad_amigos($_GET['CodUsua']))) {
    echo amigos::cantidad_amigos($_GET['CodUsua'])[0][0];
  } else {
    echo 0;
  }
?> </a></li>

Entiendo que quieres acceder a $_SESSION['CodUsua'] en vez de a $_GET['CodUsua'] para acceder a los amigos de tu código de usuario.

Además, he agregado htmlspecialchars() para recordarte que toda salida al navegador debería pasar por esta función. El contenido de la variable súper global $_GET no la controlas, por lo que podrían hacerte XSS fácilmente.

He hecho uso del operador ternario ?: para que en caso de que el índice no exista devuelva una cadena vacía, pero queda en tu mano adoptar otra solución en caso de que ésta no exista.

Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

El problema que tienes es que cuando no existe el parámetro GET CodUsua en la URL intentas acceder a él igualmente. PHP lanza una advertencia cuando tratas de acceder al índice de una matriz que no existe.

Para detectar cuándo un índice de una matriz existe o no antes de acceder a su valor debes hacer uso de isset() de la siguiente manera:

<?php if (isset($_GET['CodUsua']) && ($_GET['CodUsua'] != $_SESSION['CodUsua'])): ?>

Donde la evaluación fallará si no existe el índice antes de acceder a él. Esto es así porque en una operación booleana && (Y lógico) se detiene la evaluación del resto de operandos (se evalúan de derecha a izquierda) en cuanto hay uno que vale false.

Otra zona donde no compruebas la validez del parámetro es ésta:

<li><a class="icon-users" class="amigos" href="amigos.php?CodUsua=<?= htmlspecialchars($_SESSION['CodUsua']) ?>">Ver amigos <?php 
if(isset($_GET['CodUsua'])) && !empty(amigos::cantidad_amigos($_GET['CodUsua']))) {
  echo amigos::cantidad_amigos($_GET['CodUsua'])[0][0];
} else {
  echo 0;
}
?> </a></li>