基于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方法也是多余的，不必要的封装。