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

Además, son dos líneas redundantes, bastaría con una (de hecho, en algún escenario, no recurdorecuerdo ahora si con mysqli o con PDO), invocar a close() no me destruía al objeto, null en cambio siempre lo destruía.

static public function mdlCerrarCaja($tabla,$Hora_Cierre,$Total_Ventas,$Estado,$cajaAbierta,$Sucursal){

    if($cajaAbierta!= null){
        //Consulta UPDATE y demás...

        if($stmt->execute()){
            $mData=array(
                          "success"=>true,
                          "action"=>"update",
                          "data"=>"Filas actualizadas N..."
                         );
        }else{
            $mData=array(
                          "success"=>false,
                          "msg"=>"Error ... Conviene verificar el no de Error con errorInfo() y lanzar tu política de errores"
                         );
        }           
    }else{
        $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");
        $stmt -> execute();
        $mData=array(
                       "success"=>true,
                       "action"=>"read",
                       "data"=>$stmt -> fetchAll()
                    );

    }
    $return$stmt $mData;
= null;
   $stmt =$return null;$mData;
}

Dado que este método puede devolver dos tipos de respuestas exitosas, he puesto una clave action por la cual podrás distinguir si ocurrió un update o un read y acoplar en la respuesta: mostrar el mensaje de cantidad de filas actualizadas por ejemplo, o leer los datos del fetchAll que obtuviste en el SELECT.

En cuanto a los casos de error, cuando success sea false el mensaje de lo ocurrido se encontrará en una clave msg.

Además, son dos líneas redundantes, bastaría con una (de hecho, en algún escenario, no recurdo ahora si con mysqli o con PDO, invocar a close() no me destruía al objeto, null en cambio siempre lo destruía.

static public function mdlCerrarCaja($tabla,$Hora_Cierre,$Total_Ventas,$Estado,$cajaAbierta,$Sucursal){

    if($cajaAbierta!= null){
        //Consulta UPDATE y demás...

        if($stmt->execute()){
            $mData=array(
                          "success"=>true,
                          "action"=>"update",
                          "data"=>"Filas actualizadas N...");
        }else{
            $mData=array(
                          "success"=>false,
                          "msg"=>"Error ... Conviene verificar el no de Error con errorInfo() y lanzar tu política de errores");
        }           
    }else{
        $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");
        $stmt -> execute();
        $mData=array(
                       "success"=>true,
                       "action"=>"read",
                       "data"=>$stmt -> fetchAll()
                    );

    }
    $return $mData;
    $stmt = null;
}

Dado que este método puede devolver dos tipos de respuestas exitosas, he puesto una clave action por la cual podrás distinguir si ocurrió un update o un read y acoplar en la respuesta: mostrar el mensaje de cantidad de filas actualizadas por ejemplo, o leer los datos del fetchAll que obtuviste en el SELECT.

Además, son dos líneas redundantes, bastaría con una (de hecho, en algún escenario, no recuerdo ahora si con mysqli o con PDO), invocar a close() no me destruía al objeto, null en cambio siempre lo destruía.

static public function mdlCerrarCaja($tabla,$Hora_Cierre,$Total_Ventas,$Estado,$cajaAbierta,$Sucursal){

    if($cajaAbierta!= null){
        //Consulta UPDATE y demás...

        if($stmt->execute()){
            $mData=array(
                          "success"=>true,
                          "action"=>"update",
                          "data"=>"Filas actualizadas N..."
                         );
        }else{
            $mData=array(
                          "success"=>false,
                          "msg"=>"Error ... Conviene verificar el no de Error con errorInfo() y lanzar tu política de errores"
                         );
        }           
    }else{
        $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");
        $stmt -> execute();
        $mData=array(
                       "success"=>true,
                       "action"=>"read",
                       "data"=>$stmt -> fetchAll()
                    );

    }
    $stmt = null;
    $return $mData;
}

Dado que este método puede devolver dos tipos de respuestas exitosas, he puesto una clave action por la cual podrás distinguir si ocurrió un update o un read y acoplar en la respuesta: mostrar el mensaje de cantidad de filas actualizadas por ejemplo, o leer los datos del fetchAll que obtuviste en el SELECT.

En cuanto a los casos de error, cuando success sea false el mensaje de lo ocurrido se encontrará en una clave msg.

se añadieron 2189 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Advertencia sobre una mala práctica

En tu método, estas dos líneas al final no se ejecutarán nunca:

$stmt -> close();
$stmt = null;

¿Por quéeeee si las he puesto con muy buena intención? Pues porque tienes return más arriba y nada lo que esté después de return se ejecuta.

Además, son dos líneas redundantes, bastaría con una (de hecho, en algún escenario, no recurdo ahora si con mysqli o con PDO, invocar a close() no me destruía al objeto, null en cambio siempre lo destruía.

Quizá una forma mejor de estructurar tu método sería esta:

static public function mdlCerrarCaja($tabla,$Hora_Cierre,$Total_Ventas,$Estado,$cajaAbierta,$Sucursal){

    if($cajaAbierta!= null){
        //Consulta UPDATE y demás...

        if($stmt->execute()){
            $mData=array(
                          "success"=>true,
                          "action"=>"update",
                          "data"=>"Filas actualizadas N...");
        }else{
            $mData=array(
                          "success"=>false,
                          "msg"=>"Error ... Conviene verificar el no de Error con errorInfo() y lanzar tu política de errores");
        }           
    }else{
        $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");
        $stmt -> execute();
        $mData=array(
                       "success"=>true,
                       "action"=>"read",
                       "data"=>$stmt -> fetchAll()
                    );

    }
    $return $mData;
    $stmt = null;
}

Si observas, se ha usado un objeto $mData para controlar la lógica interna del método. Devolviendo una clave success establecida a true cuando haya algún dato o false cuando haya algún error.

Dado que este método puede devolver dos tipos de respuestas exitosas, he puesto una clave action por la cual podrás distinguir si ocurrió un update o un read y acoplar en la respuesta: mostrar el mensaje de cantidad de filas actualizadas por ejemplo, o leer los datos del fetchAll que obtuviste en el SELECT.


Advertencia sobre una mala práctica

En tu método, estas dos líneas al final no se ejecutarán nunca:

$stmt -> close();
$stmt = null;

¿Por quéeeee si las he puesto con muy buena intención? Pues porque tienes return más arriba y nada lo que esté después de return se ejecuta.

Además, son dos líneas redundantes, bastaría con una (de hecho, en algún escenario, no recurdo ahora si con mysqli o con PDO, invocar a close() no me destruía al objeto, null en cambio siempre lo destruía.

Quizá una forma mejor de estructurar tu método sería esta:

static public function mdlCerrarCaja($tabla,$Hora_Cierre,$Total_Ventas,$Estado,$cajaAbierta,$Sucursal){

    if($cajaAbierta!= null){
        //Consulta UPDATE y demás...

        if($stmt->execute()){
            $mData=array(
                          "success"=>true,
                          "action"=>"update",
                          "data"=>"Filas actualizadas N...");
        }else{
            $mData=array(
                          "success"=>false,
                          "msg"=>"Error ... Conviene verificar el no de Error con errorInfo() y lanzar tu política de errores");
        }           
    }else{
        $stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");
        $stmt -> execute();
        $mData=array(
                       "success"=>true,
                       "action"=>"read",
                       "data"=>$stmt -> fetchAll()
                    );

    }
    $return $mData;
    $stmt = null;
}

Si observas, se ha usado un objeto $mData para controlar la lógica interna del método. Devolviendo una clave success establecida a true cuando haya algún dato o false cuando haya algún error.

Dado que este método puede devolver dos tipos de respuestas exitosas, he puesto una clave action por la cual podrás distinguir si ocurrió un update o un read y acoplar en la respuesta: mostrar el mensaje de cantidad de filas actualizadas por ejemplo, o leer los datos del fetchAll que obtuviste en el SELECT.

Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Es preciso que comprendas que en las consultas preparadas se ponen marcadores, no los valores.

Por tanto esto es erróneo:

$stmt = Conexion::conectar()->prepare("UPDATE $tabla SET Hora_Cierre = :$Hora_Cierre, Total_Ventas = :$Total_Ventas, Estado = :$Estado WHERE Sucursal = :$Sucursal AND caja_sucursal = :$cajaAbierta");

Precisamente, el quid de una consulta preparada consiste en poner un marcador en el lugar donde va cada dato y luego, con un método apropiado (bindParam en este caso) pasas el valor para cada dato asociado a su marcador. Es como decirle al controlador: donde puse este marcador, debes poner este dato, pero revisa que no haya riesgo de inyección.

Entendido eso, tu código debería quedar así:

$sql="UPDATE $tabla SET 
          Hora_Cierre = :Hora_Cierre, 
          Total_Ventas = :Total_Ventas, 
          Estado = :Estado 
       WHERE 
          Sucursal = :Sucursal AND 
          caja_sucursal = :caja_sucursal";
$stmt = Conexion::conectar()->prepare($sql);

Simplemente, hemos quitado toooodas las $, teniendo así lo que se requiere, que son marcadores.

Ahora, debes decirle al SGBD a qué dato va asociado cada marcador que pusiste más arriba, algo MUY IMPORTANTE es que debes escribir cada marcador exactamente igual que como lo escribiste en la instrucción SQL, y debes vigilar que asocias en cada caso el valor correspondiente y no otro valor. Por tanto, si estás recibiendo estas variables en parámetro:

$Hora_Cierre
$Total_Ventas
$Estado
$cajaAbierta
$Sucursal

Debes hacer las vinculaciones usando esas variables:

$stmt -> bindParam(":Hora_Cierre", $Hora_Cierre, PDO::PARAM_STR);
$stmt -> bindParam(":Total_Ventas", $Total_Ventas, PDO::PARAM_STR);
$stmt -> bindParam(":Estado", $Estado, PDO::PARAM_STR);
$stmt -> bindParam(":Sucursal", $Sucursal, PDO::PARAM_STR);
$stmt -> bindParam(":caja_sucursal", $cajaAbierta, PDO::PARAM_STR);

No estabas vinculando :Sucursal ni :caja_sucursal. Yo he asumido que también son cadenas, si alguna no lo es, cambia PDO::PARAM_STR por el tipo de dato correcto.

Así debería funcionar, salvo otros errores.