重构Web Api程序(Api Controller和Entity)

Insus.NET较习惯做法，是在程序完成之后，抽一些少时间对写好的代码重新审视。比如这些天写的Web Api的程序来说，发现有很多值得重构代码。

开发ASP.NET MVC程序，与数据相关有关联的一般是在控制器与Entity实体之间。

初步重构是把Api控制器OrdersController.cs的程序移至实体OrderEntity.cs中去：
第一：
重构public IEnumerable<Order> Get()方法，


此方法移至Entity:

这样一移动，那在控制器的Get()方法，就可以改为：

第二：
 重构public void Post(Order o)方法：

在实体添加

控制器方法重构之后：



第三：
重构控制器OrderController另一个方法public void Post(Order o)：


把引方法移至实体：

在控制器中的方法被重构之后：

第四：
重构public void Put(Order order)方法，把此方法移至Entity中去，并创建一个新方法，稍作一些小修改：

 

在实体OrderEntity中新建的public void Update(Order order, string fileVirtualPath)方法：


 
重构之后，方法体内代码，可使用2句即可，并删除移走的代码：


第五，最后一个需要搬移的是Delete()方法：



在实体类中，创建一个全新的Delete()方法：

 

被重构之后，在控制器看到的Delete()方法，简洁多了。更多的逻辑处理代码不必在控制方法呈现。

 
最后OrderController.cs类别重构好的类，如下，删除注解的代码，下机标记1至标记5就是我们上面所重构过的方法：



现在我们可以关闭OrderControler.cs类，打开OrderEntity类，对这个类进行重构。找到下面三个方法中
public void AppendDataToExistingJsonFile(Order order, string fileVirtualPath)
public void Update(Order order, string fileVirtualPath)
public void Delete(Order order, string fileVirtualPath)

下面的代代码有冗余，如：



把它写成一个私有方法：



看到上面代码示例中的红色箭头否，说明三个方法内已经引用了重构好的代码了。



接下来我们又发现四个方法
public void PostAndSaveToFile(Order order, string fileVirtualPath)
public void AppendDataToExistingJsonFile(Order order, string fileVirtualPath)
public void Update(Order order, string fileVirtualPath)
 public void Delete(Order order, string fileVirtualPath)
中，下面这段代码是将List<Order>对象写成json文件，它们在这个OrderEntity类中，已经显冗余了：



写成一个私有方法：



这样四个方法均可以删除冗余部分的代码，引用这个私有的方法即可，不过第一个方法，public void PostAndSaveToFile(Order order, string fileVirtualPath)
引用此方法，有些问题，因为重构好的方法传入的参数是List<Order>，因此需要做些修改，如下：



还有
public void AppendDataToExistingJsonFile(Order order, string fileVirtualPath)
public void Update(Order order, string fileVirtualPath)
public void Delete(Order order, string fileVirtualPath)
三个方法中有一个功能是相同的，就是读取Json文件，并转换为List<Order>对象，虽然代码不一相同，但是我们是可以把它写成一个统一的：



这样子，三个方法就可以引用此方法：



在Update()和Delete()方法，还有2句，是实现删除文件的功能：

 if (System.IO.File.Exists(newFilePhysicalPath))
{
   File.Delete(newFilePhysicalPath);
}

可以重构写成一个私有方法并对程序重构如下:



OK，最后整理一下4个私有方法：