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

Aparte de eso, cuando escribas funciones conviene establecer un control estricto en ellas, de modo que puedas saber fácilmente qué ha ocurrido en tu función. Aquí he implementado una práctica que consiste en recoger la información dentro de un array $response con dos claves: status y msg. Luego, en el resultado de la función puedes verificar status y si quieres mostrar el mensaje de lo que ocurrió, hechas mano de usas la clave msg para mostrar lo que hay en ella.

$mData=addCita (....aquí los parámetros...);
if ( $mData["status")$mData["status"] ) 
{
    //Se insertó el registro
    //Puedes mostrar $mData['msg'] si quieres
}
else
{
    echo "Hubo un error: $mData[msg]";
}

Aparte de eso, cuando escribas funciones conviene establecer un control estricto en ellas, de modo que puedas saber fácilmente qué ha ocurrido en tu función. Aquí he implementado una práctica que consiste en recoger la información dentro de un array $response con dos claves: status y msg. Luego, en el resultado de la función puedes verificar status y si quieres mostrar el mensaje de lo que ocurrió, hechas mano de msg.

$mData=addCita (....aquí los parámetros...);
if ( $mData["status") ) 
{
    //Se insertó el registro
}
else
{
    echo "Hubo un error: $mData[msg]";
}

Aparte de eso, cuando escribas funciones conviene establecer un control estricto en ellas, de modo que puedas saber fácilmente qué ha ocurrido en tu función. Aquí he implementado una práctica que consiste en recoger la información dentro de un array $response con dos claves: status y msg. Luego, en el resultado de la función puedes verificar status y si quieres mostrar el mensaje de lo que ocurrió usas la clave msg para mostrar lo que hay en ella.

$mData=addCita (....aquí los parámetros...);
if ( $mData["status"] ) 
{
    //Se insertó el registro
    //Puedes mostrar $mData['msg'] si quieres
}
else
{
    echo "Hubo un error: $mData[msg]";
}
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

El problema es que no hay coherencia en tu consulta SELECT, ya que en el execute estás pasando un parámetro :Estado que no has indicado en el prepare. Imagino que quieres filtrar por el valor que se pasó en la variable $Estado. Entonces tienes que agregar ese parámetro, que ahora mismo está escrito a mano con un valor 3.

Aparte de eso, cuando escribas funciones conviene establecer un control estricto en ellas, de modo que puedas saber fácilmente qué ha ocurrido en tu función. Aquí he implementado una práctica que consiste en recoger la información dentro de un array $response con dos claves: status y msg. Luego, en el resultado de la función puedes verificar status y si quieres mostrar el mensaje de lo que ocurrió, hechas mano de msg.

Otra cosa es que, si la finalidad del SELECT es únicamente saber si el registro existe, es mejor hacer una consulta basa en COUNT y luego obtener la cantidad de filas, usando fetchColumn(). Es más ligero y más seguro que hacer un fetchAll con unos datos que al fin y al cabo no vas a usar.

Aplicando todo eso el código quedaría así.

<?php
    public function addCita($Next,$Trabajadora,$Cliente,$Concat,$Suma,$Observaciones,$Estado)
    {
        $sql= "SELECT COUNT(*) from cita WHERE Id_Trabajadora= :Trabajadora and Fecha_Hora= :Concat and Id_Estado !=:Estado";
        $query = $this->db->prepare($sql);
        $parameters = array(':Trabajadora' => $Trabajadora,':Concat' => $Concat, ':Estado' => $Estado);
        if ( !$query->execute($parameters) )
        {
            $response=array("status"=>FALSE, "msg"=>"Error en la consulta SELECT");
        }
        else
        {
            if ( $query->fetchColumn()>0 )
            {
                $response=array("status"=>FALSE, "msg"=>"$Trabajadora existe");
            }
            else
            {
                $sql = "INSERT INTO `cita`(`Id_Cita`, `Id_Trabajadora`, `Id_Cliente`, `Fecha_Hora`, `Precio_Total`, `Observaciones`, `Id_Estado`) VALUES (:Next, :Trabajadora, :Cliente, :Concat, :Suma, :Observaciones, :Estado)";
                $query = $this->db->prepare($sql);
                $parameters = array(':Next' => $Next, ':Trabajadora' => $Trabajadora, ':Cliente' => $Cliente, ':Concat' => $Concat, ':Suma' => $Suma, ':Observaciones' => $Observaciones, ':Estado' => $Estado);
                if ( !$query->execute($parameters) )
                {
                    $response=array("status"=>FALSE, "msg"=>"Error en la consulta INSERT");
                }
                else
                {
                    $response=array("status"=>TRUE, "msg"=>"$Trabajadora insertada");
                }
            }
    
        }
    return $response;       
    }
?>

##Ejemplo de uso

$mData=addCita (....aquí los parámetros...);
if ( $mData["status") ) 
{
    //Se insertó el registro
}
else
{
    echo "Hubo un error: $mData[msg]";
}