chore: Added some more logging to Service since it caused problems in Serialization in the past

Author: DevTKSS

src/DevTKSS.Uno.Samples.MvuxGallery/Models/CodeSamples/CodeSampleService.cs

@@ -11,18 +11,27 @@ public CodeSampleService(
         _logger = logger;
         _storage = storage;
         if (_logger.IsEnabled(LogLevel.Trace))
+        {
+            _logger.LogTrace("Initializing options for {serviceName}...",
+                nameof(CodeSampleService<SampleOptions>));
+        }
+        if (_logger.IsEnabled(LogLevel.Debug))
         { 
             // Log LineRanges for each sample option
             foreach (var sample in _options.Samples)
             {
-                _logger.LogTrace("SampleID: {sampleID},\nDescription: {description},\nFilePath: {filePath},\nLineRanges: {lineRanges}",
+                _logger.LogDebug("\tSampleID: {sampleID},\nDescription: {description},\nFilePath: {filePath},\nLineRanges: {lineRanges}",
                     sample.SampleID,
                     sample.Description,
                     sample.FilePath,
                     sample.LineRanges);
 
             }
         }
+        else if (_logger.IsEnabled(LogLevel.Trace))
+        {
+            _logger.LogTrace("Gathered {count} Options", _options.Samples.Length);
+        }
     }
 
     private readonly IStorage _storage;
@@ -46,8 +55,20 @@ public CodeSampleService(
     public async ValueTask<IImmutableList<string>> GetCodeSampleOptionsAsync(CancellationToken ct = default)
     {
         await Task.Delay(1);
+        _logger.LogTrace("Collecting available code sample options from appsettings.sampledata.json...");
         var sampleOptions = _options.Samples.Select(sample => sample.SampleID).ToImmutableList();
-        _logger.LogInformation("Options:\n{options}", sampleOptions.JoinBy("," + Environment.NewLine));
+        
+        if (_logger.IsEnabled(LogLevel.Debug))
+        {
+            // Log available options
+            _logger.LogDebug("Available Options:\n{options}", sampleOptions.JoinBy("," + Environment.NewLine));
+        }
+        else if (_logger.IsEnabled(LogLevel.Trace))
+        {
+            // Log available options
+            _logger.LogTrace("Gathered {count} Options", sampleOptions.Count);
+        }
+
         return sampleOptions; 
     }
 
@@ -57,7 +78,7 @@ public async ValueTask<string> GetCodeSampleAsync(string? sampleID, Cancellation
         {
             if(_logger.IsEnabled(LogLevel.Trace))
             {
-                _logger.LogTrace("SampleID: {sampleID},\nDescription: {description},\nFilePath: {filePath},\nLineRanges: {lineRanges}",
+                _logger.LogTrace("Fetching Storage Data for SampleID: {sampleID},\nDescription: {description},\nFilePath: {filePath},\nLineRanges: {lineRanges}",
                     sampleOption.SampleID,
                     sampleOption.Description,
                     sampleOption.FilePath,
@@ -66,9 +87,8 @@ public async ValueTask<string> GetCodeSampleAsync(string? sampleID, Cancellation
 
             return await _storage.ReadLinesFromPackageFile(sampleOption.FilePath,sampleOption.LineRanges.Select(lr => (lr.Start, lr.End)));
         }
-        
 
-        _logger.LogWarning("Code sample with ID {sampleID} not found", sampleID);
+        _logger.LogError("Code sample with ID {sampleID} not found", sampleID);
         return string.Empty;
     }
 }

src/DevTKSS.Uno.Samples.MvuxGallery/Models/GalleryImages/GalleryImageService.cs

@@ -1,15 +1,17 @@
 namespace DevTKSS.Uno.Samples.MvuxGallery.Models.GalleryImages;
-[Bindable]
 public partial class GalleryImageService : IGalleryImageService
 {
     private readonly ILocalizationService _localizationService;
     private readonly IStringLocalizer _stringlocalizer;
+    private readonly ILogger<GalleryImageService> _logger;
     public GalleryImageService(
         ILocalizationService localizationService,
-        IStringLocalizer _stringLocalizer)
+        IStringLocalizer _stringLocalizer,
+        ILogger<GalleryImageService> logger)
     {
         _localizationService = localizationService;
         _stringlocalizer = _stringLocalizer;
+        _logger = logger;
     }
 
     public async ValueTask<IImmutableList<GalleryImageModel>> GetGalleryImagesWithReswAsync(CancellationToken ct)
@@ -19,6 +21,7 @@ public async ValueTask<IImmutableList<GalleryImageModel>> GetGalleryImagesWithRe
     public async ValueTask<IImmutableList<GalleryImageModel>> GetGalleryImagesWithoutReswAsync(CancellationToken ct)
     {
         string cultureString = _localizationService.CurrentCulture.TwoLetterISOLanguageName;
+        _logger.LogTrace("Trying to get GalleryImages for Culture: {culture}", cultureString);
         var galleryImages = cultureString switch
         {
             "de" => await GetDEGalleryImagesAsync(ct),
@@ -33,7 +36,9 @@ public async ValueTask<IImmutableList<GalleryImageModel>> GetGalleryImagesWithou
     #region With resw
     private async ValueTask<IImmutableList<GalleryImageModel>> GetGalleryImagesAsync(CancellationToken ct)
     {
+        // Simulate a delay to mimic data fetching
         await Task.Delay(TimeSpan.FromSeconds(2), ct);
+        _logger.LogTrace("Trying to get GalleryImages from Resw");
         var galleryImages = new GalleryImageModel[]
         {
             new GalleryImageModel(
@@ -72,6 +77,7 @@ private async ValueTask<IImmutableList<GalleryImageModel>> GetDEGalleryImagesAsy
     {
         // Simulate a delay to mimic data fetching
         await Task.Delay(TimeSpan.FromSeconds(2),ct);
+        _logger.LogTrace("Creating DE GalleryImages without resw");
         var galleryImages = new GalleryImageModel[]
         {
             new GalleryImageModel(
@@ -108,6 +114,7 @@ private async ValueTask<IImmutableList<GalleryImageModel>> GetENGalleryImagesAsy
     {
         // Simulate a delay to mimic data fetching
         await Task.Delay(TimeSpan.FromSeconds(2), ct);
+        _logger.LogTrace("Creating EN GalleryImages without resw");
         var galleryImages = new GalleryImageModel[]
         {
             new GalleryImageModel(