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