Enhance documentation and null-checking in data classes

Added XML documentation comments to `DataReaderExtensions.cs` and `DynamicDataRecord.cs`. Improved null-checking using `ArgumentNullException.ThrowIfNull` for .NET 6.0 and greater. Updated method implementations for better readability and maintainability while preserving functionality.

Author: wforney

SharedCode.Data/DataReaderExtensions.cs

@@ -1,75 +1,79 @@
-
-namespace SharedCode.Data;
-
-using System;
-using System.Collections.Generic;
-using System.Data;
+using System.Data;
 using System.Data.Common;
 
+namespace SharedCode.Data;
+
 /// <summary>
 /// The data reader extensions class.
 /// </summary>
 public static class DataReaderExtensions
 {
-	/// <summary>
-	/// Enumerates the specified database data reader.
-	/// </summary>
-	/// <typeparam name="T">The type of the objects being enumerated.</typeparam>
-	/// <param name="this">The database data reader.</param>
-	/// <returns>The enumerable sequence.</returns>
-	/// <exception cref="ArgumentNullException">dbDataReader</exception>
-	public static IEnumerable<T> Enumerate<T>(this IDataReader @this) where T : new()
-	{
-		return @this is null
-			? throw new ArgumentNullException(nameof(@this))
-			: EnumerateImpl();
+    /// <summary>
+    /// Enumerates the specified database data reader.
+    /// </summary>
+    /// <typeparam name="T">The type of the objects being enumerated.</typeparam>
+    /// <param name="this">The database data reader.</param>
+    /// <returns>The enumerable sequence.</returns>
+    /// <exception cref="ArgumentNullException">dbDataReader</exception>
+    public static IEnumerable<T> Enumerate<T>(this IDataReader @this) where T : new()
+    {
+        return @this is null
+            ? throw new ArgumentNullException(nameof(@this))
+            : EnumerateImpl();
 
-		IEnumerable<T> EnumerateImpl()
-		{
-			var converter = new DataRecordConverter<T>(@this);
-			while (@this.Read())
-			{
-				yield return converter.ConvertRecordToItem();
-			}
-		}
-	}
+        IEnumerable<T> EnumerateImpl()
+        {
+            var converter = new DataRecordConverter<T>(@this);
+            while (@this.Read())
+            {
+                yield return converter.ConvertRecordToItem();
+            }
+        }
+    }
 
-	/// <summary>
-	/// Enumerates the specified database data reader as an asynchronous operation.
-	/// </summary>
-	/// <typeparam name="T">The type of the objects being enumerated.</typeparam>
-	/// <param name="this">The database data reader.</param>
-	/// <returns>The asynchronous enumerable sequence.</returns>
-	/// <exception cref="ArgumentNullException">dbDataReader</exception>
-	public static IAsyncEnumerable<T> EnumerateAsync<T>(this DbDataReader @this) where T : new()
-	{
-		return @this is null
-			? throw new ArgumentNullException(nameof(@this))
-			: EnumerateAsyncImpl();
+    /// <summary>
+    /// Enumerates the specified database data reader as an asynchronous operation.
+    /// </summary>
+    /// <typeparam name="T">The type of the objects being enumerated.</typeparam>
+    /// <param name="this">The database data reader.</param>
+    /// <returns>The asynchronous enumerable sequence.</returns>
+    /// <exception cref="ArgumentNullException">dbDataReader</exception>
+    public static IAsyncEnumerable<T> EnumerateAsync<T>(this DbDataReader @this) where T : new()
+    {
+        return @this is null
+            ? throw new ArgumentNullException(nameof(@this))
+            : EnumerateAsyncImpl();
 
-		async IAsyncEnumerable<T> EnumerateAsyncImpl()
-		{
-			var converter = new DataRecordConverter<T>(@this);
-			while (await @this.ReadAsync().ConfigureAwait(true))
-			{
-				yield return converter.ConvertRecordToItem();
-			}
-		}
-	}
+        async IAsyncEnumerable<T> EnumerateAsyncImpl()
+        {
+            var converter = new DataRecordConverter<T>(@this);
+            while (await @this.ReadAsync().ConfigureAwait(true))
+            {
+                yield return converter.ConvertRecordToItem();
+            }
+        }
+    }
 
-	/// <summary>
-	/// Maps the specified database data record to the specified object type <typeparamref
-	/// name="T" />.
-	/// </summary>
-	/// <typeparam name="T">The type of the object being mapped.</typeparam>
-	/// <param name="this">The database data record.</param>
-	/// <returns>The resulting object.</returns>
-	/// <exception cref="ArgumentNullException">dbDataRecord</exception>
-	public static T Map<T>(this IDataRecord @this) where T : new()
-	{
-		ArgumentNullException.ThrowIfNull(@this);
+    /// <summary>
+    /// Maps the specified database data record to the specified object type <typeparamref
+    /// name="T" />.
+    /// </summary>
+    /// <typeparam name="T">The type of the object being mapped.</typeparam>
+    /// <param name="this">The database data record.</param>
+    /// <returns>The resulting object.</returns>
+    /// <exception cref="ArgumentNullException">dbDataRecord</exception>
+    public static T Map<T>(this IDataRecord @this) where T : new()
+    {
+#if NET6_0_OR_GREATER
+        ArgumentNullException.ThrowIfNull(@this);
+#else
+        if (@this is null)
+        {
+            throw new ArgumentNullException(nameof(@this));
+        }
+#endif
 
-		var converter = new DataRecordConverter<T>(@this);
-		return converter.ConvertRecordToItem();
-	}
+        var converter = new DataRecordConverter<T>(@this);
+        return converter.ConvertRecordToItem();
+    }
 }

SharedCode.Data/DynamicDataRecord.cs

@@ -1,44 +1,40 @@
-
+using SharedCode.Data.Exceptions;
 
-
-namespace SharedCode.Data;
-
-using SharedCode.Data.Exceptions;
-
-using System;
 using System.Data;
 using System.Dynamic;
 
+namespace SharedCode.Data;
+
 /// <summary>
 /// The dynamic data record class.
 /// </summary>
-public class DynamicDataRecord : DynamicObject
+/// <remarks>
+/// Initializes a new instance of the <see cref="DynamicDataRecord"/> class.
+/// </remarks>
+/// <param name="record">The data record to be made dynamic.</param>
+public class DynamicDataRecord(IDataRecord record) : DynamicObject
 {
-	/// <summary>
-	/// The record
-	/// </summary>
-	private readonly IDataRecord record;
-
-	/// <summary>
-	/// Initializes a new instance of the <see cref="DynamicDataRecord" /> class.
-	/// </summary>
-	/// <param name="record">The record.</param>
-	public DynamicDataRecord(IDataRecord record) => this.record = record;
-
-	/// <inheritdoc/>
-	public override bool TryGetMember(GetMemberBinder binder, out object? result)
-	{
-		ArgumentNullException.ThrowIfNull(binder);
-
-		try
-		{
-			var rawResult = this.record.GetValue(this.record.GetOrdinal(binder.Name));
-			result = rawResult == DBNull.Value ? default : rawResult;
-			return true;
-		}
-		catch (IndexOutOfRangeException)
-		{
-			throw new ColumnIsNotInResultSetException(binder.Name);
-		}
-	}
+    /// <inheritdoc/>
+    public override bool TryGetMember(GetMemberBinder binder, out object? result)
+    {
+#if NET6_0_OR_GREATER
+        ArgumentNullException.ThrowIfNull(binder);
+#else
+        if (binder is null)
+        {
+            throw new ArgumentNullException(nameof(binder));
+        }
+#endif
+
+        try
+        {
+            var rawResult = record.GetValue(record.GetOrdinal(binder.Name));
+            result = rawResult == DBNull.Value ? default : rawResult;
+            return true;
+        }
+        catch (IndexOutOfRangeException)
+        {
+            throw new ColumnIsNotInResultSetException(binder.Name);
+        }
+    }
 }