То, что вы сделали, чем-то похоже на Value Object, но обладает следующими недостатком - CheckedValue можно создать с разными проверяющими функциями - то есть информация о виде пройденной проверки теряется.
Если внести информацию об ограничении в имя типа - как раз и получится паттерн Value Object.
Пример как может Value Object выглядеть:
public struct Temperature
{
private readonly decimal value;
private Temperature(decimal value)
{
this.value = value;
}
public decimal ToCelsius => value − 273.15m;
public decimal ToKelvins => value;
public decimal ToFahrenheit => value * 1.8m − 459.67m;
public static Temperature FromCelsius(decimal value) => FromKelvins(value + 273.15m);
public static Temperature FromFahrenheit(decimal value) => FromKelvins((value + 459.67m) / 1.8m);
public static Temperature FromKelvins(decimal value)
{
if (value < 0)
throw new ArgumentOutOfRangeException("value", "Такой температуры не бывает");
return new Temperature(value);
}
public static Temperature? TryConvertFromCelsius(decimal value) => TryConvertFromKelvins(value + 273.15m);
public static Temperature? TryConvertFromFahrenheit(decimal value) => TryConvertFromKelvins((value + 459.67m) / 1.8m);
public static Temperature? TryConvertFromKelvins(decimal value)
{
if (value < 0)
return null;
return new Temperature(value);
}
public static Temperature Zero { get; } = new Temperature(0);
}
Отмечу также важную особенность любых структур: структуру можно инициализировать нулями через конструкцию default(...)
или открытый конструктор по умолчанию (который всегда присутствует и который нельзя переопределить), поэтому если вы делаете Value Object структурой - нужно обязательно убедиться что нулевое значение будет корректным значением.
Если так сделать не получается - то Value Object надо делать классом.
GetCheckedValue
, если он не является статическим?