3

Tengo este error en PHP 7.4. He intentado varias cosas pero no me funciona.

El error es éste:

Notice: Trying to access array offset on value of type bool in C:\xampp7.47.4\htdocs\proyecto\vistas\plantilla.php on line 65

El error está en esta línea:

if($validarCorreo ["email_encriptado"] == $_GET["ruta"]){

Y esto es lo que tengo en el código en esa línea:

  $item = "email_encriptado";
  $valor = $_GET["ruta"];

  $validarCorreo = ControladorUsuarios::ctrMostrarUsuarios($item, $valor);

  if($validarCorreo ["email_encriptado"] == $_GET["ruta"]){


    $id = $validarCorreo["id_usuario"];
    $item = "verificacion";
    $valor = 1;

    $respuesta = ControladorUsuarios::ctrActualizarUsuario($id, $item, $valor);

    if($respuesta == "ok"){

      echo'<script>

          swal.fire({
              icon:"success",
                title: "¡CORRECTO!",
                text: "¡Su cuenta ha sido verificada, ya puede ingresar al sistema!",
                showConfirmButton: true,
              confirmButtonText: "Cerrar"
              
          }).then(function(result){

              if(result.value){   
                  window.location = "'.$ruta.'ingreso"
                } 
          });

        </script>';

      return;

    }
  
  }

Ésta es la consulta en el modelo:

    /*=============================================
    Actualizar usuario
    =============================================*/

    static public function mdlActualizarUsuario($tabla, $id, $item, $valor){

        $stmt = Conexion::conectar()->prepare("UPDATE $tabla SET $item = :$item WHERE id_usuario = :id_usuario");

        $stmt -> bindParam(":".$item, $valor, PDO::PARAM_STR);
        $stmt -> bindParam(":id_usuario", $id, PDO::PARAM_INT);

        if($stmt -> execute()){

            return "ok";

        }else{

            return print_r(Conexion::conectar()->errorInfo());

        }

        $stmt-> close();

        $stmt = null;
        
    }

Aquí el controlador:

  /*=============================================
  ACTUALIZAR USUARIO
  =============================================*/
  static public function ctrActualizarUsuario($id, $item, $valor){

    $tabla = "registrousuarios";

    $respuesta = ModeloUsuarios::mdlActualizarUsuario($tabla, $id, $item, $valor);

    return $respuesta;

Y el mostrar usuarios del controlador:

  /*=============================================
  Mostrar Usuarios
  =============================================*/

  static public function ctrMostrarUsuarios($item, $valor){
  
    $tabla = "registrousuarios";

    $respuesta = ModeloUsuarios::mdlMostrarUsuarios($tabla, $item, $valor);

    return $respuesta;

  }

Modelo de mostrar usuarios:

/*=============================================
    Mostrar Usuarios
    =============================================*/

    static public function mdlMostrarUsuarios($tabla, $item, $valor){

        if($item != null && $valor != null){

            $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla WHERE $item = :$item");

            $stmt->bindParam(":".$item, $valor, PDO::PARAM_STR);

            $stmt -> execute();

            return $stmt -> fetch();

        }else{

            $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");

            $stmt -> execute();

            return $stmt -> fetchAll();

        }

        $stmt-> close();

        $stmt = null;

    }
15
  • 1
    Este código parece estar correcto ¿Cuál es tu otro código?
    – Rodrigo
    Commented el 20 ene. 2021 a las 20:23
  • 1
    ¿Y cuál es la línea del problema?
    – user128299
    Commented el 20 ene. 2021 a las 20:25
  • 1
    if($validarCorreo ["email_encriptado"] == $_GET["ruta"]){ Commented el 20 ene. 2021 a las 20:27
  • 1
    Imagino que $validarCorreo es el resultado de una consulta SQL realizada en ControladorUsuarios::ctrMostrarUsuarios(), ¿verdad?. No has comprobado el resultado de dicha consulta, es muy probable que algo esté mal en ella. Commented el 20 ene. 2021 a las 20:30
  • 1
    sii pero en la version de php 7.0 y 7.2 no me sale ese error solo en la 7.3 y la 7.4 Commented el 20 ene. 2021 a las 20:31

2 respuestas 2

3

Tu problema se origina en la siguiente línea:

$validarCorreo = ControladorUsuarios::ctrMostrarUsuarios($item, $valor);

La llamada está devolviendo true o false (un valor booleano), de modo que no es posible acceder a un elemento de un valor booleano como si se tratara de una matriz justo en la línea siguiente:

if ($validarCorreo["email_encriptado"] == $_GET["ruta"]) {

Si la llamada a ControladorUsuarios::ctrMostrarUsuarios() realiza una consulta SQL es muy probable que no estés comprobando su resultado de manera adecuada.


Edito (tras agregar el código necesario):

Estás haciendo varias cosas mal en tu código, entre ellas no comprobar el resultado de la consulta SQL.

static public function mdlMostrarUsuarios($tabla, $item, $valor)
{
    /* Solo compruebas si valen null, pero podrían estar vacíos */
    if (empty($item) === false && empty($valor) === false) {
        $stmt = Conexion::conectar()->prepare(
            "SELECT * FROM $tabla WHERE $item = :$item"
        );
        $stmt->bindParam(":". $item, $valor, PDO::PARAM_STR);
        if ($stmt->execute() === false) {
            /* Si la consulta falló, habrá que hacer algo */
            die(
                htmlspecialchars(implode(", ", $stmt->errorInfo()))
            );
        }
        return $stmt->fetch();
    } else {
        $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");
        $stmt->execute();
        if ($stmt->execute() === false) {
            /* Si la consulta falló, habrá que hacer algo */
            die(
                htmlspecialchars(implode(", ", $stmt->errorInfo()))
            );
        }
        return $stmt->fetchAll();
    }
    /* Este código de aquí abajo es completamente inútil, nunca se ejecutará */
    $stmt-> close();
    $stmt = null;
}

Además, estás devolviendo tres tipos de datos en tu función (deberías evitar este tipo de prácticas):

  • Si $item y $valor NO están vacíos (antes comprobabas si eran nulos) devuelves:
    • Un único registro, una matriz unidimensional con los campos del único registro.
    • false en caso de que no hayas encontrado ninguno.
  • Si están vacíos entonces devuelves:
    • Todos los registros de la tabla, una matriz multidimensional con un elemento por registro y cada registro con sus campos de la tabla.

De modo que también podrías obtener un false en caso de que no hayas obtenido ningún registro como resultado, por lo que tu if debería contemplarlo:

if ($validarCorreo === false) {
    /* Mensaje advirtiendo que no se encontró el registro */
} else if ($validarCorreo ["email_encriptado"] == $_GET["ruta"]){
    /* Tu código habitual */
} else {
    /* ¡¡Mensaje o acción cuando no coinciden!! */
}

Última edición:

Tras haber visto varias veces el código y haber entendido el motivo por el que haces estas líneas:

<?php
$item = "email_encriptado";
$valor = $_GET["ruta"];

$validarCorreo = ControladorUsuarios::ctrMostrarUsuarios($item, $valor);

if($validarCorreo ["email_encriptado"] == $_GET["ruta"]){

Veo que se puede simplificar enormemente de esta manera:

<?php
/* No es necesario hacer uso de variables intermedias */
$validarCorreo = ControladorUsuarios::ctrMostrarUsuarios(
    "email_encriptado",
    $_GET["ruta"]
);
if ($validarCorreo === false) {
    /* No se encontró el usuario */
} else {
    /* Se encontró el usuario y no es necesario comprobar que el campo
    "email_encriptado" del registro vale lo mismo que $_GET["ruta"] porque
    precisamente esa es la condición de búsqueda "WHERE" */
}
4
  • 1
    /*============================================= Actualizar usuario =============================================*/ static public function mdlActualizarUsuario($tabla, $id, $item, $valor){ $stmt = Conexion::conectar()->prepare("UPDATE $tabla SET $item = :$item WHERE id_usuario = :id_usuario"); $stmt -> bindParam(":".$item, $valor, PDO::PARAM_STR); $stmt -> bindParam(":id_usuario", $id, PDO::PARAM_INT); if($stmt -> execute()){ return "ok"; }else{ return print_r(Conexion::conectar()->errorInfo()); } $stmt-> close(); $stmt = null; ] Commented el 20 ene. 2021 a las 20:35
  • 1
    YA LA AGREGUE A LA PREGUNTA Commented el 20 ene. 2021 a las 20:36
  • 1
    He editado la respuesta con tu código corregido (también he aplicado PSR-2, deberías acostumbrarte a usarlo siempre). Mi respuesta inicial sigue siendo válida, el problema es que falla la consulta SQL, así que con este código, a parte de explicarte que produce el error, te ayudará a encontrar el error. También te he indicado que $stmt-> close(); y $stmt = null; nunca se ejecutarán, porque los return devuelven el control a la función llamante mucho antes. Commented el 20 ene. 2021 a las 21:08
  • 1
    He hecho una última edición ahora que he tomado consciencia del conjunto (no veía la relación entre el campo email_cifrado y $_GET["ruta"]. Tu código puede reducir su complejidad ya que tu SQL está buscando un registro cuyo campo email_cifrado calga $_GET["ruta"], por lo que si se recibe un registro es porque se cumple esa condición, y si se recibe false es porque no se encontró dicho registro. Hacer de nuevo la comprobación en un if es redundante (y falla cuando no recibes un registro). Commented el 20 ene. 2021 a las 21:38
1

Encontre la respuesta añadi esta linea solucionando el error

 if(is_array($validarCorreo) && $validarCorreo["email_encriptado"] == $_GET["ruta"]){
8
  • 5
    Realmente esa no es la solución, es un parche. $validarCorreo vale false por algún motivo, y debes encontrar el motivo por el que adquirió ese valor. Commented el 20 ene. 2021 a las 21:11
  • 1
    si es que eso esta muy sospechoso Commented el 20 ene. 2021 a las 21:14
  • 3
    sería bueno que evalúes que te da errorInfo el te debe auxiliar en detectar que ocurre @luisfernando
    – user128299
    Commented el 20 ene. 2021 a las 21:15
  • 1
    Estoy con @BetaM. Es justo lo que he propuesto en mi respuesta. El origen del problema está en que la consulta SQL está fallando y devuelve false, por lo que antes de "tapar" ese fallo, deberías corregirlo, ya que estás perdiendo esa funcionalidad en tu aplicación. Commented el 20 ene. 2021 a las 21:18
  • 1
    si estoy revisando esa consulta haber si logro arreglarlo de raiz Commented el 20 ene. 2021 a las 21:23

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.