zoukankan      html  css  js  c++  java
  • ERP程序的公共代码中出现的问题 过度封装不方便维护

    在设计一个ERP程序时,提取公共的代码到一个通用类型库中是必要的。这样可以减少代码重复,提高代码利用率。

    但是,做任何事情都要有个度,有些公共的代码引起了过度封装,反而不利于代码的理解。

    举例说明如下

    public class ConfigHelper
    {
            /// <summary> /// Gets whether the specified path is a valid absolute file path. /// </summary> 
            /// <param name="path">Any path. OK if null or empty.</param> 
            static public bool IsValidPath(string path)
            {
                Regex r = new Regex(@"^(([a-zA-Z]:)|(\))(\{1}|((\{1})[^\]([^/:*?<>""|]*))+)$");
                return r.IsMatch(path);
            }
    
    
            public static string GetString(string key)
            {
                return System.Configuration.ConfigurationManager.AppSettings[key];
            }
    }

    第二个方法 GetString,我以为它的封装是不必要的。调用.NET框架的代码只有一行或简单的数行,对它进行封装,反而会引起理解上的障碍。

    再来看另一个方法,对它的封装,要依据具体的使用场景。

    public static decimal GetDecimal(string key)
    {
                decimal value = default(decimal);
                if ((decimal.TryParse(GetString(key), out value)))
                {
                    return value;
                }
                else
                {
                    return 0m;
                }
    }

    这个方法实现的功能是:把一个字符串转化为数字类型,如果它的值不为数字类型的,则返回默认值0。

    根据需要的场景,这个封装有可能是有必要的,可以减少很多重复的代码。

    欢迎大家给出意见,我觉得这个GetDecimal方法也是多余的,不必要的封装。

  • 相关阅读:
    非常牛的实现三个数交换
    assert后面如果是假则程序崩溃
    python三元操作符
    三种方法划分成绩等级
    爱因斯坦难题
    打印奇数
    判断给到的年份是否为闰年
    【bzoj1895】Pku3580 supermemo Splay
    【bzoj1050】[HAOI2006]旅行comf 并查集
    【bzoj4002】[JLOI2015]有意义的字符串 数论+矩阵乘法
  • 原文地址:https://www.cnblogs.com/JamesLi2015/p/3054122.html
Copyright © 2011-2022 走看看