diff --git a/src/Strata.SqlTools.LinqToSql/Breakdowns/LinqQueryBreakdown.cs b/src/Strata.SqlTools.LinqToSql/Breakdowns/LinqQueryBreakdown.cs index df6053e..68ad8d8 100644 --- a/src/Strata.SqlTools.LinqToSql/Breakdowns/LinqQueryBreakdown.cs +++ b/src/Strata.SqlTools.LinqToSql/Breakdowns/LinqQueryBreakdown.cs @@ -71,10 +71,7 @@ public class LinqQueryBreakdown : QueryBreakdown /// A LinqQueryBreakdown representing the query structure. public static LinqQueryBreakdown Analyze(IQueryable query) { - if (query == null) - { - throw new ArgumentNullException(nameof(query)); - } + ArgumentNullException.ThrowIfNull(query); var breakdown = new LinqQueryBreakdown { @@ -232,10 +229,7 @@ public class LinqQueryBreakdown : QueryBreakdown /// An InsertBreakdown representing the insert operation. public static Breakdowns.SqlServer.InsertBreakdown AnalyzeInsert(T entity) where T : class { - if (entity == null) - { - throw new ArgumentNullException(nameof(entity)); - } + ArgumentNullException.ThrowIfNull(entity); var breakdown = new Breakdowns.SqlServer.InsertBreakdown(); breakdown.TableName.Clause = typeof(T).Name; @@ -312,10 +306,7 @@ public class LinqQueryBreakdown : QueryBreakdown /// A DeleteBreakdown representing the delete operation. public static Breakdowns.SqlServer.DeleteBreakdown AnalyzeDelete(Expression> filterExpression) where T : class { - if (filterExpression == null) - { - throw new ArgumentNullException(nameof(filterExpression)); - } + ArgumentNullException.ThrowIfNull(filterExpression); var breakdown = new Breakdowns.SqlServer.DeleteBreakdown(); breakdown.FromClause.Clause = typeof(T).Name; @@ -343,14 +334,8 @@ public class LinqQueryBreakdown : QueryBreakdown Expression> filterExpression, Expression> updateExpression) where T : class { - if (filterExpression == null) - { - throw new ArgumentNullException(nameof(filterExpression)); - } - if (updateExpression == null) - { - throw new ArgumentNullException(nameof(updateExpression)); - } + ArgumentNullException.ThrowIfNull(filterExpression); + ArgumentNullException.ThrowIfNull(updateExpression); var breakdown = new Breakdowns.SqlServer.UpdateBreakdown(); breakdown.TableName.Clause = typeof(T).Name; @@ -425,10 +410,7 @@ public class LinqQueryBreakdown : QueryBreakdown /// A string representation of the trace analysis. public static string AnalyzeTrace(IQueryable query, string? executionContext = null) where T : class { - if (query == null) - { - throw new ArgumentNullException(nameof(query)); - } + ArgumentNullException.ThrowIfNull(query); var lines = new List { diff --git a/src/Strata.SqlTools.LinqToSql/Converters/ReverseConverterExtensions.cs b/src/Strata.SqlTools.LinqToSql/Converters/ReverseConverterExtensions.cs index d684c1a..9f5cc55 100644 --- a/src/Strata.SqlTools.LinqToSql/Converters/ReverseConverterExtensions.cs +++ b/src/Strata.SqlTools.LinqToSql/Converters/ReverseConverterExtensions.cs @@ -18,10 +18,7 @@ public static class ReverseConverterExtensions /// A new LinqQueryBreakdown with the same clauses. public static LinqQueryBreakdown ToLinqQueryBreakdown(this QueryBreakdown breakdown) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); var linq = new LinqQueryBreakdown( breakdown.SelectClause?.Clause ?? "*", @@ -54,10 +51,7 @@ public static class ReverseConverterExtensions /// A new LinqQueryBreakdown with the same clauses. public static LinqQueryBreakdown ToLinqQueryBreakdown(this PostgreSqlBreakdown breakdown) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); var linq = new LinqQueryBreakdown( breakdown.SelectClause?.Clause ?? "*", @@ -90,10 +84,7 @@ public static class ReverseConverterExtensions /// A new LinqQueryBreakdown with the same clauses. public static LinqQueryBreakdown ToLinqQueryBreakdown(this SnowflakeBreakdown breakdown) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); var linq = new LinqQueryBreakdown( breakdown.SelectClause?.Clause ?? "*", @@ -127,10 +118,7 @@ public static class ReverseConverterExtensions /// A new breakdown in the target dialect format. public static object ConvertToDialect(this QueryBreakdown breakdown, string targetDialect) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); return targetDialect.ToLowerInvariant() switch { @@ -150,10 +138,7 @@ public static class ReverseConverterExtensions /// A new breakdown in the target dialect format. public static object ConvertToDialect(this PostgreSqlBreakdown breakdown, string targetDialect) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); return targetDialect.ToLowerInvariant() switch { @@ -173,10 +158,7 @@ public static class ReverseConverterExtensions /// A new breakdown in the target dialect format. public static object ConvertToDialect(this SnowflakeBreakdown breakdown, string targetDialect) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); return targetDialect.ToLowerInvariant() switch { diff --git a/src/Strata.SqlTools.LinqToSql/Validators/QueryValidator.cs b/src/Strata.SqlTools.LinqToSql/Validators/QueryValidator.cs index 3085adb..6170410 100644 --- a/src/Strata.SqlTools.LinqToSql/Validators/QueryValidator.cs +++ b/src/Strata.SqlTools.LinqToSql/Validators/QueryValidator.cs @@ -71,10 +71,7 @@ public class QueryValidator /// This validator for method chaining. public QueryValidator Validate(LinqQueryBreakdown breakdown) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); _issues.Clear(); diff --git a/src/Strata.SqlTools.SqlBreakdown/Classes/SqlBreakdownCollection.cs b/src/Strata.SqlTools.SqlBreakdown/Classes/SqlBreakdownCollection.cs index ae378df..0abfdee 100644 --- a/src/Strata.SqlTools.SqlBreakdown/Classes/SqlBreakdownCollection.cs +++ b/src/Strata.SqlTools.SqlBreakdown/Classes/SqlBreakdownCollection.cs @@ -88,10 +88,7 @@ public class SqlBreakdownCollection : ICollection /// Thrown when breakdown is null. public void Add(ISqlBreakdown breakdown) { - if (breakdown == null) - { - throw new ArgumentNullException(nameof(breakdown)); - } + ArgumentNullException.ThrowIfNull(breakdown); _breakdowns.Add(breakdown); } @@ -103,10 +100,7 @@ public class SqlBreakdownCollection : ICollection /// Thrown when breakdowns is null. public void AddRange(IEnumerable breakdowns) { - if (breakdowns == null) - { - throw new ArgumentNullException(nameof(breakdowns)); - } + ArgumentNullException.ThrowIfNull(breakdowns); _breakdowns.AddRange(breakdowns); } @@ -142,10 +136,7 @@ public class SqlBreakdownCollection : ICollection /// Thrown when sqlBatch is null. public void ParseBatch(string sqlBatch, string? separator = null) { - if (sqlBatch == null) - { - throw new ArgumentNullException(nameof(sqlBatch)); - } + ArgumentNullException.ThrowIfNull(sqlBatch); Clear(); diff --git a/src/Strata.SqlTools.SqlBreakdown/Expressions/InputPropertyExpression.cs b/src/Strata.SqlTools.SqlBreakdown/Expressions/InputPropertyExpression.cs index f984759..fd53385 100644 --- a/src/Strata.SqlTools.SqlBreakdown/Expressions/InputPropertyExpression.cs +++ b/src/Strata.SqlTools.SqlBreakdown/Expressions/InputPropertyExpression.cs @@ -269,10 +269,7 @@ public static class FlatDataUtils public static object GetValue(this IFlatData data, string key) { - if (data == null) - { - throw new ArgumentNullException(nameof(data)); - } + ArgumentNullException.ThrowIfNull(data); if (!data.ContainsKey(key)) { diff --git a/src/Strata.SqlTools.SqlServer/Breakdowns/QueryBreakdownCollectionBase.cs b/src/Strata.SqlTools.SqlServer/Breakdowns/QueryBreakdownCollectionBase.cs index 5a5d0f9..88ad272 100644 --- a/src/Strata.SqlTools.SqlServer/Breakdowns/QueryBreakdownCollectionBase.cs +++ b/src/Strata.SqlTools.SqlServer/Breakdowns/QueryBreakdownCollectionBase.cs @@ -51,10 +51,7 @@ public abstract class QueryBreakdownCollectionBase : SqlBreakdownCollect /// Thrown when is null. public void Add(TQuery queryBreakdown) { - if (queryBreakdown == null) - { - throw new ArgumentNullException(nameof(queryBreakdown)); - } + ArgumentNullException.ThrowIfNull(queryBreakdown); QueryBreakdownList.Add(queryBreakdown); base.Add(queryBreakdown); @@ -67,10 +64,7 @@ public abstract class QueryBreakdownCollectionBase : SqlBreakdownCollect /// Thrown when is null. public void AddRange(IEnumerable queryBreakdowns) { - if (queryBreakdowns == null) - { - throw new ArgumentNullException(nameof(queryBreakdowns)); - } + ArgumentNullException.ThrowIfNull(queryBreakdowns); foreach (var breakdown in queryBreakdowns) {