update (#400)

Author: zhiyuanliang-ms

src/Microsoft.FeatureManagement/FeatureFilters/PercentageFilter.cs

@@ -51,10 +51,7 @@ public Task<bool> EvaluateAsync(FeatureFilterEvaluationContext context)
 
             if (settings.Value < 0)
             {
-                if (_logger != null)
-                {
-                    _logger.LogWarning($"The '{Alias}' feature filter does not have a valid '{nameof(settings.Value)}' value for feature '{context.FeatureName}'");
-                }
+                _logger?.LogWarning($"The '{Alias}' feature filter does not have a valid '{nameof(settings.Value)}' value for feature '{context.FeatureName}'");
 
                 result = false;
             }

src/Microsoft.FeatureManagement/FeatureFilters/TimeWindowFilter.cs

@@ -51,10 +51,7 @@ public Task<bool> EvaluateAsync(FeatureFilterEvaluationContext context)
 
             if (!settings.Start.HasValue && !settings.End.HasValue)
             {
-                if (_logger != null)
-                {
-                    _logger.LogWarning($"The '{Alias}' feature filter is not valid for feature '{context.FeatureName}'. It must have have specify either '{nameof(settings.Start)}', '{nameof(settings.End)}', or both.");
-                }
+                _logger?.LogWarning($"The '{Alias}' feature filter is not valid for feature '{context.FeatureName}'. It must have have specify either '{nameof(settings.Start)}', '{nameof(settings.End)}', or both.");
 
                 return Task.FromResult(false);
             }

src/Microsoft.FeatureManagement/Targeting/TargetingFilter.cs

@@ -64,10 +64,7 @@ public async Task<bool> EvaluateAsync(FeatureFilterEvaluationContext context)
             // Ensure targeting can be performed
             if (targetingContext == null)
             {
-                if (_logger != null)
-                {
-                    _logger.LogWarning("No targeting context available for targeting evaluation.");
-                }
+                _logger?.LogWarning("No targeting context available for targeting evaluation.");
 
                 return false;
             }