基于ERP程序的公共代碼中出現(xiàn)的問題及過度封裝不方便維護的解決辦法

更新時間：2013年05月02日 11:30:40 作者：

本篇文章介紹了，基于ERP程序的公共代碼中出現(xiàn)的問題及過度封裝不方便維護的解決辦法。需要的朋友參考下

在設計一個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框架的代碼只有一行或簡單的數(shù)行，對它進行封裝，反而會引起理解上的障礙。

再來看另一個方法，對它的封裝，要依據(jù)具體的使用場景。

復制代碼代碼如下:

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

這個方法實現(xiàn)的功能是：把一個字符串轉化為數(shù)字類型，如果它的值不為數(shù)字類型的，則返回默認值0。

根據(jù)需要的場景，這個封裝有可能是有必要的，可以減少很多重復的代碼。

歡迎大家給出意見，我覺得這個GetDecimal方法也是多余的，不必要的封裝。