lmasi/webzine
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: amélioration de la journalisation et la gestion des exceptions dans les repositories de styles et titres

Browse Source
This commit is contained in:
mirage
2026-03-26 11:29:53 +01:00
parent 4deb7af6f0
commit 2e9373a853
7 changed files with 738 additions and 385 deletions
Download Patch File Download Diff File Expand all files Collapse all files

295
Webzine.Repository/LocalTitreRepository.cs
View File

@@ -21,169 +21,254 @@ public class LocalTitreRepository : ITitreRepository
{
this.logger = logger;
this.titres = titres;
this.logger.LogDebug(1, "NLog injected into LocalTitreRepository");
this.logger.LogDebug(1, "NLog injecté dans LocalTitreRepository");
}
/// <summary>
/// Ajoute un titre à la liste des titres.
/// </summary>
/// <param name="titre">L'objet titre à ajouter.</param>
/// <inheritdoc/>
public void Add(Titre titre)
{
this.titres.Add(titre);
try
{
this.logger.LogDebug("Ajout du titre: {Libelle}", titre.Libelle);
this.titres.Add(titre);
this.logger.LogDebug("Titre ajouté avec succès, ID: {IdTitre}", titre.IdTitre);
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de l'ajout du titre: {Libelle}", titre.Libelle);
throw;
}
}
/// <summary>
/// Remonte le nombre de titres.
/// </summary>
/// <returns>Le nombre total de titres présents dans la liste après l'incrémentation du nombre de lectures.</returns>
/// <inheritdoc/>
public int Count()
{
return this.titres.Count;
try
{
var count = this.titres.Count;
this.logger.LogDebug("Nombre total de titres: {Count}", count);
return count;
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors du comptage des titres");
throw;
}
}
/// <summary>
/// Supprime un titre de la liste des titres.
/// </summary>
/// <param name="titre">L'objet titre à supprimer.</param>
/// <inheritdoc/>
public void Delete(Titre titre)
{
if (!this.titres.Contains(titre))
try
{
this.logger.LogWarning("Le titre avec l'identifiant {IdTitre} n'existe pas dans la liste.", titre.IdTitre);
}
this.logger.LogDebug("Suppression du titre ID: {IdTitre}", titre.IdTitre);
this.titres.Remove(titre);
if (!this.titres.Contains(titre))
{
this.logger.LogWarning("Le titre avec l'identifiant {IdTitre} n'existe pas dans la liste.", titre.IdTitre);
}
this.titres.Remove(titre);
this.logger.LogDebug("Titre supprimé avec succès, ID: {IdTitre}", titre.IdTitre);
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de la suppression du titre ID: {IdTitre}", titre.IdTitre);
throw;
}
}
/// <summary>
/// Recherche les titres dans la liste des titres en fonction de l'offset et de la limite spécifiés, permettant ainsi une pagination des résultats.
/// </summary>
/// <param name="offset">Le nombre de titres à ignorer avant de commencer à retourner les résultats.</param>
/// <param name="limit">Le nombre maximum de titres à retourner.</param>
/// <returns>Une collection de titres correspondant au critère de pagination, triée par libellé.</returns>
/// <inheritdoc/>
public IEnumerable<Titre> FindTitres(int offset, int limit)
{
if (offset < 0 || limit <= 0)
try
{
this.logger.LogWarning("FindTitres called with invalid offset {Offset} or limit {Limit}.", offset, limit);
return Enumerable.Empty<Titre>();
}
this.logger.LogDebug("Recherche des titres avec offset: {Offset}, limit: {Limit}", offset, limit);
return this.titres.Skip(offset).Take(limit).ToList();
if (offset < 0 || limit <= 0)
{
this.logger.LogWarning("FindTitres appelé avec offset {Offset} ou limit {Limit} invalide.", offset, limit);
return Enumerable.Empty<Titre>();
}
var result = this.titres.Skip(offset).Take(limit).ToList();
this.logger.LogDebug("{Count} titres trouvés", result.Count);
return result;
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de la recherche des titres avec offset: {Offset}, limit: {Limit}", offset, limit);
throw;
}
}
/// <summary>
/// Incrémente le nombre de lectures d'un titre donné. Si le titre est null, un message d'avertissement est enregistré dans les logs et aucune action n'est effectuée.
/// </summary>
/// <param name="titre">L'objet titre dont le nombre de lectures doit être incrémenté.</param>
/// <inheritdoc/>
public void IncrementNbLectures(Titre titre)
{
titre.NbLectures++;
try
{
this.logger.LogDebug("Incrémentation du nombre de lectures pour le titre ID: {IdTitre}", titre.IdTitre);
titre.NbLectures++;
this.logger.LogDebug("Nouveau nombre de lectures: {NbLectures}", titre.NbLectures);
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de l'incrémentation des lectures pour le titre ID: {IdTitre}", titre.IdTitre);
throw;
}
}
/// <summary>
/// Incrémente le nombre de likes d'un titre donné. Si le titre est null, un message d'avertissement est enregistré dans les logs et aucune action n'est effectuée.
/// </summary>
/// <param name="titre">L'objet titre dont le nombre de likes doit être incrémenté.</param>
/// <inheritdoc/>
public void IncrementNbLikes(Titre titre)
{
titre.NbLikes++;
try
{
this.logger.LogDebug("Incrémentation du nombre de likes pour le titre ID: {IdTitre}", titre.IdTitre);
titre.NbLikes++;
this.logger.LogDebug("Nouveau nombre de likes: {NbLikes}", titre.NbLikes);
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de l'incrémentation des likes pour le titre ID: {IdTitre}", titre.IdTitre);
throw;
}
}
/// <summary>
/// Recherche les titres dont le libellé contient le mot spécifié, en ignorant la casse.
/// </summary>
/// <param name="mot">Le mot à rechercher dans les libellés des titres.</param>
/// <returns>Une collection de titres correspondant au critère de recherche, triée par libellé.</returns>
/// <inheritdoc/>
public IEnumerable<Titre> Search(string mot)
{
if (string.IsNullOrWhiteSpace(mot))
try
{
this.logger.LogWarning("Search called with an empty or whitespace string.");
return Enumerable.Empty<Titre>();
this.logger.LogDebug("Recherche de titres avec le mot-clé: {Mot}", mot);
if (string.IsNullOrWhiteSpace(mot))
{
this.logger.LogWarning("Search appelé avec une chaîne vide ou contenant uniquement des espaces.");
return Enumerable.Empty<Titre>();
}
IEnumerable<Titre> list = this.titres
.Where(t => !string.IsNullOrWhiteSpace(t.Libelle)
&& t.Libelle.Contains(mot, StringComparison.OrdinalIgnoreCase))
.OrderBy(t => t.Libelle)
.ToList();
if (!list.Any())
{
this.logger.LogInformation("Aucun titre trouvé correspondant au terme de recherche '{Mot}'.", mot);
}
else
{
this.logger.LogDebug("{Count} titres trouvés pour le mot-clé '{Mot}'", list.Count(), mot);
}
return list;
}
IEnumerable<Titre> list = this.titres
.Where(t => !string.IsNullOrWhiteSpace(t.Libelle)
&& t.Libelle.Contains(mot, StringComparison.OrdinalIgnoreCase))
.OrderBy(t => t.Libelle)
.ToList();
if (!list.Any())
catch (Exception ex)
{
this.logger.LogInformation("No titres found matching the search term '{Mot}'.", mot);
this.logger.LogError(ex, "Erreur lors de la recherche des titres avec le mot-clé: {Mot}", mot);
throw;
}
return list;
}
/// <summary>
/// Trouve un titre dans la liste des titres en fonction de son identifiant.
/// </summary>
/// <param name="idTitre">L'identifiant du titre à trouver.</param>
/// <returns>Le titre correspondant à l'identifiant fourni, ou null si aucun titre n'est trouvé.</returns>
public Titre? Find(int idTitre)
/// <inheritdoc/>
public Titre Find(int idTitre)
{
Titre? titre = titres.FirstOrDefault(t => t.IdTitre == idTitre);
if (titre == null)
try
{
this.logger.LogInformation("No titre found with IdTitre {IdTitre}.", idTitre);
this.logger.LogDebug("Recherche du titre avec ID: {IdTitre}", idTitre);
Titre titre = this.titres.First(t => t.IdTitre == idTitre);
this.logger.LogDebug("Titre trouvé: {Libelle}", titre.Libelle);
return titre;
}
catch (InvalidOperationException ex)
{
this.logger.LogWarning(ex, "Aucun titre trouvé avec l'ID: {IdTitre}", idTitre);
throw;
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de la recherche du titre avec ID: {IdTitre}", idTitre);
throw;
}
return titre;
}
/// <summary>
/// Trouve tous les titres dans la liste des titres.
/// </summary>
/// <returns>Une collection de tous les titres présents dans la liste.</returns>
/// <inheritdoc/>
public IEnumerable<Titre> FindAll()
{
return this.titres;
try
{
this.logger.LogDebug("Récupération de tous les titres");
var result = this.titres;
this.logger.LogDebug("{Count} titres récupérés", result.Count);
return result;
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de la récupération de tous les titres");
throw;
}
}
/// <summary>
/// Recherche les titres associés à un style dont le libellé contient la chaîne spécifiée, en ignorant la casse.
/// </summary>
/// <param name="libelle">Le libellé du style à rechercher dans les titres.</param>
/// <returns>Une collection de titres correspondant au critère de recherche, triée par libellé.</returns>
/// <inheritdoc/>
public IEnumerable<Titre> SearchByStyle(string libelle)
{
if (string.IsNullOrWhiteSpace(libelle))
try
{
this.logger.LogWarning("SearchByStyle called with an empty or whitespace string.");
return Enumerable.Empty<Titre>();
this.logger.LogDebug("Recherche des titres par style: {Libelle}", libelle);
if (string.IsNullOrWhiteSpace(libelle))
{
this.logger.LogWarning("SearchByStyle appelé avec une chaîne vide ou contenant uniquement des espaces.");
return Enumerable.Empty<Titre>();
}
IEnumerable<Titre> list = this.titres
.Where(t => t.Styles.Any(s => !string.IsNullOrWhiteSpace(s.Libelle)
&& s.Libelle.Contains(libelle, StringComparison.OrdinalIgnoreCase)))
.OrderBy(t => t.Libelle)
.ToList();
if (!list.Any())
{
this.logger.LogInformation("Aucun titre trouvé correspondant au style '{Libelle}'.", libelle);
}
else
{
this.logger.LogDebug("{Count} titres trouvés pour le style '{Libelle}'", list.Count(), libelle);
}
return list;
}
IEnumerable<Titre> list = this.titres
.Where(t => t.Styles.Any(s => !string.IsNullOrWhiteSpace(s.Libelle)
&& s.Libelle.Contains(libelle, StringComparison.OrdinalIgnoreCase)))
.OrderBy(t => t.Libelle)
.ToList();
if (!list.Any())
catch (Exception ex)
{
this.logger.LogInformation("No titres found matching the style '{Libelle}'.", libelle);
this.logger.LogError(ex, "Erreur lors de la recherche des titres par style: {Libelle}", libelle);
throw;
}
return list;
}
/// <summary>
/// Met à jour un titre dans la liste des titres en fonction de son identifiant. Si aucun titre correspondant à l'identifiant du titre fourni n'est trouvé, un message d'avertissement est enregistré dans les logs et aucune mise à jour n'est effectuée.
/// </summary>
/// <param name="titre">L'objet titre à mettre à jour.</param>
/// <inheritdoc/>
public void Update(Titre titre)
{
int index = this.titres.FindIndex(t => t.IdTitre == titre.IdTitre);
if (index == -1)
try
{
this.logger.LogWarning("Aucun titre trouvé avec l'identifiant {IdTitre}.", titre.IdTitre);
}
this.logger.LogDebug("Mise à jour du titre ID: {IdTitre}", titre.IdTitre);
this.titres[index] = titre;
int index = this.titres.FindIndex(t => t.IdTitre == titre.IdTitre);
if (index == -1)
{
this.logger.LogWarning("Aucun titre trouvé avec l'identifiant {IdTitre}.", titre.IdTitre);
throw new InvalidOperationException($"Aucun titre trouvé avec l'identifiant {titre.IdTitre}.");
}
this.titres[index] = titre;
this.logger.LogDebug("Titre mis à jour avec succès, ID: {IdTitre}", titre.IdTitre);
}
catch (Exception ex)
{
this.logger.LogError(ex, "Erreur lors de la mise à jour du titre ID: {IdTitre}", titre.IdTitre);
throw;
}
}
}