From 5673d60e4fb2ee4c0299ccae6fbdc345044aba58 Mon Sep 17 00:00:00 2001
From: fede <federico.nicolas.polidoro@gmail.com>
Date: Sun, 11 Aug 2024 18:21:15 -0300
Subject: [PATCH] refactor: Cambiada la estructura de las controladoras

---
 Controladora/ControladoraBase.cs              | 35 -------------------
 Controladora/ControladoraCategorias.cs        | 10 +++---
 Controladora/ControladoraClientes.cs          | 10 +++---
 Controladora/ControladoraFacturas.cs          | 10 +++---
 Controladora/ControladoraOrdenesDeCompra.cs   | 12 ++++---
 Controladora/ControladoraPedidoPresupuesto.cs | 10 +++---
 Controladora/ControladoraPresupuestos.cs      | 10 +++---
 Controladora/ControladoraProductos.cs         | 10 +++---
 Controladora/ControladoraProveedores.cs       | 16 +++------
 Controladora/ControladoraRemito.cs            | 10 +++---
 Controladora/Singleton.cs                     | 20 +++++++++++
 11 files changed, 67 insertions(+), 86 deletions(-)
 delete mode 100644 Controladora/ControladoraBase.cs
 create mode 100644 Controladora/Singleton.cs

diff --git a/Controladora/ControladoraBase.cs b/Controladora/ControladoraBase.cs
deleted file mode 100644
index 92fd9f2..0000000
--- a/Controladora/ControladoraBase.cs
+++ /dev/null
@@ -1,35 +0,0 @@
-using System.Collections.ObjectModel;
-using System.Runtime.InteropServices;
-using Modelo;
-
-namespace Controladora
-{
-    public abstract class ControladoraBase<T /*Tipo de Dato*/ ,
-                                           J /*Singleton*/>
-        where J : new()
-    {
-
-
-        // Singleton thread-safe por si quiero usar "Parallel"
-        private static J instance = new J();
-        public static J Instance
-        {
-            get
-            {                
-                return instance;
-            }   
-        }
-
-        // Lista los contenidos del repositorio
-        abstract public ReadOnlyCollection<T> Listar();
-
-        // Recibe mensajes de la Vista para dar de alta
-        abstract public string Añadir(T t);
-
-        // Recibe mensajes para modificar
-        abstract public string Modificar(T t);
-
-        // Recibe mensajes para eliminar
-        abstract public string Eliminar(T t);
-    }
-}
\ No newline at end of file
diff --git a/Controladora/ControladoraCategorias.cs b/Controladora/ControladoraCategorias.cs
index 6ed2ff6..80f0125 100644
--- a/Controladora/ControladoraCategorias.cs
+++ b/Controladora/ControladoraCategorias.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraCategorias : ControladoraBase<Categoria, ControladoraCategorias>
+    public class ControladoraCategorias : Singleton<ControladoraCategorias>
     {
-        public override string Añadir(Categoria t)
+        public string Añadir(Categoria t)
         {
             if (t == null) return "El Categoria es nulo fallo la carga";
 
@@ -15,7 +15,7 @@ namespace Controladora
                 $"Fallo la carga del Categoria {t.Descripcion}"; 
         }
 
-        override public string Eliminar(Categoria t)
+        public string Eliminar(Categoria t)
         {
            if (t == null) return "El Categoria es nulo fallo la carga";
 
@@ -24,7 +24,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del Categoria {t.Descripcion}";         
         }
 
-        override public string Modificar(Categoria t)
+        public     string Modificar(Categoria t)
         {
             if (t == null) return "El Categoria es nulo fallo la carga";
 
@@ -33,7 +33,7 @@ namespace Controladora
                 $"Fallo la Modificacion del Categoria {t.Descripcion}";    
         }
 
-        public override ReadOnlyCollection<Categoria> Listar()
+        public ReadOnlyCollection<Categoria> Listar()
         {
             return RepositorioCategoria.Instance.Listar();
         }
diff --git a/Controladora/ControladoraClientes.cs b/Controladora/ControladoraClientes.cs
index a786840..925c5ea 100644
--- a/Controladora/ControladoraClientes.cs
+++ b/Controladora/ControladoraClientes.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraClientes : ControladoraBase<Cliente, ControladoraClientes>
+    public class ControladoraClientes : Singleton<ControladoraClientes>
     {
-        public override string Añadir(Cliente t)
+        public   string Añadir(Cliente t)
         {
             if (t == null) return "El Cliente es nulo fallo la carga";
 
@@ -15,7 +15,7 @@ namespace Controladora
                 $"Fallo la carga del Cliente {t.Nombre}"; 
         }
 
-        override public string Eliminar(Cliente t)
+        public string Eliminar(Cliente t)
         {
            if (t == null) return "El Cliente es nulo fallo la carga";
 
@@ -24,7 +24,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del Cliente {t.Nombre}";         
         }
 
-        override public string Modificar(Cliente t)
+        public string Modificar(Cliente t)
         {
             if (t == null) return "El Cliente es nulo fallo la carga";
 
@@ -33,7 +33,7 @@ namespace Controladora
                 $"Fallo la Modificacion del Cliente {t.Nombre}";    
         }
 
-        public override ReadOnlyCollection<Cliente> Listar()
+        public ReadOnlyCollection<Cliente> Listar()
         {
             return RepositorioClientes.Instance.Listar();
         }
diff --git a/Controladora/ControladoraFacturas.cs b/Controladora/ControladoraFacturas.cs
index 1936dba..74bd2fe 100644
--- a/Controladora/ControladoraFacturas.cs
+++ b/Controladora/ControladoraFacturas.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraFacturas : ControladoraBase<Factura, ControladoraFacturas>
+    public class ControladoraFacturas : Singleton<ControladoraFacturas>
     {
-        public override string Añadir(Factura t)
+        public string Añadir(Factura t)
         {
             if (t == null) return "El Factura es nulo fallo la carga";
 
@@ -15,7 +15,7 @@ namespace Controladora
                 $"Fallo la carga del Factura {t.Id}"; 
         }
 
-        override public string Eliminar(Factura t)
+        public string Eliminar(Factura t)
         {
            if (t == null) return "El Factura es nulo fallo la carga";
 
@@ -24,7 +24,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del Factura {t.Id}";         
         }
 
-        override public string Modificar(Factura t)
+        public string Modificar(Factura t)
         {
             if (t == null) return "El Factura es nulo fallo la carga";
 
@@ -33,7 +33,7 @@ namespace Controladora
                 $"Fallo la Modificacion del Factura {t.Id}";    
         }
 
-        public override ReadOnlyCollection<Factura> Listar()
+        public ReadOnlyCollection<Factura> Listar()
         {
             return RepositorioFactura.Instance.Listar();
         }
diff --git a/Controladora/ControladoraOrdenesDeCompra.cs b/Controladora/ControladoraOrdenesDeCompra.cs
index c006891..f652140 100644
--- a/Controladora/ControladoraOrdenesDeCompra.cs
+++ b/Controladora/ControladoraOrdenesDeCompra.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraOrdenDeCompras : ControladoraBase<OrdenDeCompra, ControladoraOrdenDeCompras>
+    public class ControladoraOrdenDeCompras : Singleton<ControladoraOrdenDeCompras>
     {
-        public override string Añadir(OrdenDeCompra t)
+        public string Añadir(OrdenDeCompra t)
         {
             if (t == null) return "El OrdenDeCompra es nulo fallo la carga";
             if (t.Proveedor == null) return "No se cargo el proveedor";
@@ -19,6 +19,7 @@ namespace Controladora
                 $"Fallo la carga del OrdenDeCompra {t.Id}"; 
         }
 
+<<<<<<< HEAD
         private bool ProductoCheck(ReadOnlyCollection<DetalleOrdenDeCompra> ldetalles)
         {
             bool ret = false;
@@ -32,6 +33,9 @@ namespace Controladora
             return ret;
         }
         override public string Eliminar(OrdenDeCompra t)
+=======
+        public string Eliminar(OrdenDeCompra t)
+>>>>>>> 675d86f (refactor: Cambiada la estructura de las controladoras)
         {
            if (t == null) return "El OrdenDeCompra es nulo fallo la carga";
 
@@ -40,7 +44,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del OrdenDeCompra {t.Id}";         
         }
 
-        override public string Modificar(OrdenDeCompra t)
+        public string Modificar(OrdenDeCompra t)
         {
             if (t == null) return "El OrdenDeCompra es nulo fallo la carga";
 
@@ -49,7 +53,7 @@ namespace Controladora
                 $"Fallo la Modificacion del OrdenDeCompra {t.Id}";    
         }
 
-        public override ReadOnlyCollection<OrdenDeCompra> Listar()
+        public ReadOnlyCollection<OrdenDeCompra> Listar()
         {
             return RepositorioOrdenDeCompra.Instance.Listar();
         }
diff --git a/Controladora/ControladoraPedidoPresupuesto.cs b/Controladora/ControladoraPedidoPresupuesto.cs
index 77af1b2..0d59bae 100644
--- a/Controladora/ControladoraPedidoPresupuesto.cs
+++ b/Controladora/ControladoraPedidoPresupuesto.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraPedidoDePresupuestos : ControladoraBase<PedidoDePresupuesto, ControladoraPedidoDePresupuestos>
+    public class ControladoraPedidoDePresupuestos : Singleton<ControladoraPedidoDePresupuestos>
     {
-        public override string Añadir(PedidoDePresupuesto t)
+        public string Añadir(PedidoDePresupuesto t)
         {
             if (t == null) return "El PedidoDePresupuesto es nulo fallo la carga";
 
@@ -15,7 +15,7 @@ namespace Controladora
                 $"Fallo la carga del PedidoDePresupuesto {t.Id}"; 
         }
 
-        override public string Eliminar(PedidoDePresupuesto t)
+        public string Eliminar(PedidoDePresupuesto t)
         {
            if (t == null) return "El PedidoDePresupuesto es nulo fallo la carga";
 
@@ -24,7 +24,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del PedidoDePresupuesto {t.Id}";         
         }
 
-        override public string Modificar(PedidoDePresupuesto t)
+        public string Modificar(PedidoDePresupuesto t)
         {
             if (t == null) return "El PedidoDePresupuesto es nulo fallo la carga";
 
@@ -33,7 +33,7 @@ namespace Controladora
                 $"Fallo la Modificacion del PedidoDePresupuesto {t.Id}";    
         }
 
-        public override ReadOnlyCollection<PedidoDePresupuesto> Listar()
+        public ReadOnlyCollection<PedidoDePresupuesto> Listar()
         {
             return RepositorioPedidoDePresupuesto.Instance.Listar();
         }
diff --git a/Controladora/ControladoraPresupuestos.cs b/Controladora/ControladoraPresupuestos.cs
index 2a43fdf..e5bff16 100644
--- a/Controladora/ControladoraPresupuestos.cs
+++ b/Controladora/ControladoraPresupuestos.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraPresupuestos : ControladoraBase<Presupuesto, ControladoraPresupuestos>
+    public class ControladoraPresupuestos : Singleton<ControladoraPresupuestos>
     {
-        public override string Añadir(Presupuesto t)
+        public string Añadir(Presupuesto t)
         {
             if (t == null) return "El Presupuesto es nulo fallo la carga";
 
@@ -15,7 +15,7 @@ namespace Controladora
                 $"Fallo la carga del Presupuesto {t.Id}"; 
         }
 
-        override public string Eliminar(Presupuesto t)
+        public string Eliminar(Presupuesto t)
         {
            if (t == null) return "El Presupuesto es nulo fallo la carga";
 
@@ -24,7 +24,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del Presupuesto {t.Id}";         
         }
 
-        override public string Modificar(Presupuesto t)
+        public string Modificar(Presupuesto t)
         {
             if (t == null) return "El Presupuesto es nulo fallo la carga";
 
@@ -33,7 +33,7 @@ namespace Controladora
                 $"Fallo la Modificacion del Presupuesto {t.Id}";    
         }
 
-        public override ReadOnlyCollection<Presupuesto> Listar()
+        public ReadOnlyCollection<Presupuesto> Listar()
         {
             return RepositorioPresupuesto.Instance.Listar();
         }
diff --git a/Controladora/ControladoraProductos.cs b/Controladora/ControladoraProductos.cs
index 1c399e7..87aea0a 100644
--- a/Controladora/ControladoraProductos.cs
+++ b/Controladora/ControladoraProductos.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraProductos : ControladoraBase<Producto, ControladoraProductos>
+    public class ControladoraProductos : Singleton<ControladoraProductos>
     {
-        public override string Añadir(Producto t)
+        public string Añadir(Producto t)
         {
             if (t == null) return "El Producto es nulo fallo la carga";
 
@@ -15,7 +15,7 @@ namespace Controladora
                 $"Fallo la carga del Producto {t.Nombre}"; 
         }
 
-        public override string Eliminar(Producto t)
+        public string Eliminar(Producto t)
         {
            if (t == null) return "El Producto es nulo fallo la carga";
 
@@ -24,7 +24,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del Producto {t.Nombre}";         
         }
 
-        public override string Modificar(Producto t)
+        public string Modificar(Producto t)
         {
             if (t == null) return "El Producto es nulo fallo la carga";
 
@@ -33,7 +33,7 @@ namespace Controladora
                 $"Fallo la Modificacion del Producto {t.Nombre}";    
         }
 
-        public override ReadOnlyCollection<Producto> Listar()
+        public ReadOnlyCollection<Producto> Listar()
         {
             return RepositorioProductos.Instance.Listar();
         }
diff --git a/Controladora/ControladoraProveedores.cs b/Controladora/ControladoraProveedores.cs
index 2b91c42..a5df5c0 100644
--- a/Controladora/ControladoraProveedores.cs
+++ b/Controladora/ControladoraProveedores.cs
@@ -4,9 +4,9 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraProveedores : ControladoraBase<Proveedor, ControladoraProveedores>
+    public class ControladoraProveedores : Singleton<ControladoraProveedores>
     {
-        public override string Añadir(Proveedor t)
+        public string Añadir(Proveedor t)
         {
             if (t == null) return "El Proveedor es nulo fallo la carga";
 
@@ -15,14 +15,6 @@ namespace Controladora
                 $"Fallo la carga del Proveedor {t.Nombre}"; 
         }
 
-        public override string Eliminar(Proveedor t)
-        {
-           if (t == null) return "El Proveedor es nulo fallo la baja";
-
-            return (RepositorioProveedor.Instance.Del(t)) ? 
-                $"El Proveedor {t.Nombre} se Elimino correctamente":
-                $"Fallo la Eliminacion del Proveedor {t.Nombre}";         
-        }
         public string Eliminar(long t)
         {
             var proveedor = RepositorioProveedor.Instance.Listar().First(x => x.Cuit == t);
@@ -34,7 +26,7 @@ namespace Controladora
                 $"Fallo la Eliminacion del Proveedor {t}";
         }
 
-        public override string Modificar(Proveedor t)
+        public string Modificar(Proveedor t)
         {
             if (t == null) return "El Proveedor es nulo fallo la modificacion";
 
@@ -43,7 +35,7 @@ namespace Controladora
                 $"Fallo la Modificacion del Proveedor {t.Nombre}";    
         }
 
-        public override ReadOnlyCollection<Proveedor> Listar()
+        public ReadOnlyCollection<Proveedor> Listar()
         {
             return RepositorioProveedor.Instance.Listar();
         }
diff --git a/Controladora/ControladoraRemito.cs b/Controladora/ControladoraRemito.cs
index 5acca08..9341725 100644
--- a/Controladora/ControladoraRemito.cs
+++ b/Controladora/ControladoraRemito.cs
@@ -4,14 +4,14 @@ using Modelo;
 
 namespace Controladora
 {
-    public class ControladoraRemito : ControladoraBase<Remito, ControladoraRemito>
+    public class ControladoraRemito : Singleton<ControladoraRemito>
     {
-        public override ReadOnlyCollection<Remito> Listar()
+        public ReadOnlyCollection<Remito> Listar()
         {
             return RepositorioRemito.Instance.Listar();
         }
 
-        override public string Añadir(Remito t)
+        public string Añadir(Remito t)
         {
             if (t == null) return "El Remito es nulo fallo la carga";
 
@@ -20,7 +20,7 @@ namespace Controladora
                 $"Fallo la carga del remito {t.Id}"; 
 
         }
-        override public string Modificar(Remito t)
+        public string Modificar(Remito t)
         {
             if (t == null) return "El Remito es nulo fallo la carga";
 
@@ -29,7 +29,7 @@ namespace Controladora
                 $"Fallo la carga del remito {t.Id}"; 
 
         }
-        override public string Eliminar(Remito t)
+        public string Eliminar(Remito t)
         {
             if (t == null) return "El Remito es nulo fallo la carga";
 
diff --git a/Controladora/Singleton.cs b/Controladora/Singleton.cs
new file mode 100644
index 0000000..8b25314
--- /dev/null
+++ b/Controladora/Singleton.cs
@@ -0,0 +1,20 @@
+using System.Collections.ObjectModel;
+using System.Runtime.InteropServices;
+using Modelo;
+
+namespace Controladora
+{
+    public abstract class Singleton<T> where T : new()
+    {
+        // Singleton thread-safe por si quiero usar "Parallel"
+        private static T instance = new T();
+        public static T Instance
+        {
+            get
+            {                
+                return instance;
+            }   
+        }
+
+    }
+}
\ No newline at end of file