我们将如何重构这两个类以抽象出相似之处？抽象类？简单继承？重构的类是什么样的？

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;
        }
    }
}

解决方案

回答

这取决于，如果他们不打算做更多的事情，那么在这种情况下，我可能会把它们留在原处，因为恕我直言，将东西分解出来可能会更复杂。

回答

这是一个非常简单的问题，对我来说，这就像是一项家庭作业。

我们显然可以在代码中看到常见的地方，而且我敢肯定，我们可以通过将此类内容放入超类来自己尝试一下。

回答

我们可以将它们组合成一个Locale类，该类存储语言代码和区域代码，具有用于区域和语言的访问器，以及一个解析函数，该函数还允许使用诸如" en_gb"之类的字符串...

这就是我看到在各种框架中处理语言环境的方式。

回答

就目前的情况而言，由于静态方法，这两种方法无法很好地重构。

我们可能会在基类上得到某种工厂方法，该方法会返回该基类的一种类型(随后将需要进行强制转换)，或者我们需要某种其他的辅助类。

考虑到额外的代码量以及随后转换为适当类型的代码，这是不值得的。

回答

我确定有一个更好的基于泛型的解决方案。但是还是给了它一个机会。

编辑：正如评论所说，静态方法不能被覆盖，因此一种选择是保留它并在周围使用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);
    }
}

回答

• 创建一个通用基类(例如，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;
}

那将重构两者。克里斯·金普顿(Chris Kimpton)确实提出了一个重要的问题，即这种努力是否值得。

回答

除非我们有充分的理由进行重构(因为我们将在不久的将来添加更多的类)，否则为这种小巧的示例更改设计的代价将克服这种情况下维护或者开销的增加。无论如何，这里是基于通用和lambda表达式的可能设计。

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));
    }
}