Les erreurs sont indiquées en TODO. A corriger puis supprimer les commentaires

Webzine.WebApplication/Areas/Administration/Controllers/CommentaireController.cs

@@ -22,7 +22,8 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
 
             _logger.LogInformation("Initialisation du contrôleur CommentaireController.");
 
-            var factory = new DataFactory();
+            var factory = new DataFactory(); // TODO injecter le factory via DI pour éviter de le recréer à chaque fois
+            // faire une classe statique
 
             var _artistes = factory.GenerateArtists(10);
             var _styles = factory.GenerateStyles(10);
@@ -84,13 +85,14 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
         /// </summary>
         /// <param name="id">L'identifiant du commentaire à supprimer.</param>
         /// <returns>La vue de confirmation de suppression avec le ViewModel contenant les détails du commentaire à supprimer, ou une réponse NotFound si le commentaire n'existe pas.</returns>
-        public ActionResult Delete(int id)
+        public ActionResult Delete(int id) // TODO IActionResult
         {
             var commentaire = _commentaires
                 .FirstOrDefault(c => c.IdCommentaire == id);
 
             if (commentaire == null)
-                return NotFound();
+                return NotFound(); // TODO faire une page d'erreur personnalisée pour les 404, et pas juste un message d'erreur dans la console
+            // mettre des accolades dans les if pour éviter les erreurs de maintenance, même pour une seule ligne de code
 
             var vm = new CommentaireDeleteViewModel
             {
@@ -111,12 +113,13 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
         /// <param name="model">Le ViewModel contenant les détails du commentaire à supprimer, utilisé pour afficher les informations en cas d'erreur.</param>
         /// <returns>Redirection vers la liste des commentaires en cas de succès, ou la vue de confirmation avec le message d'erreur en cas d'échec.</returns>
         [HttpPost]
-        [ValidateAntiForgeryToken]
-        public ActionResult Delete(int id, CommentaireDeleteViewModel model)
+        [ValidateAntiForgeryToken] // TODO pas utilisée encore, à virer. tag helper form le fait déjà auto
+        public ActionResult Delete(int id, CommentaireDeleteViewModel model) //TODO IActionResult
         {
+            // TODO ça boucle à l'infini si on supprime pas le commentaire, parce que la vue Delete.cshtml affiche les détails du commentaire à supprimer, et que le commentaire n'est pas supprimé en base de données, donc il réaffiche la même page avec les mêmes données, et ça boucle à l'infini
             try
             {
-                return RedirectToAction();
+                return RedirectToAction(); // TODO rediriger vers la liste des commentaires, et pas juste RedirectToAction() qui va faire une erreur
             }
             catch (Exception e)
             {

Webzine.WebApplication/Areas/Administration/Controllers/StyleController.cs

@@ -36,7 +36,7 @@ namespace Webzine.WebApplication.Areas.Administration.Controllers
         public ActionResult Index()
         {
             // Création de données "bouchon" (mock) pour tester l'affichage
-            var listeStyles = new List<Style>
+            var listeStyles = new List<Style> // TODO remplacer par les données générées dans le constructeur (la Factory)
             {
                 new Style
                 {

Webzine.WebApplication/Areas/Administration/Controllers/TitreController.cs

@@ -90,7 +90,7 @@ public class TitreController : Controller
     /// <returns>Redirige vers l'action Index après la soumission du formulaire. En cas d'erreur, retourne la vue Create pour permettre à l'utilisateur de corriger les données.</returns>
     [HttpPost]
     [ValidateAntiForgeryToken]
-    public ActionResult Create(IFormCollection collection)
+    public ActionResult Create(IFormCollection collection) // TODO IFormCollection c'est quoi
     {
         try
         {

Webzine.WebApplication/Areas/Administration/ViewModels/Artiste/GroupeArtisteViewModel.cs

@@ -4,7 +4,7 @@
     /// <summary>
     /// ViewModel pour afficher un groupe d'artiste.
     /// </summary>
-    public class GroupeArtisteViewModel
+    public class GroupeArtisteViewModel // TODO groupe view model pas bien, pas pertinent
     {
         /// <summary>
         /// Liste d'artistes.

Webzine.WebApplication/Areas/Administration/Views/Commentaire/Delete.cshtml

@@ -16,9 +16,9 @@
     <div class="mb-4">
         <h4>@Model.Contenu</h4>
 
-        <div class="text-muted">
+        <div class="text-muted"> // TODO y a des balises pour les citations, <blockquote></blockquote>
             — <strong>@Model.Auteur</strong>
-            le @Model.DateCreation.ToString("dd/MM/yyyy HH:mm:ss")
+            le @Model.DateCreation.ToString("dd/MM/yyyy HH:mm:ss") // TODO à virer, c'est pas très lisible, trouver un format de date plus sympa, ou même afficher "il y a X minutes/heures/jours" comme sur les réseaux sociaux
             sur <em>@Model.TitreLibelle</em>
         </div>
     </div>

Webzine.WebApplication/Areas/Administration/Views/Dashboard/Index.cshtml

@@ -29,17 +29,17 @@
             </a>
         </div>
 
-        <!-- L'ARTIST LE PLUS CHRONICLED -->
+        <!-- L'ARTIST LE PLUS CHRONICLED --> // TODO faute de frappe, à virer
         <div class="col-md-4">
             <a asp-area=""
                asp-controller="Artiste"
-               asp-route-nom="@Model.MostChronicledArtistName"
+               asp-route-nom="@Model.MostChronicledArtistName" // TODO enlever texte décorationnel, pas maintenable 
                class="text-decoration-none">
 
                 <div class="card shadow-sm p-4 bg-light h-100 dashboard-card">
                     <i class="fa fa-user fa-3x text-primary mb-3"></i>
 
-                    <h3 class="text-primary">
+                    <h3 class="text-primary"> // TODO c'est déjà par défaut en couleur primaire, pas besoin de le redéfinir
                         @Model.MostChronicledArtistName
                     </h3>
 

Webzine.WebApplication/Areas/Administration/Views/Style/Delete.cshtml

@@ -14,7 +14,7 @@
     </p>
 
     <div class="mb-4">
-        @* On affiche le Libellé en gros *@
+        @* On affiche le Libellé en gros *@ // TODO c'est quoi ces commentaires Baptiste
         <h4>@Model.Libelle</h4>
 
         @* On affiche l'ID discrètement en dessous *@

Webzine.WebApplication/Controllers/TitreController.cs

@@ -100,14 +100,17 @@ public class TitreController : Controller
     [HttpPost("like")]
     public IActionResult Like(TitreLike model)
     {
-        _logger.LogInformation("Ajout d'un like pour le titre ID {Id}.", model.IdTitre);
+        this._logger.LogInformation("Ajout d'un like pour le titre ID {Id}.", model.IdTitre);
 
-        var titre = FindById(model.IdTitre);
+        var titre = FindById(model.IdTitre); //TODO appeler directement le repository pour éviter une requête supplémentaire
 
+        // TODO faut mettre des autres cas d'erreur, ça va faire un 404
         if (titre == null)
         {
             _logger.LogWarning("Impossible d'ajouter un like. Titre ID {Id} introuvable.", model.IdTitre);
-            return NotFound();
+            return NotFound(); // TODO va pas faire une page d'erreur personnalisée, ça va faire un 404
+            // faudrait faire une page d'erreur personnalisée pour les 404, et pas juste un message d'erreur dans la console
+            // ou rediriger vers page accueil si le titre est pas trouvé, ça serait plus user-friendly
         }
 
         titre.NbLikes++;
@@ -152,7 +155,7 @@ public class TitreController : Controller
         return RedirectToAction("Details", new { id = model.IdTitre });
     }
 
-    private Titre? FindById(int id)
+    private Titre? FindById(int id) // TODO virer ça vu que ça appelle directement le repository
     {
         return _titreRepository.Find(id);
     }

Webzine.WebApplication/ViewModels/Artiste/ArtisteModel.cs

@@ -5,7 +5,7 @@ namespace Webzine.WebApplication.ViewModels.Artiste
     /// <summary>
     /// ViewModel pour afficher les détails d'un artiste, incluant les informations de l'artiste et la liste de ses titres.
     /// </summary>
-    public class ArtisteModel
+    public class ArtisteModel // TODO viewmodel sert à rien, c'est juste un artiste et une liste de titres, on peut faire ça directement dans la vue non ?
     {
         /// <summary>
         /// Artiste dont on affiche les détails.
@@ -15,6 +15,6 @@ namespace Webzine.WebApplication.ViewModels.Artiste
         /// <summary>
         /// Liste des titres de l'artiste.
         /// </summary>
-        public List<Entity.Titre> Titres { get; set; }
+        public List<Entity.Titre> Titres { get; set; } 
     }
 }

Webzine.WebApplication/Views/Accueil/Index.cshtml

@@ -5,7 +5,7 @@
 
 <h1>Derniers titres chroniqués</h1>
 
-@* TEMPLATE  *@
+@* TEMPLATE TODO: virer *@
 @* <div class="container">
     <div class="container bg-light row p-3 mt-3">
         <div class="col-auto">
@@ -41,7 +41,7 @@
         <div class="container bg-light row p-3 mt-3">
             <div class="col-auto">
                 <img class="img-thumbnail img-fluid"
-                     style="max-width:200px;"
+                     style="max-width:200px;" @* TODO à virer pour que ça soit responsive *@
                      @* UrlJaquette *@
                      src="@titre.UrlJaquette" />
             </div>

Webzine.WebApplication/Views/Recherche/Index.cshtml

@@ -2,7 +2,7 @@
 
 @{
     ViewData["Title"] = "Recherche";
-    Layout = "_Layout";
+    Layout = "_Layout"; //TODO à virer c'est déjà dans le _ViewStart
 }
 
 <div class="container mt-4">

Webzine.WebApplication/Views/Shared/_Layout.cshtml

@@ -6,12 +6,12 @@
     <title>@ViewData["Title"] - Webzine</title>
 
     @* Ajout de bootstrap *@
-    <script src="~/js/bootstrap.min.js" defer></script>
-    <script src="~/js/bootstrap.bundle.js" defer></script>
-    <link rel="stylesheet" href="~/css/app.css">
+    <script src="~/js/bootstrap.min.js" defer></script> //TODO virer le bootstrap.bundle.js qui est en double
+    <script src="~/js/bootstrap.bundle.js" defer></script> //TODO script pas bien placé
+    <link rel="stylesheet" href="~/css/app.css"> // TODO mettre le app.css après le bootstrap pour pouvoir override les styles de bootstrap si besoin
     <link rel="stylesheet" href="~/css/bootstrap.min.css">
 
-    @* Ajout de font-awesome *@
+    @* Ajout de font-awesome, TODO cdn à virer, mettre font awesome dans le projet *@
     <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.5.1/css/all.min.css">
 </head>
 <body>

Webzine.WebApplication/Views/Titre/Style.cshtml

@@ -2,7 +2,7 @@
 
 @{
     ViewData["Title"] = $"Titres - {Model.StyleName}";
-    Layout = "_Layout";
+    Layout = "_Layout"; //TODO à virer c'est déjà dans le _ViewStart
 }
 <div class="container mt-4">
     <div class="row">