Question

Hola, recien estoy usando mysqli, y queria que ustedes revisen mi codigo ya que es la primera vez que uso transacciones en php

 DELIMITER $$

CREATE DEFINER=`root`@`localhost` PROCEDURE `insertarUsuario`(in dni int, in nombre varchar(50),in apellido varchar(150)
, in password varchar(32), in puesto varchar(30), in priv int,in habilitado int, in id_sucursal int)
begin

DECLARE EXIT HANDLER FOR SQLEXCEPTION 
BEGIN 
SELECT 1 as error; 
ROLLBACK; 
END; 

/*Handler para error SQL*/ 
DECLARE EXIT HANDLER FOR SQLWARNING 
BEGIN 
SELECT 1 as error; 
ROLLBACK; 
END; 

/*Inicia transaccion*/ 
START TRANSACTION; 

    insert into usuarios(dni,nombre,apellido,password,puesto,priv,habilitado,id_sucursal) values (dni,nombre,apellido,password,puesto,priv,habilitado,id_sucursal);

commit;

SELECT 0 as error; 
end

De hecho lo he puesto en un procedimiento almacenado, tampoco se si sea buena idea, agradeceria opiniones :)

Answer 1

Hola Christian,

a simple vista, el código no tiene nada de mal, sin embargo es innecesario.

La idea de una transacción es englobar varias operaciones sobre la DB como si fueran una sola, para no dejar una inserción, una actualización o un borrado a medias cuando están afectadas varias tablas o registros.

En tu caso, solo hay un INSERT, que por definición es una tarea atómica, así que todo esto (al menos así como está) es superfluo y complica el código haciéndolo más difícil de mantener.

Si solo es una prueba de concepto, adelante! Es este el camino.

Por otra parte, podés desde PHP ahorrarte escribir procedimientos almacenados y todo eso, porque podés usar el método $mysqli->autocommit(FALSE);, luego poner un montód de querys, como siempre, y al final llamar a $mysqli->commit(). Esto hace que todas las querys sean parte de una transacción.

Acá hay un ejemplo de código: http://www.php.net/manual/es/mysqli.commit.php

Saludos!