clean code + add interface

Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/AgenceController.cs

@@ -1,12 +1,11 @@
 using espacecollab.backend.appservices;
 using espacecollab.backend.appservices.dtos;
 
-namespace espacecollab.backend.api.Controllers
+namespace espacecollab.backend.api.Controllers;
+
+public class AgenceController : BaseController<AgenceApiDto>
 {
-    public class AgenceController : BaseController<AgenceApiDto>
+    public AgenceController(AgenceService agenceService) : base(agenceService)
     {
-        public AgenceController(AgenceService agenceService) : base(agenceService)
-        {
-        }
     }
-}
+}

Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/BaseController.cs

@@ -6,7 +6,7 @@ namespace espacecollab.backend.api.Controllers
 {
     [Route("api/[controller]")]
     [ApiController]
-    public abstract class BaseController<TO> : Controller where TO : class, IGenericIdApiDto
+    public abstract class BaseController<TO> : Controller, IBaseController<TO> where TO : class, IGenericIdApiDto
     {
         protected IGenericsServices<TO> Services { get; }
 

Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/BusinessUnitController.cs

@@ -1,13 +1,11 @@
 using espacecollab.backend.appservices;
 using espacecollab.backend.appservices.dtos;
-using espacecollab.backend.infrastructure.sql.dtos;
 
-namespace espacecollab.backend.api.Controllers
+namespace espacecollab.backend.api.Controllers;
+
+public class BusinessUnitController : BaseController<BusinessUnitApiDto>
 {
-    public class BusinessUnitController : BaseController<BusinessUnitApiDto>
+    public BusinessUnitController(BusinessUnitService businessUnitService) : base(businessUnitService)
     {
-        public BusinessUnitController(BusinessUnitService businessUnitService) : base(businessUnitService)
-        {
-        }
     }
-}
+}

Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/CollaborateursController.cs

@@ -2,47 +2,44 @@
 using espacecollab.backend.appservices.dtos;
 using Microsoft.AspNetCore.Mvc;
 
-namespace espacecollab.backend.api.Controllers
+namespace espacecollab.backend.api.Controllers;
+
+public class CollaborateursController : BaseController<CollaborateurApiDto>
 {
-    //[Route("api/[controller]")]
+    private CollaborateursService CollaborateursServices { get; }
-    //[ApiController]
+
-    public class CollaborateursController : BaseController<CollaborateurApiDto>
+    public CollaborateursController(CollaborateursService collaborateursServices) : base(collaborateursServices)
+    {
+        CollaborateursServices = collaborateursServices;
+    }
+
+    [HttpGet("businessunit/{businessUnitId:int:min(1)}")]
+    public ActionResult<IEnumerable<CollaborateurApiDto>> GetCollaborateursByBusinessUnit(uint businessUnitId)
+    {
+        IEnumerable<CollaborateurApiDto> collaborateurs = CollaborateursServices.GetCollaborateursByBusinessUnit(businessUnitId);
+        if (!collaborateurs.Any())
+            return NotFound();
+
+        return Ok(collaborateurs);
+    }
+
+    [HttpGet("referrer/{referrerId:int:min(1)}")]
+    public ActionResult<IEnumerable<CollaborateurApiDto>> GetCollaborateursByReferrer(uint referrerId)
     {
-        private CollaborateursService CollaborateursServices { get; }
+        IEnumerable<CollaborateurApiDto> collaborateurs = CollaborateursServices.GetCollaborateursByReferrer(referrerId);
-
+        if (!collaborateurs.Any())
-        public CollaborateursController(CollaborateursService collaborateursServices) : base(collaborateursServices)
+            return NotFound();
-        {
+
-            CollaborateursServices = collaborateursServices;
+        return Ok(collaborateurs);
-        }
+    }
-
+
-        [HttpGet("businessunit/{businessUnitId:int:min(1)}")]
+    [HttpGet("apsidemail/{apsideMail:minlength(1):regex(^\\S.*)}")]
-        public ActionResult<IEnumerable<CollaborateurApiDto>> GetCollaborateursByBusinessUnit(uint businessUnitId)
+    public ActionResult<CollaborateurApiDto> GetCollaborateurByApsideMail(string apsideMail)
-        {
+    {
-            IEnumerable<CollaborateurApiDto> collaborateurs = CollaborateursServices.GetCollaborateursByBusinessUnit(businessUnitId);
+        CollaborateurApiDto? collaborateur = CollaborateursServices.GetCollaborateurByApsideMail(apsideMail);
-            if (!collaborateurs.Any())
+        if (collaborateur == null)
-                return NotFound();
+            return NotFound();
-
+
-            return Ok(collaborateurs);
+        return Ok(collaborateur);
-        }
-
-        [HttpGet("referrer/{referrerId:int:min(1)}")]
-        public ActionResult<IEnumerable<CollaborateurApiDto>> GetCollaborateursByReferrer(uint referrerId)
-        {
-            IEnumerable<CollaborateurApiDto> collaborateurs = CollaborateursServices.GetCollaborateursByReferrer(referrerId);
-            if (!collaborateurs.Any())
-                return NotFound();
-
-            return Ok(collaborateurs);
-        }
-
-        [HttpGet("apsidemail/{apsideMail:minlength(1):regex(^\\S.*)}")]
-        public ActionResult<CollaborateurApiDto> GetCollaborateurByApsideMail(string apsideMail)
-        {
-            CollaborateurApiDto? collaborateur = CollaborateursServices.GetCollaborateurByApsideMail(apsideMail);
-            if (collaborateur == null)
-                return NotFound();
-
-            return Ok(collaborateur);
-        }
     }
-}
+}

Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/FonctionController.cs

@@ -1,12 +1,11 @@
 using espacecollab.backend.appservices;
 using espacecollab.backend.appservices.dtos;
 
-namespace espacecollab.backend.api.Controllers
+namespace espacecollab.backend.api.Controllers;
+
+public class FonctionController : BaseController<FonctionApiDto>
 {
-    public class FonctionController : BaseController<FonctionApiDto>
+    public FonctionController(FonctionService fonctionService) : base(fonctionService)
     {
-        public FonctionController(FonctionService fonctionService) : base(fonctionService)
-        {
-        }
     }
-}
+}

Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/IBaseController.cs

@@ -0,0 +1,16 @@
+using Microsoft.AspNetCore.Mvc;
+
+namespace espacecollab.backend.api.Controllers;
+
+public interface IBaseController<TO>
+{
+    ActionResult<IEnumerable<TO>> GetAll();
+
+    ActionResult<TO> GetById(uint id);
+
+    ActionResult<TO> Add(TO apiDto);
+
+    ActionResult Delete(uint id);
+
+    ActionResult<TO> Update(uint id, TO apiDto);
+}