From b5c3b0a88cb43eee3b0fb3ab8bc9b87d7ee937c3 Mon Sep 17 00:00:00 2001 From: Matt Nadareski Date: Sun, 5 Oct 2025 15:35:08 -0400 Subject: [PATCH] Reduce two places to TryGet to avoid double access --- SabreTools.CommandLine/Inputs/UserInput.cs | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/SabreTools.CommandLine/Inputs/UserInput.cs b/SabreTools.CommandLine/Inputs/UserInput.cs index 1dc4709..fd55aec 100644 --- a/SabreTools.CommandLine/Inputs/UserInput.cs +++ b/SabreTools.CommandLine/Inputs/UserInput.cs @@ -69,7 +69,13 @@ namespace SabreTools.CommandLine.Inputs /// public UserInput? this[string name] { - get { return Children.ContainsKey(name) ? Children[name] : null; } + get + { + if (!Children.TryGetValue(name, out var input)) + return null; + + return input; + } } /// @@ -77,7 +83,13 @@ namespace SabreTools.CommandLine.Inputs /// public UserInput? this[UserInput subfeature] { - get { return Children.ContainsKey(subfeature.Name) ? Children[subfeature.Name] : null; } + get + { + if (!Children.TryGetValue(subfeature.Name, out var input)) + return null; + + return input; + } } ///