扎实的原则/建造者模式

时间:2018-11-11 10:20:38

标签: .net design-patterns interface .net-core solid-principles

我正在创建一个非常小的应用程序以演示扎实的原则以及构建器模式的简短实现,是否有人对如何改进此方法或如何打破扎实的原则有任何反馈?该应用是一个小型应用,仅可转换单位,例如码到米,英寸到厘米。

界面

public interface IConverter
{
    double ConversionRate { get; set; }
    string[] ConvertedUnits { get; set; }
    string DisplayText { get; set; }

    string[] Convert(string input);
}

类实现接口

 public class Converter : IConverter
    {
        public double ConversionRate { get; set; }
        public string[] ConvertedUnits { get; set; }
        public string DisplayText { get; set; }

        public Converter(double conversionRate, string[] convertedUnits, string displayText)
        {
            ConversionRate = conversionRate;
            ConvertedUnits = convertedUnits;
            DisplayText = displayText;
        }

        public string[] Convert(string input)
        {
            if (!input.Contains('\n'))
            {
                double d = double.Parse(input, CultureInfo.InvariantCulture) * ConversionRate;

                string[] array = new string[1];
                array[0] = d.ToString();

                return array;
            }
            else
            {
                double[] doubles = new double[input.Split('\n').Count()];

                for (int i = 0; i < input.Split('\n').Count(); i++)
                {
                    double value = double.Parse(input.Split('\n')[i]);
                    doubles[i] = value;

                    string.Format("{0}", value * ConversionRate);
                }

                string[] strings = new string[doubles.Length];

                for (int i = 0; i < input.Split('\n').Length; i++)
                {
                    strings[i] = string.Format("{0}", doubles[i] * ConversionRate);
                }

                return strings;
            }
        }
    }

Builder(抽象类)

public abstract class ConverterBuilder
    {
        protected double _conversionRate;
        protected string[] _convertedUnits;
        protected string _displayText;

        public ConverterBuilder AddConversionRate(double conversionRate)
        {
            _conversionRate = conversionRate;
            return this;
        }

        public ConverterBuilder AddConversionRate(string[] convertedUnits)
        {
            _convertedUnits = convertedUnits;
            return this;
        }

        public ConverterBuilder AddDisplayText(string displayText)
        {
            _displayText = displayText;
            return this;
        }

        public Converter Build()
        {
            return new Converter(_conversionRate, _convertedUnits, _displayText);
        }

    }

构建器实现抽象构建器类

public class UnitConverterBuilder : ConverterBuilder
{
    public UnitConverterBuilder()
    {

    }
}

控制器

public IActionResult Convert(string text, double conversionRate)
{
    Converter converter = new UnitConverterBuilder()
    .AddConversionRate(conversionRate)
    .Build();

    string output = "";
    for (int i = 0; i < converter.Convert(text).Count(); i++)
    {
        output += converter.Convert(text)[i] + Environment.NewLine;
    }

    return Content(output);

}

查看

    

输入要转换的值

<form asp-controller="Home" asp-action="Convert" method="post">
    <textarea id="text" name="text" rows="10" cols="40"></textarea>
    <select name="conversionRate">
        <option value="">Please Select</option>
        <option value="0.9144">Yards to Meters</option>
        <option value="2.54">Inches To Centimeters</option>
    </select>
    <button>Convert</button>
</form>

该应用是使用.net核心构建的,任何反馈都非常感激:)

2 个答案:

答案 0 :(得分:1)

这些不一定与SOLID原则相关,但是如果我在工作中查看此代码,我会提到以下几件事

  • 过多使用了字符'/n'。如果要从'/n'更改为'/r/n',则必须在Converter.Convert中将其更改5次。处理此问题的更好方法是将其存储在变量中,或者允许通过构造函数进行设置。
  • 您可以使用var代替明确声明变量类型,例如var d = double.Parse(input, CultureInfo.InvariantCulture) * ConversionRate;
  • 可变名称,简洁的代码应该易于阅读,因此不要使用像d和double这样的名称,而使用易于阅读代码的人理解的名称
  • 您不需要在IConverter接口上指定Converter的属性,这是因为该接口只需要公开对象的行为,并且这些属性更多地是实现细节。删除这些属性将使您拥有多个实现IConvert的对象,而不必强行拥有这些特定的属性(接口隔离)

我希望这会有所帮助:)

答案 1 :(得分:0)

首先,看起来像是复制和粘贴错误...您的第二个AddConversionRate方法应该是AddConversionUnits。

关于SOLID原则,您不可能总是在像这样的小代码示例中显示所有原则的证据。

  1. 单一责任-如前所述,转换器中的显示文本可能违反了这一原则。
  2. 打开/关闭-您在这里没有扩展任何类,因此很难在您的示例中显示。
  3. 利斯科夫(Liskov)的换人-没有子类,因此也没有违反这一证据的证据。
  4. 接口隔离-DisplayText应该从IConverter中拉出。
  5. 依赖倒置-最大的违规在这里。详细信息是针对具体类型而不是抽象实现的。在ConverterBuilder上构建应该返回IConverter而不是具体的Converter。您的控制器Convert操作应将转换器变量声明为IConverter而不是Converter具体类型,并且我还将该依赖项注入到构造函数中的控制器类中(再次作为IConverter),然后在操作中使用该依赖项,而不是声明局部变量。

我写了an article on the SOLID principles,您可能会觉得有趣