[Aporte] Sistema de Ventas en C# y SQL Server

[hernandemczuk]

Que tal, colegas. Quería contarles acerca de un proyecto en el que estuve trabajando estos últimos días. Se trata de un Sistema de Ventas desarrollado en C#.NET y SQL Server para gestionar la actividad de un negocio, generar reportes y controlar distintas áreas.

Se me ocurrió la idea de crear un videotutorial en YouTube contando cómo lo hice paso a paso, ya que no ví mucho de esto, y todo lo que hay está a la venta. Cualquier consulta o duda me la pueden transmitir!

Ah, el sistema terminado está online para descargar de forma libre y gratuita junto con el código fuente para que lo puedan chequear y crear uno acorde a sus propias necesidades.


Gráficas de los Reportes


Alta de Registros

Repositorio para descargar en GitHub:
https://github.com/hernandemczuk

Si no quieren GitHub, en mi blog tengo los archivos disponibles también:
http://hernanmartin.me/proyecto-sistema-de-gestion-para-una-empresa/

Aquí tienen los videos de cómo lo hice:
https://www.youtube.com/playlist?list=PL0ENtl4yA9gON4hQKIfLOhdEphoHncLOX

Saludos a todos y espero su feedback!

[uagrm]

Ante nada, felicidades, muy pocos comparten su código, aún no lo probé el sistema, pero debe de funcionar...
Estoy viendo tu código de la clase DEmpleados, DCategorias y DProductos (los demás no los vi aún) pero debo hacerte algunas observaciones, en estas clases tienes métodos como Insertar, Eliminar, Editar, Mostrar y Buscar, a única diferencia entre tus métodos son los procedimientos almacenados por que accedes a través de ellos a las diferentes tablas.

Entoces, te recomiendo para una próxima versión de tu sistema, puedas crearte una clase genérica que te permita Insertar, Eliminar, Editar, Mostrar y Buscar. Cuando necesites Insertar un empleado, categoría o producto..utilizas el método Insertar de tu clase genérica. Así no vas a repetir tu código.

Es recomendable que las validaciones las hagas en el procedimiento almacenado por ejemplo: Respuesta = "Ya existe un producto con ese Nombre"; este mensaje debería venir de la BD. Supongamos que ya lo tienes instalado tu sistema, y al cliente no le gusta ese mensaje, lo que vas hacer es ir al código modificar el mensaje, volver a compilar y instalar (tardas mucho tiempo), si la BD te manda ese mensaje pues es solo vas al procedimiento almacenado cambias el mensaje y ejecutas nuevamente y listo..

Bueno, saludos
Espero que no lo tomes a mal mis observaciones.

[Malenko]

Un aporte interesante, aunque me gustaría comentar varios puntos...

Capa de datos:
- En Dconexion tienes la cadena de connexión hard-coded. Lo más correcto sería obtenerla del app.config. Además te recuerdo que los usuarios que aparezcan en la cadena de conexión han de tener la menor cantidad de privilegios posibles y, opcionalmente, puedes encriptar la cadena de conexión.
- Tendrías que separar las entidades en POCO's e implementar el acceso a BBDD mediante repositorios. Mi recomendación además sería que usaras toda la potencia que te pueden dar los ORM como NHibernate o Entity Framework. Por otro lado, veo que no estás haciendo el Dispose de las connexiones ni de los commands (SqlConnection y SqlCommand).
- Otro tema aparte es que tendrías que evitar tener lógica de aplicación/negocio en la BBDD. Tendrías que mirar de pasar, en la medida de lo posible, los storeds a métodos de tu aplicación.

Capa de Negocio:
- Tienes un error bastante importante y es que en la capa de datos en caso de excepción la lanzas hacia la capa de arriba, pero en la capa superior (la de Negocio) no las estás tratando.
- Los métodos insertar tienen una gran cantidad de parámetros con los que luego montas la entidad. Sería bastante más limpio recibir la entidad (de ahí la importancia de usar POCOs para usarlos como DTO's entre capas de la aplicación).
- En cambio con el eliminar el método obtiene un Id pero luego montas una entidad "dummy" para pasarla a la capa de datos y de ahí solo volver a obtener el Id. No sería mejor pasar directamente el Id?

Capa de Presentación:
- No la he podido revisar a fondo

Notas generales:
- No veo ningún summary ni ningún tipo de documentación del código
- No hay tests de ningún tipo.
- Tendrías que usar tu propia implementación de la Excepción (extender la clase base)
- Personalmente, por trazabilidad, guardaría un registro de los errores en un log.
- No usas interfaces por ningún lado y tampoco contenedores de inyección de dependencias. Usando ambas cosas podrías hacer la aplicación más extensible y poder cambiar modulos "en caliente", etc.

Como me parece un "ejercicio educativo" interesante y dará bastante valor añadido al hilo, voy a mirar de crear una versión "mejorada" (según mi punto de vista, claro) y la subiré aquí mismo (bueno, el enlace al github).

[hernandemczuk]

Gracias por las observaciones. Les cuento que trabajo principalmente (por no decir solamente) como admin de bases de datos y recién ahora estoy volviendo a tocar la parte de desarrollo que hace mucho tiempo no veía y había dejado atrás.

Voy a tratar de implementar cada uno de los puntos que recomendaron, si bien muchos tendré que investigar al respecto.

Gracias y saludos!

[hernandemczuk]

Iniciado por uagrm

Ante nada, felicidades, muy pocos comparten su código, aún no lo probé el sistema, pero debe de funcionar...
Estoy viendo tu código de la clase DEmpleados, DCategorias y DProductos (los demás no los vi aún) pero debo hacerte algunas observaciones, en estas clases tienes métodos como Insertar, Eliminar, Editar, Mostrar y Buscar, a única diferencia entre tus métodos son los procedimientos almacenados por que accedes a través de ellos a las diferentes tablas.

Entoces, te recomiendo para una próxima versión de tu sistema, puedas crearte una clase genérica que te permita Insertar, Eliminar, Editar, Mostrar y Buscar. Cuando necesites Insertar un empleado, categoría o producto..utilizas el método Insertar de tu clase genérica. Así no vas a repetir tu código.

Es recomendable que las validaciones las hagas en el procedimiento almacenado por ejemplo: Respuesta = "Ya existe un producto con ese Nombre"; este mensaje debería venir de la BD. Supongamos que ya lo tienes instalado tu sistema, y al cliente no le gusta ese mensaje, lo que vas hacer es ir al código modificar el mensaje, volver a compilar y instalar (tardas mucho tiempo), si la BD te manda ese mensaje pues es solo vas al procedimiento almacenado cambias el mensaje y ejecutas nuevamente y listo..

Bueno, saludos
Espero que no lo tomes a mal mis observaciones.

Que tal, gracias por el feedback. Al contrario, estas observaciones estaba buscando!. Esa validación está también en la base de datos mediante constraints, sucede que para no tener que esperar una respuesta de la bd, cree otra validación en la capa presentación para que responda más rápido hacia el usuario.

Encontré varios comentarios acerca de la validaciones y muchos concuerdan en utilizar esta "doble verificación".

En cuanto a la clase genérica, me interesa mucho la idea, aunque me resulta un poco confuso como implementarlo. Investigaré más al respecto.

Gracias nuevamente

[uagrm]

La idea es tener métodos (insertar, actualizar, buscar..etc) que luego podas reutilizar en las clases que sea necesaria.

class ADO

Código PHP:

/*Este método permite registrar, funciona para todos los Insert*/
        public int Create(string stored, ArrayList sqlParametros)
        {
            SqlCommand commmand = new SqlCommand();
            SqlTransaction transaccion;
            Conectar();
            transaccion = conexion.BeginTransaction();
            commmand.Connection = conexion;
            commmand.Transaction = transaccion;
            commmand.CommandType = CommandType.StoredProcedure;
            commmand.CommandText = stored;
            SqlCommandBuilder.DeriveParameters(commmand);
            for (int i = 1; i <= commmand.Parameters.Count - 1; i++)
            {
                commmand.Parameters[i].Value = sqlParametros[i - 1];
            }
 
            string filasaceptadas = commmand.ExecuteScalar().ToString();
 
            if (Convert.ToInt32(filasaceptadas) != 0)
            {
                transaccion.Commit();
            }
            else
            {
                transaccion.Rollback();
            }
 
            Desconcestar();
            return Convert.ToInt32(filasaceptadas);
        }
 

Código PHP:

/*Insertar de tu class DCategorias*/
        public static int Insertar(String parNombre_Categoria, String parDescripcion)
        {
            int resultado = 0;
            ArrayList array = new ArrayList();
            array.Add(parNombre_Categoria);
            array.Add(String parDescripcion);
            resultado = ado.Create("Produccion.InsertarCategoria", array);
 
            return resultado;
        }
 

Código PHP:

/*Insertar de tu class NEmpleados*/
        public static int Insertar(String parNombre_Empleado, String parApellido_Empleado,
            String parDireccion, String parCiudad, String parRegion, String parPais, String parTelefono)
        {
            int resultado = 0;
            ArrayList array = new ArrayList();
            array.Add(parNombre_Empleado);
            array.Add(parApellido_Empleado);
            array.Add(parDireccion);
            array.Add(parCiudad);
            array.Add(parRegion);
            array.Add(parPais);
            array.Add(parTelefono);
            resultado = ado.Create("RecursosHumanos.InsertarEmpleado", array);
 
            return resultado;
        }