求问一道 C#笔试题，想知道最好的解答是什么

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<Shape> 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<ICalc> shapes)     {         double totalArea = 0;         if (shapes != null)         {             totalArea = shapes.Sum(shape => shape.GetAera());         }         return totalArea;     } } 

• 資深大佬 : across

  我猜原意不止 oop，还要你改几个细节，比如 int 应该是 float，面积 float 精度就足够。

• 資深大佬 : forgottencoast

  源代码最主要的问题就是每增加一个 Shape，都要改动 CalculateTotalArea 方法。
  写法很多，你这个写法可以了。
  剩下的就是细节问题了，比如#1 所说的，Width 和 Length 用 int 肯定不妥。
  CalculateTotalArea 接收的参数类型要更抽象一点，List<T>肯定不好。
  你定义的接口 ICalc 命名太宽，和方法名 GetArea 不够匹配。

• 主 資深大佬 : richards64

  @forgottencoast CalculateTotalArea(IEnumerable<ICalc> shapes) 好一点？

• 資深大佬 : andrewpsy

  上面都说的很好了，还有个可能是面试官想测你知不知道 strategy pattern 。你要是想 over engineering 可以实现一下玩玩。

• 主 資深大佬 : richards64

  @andrewpsy 实际情况是面试官好像对我的回答没有什么异议，对我的 C#基础还是挺满意的，但是最后拒掉了我的理由是因为这个岗位需要精通 SQL 优化，这我就不会了……

• 資深大佬 : yazoox

  这就是“十动然拒”？
  奇怪，如果岗位是对 SQL 技能有特别需求，一开始知道你不会（比如：简历），那还面试啥？这不是浪费大家时间么？
• 資深大佬 : Rwing

  挺好的，我觉得 HR 只是不想招人，换公司继续面吧

• 資深大佬 : forgottencoast

  @richards64 #3
  是的。
  越抽象的接口适用范围越广，如果你用不到更具体接口的其他成员，就应该用更抽象的接口。
  IEnumerable<T>是最抽象的接口了，所有的泛型集合都实现了它，一般设计时可以考虑先用它，除非它不符合需求，再考虑更具体的接口。

• 資深大佬 : yamatamain

  @richards64 总会有一个理由。

• 資深大佬 : SWALLOWW

  第一眼 shapes.Count()有问题…

• 資深大佬 : beingWH

  感觉问题不大。

• 資深大佬 : beingWH

  “`
  interface ICalc
  {
  float GetAera();
  }

  class Circle : ICalc
  {
  public float Radius { get; set; }

  public float GetAera()
  {
  return Math.PI * Radius * Radius;
  }
  }

  class Rectangle : ICalc
  {
  public float Length { get; set; }
  public float Width { get; set; }

  public float GetAera()
  {
  return Length * Width;
  }
  }

  class Triangle : ICalc
  {
  public float Bottom { get; set; }
  public float Height { get; set; }

  public float GetAera()
  {
  return Bottom * Height * 0.5;
  }
  }

  static class ShapeCalculator
  {
  public static float CalculateTotalArea(this List<ICalc> shapes)
  {
  float totalArea = 0;
  if (shapes != null)
  {
  totalArea = shapes.Sum(shape => shape.GetAera());
  }
  return totalArea;
  }
  }
  “`

• 資深大佬 : kop1989

  我有个想法不知道对不对。

  觉得可以在你的基础上再添加一个 Shape 转换为 ICalc 实现类的逻辑。
  因为从实际项目的角度讲，你再怎么重构，数据源也是不变的。
  你这个重构最终丢了逻辑。既你把 ICalc 的实现类抉择抛给了调用 CalculateTotalArea 的人。
  原来的数据源是 List<Shape>。现在传进来的还应该是 List<Shape>。

• 資深大佬 : Caskia

  @richards64 拒绝的问题应该不在你这个回答上面，敢问这是哪家的面试题？

• 主 資深大佬 : richards64

  @kop1989 这个没太看明白，能具体演示一下吗

• 主 資深大佬 : richards64

  @Caskia 确实挺奇怪的，我看 JD 上有要求掌握 WPF 就投了简历，简历上也表示出了我只有 WPF 经历，然后就让我去面试了，去了之后发现实际的主要工作内容是.NET Core 的后端开发，只是个别项目组有少量的老旧项目涉及到 WPF