Рефакторинг двух базовых классов
-
09-06-2019 - |
Вопрос
Как бы вы провели рефакторинг этих двух классов, чтобы абстрагироваться от сходства?Абстрактный класс?Простое наследование?Как будут выглядеть реорганизованные классы?
public class LanguageCode
{
/// <summary>
/// Get the lowercase two-character ISO 639-1 language code.
/// </summary>
public readonly string Value;
public LanguageCode(string language)
{
this.Value = new CultureInfo(language).TwoLetterISOLanguageName;
}
public static LanguageCode TryParse(string language)
{
if (language == null)
{
return null;
}
if (language.Length > 2)
{
language = language.Substring(0, 2);
}
try
{
return new LanguageCode(language);
}
catch (ArgumentException)
{
return null;
}
}
}
public class RegionCode
{
/// <summary>
/// Get the uppercase two-character ISO 3166 region/country code.
/// </summary>
public readonly string Value;
public RegionCode(string region)
{
this.Value = new RegionInfo(region).TwoLetterISORegionName;
}
public static RegionCode TryParse(string region)
{
if (region == null)
{
return null;
}
if (region.Length > 2)
{
region = region.Substring(0, 2);
}
try
{
return new RegionCode(region);
}
catch (ArgumentException)
{
return null;
}
}
}
Решение
Если у вас нет веской причины для рефакторинга (поскольку вы собираетесь добавить больше классов, подобных тем, которые есть в ближайшем будущем), штраф за изменение дизайна для такого маленького и надуманного примера перевесит выигрыш в обслуживании или накладных расходах в этом сценарии.В любом случае, вот возможный дизайн, основанный на общих и лямбда-выражениях.
public class TwoLetterCode<T>
{
private readonly string value;
public TwoLetterCode(string value, Func<string, string> predicate)
{
this.value = predicate(value);
}
public static T TryParse(string value, Func<string, T> predicate)
{
if (value == null)
{
return default(T);
}
if (value.Length > 2)
{
value = value.Substring(0, 2);
}
try
{
return predicate(value);
}
catch (ArgumentException)
{
return default(T);
}
}
public string Value { get { return this.value; } }
}
public class LanguageCode : TwoLetterCode<LanguageCode> {
public LanguageCode(string language)
: base(language, v => new CultureInfo(v).TwoLetterISOLanguageName)
{
}
public static LanguageCode TryParse(string language)
{
return TwoLetterCode<LanguageCode>.TryParse(language, v => new LanguageCode(v));
}
}
public class RegionCode : TwoLetterCode<RegionCode>
{
public RegionCode(string language)
: base(language, v => new CultureInfo(v).TwoLetterISORegionName)
{
}
public static RegionCode TryParse(string language)
{
return TwoLetterCode<RegionCode>.TryParse(language, v => new RegionCode(v));
}
}
Другие советы
Это зависит от того, если они не собираются делать намного больше, то я, вероятно, оставлю их как есть - ИМХО, в этом случае выделение вещей, вероятно, будет более сложным.
Это довольно простой вопрос, и для меня он ужасно пахнет домашним заданием.
Очевидно, вы можете увидеть общие элементы кода, и я почти уверен, что вы можете попытаться сделать это самостоятельно, поместив такие вещи в суперкласс.
Возможно, вы могли бы объединить их в Locale
Класс, который хранит как код языка, так и код региона, имеет средства доступа для региона и языка, а также одну функцию синтаксического анализа, которая также позволяет использовать такие строки, как "en_gb"...
Я видел, как локали обрабатываются в различных средах.
Эти два, в их нынешнем виде, не будут хорошо подвергаться рефакторингу из-за статических методов.
В конечном итоге вы либо получите какой-то фабричный метод базового класса, который возвращает тип этого базового класса (который впоследствии потребует приведения), либо вам понадобится какой-то дополнительный вспомогательный класс.
Учитывая количество лишнего кода и последующее приведение к соответствующему типу, оно того не стоит.
- Создайте общий базовый класс (например,
AbstractCode<T>
) добавьте абстрактные методы, такие как
protected T GetConstructor(string code);
переопределить в базовых классах, например
protected override RegionCode GetConstructor(string code) { return new RegionCode(code); }
Наконец, сделайте то же самое с
string GetIsoName(string code)
, напримерprotected override GetIsoName(string code) { return new RegionCode(code).TowLetterISORegionName; }
Это приведет к рефакторингу обоих.Крис Кимптон поднимает важный вопрос о том, стоят ли затраченные усилия.
Я уверен, что есть лучшее решение на основе дженериков.Но все же попробовал.
РЕДАКТИРОВАТЬ:Как говорится в комментарии, статические методы не могут быть переопределены, поэтому одним из вариантов было бы сохранить их, использовать объекты TwoLetterCode и приводить их, но, как уже отмечал другой человек, это довольно бесполезно.
Как насчет этого?
public class TwoLetterCode {
public readonly string Value;
public static TwoLetterCode TryParseSt(string tlc) {
if (tlc == null)
{
return null;
}
if (tlc.Length > 2)
{
tlc = tlc.Substring(0, 2);
}
try
{
return new TwoLetterCode(tlc);
}
catch (ArgumentException)
{
return null;
}
}
}
//Likewise for Region
public class LanguageCode : TwoLetterCode {
public LanguageCode(string language)
{
this.Value = new CultureInfo(language).TwoLetterISOLanguageName;
}
public static LanguageCode TryParse(string language) {
return (LanguageCode)TwoLetterCode.TryParseSt(language);
}
}