在設計一個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方法也是多余的,不必要的封裝。