项目 LZJYXT 感触 ---- 瘦方法

       最近刚开始真正的接触到系统型的项目, 要帮一个学校搞一个教研系统, 主要是对一次考试中学生的成绩进行大量的分析! 涉及很多计算!

在完成类库到类库的调用时, 体会到之前在书上看到的关于瘦方法和胖方法的讨论! 在这里做一下记录:

       比如说, 写一个计算方差的函数, 我的第一反应是这样写的:

 1 /// <summary> 
 2 /// 计算方差 
 3 /// </summary> 
 4 /// <param name="mark"></param> 
 5 /// <returns></returns> 
 6 public static double Variance(List<float> mark) 
 7 { 
 8     int markCount = mark.Count; //求分数个数
 9 
10     double markSum = 0; // 总分数 
11     double markLoop = 0; // 累加数
12 
13 
14     for (int i = 0; i < markCount; i++) 
15     { 
16         markSum += mark[i]; 
17     }
18 
19     double aver= markSum / markCount; // 平均数
20 
21  
22 
23     for (int i = 0; i < markCount; i++) 
24     { 
25         markLoop += Math.Pow((mark[i] - aver), 2); //两个数相减 再平方 再累加 
26     }
27 
28     return markLoop / markCount; 
29 }

 

      后来发现很多计算都是要先求平均数, 然后做其它处理, 于是把求平均数的方法给抽象出来:

 1 /// <summary> 
 2 /// 计算算术平均数 
 3 /// </summary> 
 4 /// <param name="mark"></param> 
 5 /// <returns></returns> 
 6 public static double Average(List<float> mark) 
 7 { 
 8     CheckList(mark);
 9 
10     int markCount = mark.Count; //分数个数 
11     double markSum = 0; //计分数
12 
13     for (int i = 0; i < markCount; i++) 
14     { 
15         markSum += mark[i]; 
16     }
17 
18     double markAverage = markSum / markCount; 
19         
20     return markAverage; 
21 }
22 
23 
24 /// <summary> 
25 /// 计算方差 
26 /// </summary> 
27 /// <param name="mark"></param> 
28 /// <returns></returns> 
29 public static double Variance(List<float> mark) 
30 { 
31     int markCount = mark.Count; //求分数个数 
32     double markLoop = 0; //计分数
33 
34     double aver = Average(mark);
35 
36     for (int i = 0; i < markCount; i++) 
37     { 
38         markLoop += Math.Pow((mark[i] - aver), 2); //两个数相减 再平方 再累加 
39     }
40 
41     return markLoop / markCount; 
42 }

      在实际类库调用时, 又发现, 在数据显示时, 一般是要求显示 “平均数” 这个数值的, 因此就会发生这样的类似情况:

1 Average(mark);   // 求平均数
2 
3 Variance(mark);   // 求方差

      这样子, 其实 mark 列表被遍历了三次, 求了两次平均数, 这不是一个很大的浪费么? 所以, 左思右想, 又对上面的方法做了改造:

 1 /// <summary> 
 2 /// 计算算术平均数 
 3 /// </summary> 
 4 /// <param name="mark"></param> 
 5 /// <returns></returns> 
 6 public static double Average(List<float> mark) 
 7 { 
 8     CheckList(mark);
 9 
10     int markCount = mark.Count; //分数个数 
11     double markSum = 0; //计分数
12 
13     for (int i = 0; i < markCount; i++) 
14     { 
15         markSum += mark[i]; 
16     }
17 
18     double markAverage = markSum / markCount; 
19         
20     return markAverage; 
21 }
22 
23 /// <summary> 
24 /// 计算方差 
25 /// </summary> 
26 /// <param name="mark"></param> 
27 /// <returns></returns> 
28 public static double Variance(List<float> mark, double aver) 
29 { 
30     int markCount = mark.Count; //求分数个数 
31     double markLoop = 0; //计分数
32 
33     for (int i = 0; i < markCount; i++) 
34     { 
35         markLoop += Math.Pow((mark[i] - aver), 2); //两个数相减 再平方 再累加 
36     }
37 
38     return markLoop / markCount; 
39 }
40 
41 /// <summary> 
42 /// 计算方差 
43 /// </summary> 
44 /// <param name="mark"></param> 
45 /// <returns></returns> 
46 public static double Variance(List<float> mark) 
47 { 
48     double aver = Average(mark); //求算数平均数 
49     return Variance(mark, aver);
50 
51 }

      其实 Variance(List<float> mark, double aver); 才是求方差函数正确的形式, (呃, 至少在我项目中的这种情况下是这样的.) 因为方差, 本身就是将每个数与平均数做比较得出来的结果, 求平均数不是它应该有的功能, 也不是它的职责, 所以在它的计算方法中, 平均数应该已参数的形式出现, 而不是等着它自己去求! 而 Variance(List<float> mark) 则是为了类库的调用者的方便, 写出的一个重载!

 

      以上就是我这两天在类库的设计和调用的时候对 “瘦方法” 这一词的亲身体会! 一个设计合理的方法, 它的职责和功能应该得到严格的控制, 不能把所有的功能代码都过程式的写到一个函数里面, 这样就会使函数变得臃肿, 调用的时候难免会有些多余的功能造成资源浪费!

      这其中没有什么绝对的准则界限, 要靠 Coder 依靠自己的经验, 结合实际情况去判断, 就好像上面的例子, 如果在显示方差时, 不需要显示 “平均数”, 则完全没必要去把一个计算方差函数分得那么细, 直接按第一种情况写出就够了! 所以, 具体情况还得具体分析! 同时也说明, 开始敲键盘前, 动动笔或动动鼠标为代码设计一个框架的重要性!

转载于:https://www.cnblogs.com/mend/archive/2011/10/02/2197995.html