这是一道关于C#笔试题的讨论，涉及重构计算几何形状（圆形、矩形、三角形）面积的代码。原始代码使用Shape类和ShapeCalculator类，存在冗余检查和条件判断。提出的问题是：如何改进这段代码以更优雅地计算多种形状的面积？回答者提出了使用接口ICalc和具体形状类（Circle、Rectangle、Triangle）的实现，通过LINQ简化了计算总区域的逻辑。寻求是否有更好的解决方案。
题目如下：
这是一段用于计算多个几何形状面积之和的程序
指出程序段存在的不妥之处，并重构代码
在 1 的基础上增加一种可计算的几何形状类型三角形，其面积为 底×高×1/2
public class Shape
{
public int Type { get; set; }
public int Radius { get; set; }
public int Width { get; set; }
public int Length { get; set; }
}
public class ShapeCalculator
{
public static double CalculateTotalArea(List shapes)
{
double totalArea = 0;
if (shapes != null && shapes.Count() > 0)
{
foreach (Shape shape in shapes)
{
if (shape.Type == 1)
{
totalArea += Math.PI * shape.Radius * shape.Radius;
}
else if (shape.Type == 2)
{
totalArea += shape.Length * shape.Width;
}
}
}
return totalArea;
}
}
我的回答是这样的：
interface ICalc
{
double GetAera();
}
class Circle : ICalc
{
public int Radius { get; set; }
public double GetAera()
{
return Math.PI * Radius * Radius;
}
}
class Rectangle : ICalc
{
public int Length { get; set; }
public int Width { get; set; }
public double GetAera()
{
return Length * Width;
}
}
class Triangle : ICalc
{
public int Bottom { get; set; }
public int Height { get; set; }
public double GetAera()
{
return Bottom * Height * 0.5;
}
}
class ShapeCalculator
{
public static double CalculateTotalArea(List shapes)
{
double totalArea = 0;
if (shapes != null)
{
totalArea = shapes.Sum(shape => shape.GetAera());
}
return totalArea;
}
}
凭我的水平和理解能力就只有写到这种程度了……有什么更好的写法吗？
扫一扫
1875
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
