Fix typos & minor code style improvements

configcat · Oct 26, 2023 · 537a906 · 537a906
1 parent 7306e00
commit 537a906
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 12 deletions.
diff --git a/src/ConfigCatClient/Evaluation/EvaluateLogHelper.cs b/src/ConfigCatClient/Evaluation/EvaluateLogHelper.cs
@@ -203,11 +203,10 @@ private static IndentedTextBuilder AppendPercentageOptions(this IndentedTextBuil
 
     private static IndentedTextBuilder AppendTargetingRuleThenPart(this IndentedTextBuilder builder, TargetingRule targetingRule, bool newLine, bool appendPercentageOptions = false, string? percentageOptionsAttribute = null)
     {
-        var percentageOptions = targetingRule.PercentageOptions;
-
         (newLine ? builder.NewLine() : builder.Append(" "))
             .Append("THEN");
 
+        var percentageOptions = targetingRule.PercentageOptions;
         if (percentageOptions is not { Length: > 0 })
         {
             return builder.Append($" '{targetingRule.SimpleValue?.Value ?? default}'");

diff --git a/src/ConfigCatClient/Evaluation/RolloutEvaluator.cs b/src/ConfigCatClient/Evaluation/RolloutEvaluator.cs
@@ -61,7 +61,7 @@ public EvaluateResult Evaluate<T>(T defaultValue, ref EvaluateContext context, [
                 if (context.Setting.SettingType != Setting.UnknownType && context.Setting.SettingType != expectedSettingType)
                 {
                     throw new InvalidOperationException(
-                        "The type of a setting must match the type of the specified default value "
+                        "The type of a setting must match the type of the specified default value. "
                         + $"Setting's type was {context.Setting.SettingType} but the default value's type was {typeof(T)}. "
                         + $"Please use a default value which corresponds to the setting type {context.Setting.SettingType}.");
                 }
@@ -158,13 +158,10 @@ private bool TryEvaluateTargetingRules(TargetingRule[] targetingRules, ref Evalu
                 logBuilder?.DecreaseIndent();
                 return true;
             }
-            else
-            {
-                logBuilder?
-                    .NewLine(TargetingRuleIgnoredMessage)
-                    .DecreaseIndent();
-                continue;
-            }
+
+            logBuilder?
+                .NewLine(TargetingRuleIgnoredMessage)
+                .DecreaseIndent();
         }
 
         result = default;
@@ -236,14 +233,14 @@ private bool TryEvaluatePercentageOptions(PercentageOption[] percentageOptions,
                 continue;
             }
 
-            var percentageOptionValue = percentageOption.Value.GetValue(context.Setting.SettingType, throwIfInvalid: false);
+            var percentageOptionValue = percentageOption.Value.GetValue(throwIfInvalid: false);
             logBuilder?.NewLine().Append($"- Hash value {hashValue} selects % option {i + 1} ({percentageOption.Percentage}%), '{percentageOptionValue ?? EvaluateLogHelper.InvalidValuePlaceholder}'.");
 
             result = new EvaluateResult(percentageOption, matchedTargetingRule: targetingRule, matchedPercentageOption: percentageOption);
             return true;
         }
 
-        throw new InvalidOperationException("Sum of percentage option percentages are less than 100).");
+        throw new InvalidOperationException("Sum of percentage option percentages are less than 100.");
     }
 
     private bool TryEvaluateConditions<TCondition>(TCondition[] conditions, TargetingRule? targetingRule, string contextSalt, ref EvaluateContext context, out bool result)