2

Estoy desarrollando un programa donde envío un activo que puede o no estar asociado a una habitación. Si no se asocia a una habitación idHabitación = 0, si no contiene la clave primaria de esa habitación. Creé un procedimiento donde valido primero si el activo existe con el número de serie, si no existe, valido si idHabitacion es 0, si lo es hace un insert sin llenar el campo idHabitacion (null), sino si lo llena (llave primaria), pero esto no me está funcionando.

create proc procCrearNuevoActivoPagNuevoActivo
@nomActivo varchar(255),
@numSerie as varchar(255),
@idTipo  int,
@marcaActivo varchar(255), 
@modeloActivo varchar(255),
@fechaCompra date,
@fechaVenceGarantia date,
@estadoActivo varchar(255),
@descripcion text,
@idUsuario int,
@idHabitacion int
as 
if (not exists(select * from IH_ACTIVOS where numSerie=@numSerie))
    begin 
        if (@idHabitacion=0)
        begin 
            insert into IH_ACTIVOS(nomActivo, fechaCreacion, numSerie, idTipos, marcaActivo, modeloActivo, 
            fechaCompra, fechaVenceGarantia,estadoActivo, descripcion, idUsuario) 
            values(@nomActivo,GETDATE(),@numSerie,@idTipo,@marcaActivo, @modeloActivo,
            @fechaCompra,@fechaVenceGarantia,@estadoActivo,@descripcion,@idUsuario)
        end
            else
                begin
                    insert into IH_ACTIVOS(nomActivo, fechaCreacion, numSerie, idTipos, marcaActivo, modeloActivo, 
                    fechaCompra, fechaVenceGarantia,estadoActivo, descripcion, idUsuario, idHabitacion) 
                    values(@nomActivo,GETDATE(),@numSerie,@idTipo,@marcaActivo, @modeloActivo,
                    @fechaCompra,@fechaVenceGarantia,@estadoActivo,@descripcion,@idUsuario, @idHabitacion)
                end
    end

C#:

 public string procCrearNuevoActivoPagNuevoActivo(Activo objActivo)
        {
            string mensaje = "";
            cn = conexion.getConexion();

            using (SqlTransaction tr = cn.BeginTransaction(IsolationLevel.Serializable))
            {

                SqlCommand cmd = new SqlCommand("procCrearNuevoActivoPagNuevoActivo", cn, tr);
                cmd.CommandType = CommandType.StoredProcedure;
                cmd.Parameters.Add("nomActivo", SqlDbType.VarChar).Value = objActivo.nomActivo;
                cmd.Parameters.Add("numSerie", SqlDbType.VarChar).Value = objActivo.numSerie;
                cmd.Parameters.Add("idTipo", SqlDbType.Int).Value = objActivo.idTipo;
                cmd.Parameters.Add("marcaActivo", SqlDbType.VarChar).Value = objActivo.marcaActivo;
                cmd.Parameters.Add("modeloActivo", SqlDbType.VarChar).Value = objActivo.modeloActivo;
                cmd.Parameters.Add("fechaCompra", SqlDbType.Date).Value = objActivo.fechaCompra;
                cmd.Parameters.Add("fechaVenceGarantia", SqlDbType.Date).Value = objActivo.fechaVenceGarantia;
                cmd.Parameters.Add("estadoActivo", SqlDbType.VarChar).Value = objActivo.estadoActivo;
                cmd.Parameters.Add("descripcion", SqlDbType.Text).Value = objActivo.descripcion;
                cmd.Parameters.Add("idHabitacion", SqlDbType.Int).Value = objActivo.idHabitacion;
                cmd.Parameters.Add("idUsuario", SqlDbType.Int).Value = objActivo.idUsuario;

                try
                {
                    int i = cmd.ExecuteNonQuery();
                    if (i== 1)
                    {
                        mensaje = "Se ingresó " + i + " nuevo activo";
                        tr.Commit();
                    }
                    else {
                        mensaje = "El número de serie " + objActivo.numSerie + " ya existe en inventario !";
                    }

                }
                catch(Exception e)
                {
                    mensaje = "Error en el registro del activo: "+e;
                    tr.Rollback();
                }
                finally
                {
                    cn.Close();
                }
                return mensaje;
            }
        }

Diagrama:

Diagrama de las tablas

Lo que quiero en el procedimiento almacenado es básicamente: " SI Activo no existe entonces, si idHabitacion=0 entonces inserte todo menos idHabitación (queda null), sino entonces inserte todo (con idHabitaion = valor correspondiente).

El problema es que cuando guardo un activo con un nuevo número de serie, por ejemplo sk123, este se guarda pero me indica que "El número de serie sk123 ya existe en inventario !" en lugar de dicirme "Se ingresó 1 nuevo activo" que sería lo correcto. Lo que creo que pasa es que se están ejecutando ambos update y no está obedeciendo los If Else que puse para validar @idHabitacion.

A veces me pasa que aparece un mensaje de interbloqueo lo cual no debería pasar, lo que me hace pensar que se ejecutan ambos update.

Quedo atento. Gracias !

1
  • Recomendación: La lógica que está en tu base de datos, pásala a la capa de negocios, el motivo es mantener la sencillez del desarrollo. Saludos :D
    – fredyfx
    Commented el 3 jun. 2019 a las 6:35

1 respuesta 1

0

Dejando de lado la pésima idea de pasar toda la lógica a la "capa de negocios", te recomiendo que no confíes en el conteo de filas ya que puede tener inconsistencias en su comportamiento. De hecho, la recomendación general es deshabilitar el conteo de filas al momento de llamar los procedimientos desde la aplicación mediante el uso de SET NOCOUNT ON. Una opción que tienes, es la de generar un error desde tu procedimiento que sea capturado por el TRY de tu código en C#.

CREATE PROCEDURE procCrearNuevoActivoPagNuevoActivo(
    @nomActivo varchar(255),
    @numSerie as varchar(255),
    @idTipo  int,
    @marcaActivo varchar(255), 
    @modeloActivo varchar(255),
    @fechaCompra date,
    @fechaVenceGarantia date,
    @estadoActivo varchar(255),
    @descripcion text,
    @idUsuario int,
    @idHabitacion int
)
AS 
SET NOCOUNT ON;

IF (NOT EXISTS(SELECT * FROM IH_ACTIVOS WHERE numSerie = @numSerie))
BEGIN 
    INSERT INTO IH_ACTIVOS(
            nomActivo, 
            fechaCreacion, 
            numSerie, 
            idTipos, 
            marcaActivo, 
            modeloActivo, 
            fechaCompra, 
            fechaVenceGarantia,
            estadoActivo, 
            descripcion, 
            idUsuario,
            idHabitacion
    ) 
    VALUES(
            @nomActivo,
            GETDATE(),
            @numSerie,
            @idTipo,
            @marcaActivo, 
            @modeloActivo,
            @fechaCompra,
            @fechaVenceGarantia,
            @estadoActivo,
            @descripcion,
            @idUsuario,
            NULLIF( @idHabitacion, 0)
    );
END
ELSE
    RAISERROR(  '¡El número de serie %s ya existe en inventario !', 11, 1, @numSerie);
GO

No tengas miedo de usar espacio suficiente para mejorar la lectura de tus procedimientos almacenados. También recuerda utiliza el principio DRY (Don't Repeat Yourself / No te repitas).

Aun mejor, sería que la tabla IH_ACTIVOS tuviera un UNIQUE CONSTRAINT en la columna numSerie para no necesitar la validación inicial y todo quedaría en un simple INSERT y capturas el error para regresar el mensaje deseado en vez del mensaje por default.

ALTER TABLE IH_ACTIVOS ADD CONSTRAINT UQ_IH_ACTIVOS_NumSerie UNIQUE ( numSerie);
9
  • Esto me sirve para validar la existencia del activo, pero no si voy a ingresar con un valor de idHabitacion o no. La idea es que si {@idHabitacion = 0} se ejecute un insert sin el campo @idHabitación, de lo contrario, que sí lo tome en cuenta.
    – DCampos
    Commented el 3 jun. 2019 a las 19:55
  • Eso mismo hace. La función NULLIF convertirá el zero en un nulo. Así será como si no insertaras nada sin necesidad de un IF. Commented el 3 jun. 2019 a las 19:58
  • En este caso Luis, ¿ Cómo puedo interpretar la ejecución correcta del procedimiento en C# para saber que sí se agregó el activo correctamente?
    – DCampos
    Commented el 4 jun. 2019 a las 13:41
  • Si no genera ningún error, entonces se agregó correctamente. Alternativamente, puedes usar el comando RETURN después del INSERT para regresar @@ROWCOUNT. Ejemplo: RETURN @@ROWCOUNT;. O también puedes utilizar un parámetro con OUTPUT para regresar algún valor de cualquier tipo (RETURN solo maneja enteros). Commented el 4 jun. 2019 a las 15:23
  • Pero entonces en el C# puedo seguir utilizando cmd.ExcecuteNonQuery() para validar el correcto ingreso del dato ? o como sería si quiero devolver un mensaje con output para leerlo en C#.
    – DCampos
    Commented el 4 jun. 2019 a las 15:50

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.