专栏首页哲学驱动设计重构实践:体验interface的威力(二)

重构实践:体验interface的威力(二)

前一篇博客 写了在这次重构中,如何找到关键的概念,并将它提取成为接口。这样,重构的方案基本上就已经被确定了。这篇博客主要说一些有意义的细节:

抽象实现

    提取接口模型后,各“指标”类已经呈现出一种正确的逻辑关系。那么,现在更重要的就是重构上文中提到的“普通计算过程”。由于计算过程依然有很多种,并且有通用的抽象部分。再加上接口模型已经定义出了大量重要的业务属性,所以我在这里使用约束的泛型抽象类来定义最上层基类。这里捡重点的几个计算类,组织类图如下:

图1 计算过程的类的结构

重点细节

    最IndicatorCalcBase<TModel>作为最上层基类,定义如下:

public abstract class IndicatorCalcBase<TModel> where TModel : BusinessBase, IProjectIndicator {……}

    在这里使用了IProjectIndicator来约束TModel泛型,使得IndicatorCalcBase类的计算能力十分强大。这是因为这个类只处理抽象的“指标”概念IProjectIndicator,而这个接口中定义了计算业务所需要使用的95%的属性、方法。所以原来的那些重复的计算类的代码,基本上都已经全部在这个类实现了。该类使用模板方法定义了一个固定的处理过程:

/// <summary>
/// 计算指标,不需要返回指标。
/// </summary>
public void Caculate()
{
    this.PreCollect();

    this.Collect();

    this.Save();

    this.CalculateEnd();
}

看上去,感觉是不是有点象ASP.NET的WebForm的PreLoad、Load、Unload啊?:) 这些方法在基类都已经默认实现,并标记为虚方法,这样子类就可以重写它们以添加新的功能,如下:

图2 IndicatorCalcBase<TModel>类的设计

这里把部分方法设置为protected,为子类开放出必要的使用权限,以满足特定的计算需求。而它无法实现,必须延迟到子类实现的抽象方法,就被提取为abstract方法。这个类的设计,成了整个重构的核心,这里需要深刻理解原有代码并有一定的重构经验和设计经验。

另外,这个类中的计算过程为排序所构建的内部类PBSIdComparer,实现了多种IComparer接口,以满足对多种不同的对象进行同一功能(根据PBSId)的排序。这里也把它的设计贴一下,我觉得这里比较好的设计是Convert方法的转换模式、私有的Compare方法的API思想(虽然这并不是本次重构的重点,不过暂时在这做一记录。 :))。见图:

图3 统一实现的比较类

接下来,是计算所有量指标的类QuantityCalcBase<TModel>:

图4 量指标计算基类。

注意到,这里面的类的定义中,已经把TModel约束为IProjectQuantityIndicator,使得计算过程可以对量指标进行特殊处理。

有了以上两个基类,其它子类的实现变得十分简单,几个量指标类都实现如下:

public class QuantityCalc_Entity : QuantityCalcBase<ProjectIndicatorEntity>
{
    public QuantityCalc_Entity(Project project, BudgetPhaseType budgetType, PBSs pbss)
        : base(project, budgetType, pbss) { }

    protected override string GetPreCollectProcedure()
    {
        return "dbo.CreateBudgetEntityQuantityQuery";
    }
    protected override ExtendedBindingList<ProjectIndicatorEntity> CreateAddList()
    {
        return ProjectIndicatorEntityList.GetNew();
    }
    protected override IList<ProjectIndicatorEntity> GetListByProject()
    {
        var models = ProjectIndicatorEntityList.GetListByProjectId(this.Project.Id, this.BudgetType);
        return models;
    }
}

SQL优化

    我曾经尝试不进行代码重构,而直接使用profiler寻找性能点来对原来的代码进行优化。只可惜损耗和结构也息息相关,结果那次优化只给性能带来较小的提升。现在重构过后,类设计得比较通畅了,那么整个运算过程中性能的损耗点就更容易被找到,做起优化来也更得心应手。比如,我可以把一些分散的没必要的多次查询,合并在一起进行一次查询……

    举个具体的例子,原代码中有个指标的计算过程中,会递归+循环去调用数据库中一个冗长的存储过程,来计算每个树状结点的指标值,那个存储过程我看着就头大了,大概150行吧 :(。而现在,在子类中,实现父类方法中,我只写了一个SQL语句,不但不使用存储过程,而且也不再需要多次访问数据库。这样的效果就是,原来的时间是14S,现在只要:1S。代码如下:

        protected override IList<ProjectCostIndicator> GetListByProject()
        {
            var models = ProjectCostIndicatorList.GetListByProjectId(this.Project.Id, this.BudgetType);
            this.PrepareAllModels(models);
            return models;
        }
        private void PrepareAllModels(IList<ProjectCostIndicator> models)
        {
            string sql = this.QueryTotalCostSql();
            //执行sql,并对模型进行赋值。
         }
        private string QueryTotalCostSql()
        {
            return string.Format(@"
SELECT 
ISNULL(SUM(CBFBFXBQItem.Amount), 0)
+ ISNULL(SUM(CBFGQBQItem.Amount), 0)
+ ISNULL(SUM(CBMeasureItem.Amount), 0)
+ ISNULL(SUM(CBOtherItem.Amount), 0)
AS TotalCost, Q.PBSId
FROM BudgetgetCostItemQuery as Q
LEFT OUTER JOIN CBFBFXBQItem ON CBFBFXBQItem.ID=Q.ItemId
LEFT OUTER JOIN CBFGQBQItem ON CBFGQBQItem.ID=Q.ItemId
LEFT OUTER JOIN CBMeasureItem ON CBMeasureItem.ID=Q.ItemId AND CBMeasureItem.Name <> ''
LEFT OUTER JOIN CBOtherItem ON CBOtherItem.ID=Q.ItemId AND CBOtherItem.Name <> '' AND (CBOtherItem.IsNotInPBS=0 OR CBOtherItem.IsNotInPBS is null) 
WHERE BudgetType = {1} AND ProjectId = '{0}'
GROUP BY Q.PBSId
HAVING NOT (Q.PBSId IS NULL)
", this.Project.Id.ToString(), (int)this.BudgetType);
        }

总结

    目前的整个普通计算过程,耗时只需要5秒种。其实还有很多地方我并没有对其进行优化,不过相对于原来的2分钟,已经是完全能让客户接受了。所以暂时就优化至此。而20分钟的复杂计算过程,估计也能够降低很多,不过暂时客户并不使用,且项目的其它需求也比较紧,所以只有先不管了。:)

接口能够抽象的、逻辑的的表示现实世界的概念。合理使用它,会给我们的编程带来很多好处和一些意想不到的效果。

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

我来说两句

0 条评论
登录 后参与评论

相关文章

  • 产品前端重构(TypeScript、MVC框架设计)

    最近两周完成了对公司某一产品的前端重构,本文记录重构的主要思路及相关的设计内容。 公司期望把某一管理类信息系统从项目代码中抽取、重构为一个可复用的产品。该系统的...

    用户1172223
  • 重构实践:体验interface的威力(一)

    背景     GIX4是一个建筑行业的指标计算软件,用于数据统计、分析。导入的大量数据,大部分呈现逻辑上的树状结构(关于它的重构,见:《重构一个繁琐的数据结构》...

    用户1172223
  • WPF Command CanExecute 的执行逻辑

    WPF 中命令的 CanExecute 执行逻辑,我一直觉得比较麻烦,画出来就清晰多了,备忘: ?

    用户1172223
  • 专家谈:2014年云计算市场大气候

    去年,我们的云计算专家们对2013年云计算市场的大环境进行了预测。有些预言得到了市场的验证,例如亚马逊公司网络服务(AWS)继续占据了市场的主导地...

    小莹莹
  • 大数据技术之_18_大数据离线平台_04_数据分析 + Hive 之 hourly 分析 + 常用 Maven 仓库地址

    通过表结构可以发现,只要维度id确定了,那么 new_install_users 也就确定了。

    黑泽君
  • 2020:我们处在一个什么样的技术浪潮当中?

    在软件架构领域,经历了从单体应用到 SOA 再到微服务; 在云计算领域,经历了从虚拟机到容器; 在数据库领域,从关系数据库到 NoSQL 再到 NewSQL...

    一个会写诗的程序员
  • 雾计算简史(转载)

    1961 年,人工智能之父麦卡锡在一次会议上提出了「效用计算」这个概念,第一次体现了这种共享资源的思想。当时计算设备的价格非常昂贵,远非普通企业和机构所能承受,...

    飞天小子
  • 如何将 Web 框架迁移到 Serverless

    Serverless 通常翻译为「无服务架构」,是一种软件系统设计架构思想和方法,并不是一个开发框架或者工具。他的出现是为了让开发者更加关注业务的开发,而将繁...

    腾讯云serverless团队
  • javascript基础练习:借用原型对象实现继承 & 排他思想 & 页面带参数自动跳转

    定义两个构造函数Student和Person,利用原型对象模拟上面类的继承关系。就是用对象模拟类,然后用call方法在函数内调用父类的方法即可。 注意call...

    Enterprise_
  • ionic 登录 注销 (ts格式) 原

    在输入账号与密码点击登录时如果成功,登录页面隐藏,显示出登录成功后的用户中心,然后点击用户中心的注销,弹出登录页面

    tianyawhl

扫码关注云+社区

领取腾讯云代金券