refactor: Cambiada la estructura de las controladoras

Controladora/ControladoraBase.cs

@@ -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);
-    }
-}

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();
         }

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();
         }

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();
         }

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";
 
@@ -15,7 +15,7 @@ namespace Controladora
                 $"Fallo la carga del OrdenDeCompra {t.Id}"; 
         }
 
-        override public string Eliminar(OrdenDeCompra t)
+        public string Eliminar(OrdenDeCompra t)
         {
            if (t == null) return "El OrdenDeCompra es nulo fallo la carga";
 
@@ -24,7 +24,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";
 
@@ -33,7 +33,7 @@ namespace Controladora
                 $"Fallo la Modificacion del OrdenDeCompra {t.Id}";    
         }
 
-        public override ReadOnlyCollection<OrdenDeCompra> Listar()
+        public ReadOnlyCollection<OrdenDeCompra> Listar()
         {
             return RepositorioOrdenDeCompra.Instance.Listar();
         }

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();
         }

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();
         }

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();
         }

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();
         }

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";
 

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;
+            }   
+        }
+
+    }
+}