我正在创建一个非常小的应用程序以演示扎实的原则以及构建器模式的简短实现,是否有人对如何改进此方法或如何打破扎实的原则有任何反馈?该应用是一个小型应用,仅可转换单位,例如码到米,英寸到厘米。
界面
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核心构建的,任何反馈都非常感激:)
答案 0 :(得分:1)
这些不一定与SOLID原则相关,但是如果我在工作中查看此代码,我会提到以下几件事
'/n'
。如果要从'/n'
更改为'/r/n'
,则必须在Converter.Convert
中将其更改5次。处理此问题的更好方法是将其存储在变量中,或者允许通过构造函数进行设置。var
代替明确声明变量类型,例如var d = double.Parse(input, CultureInfo.InvariantCulture) * ConversionRate;
我希望这会有所帮助:)
答案 1 :(得分:0)
首先,看起来像是复制和粘贴错误...您的第二个AddConversionRate方法应该是AddConversionUnits。
关于SOLID原则,您不可能总是在像这样的小代码示例中显示所有原则的证据。
我写了an article on the SOLID principles,您可能会觉得有趣