.net - 坚实的原则/建造者模式

问题描述

我正在创建一个非常小的应用程序来演示可靠的原则以及构建器模式的简要实现，是否有人对如何改进或如何打破可靠原则有任何反馈？该应用程序是一个小型应用程序，它只是转换单位，例如码到米，英寸到厘米。

界面

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

生成器（抽象类）

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 核心构建的，非常感谢任何反馈 :)

标签： .netdesign-patternsinterface.net-coresolid-principles

这些不一定与 SOLID 原则有关，但如果我在工作中查看此代码，我会提到以下几点

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

我希望这有帮助:)

.net - 坚实的原则/建造者模式

问题描述

解决方案

推荐阅读