Fix: Code Cleanup

**Twitch Redeems**
- brought to you by `Ydouwantjustasking` with : "Let's do update on today's stream -🦆"

Author: AnnaSasDev

.editorconfig

@@ -71,7 +71,7 @@ dotnet_naming_symbols.private_instance_fields_symbols.resharper_applicable_kinds
 dotnet_naming_symbols.private_instance_fields_symbols.resharper_required_modifiers = instance
 dotnet_naming_symbols.private_instance_fields_symbols_1.applicable_accessibilities = private
 dotnet_naming_symbols.private_instance_fields_symbols_1.applicable_kinds = field
-dotnet_naming_symbols.private_instance_fields_symbols_1.resharper_applicable_kinds = field,readonly_field
+dotnet_naming_symbols.private_instance_fields_symbols_1.resharper_applicable_kinds = field, readonly_field
 dotnet_naming_symbols.private_instance_fields_symbols_1.resharper_required_modifiers = instance
 dotnet_naming_symbols.private_static_fields_symbols.applicable_accessibilities = private
 dotnet_naming_symbols.private_static_fields_symbols.applicable_kinds = field

src/CodeOfChaos.Parsers.Csv/CodeOfChaos.Parsers.Csv.csproj

@@ -23,13 +23,13 @@
     </PropertyGroup>
 
     <ItemGroup>
-      <PackageReference Include="CodeOfChaos.Extensions" Version="0.27.1" />
+        <PackageReference Include="CodeOfChaos.Extensions" Version="0.27.1"/>
     </ItemGroup>
 
     <ItemGroup>
-        <None Include="..\..\LICENSE" Pack="true" PackagePath="" />
-        <None Include="README.md" Pack="true" PackagePath="" />
-        <None Include="../../assets/icon.png" Pack="true" PackagePath="" />
+        <None Include="..\..\LICENSE" Pack="true" PackagePath="" Visible="false"/>
+        <None Include="README.md" Pack="true" PackagePath="" Visible="false"/>
+        <None Include="../../assets/icon.png" Pack="true" PackagePath="" Visible="false"/>
     </ItemGroup>
 
 </Project>

src/CodeOfChaos.Parsers.Csv/Contracts/ICsvReader.cs

@@ -16,12 +16,13 @@ namespace CodeOfChaos.Parsers.Csv;
 ///     Provides functionality to parse CSV files into various collection types.
 /// </summary>
 public class CsvParser(CsvParserConfig config) : ICsvParser {
-    protected readonly ConcurrentDictionary<Type, PropertyInfo[]> PropertyCache = new();
     protected readonly ConcurrentDictionary<Type, string[]> HeaderCache = new();
-
-    // -----------------------------------------------------------------------------------------------------------------
-    // Constructors
-    // -----------------------------------------------------------------------------------------------------------------
+    protected readonly ConcurrentDictionary<Type, PropertyInfo[]> PropertyCache = new();
+    /// <inheritdoc />
+    public void ClearCaches() {
+        PropertyCache.Clear();
+        HeaderCache.Clear();
+    }
     /// <summary>
     ///     Creates an instance of <see cref="CsvParser" /> using the specified configuration action.
     /// </summary>
@@ -38,15 +39,50 @@ public static CsvParser FromConfig(Action<CsvParserConfig> configAction) {
         configAction(config);
         return new CsvParser(config);
     }
-
     /// <summary>
-    /// Creates an instance of <see cref="CsvParser" /> using the default configuration.
+    ///     Creates an instance of <see cref="CsvParser" /> using the default configuration.
     /// </summary>
     /// <returns>
-    /// A new instance of <see cref="CsvParser" /> initialized with the default settings, ready
-    /// to parse CSV data.
+    ///     A new instance of <see cref="CsvParser" /> initialized with the default settings, ready
+    ///     to parse CSV data.
     /// </returns>
     public static CsvParser FromDefaultConfig() => new(new CsvParserConfig());
+    protected PropertyInfo[] GetCsvProperties<T>() {
+        Type type = typeof(T);
+        if (PropertyCache.TryGetValue(type, out PropertyInfo[]? propertyInfos)) return propertyInfos;
+
+        PropertyInfo[] propertyInfosArray = type.GetProperties().ToArray();
+        PropertyCache[type] = propertyInfosArray;
+        return propertyInfosArray;
+    }
+    protected string[] GetCsvHeaders<T>() {
+        Type type = typeof(T);
+        if (HeaderCache.TryGetValue(type, out string[]? headers)) return headers;
+
+        string[] headersArray = GetCsvProperties<T>()
+            .Select(p => {
+                if (p.GetCustomAttribute<CsvColumnAttribute>() is not {} attribute)
+                    return config.UseLowerCaseHeaders ? p.Name.ToLowerInvariant() : p.Name;
+
+                return config.UseLowerCaseHeaders
+                    ? attribute.NameLowerInvariant
+                    : attribute.Name;
+            })
+            .ToArray();
+
+        HeaderCache[type] = headersArray;
+        return headersArray;
+    }
+    protected IEnumerable<string> GetCsvValues<T>(T? obj) {
+        if (obj is null) return [];
+
+        return GetCsvProperties<T>()
+            .Select(p => p.GetValue(obj)?.ToString() ?? string.Empty);
+    }
+
+    // -----------------------------------------------------------------------------------------------------------------
+    // Constructors
+    // -----------------------------------------------------------------------------------------------------------------
 
     // -----------------------------------------------------------------------------------------------------------------
     // Input Methods
@@ -173,6 +209,7 @@ public async ValueTask<List<T>> ToListAsync<T>(TextReader reader, CancellationTo
         await foreach (Dictionary<string, string?> item in FromTextReaderToDictionaryAsync(reader, ct)) {
             results.Add(item);
         }
+
         return results.ToArray();
     }
 
@@ -184,7 +221,7 @@ public async ValueTask<List<T>> ToListAsync<T>(TextReader reader, CancellationTo
         await foreach (Dictionary<string, string?> item in FromTextReaderToDictionaryAsync(reader, ct)) {
             results.Add(item);
         }
-        
+
         results.TrimExcess();
         return results.ToArray();
     }
@@ -205,7 +242,7 @@ public async ValueTask<List<T>> ToListAsync<T>(TextReader reader, CancellationTo
         await foreach (Dictionary<string, string?> item in FromTextReaderToDictionaryAsync(reader, ct)) {
             results.Add(item);
         }
-        
+
         results.TrimExcess();
         return results;
     }
@@ -217,7 +254,7 @@ public async ValueTask<List<T>> ToListAsync<T>(TextReader reader, CancellationTo
         await foreach (Dictionary<string, string?> item in FromTextReaderToDictionaryAsync(reader, ct)) {
             results.Add(item);
         }
-        
+
         results.TrimExcess();
         return results;
     }
@@ -355,14 +392,14 @@ private async IAsyncEnumerable<T> FromTextReaderAsync<T>(TextReader reader, [Enu
             }
 
             batch.Clear();
-            ct.ThrowIfCancellationRequested(); // After a batch is done, check if the cancellation token was requested
+            ct.ThrowIfCancellationRequested();// After a batch is done, check if the cancellation token was requested
             if (line == null) break;
         }
     }
 
     private void SetPropertyFromCsvColumn<T>(T? value, string[] headerColumns, string[] values) where T : class, new() {
         if (value is null) return;
-        
+
         foreach (PropertyInfo prop in GetCsvProperties<T>()) {
             int columnIndex = Attribute.GetCustomAttribute(prop, typeof(CsvColumnAttribute)) is CsvColumnAttribute attribute
                 ? Array.IndexOf(headerColumns, attribute.Name)
@@ -401,6 +438,7 @@ private async IAsyncEnumerable<T> FromTextReaderAsync<T>(TextReader reader, [Enu
                     string value = values[j];
                     dict[headerColumns[j]] = value.IsNotNullOrEmpty() ? value : null;
                 }
+
                 batch.Add(dict);
             }
 
@@ -431,6 +469,7 @@ private async IAsyncEnumerable<T> FromTextReaderAsync<T>(TextReader reader, [Enu
                     string value = values[j];
                     dict[headerColumns[j]] = value.IsNotNullOrEmpty() ? value : null;
                 }
+
                 batch.Add(dict);
             }
 
@@ -439,12 +478,11 @@ private async IAsyncEnumerable<T> FromTextReaderAsync<T>(TextReader reader, [Enu
             }
 
             batch.Clear();
-            ct.ThrowIfCancellationRequested(); // After a batch is done, check if the cancellation token was requested
+            ct.ThrowIfCancellationRequested();// After a batch is done, check if the cancellation token was requested
             if (line == null) break;
         }
     }
     #endregion
-
     #region Generic Type Writer
     private void FromDataToTextWriter<T>(TextWriter writer, IEnumerable<T> data) {
         // Write header row
@@ -457,6 +495,7 @@ private void FromDataToTextWriter<T>(TextWriter writer, IEnumerable<T> data) {
 
             writer.Write(Environment.NewLine);
         }
+
         // Write data rows
         foreach (T? obj in data) {
             string[] values = GetCsvValues(obj).ToArray();
@@ -481,6 +520,7 @@ private async Task FromDataToTextWriterAsync<T>(TextWriter writer, IEnumerable<T
             await writer.WriteAsync(Environment.NewLine);
             ct.ThrowIfCancellationRequested();
         }
+
         // Write data rows
         foreach (T? obj in data) {
             string[] values = GetCsvValues(obj).ToArray();
@@ -522,15 +562,15 @@ private async Task FromDictionaryToTextWriterAsync(TextWriter writer, IEnumerabl
             IDictionary<string, string?> firstDictionary = records.First();
             IEnumerable<string> headers = firstDictionary.Keys;
             await writer.WriteLineAsync(string.Join(config.ColumnSplit, headers));
-            
+
             ct.ThrowIfCancellationRequested();
         }
 
         // Write data rows
         foreach (IDictionary<string, string?> dictionary in records) {
             IEnumerable<string> values = dictionary.Values.Select(value => value?.ToString() ?? string.Empty);
             await writer.WriteLineAsync(string.Join(config.ColumnSplit, values));
-            
+
             ct.ThrowIfCancellationRequested();
         }
     }
@@ -539,43 +579,4 @@ private async Task FromDictionaryToTextWriterAsync(TextWriter writer, IEnumerabl
     // -----------------------------------------------------------------------------------------------------------------
     // Methods
     // -----------------------------------------------------------------------------------------------------------------
-    /// <inheritdoc />
-    public void ClearCaches() {
-        PropertyCache.Clear();
-        HeaderCache.Clear();
-    }
-    
-    protected PropertyInfo[] GetCsvProperties<T>() {
-        Type type = typeof(T);
-        if (PropertyCache.TryGetValue(type, out PropertyInfo[]? propertyInfos)) return propertyInfos;
-        PropertyInfo[] propertyInfosArray = type.GetProperties().ToArray();
-        PropertyCache[type] = propertyInfosArray;
-        return propertyInfosArray;
-    }
-
-    protected string[] GetCsvHeaders<T>() {
-        Type type = typeof(T);
-        if (HeaderCache.TryGetValue(type, out string[]? headers)) return headers;
-        
-        string[] headersArray =  GetCsvProperties<T>()
-            .Select(p => {
-                if (p.GetCustomAttribute<CsvColumnAttribute>() is not {} attribute)
-                    return config.UseLowerCaseHeaders ? p.Name.ToLowerInvariant() : p.Name;
-
-                return config.UseLowerCaseHeaders
-                    ? attribute.NameLowerInvariant
-                    : attribute.Name;
-            })
-            .ToArray();
-        
-        HeaderCache[type] = headersArray;
-        return headersArray;
-    }
-
-    protected IEnumerable<string> GetCsvValues<T>(T? obj) {
-        if (obj is null) return [];
-        
-        return GetCsvProperties<T>()
-            .Select(p => p.GetValue(obj)?.ToString() ?? string.Empty);
-    }
 }

src/CodeOfChaos.Parsers.Csv/Contracts/ICsvWriter.cs

@@ -6,10 +6,10 @@ namespace CodeOfChaos.Parsers.Csv;
 // Code
 // ---------------------------------------------------------------------------------------------------------------------
 public class CsvParserConfig {
+    public int BatchSize = 100;
     public string ColumnSplit = ",";
     public bool IncludeHeader = true;
+    public int InitialCapacity = 1000;
     public bool LogErrors = false;
     public bool UseLowerCaseHeaders = false;
-    public int BatchSize = 100;
-    public int InitialCapacity = 1000;
 }