Refactor out checking logic
This commit is contained in:
parent
fe59799e1f
commit
f0d01f1a65
@ -54,13 +54,7 @@ namespace ChaosBot.ConfigHelpers
|
|||||||
*/
|
*/
|
||||||
public T GetValue<T>(string key, T defaultValue, ulong? guildId = null)
|
public T GetValue<T>(string key, T defaultValue, ulong? guildId = null)
|
||||||
{
|
{
|
||||||
bool keyExists = TryGetKeyFromRegexMatch(key, out string realKey);
|
string realKey = EnsureKeyExistsAndTypeCorrect<T>(key);
|
||||||
|
|
||||||
if (!keyExists)
|
|
||||||
throw new ArgumentException($"Configuration does not contain key '{key}'");
|
|
||||||
|
|
||||||
if (!(ConfigurationFlags[realKey].type == typeof(T)))
|
|
||||||
throw new ArgumentException($"Configuration flag '{realKey}<{ConfigurationFlags[realKey]}>' does not have type '{typeof(T)}'");
|
|
||||||
|
|
||||||
if (guildId.HasValue)
|
if (guildId.HasValue)
|
||||||
return ConfigurationRepository.GetValue(realKey, guildId.Value, defaultValue);
|
return ConfigurationRepository.GetValue(realKey, guildId.Value, defaultValue);
|
||||||
@ -68,6 +62,11 @@ namespace ChaosBot.ConfigHelpers
|
|||||||
return _appSettingsWrapper.GetValue(realKey, defaultValue);
|
return _appSettingsWrapper.GetValue(realKey, defaultValue);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public T GetValue<T>(string key)
|
||||||
|
{
|
||||||
|
return GetValue<T>(key, default);
|
||||||
|
}
|
||||||
|
|
||||||
public T GetValueGlobalDefault<T>(string key, ulong? guildId = null)
|
public T GetValueGlobalDefault<T>(string key, ulong? guildId = null)
|
||||||
{
|
{
|
||||||
bool keyExists = TryGetKeyFromRegexMatch(key, out string realKey);
|
bool keyExists = TryGetKeyFromRegexMatch(key, out string realKey);
|
||||||
@ -80,11 +79,6 @@ namespace ChaosBot.ConfigHelpers
|
|||||||
return GetValue(key, defaultValue);
|
return GetValue(key, defaultValue);
|
||||||
}
|
}
|
||||||
|
|
||||||
public T GetValue<T>(string key)
|
|
||||||
{
|
|
||||||
return GetValue<T>(key, default);
|
|
||||||
}
|
|
||||||
|
|
||||||
public IConfigurationSection GetSection(string key)
|
public IConfigurationSection GetSection(string key)
|
||||||
{
|
{
|
||||||
return _appSettingsWrapper.GetSection(key);
|
return _appSettingsWrapper.GetSection(key);
|
||||||
@ -99,6 +93,19 @@ namespace ChaosBot.ConfigHelpers
|
|||||||
return ConfigurationFlags.ToImmutableDictionary();
|
return ConfigurationFlags.ToImmutableDictionary();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private string EnsureKeyExistsAndTypeCorrect<T>(string key)
|
||||||
|
{
|
||||||
|
bool keyExists = TryGetKeyFromRegexMatch(key, out string realKey);
|
||||||
|
|
||||||
|
if (!keyExists)
|
||||||
|
throw new ArgumentException($"Configuration does not contain key '{key}'");
|
||||||
|
|
||||||
|
if (!(ConfigurationFlags[realKey].type == typeof(T)))
|
||||||
|
throw new ArgumentException(
|
||||||
|
$"Configuration flag '{realKey}<{ConfigurationFlags[realKey]}>' does not have type '{typeof(T)}'");
|
||||||
|
return realKey;
|
||||||
|
}
|
||||||
|
|
||||||
private bool TryGetKeyFromRegexMatch(string key, out string realKey)
|
private bool TryGetKeyFromRegexMatch(string key, out string realKey)
|
||||||
{
|
{
|
||||||
if (ConfigurationFlags.ContainsKey(key))
|
if (ConfigurationFlags.ContainsKey(key))
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user