Actualizando mi capita de datos...

[moNTeZIon]

Buenas a todos.
Me estaba planteando cómo cambiar esto, para mejorarlo. La cosa es que en cada una de mis clases tengo un método parecido al siguiente, que corresponde al método "Eliminar" de mi clase "Distritos":

Código vb.net:

Public Sub Eliminar(ByVal CodigoDistrito As String)
            Dim sSQL As String
            Dim Conn As New SqlConnection(ConfigurationManager.ConnectionStrings("ConnStr").ConnectionString)
            Conn.Open()
            Dim cM As New SqlCommand
            Dim miTrans As SqlTransaction = Conn.BeginTransaction
            Try
                'Eliminamos Actuaciones
                sSQL = "DELETE FROM [Actuaciones] WHERE CodigoDistrito=@CodigoDistrito"
                cM = New SqlCommand(sSQL, Conn, miTrans)
                cM.Parameters.Add("@CodigoDistrito", SqlDbType.VarChar, 15).Value = CodigoDistrito
                cM.ExecuteNonQuery()
                'Eliminamos Distrito
                sSQL = "DELETE FROM [Distritos] WHERE Codigo=@Codigo"
                cM = New SqlCommand(sSQL, Conn, miTrans)
                cM.Parameters.Add("@Codigo", SqlDbType.VarChar, 15).Value = CodigoDistrito
                cM.ExecuteNonQuery()
                miTrans.Commit()
            Catch ex As Exception
                miTrans.Rollback()
                Throw New Exception(ex.Message)
            End Try
            Conn.Close()
            Conn.Dispose()
        End Sub
 

Luego, dispongo también de mi clase "Actuaciones", que se trata de una colección de "Distritos". Dicho de otro modo, las "Actuaciones" están relacionadas con un "Distrito". Es por eso, que al eliminar un "Distrito", antes se eliminan todas sus "Actuaciones" y finalmente, se elimina el "Distrito", todo esto dentro de una misma SqlTransaction.
Bien, ahora cuál es la modificación que me gustaría realizar...
Verán, en mi clase "Actuaciones" ya dispongo del método "Eliminar" pero en ese caso sin usar Transacciones, puesto que sólo realizo una eliminación de una Tabla. Pero el código utilizado, es exactamente el mismo que la parte correspondiente a eliminar las "Actuaciones", de la funcion anterior.
Por lo tanto, estoy repitiendo código.
Me gustaría sustituir la parte de código que elimina las "Actuaciones" en esta función, por una llamada a la función "Eliminar" de la clase "Actuaciones", de forma que le pueda pasar el obteto SqlTransaction, y todo se siga realizando en una misma transacción, así evitaría repetir código.
Pero no sé como realizar esta modificación por lo siguiente.
En la función "Eliminar" de la clase "Actuaciones" también se crea un nuevo objeto SqlConnection, que se abre, se asigna a un SqlCommand y se ejecuta la SQL. Luego, se cierra la conexión y se hace el Dispose.
Pero todo esto no se debería hacer si la operación forma parte de una transacción, puesto que la misma conexión se debe seguir utilizando, una vez eliminadas las "Actuaciones", para eliminar los "Distritos", de forma que se haga en una misma transacción.
Pero en el momento que llamamos a la funcion "Eliminar" de la clase "Actuaciones", solamente para eliminar una "Actuación" puntual, entonces si debería crear una conexión nueva, y al ejecutar la SQL, cerrar la conexión y hacerle el Dispose. Así que en este caso, no debería cerrarse y hacer el Dispose, sino quedar abierta para las siguientes instrucciones.
¿Como puedo compaginar una cosa con la otra? ¿Cuál es la forma correcta de implementar estas cosas?
Gracias por vuestras opiniones.
Saludos.

[dwaks]

OK a ver si mi idea te gusta.

Te expondré la idea lo mejor que pueda:
- El punto es tener el proceso de SqlTransaction en un método aparte
- A este método le pasas ya sea un Array de SqlCommand[] ó que tengas una lista
List<SqlCommand> donde cargues los comandos allí
- Después el método procesa los Comandos que mandes y los liga a la conexión que
el SqlTransaction maneja.
- Todos tus métodos actuales solo preparan el SqlCommand y usan siempre el otro método
que procesa el SqlTransaction
- Como tu usas Parameters puedes tener ya los querys definidos en un HasTable ó Dictionary por Key y Value donde el Key sea un nombre que indentifique el query que esta en el Value, así tienes todos tus query en un solo lugar y cuando los necesites entre los métodos solo los llamas por el Key y entonces le mandas sus parametros cuando prepares el SqlCommand.
- Creo que así tendrías: Centralizado el SqlTransaction en un solo método, Centralizado los Query en un Dictionary, Solo preparas los SqlCommand en cada método tuyo.

Saludos,

[moNTeZIon]

Muchísimas gracias por tu opinión, dwaks
Antes de hacer este cambio de centralizar las querys, voy centrarme en el tema de la transaction.
A ver si este resumen vale para explicar el método que me has explicado.
Representa que en el método "Eliminar" de "Actuaciones" solamente construyo un SQLCommand, con la query correspondiente para eliminar una o varias actuaciones. Una vez lo tengo, lo paso como parámetro a ese método independiente, en el que siempre se crea una conexión, una transacción, y el mismo método es el que ejecuta el "Connection.Open", el "Conection.BeginTransaction", y los "Execute" de los Comandos (en este caso un solo comando).
La diferencia en el caso de eliminar un "Distrito" es que este método va a recibir más comandos, de forma que los ejecutará en la misma transacción.
O sea que reciba un Comando, o reciba varios, siempre se va a ejecutar en una transacción.
Lo he entendido así, y la verdad es que ya me están entrando ganas de implementarlo.
Digamos que yo las transacciones las utilizo solo cuando tengo que ejecutar varios comandos. Cuando solo quiero ejecutar uno, no utilizo ninguna transacción.+
Imagino que tampoco hay problema por ejecutar estas querys aisladas en una transaction, aunque no sea necesario hacerlo, verdad?
Gracias una vez más por la explicación, dwaks. Creo que me has hecho ver la luz
Un saludo!

[moNTeZIon]

Buenas!
Este es el método que hice para ejecutar el uno o varios comandos en una misma transacción:

Código vb.net:

Public Shared Sub EjecutarTransaccion(ByVal ListaComandosSQL As List(Of SqlCommand))
        Dim Conn As New SqlConnection(Entorno.ConnStr)
        Conn.Open()
        Dim miTrans As SqlTransaction = Conn.BeginTransaction
        Try
            For Each miComando As SqlCommand In ListaComandosSQL
                miComando.Connection = Conn
                miComando.Transaction = miTrans
                miComando.ExecuteNonQuery()
            Next
            miTrans.Commit()
        Catch ex As Exception
            miTrans.Rollback()
            Throw New Exception(ex.Message)
        End Try
        Conn.Close()
        Conn.Dispose()
    End Sub
 

Ahora la cosa será, desde los demás métodos, simplemente construir los comandos con sus parámetros, y cuando los tenga, pasárselo/s a este método para que los ejecute en una transacción.
Cualquier cosa "rara" que veais, o cualquier consejo al respecto, es bien recibido.
Un saludo!

[dwaks]

Me gusta tu método y me alegra que me entendieras bien.

Saludos,