Skip to main content
se añadieron 1685 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Si te interesa mostrar al usuario posibles mensajes de errores de la consulta, entonces tendremos tres posibles resultados:

  1. Que la consulta no arroje datos
  2. Que haya un error en la consulta
  3. Que la consulta arroje datos

Una forma coherente de programar aquí sería devolver en todos los casos un array que podría ser NULL, tener el mensaje de error con una clave llamada error o un array con nuestros datos.

Dado que aquí me parece que la consulta no arrojará demasiadas filas, creo que podemos usar fetchAll para simplificar. Así traemos todos los resultados en una variable $resultado. También se puede hacer como antes, con el while.

La función quedaría así:

public function InfoRest(){
    session_start();
    $id = $_SESSION['usuario']['id_usuario'];
    $sql = $this->pdo->prepare("SELECT * FROM restaurants WHERE id_usuario = ?");
    $arrParams=array($id);
    $sql->execute($arrParams);

    if($sql){
        $resultado=$sql->fetchAll(PDO::FETCH_ASSOC);
        $this->datos[]=$resultado;
        return $this->datos;
    }else{
        return array ('error'=>$sql->errorInfo()[2]);
    }
}

Y cuando recibes los datos, evalúas los tres posibles resultados así:

$obj = new Models();
$id = $_GET['id_restaurant'];
$details = $obj->DetailsRestaurant($id);

if(!$detail){
    echo "No se encontraron datos"; 
}elseif (array_key_exists('error', $detail){
    echo $detail['error']; //Imprimirá el mensaje de error  
}else{
      foreach($details as $result){
          echo $result['name_rest'];
      }
}

Si te interesa mostrar al usuario posibles mensajes de errores de la consulta, entonces tendremos tres posibles resultados:

  1. Que la consulta no arroje datos
  2. Que haya un error en la consulta
  3. Que la consulta arroje datos

Una forma coherente de programar aquí sería devolver en todos los casos un array que podría ser NULL, tener el mensaje de error con una clave llamada error o un array con nuestros datos.

Dado que aquí me parece que la consulta no arrojará demasiadas filas, creo que podemos usar fetchAll para simplificar. Así traemos todos los resultados en una variable $resultado. También se puede hacer como antes, con el while.

La función quedaría así:

public function InfoRest(){
    session_start();
    $id = $_SESSION['usuario']['id_usuario'];
    $sql = $this->pdo->prepare("SELECT * FROM restaurants WHERE id_usuario = ?");
    $arrParams=array($id);
    $sql->execute($arrParams);

    if($sql){
        $resultado=$sql->fetchAll(PDO::FETCH_ASSOC);
        $this->datos[]=$resultado;
        return $this->datos;
    }else{
        return array ('error'=>$sql->errorInfo()[2]);
    }
}

Y cuando recibes los datos, evalúas los tres posibles resultados así:

$obj = new Models();
$id = $_GET['id_restaurant'];
$details = $obj->DetailsRestaurant($id);

if(!$detail){
    echo "No se encontraron datos"; 
}elseif (array_key_exists('error', $detail){
    echo $detail['error']; //Imprimirá el mensaje de error  
}else{
      foreach($details as $result){
          echo $result['name_rest'];
      }
}
se añadieron 725 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Hay un error de planteamiento en tu función. Si quieres verificar que el usuario existe, no necesitas el uso de numRows, sino que puedes llenar tu array directamente.

Como en el else puedes retornar el mensaje de error, entonces puedes verificar si el array está vacío en el destino.

La función podría quedar así (la he blindado contra inyección SQL).

public function InfoRest(){

    session_start();
    $id = $_SESSION['usuario']['id_usuario'];
    $sql = $this->pdo->prepare("SELECT * FROM restaurants WHERE id_usuario = ?");
    $arrParams=array($id);
    $sql->execute($arrParams);
    
    if($sql){
        while($row=$sql->fetch()){
            $this->datos[]=$row;
        }
        return $this->datos;
    }else{
        return $sql->errorInfo()[2];
    }
}

Y en el código para leerla:

$obj = new Models();
 $id = $_GET['id_restaurant'];
 $details = $obj->DetailsRestaurant($id);
 print_r($details);

 foreach($details as $result){
      <?php echo $result['name_rest'] ?>$result['name_rest'];
 }

NOTA: En InfoRest yo implementaría otra forma más lógica de devolver los posibles errores, que me permita verificar al recibir los datos si hay alguna clave con error.

En una futura edición de la respuesta podría proponer esa solución si te interesa. No lo hice aquí para no alejarme demasiado de tu código entrando en otros asuntos. La idea es escribir un código que sea coherente.

Si no te interesa en este caso el mensaje de error, puedes hacer lo siguiente:

    if($sql){
        while($row=$sql->fetch()){
            $this->datos[]=$row;
        }
        return $this->datos;
    }else{
        return NULL;
    }

De ese modo, tanto si no hay datos, como si la consulta falla, retornará NULL y al recibir los datos sólo tienes que evaluar la variable:

 $obj = new Models();
 $id = $_GET['id_restaurant'];
 $details = $obj->DetailsRestaurant($id);

 if($details){
      foreach($details as $result){
          echo $result['name_rest'];
      }
 }else{
     echo "No se encontraron datos o la consulta falló";
 }

Hay un error de planteamiento en tu función. Si quieres verificar que el usuario existe, no necesitas el uso de numRows, sino que puedes llenar tu array directamente.

Como en el else puedes retornar el mensaje de error, entonces puedes verificar si el array está vacío en el destino.

La función podría quedar así (la he blindado contra inyección SQL).

public function InfoRest(){

    session_start();
    $id = $_SESSION['usuario']['id_usuario'];
    $sql = $this->pdo->prepare("SELECT * FROM restaurants WHERE id_usuario = ?");
    $arrParams=array($id);
    $sql->execute($arrParams);
    
    if($sql){
        while($row=$sql->fetch()){
            $this->datos[]=$row;
        }
        return $this->datos;
    }else{
        return $sql->errorInfo()[2];
    }
}

Y en el código para leerla:

$obj = new Models();
 $id = $_GET['id_restaurant'];
 $details = $obj->DetailsRestaurant($id);
 print_r($details);

 foreach($details as $result){
      <?php echo $result['name_rest'] ?>
 }

NOTA: En InfoRest yo implementaría otra forma más lógica de devolver los posibles errores, que me permita verificar al recibir los datos si hay alguna clave con error.

En una futura edición de la respuesta podría proponer esa solución si te interesa. No lo hice aquí para no alejarme demasiado de tu código entrando en otros asuntos. La idea es escribir un código que sea coherente.

Hay un error de planteamiento en tu función. Si quieres verificar que el usuario existe, no necesitas el uso de numRows, sino que puedes llenar tu array directamente.

Como en el else puedes retornar el mensaje de error, entonces puedes verificar si el array está vacío en el destino.

La función podría quedar así (la he blindado contra inyección SQL).

public function InfoRest(){

    session_start();
    $id = $_SESSION['usuario']['id_usuario'];
    $sql = $this->pdo->prepare("SELECT * FROM restaurants WHERE id_usuario = ?");
    $arrParams=array($id);
    $sql->execute($arrParams);
    
    if($sql){
        while($row=$sql->fetch()){
            $this->datos[]=$row;
        }
        return $this->datos;
    }else{
        return $sql->errorInfo()[2];
    }
}

Y en el código para leerla:

$obj = new Models();
 $id = $_GET['id_restaurant'];
 $details = $obj->DetailsRestaurant($id);
 print_r($details);

 foreach($details as $result){
      echo $result['name_rest'];
 }

NOTA: En InfoRest yo implementaría otra forma más lógica de devolver los posibles errores, que me permita verificar al recibir los datos si hay alguna clave con error.

En una futura edición de la respuesta podría proponer esa solución si te interesa. No lo hice aquí para no alejarme demasiado de tu código entrando en otros asuntos. La idea es escribir un código que sea coherente.

Si no te interesa en este caso el mensaje de error, puedes hacer lo siguiente:

    if($sql){
        while($row=$sql->fetch()){
            $this->datos[]=$row;
        }
        return $this->datos;
    }else{
        return NULL;
    }

De ese modo, tanto si no hay datos, como si la consulta falla, retornará NULL y al recibir los datos sólo tienes que evaluar la variable:

 $obj = new Models();
 $id = $_GET['id_restaurant'];
 $details = $obj->DetailsRestaurant($id);

 if($details){
      foreach($details as $result){
          echo $result['name_rest'];
      }
 }else{
     echo "No se encontraron datos o la consulta falló";
 }
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Hay un error de planteamiento en tu función. Si quieres verificar que el usuario existe, no necesitas el uso de numRows, sino que puedes llenar tu array directamente.

Como en el else puedes retornar el mensaje de error, entonces puedes verificar si el array está vacío en el destino.

La función podría quedar así (la he blindado contra inyección SQL).

public function InfoRest(){

    session_start();
    $id = $_SESSION['usuario']['id_usuario'];
    $sql = $this->pdo->prepare("SELECT * FROM restaurants WHERE id_usuario = ?");
    $arrParams=array($id);
    $sql->execute($arrParams);
    
    if($sql){
        while($row=$sql->fetch()){
            $this->datos[]=$row;
        }
        return $this->datos;
    }else{
        return $sql->errorInfo()[2];
    }
}

Y en el código para leerla:

$obj = new Models();
 $id = $_GET['id_restaurant'];
 $details = $obj->DetailsRestaurant($id);
 print_r($details);

 foreach($details as $result){
      <?php echo $result['name_rest'] ?>
 }

NOTA: En InfoRest yo implementaría otra forma más lógica de devolver los posibles errores, que me permita verificar al recibir los datos si hay alguna clave con error.

En una futura edición de la respuesta podría proponer esa solución si te interesa. No lo hice aquí para no alejarme demasiado de tu código entrando en otros asuntos. La idea es escribir un código que sea coherente.