在设计一个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方法也是多余的，不必要的封装。
下载本文