基於ERP程式的公用代碼中出現的問題及過度封裝不方便維護的解決辦法

來源:互聯網
上載者:User

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

聯繫我們

該頁面正文內容均來源於網絡整理,並不代表阿里雲官方的觀點,該頁面所提到的產品和服務也與阿里云無關,如果該頁面內容對您造成了困擾,歡迎寫郵件給我們,收到郵件我們將在5個工作日內處理。

如果您發現本社區中有涉嫌抄襲的內容,歡迎發送郵件至: info-contact@alibabacloud.com 進行舉報並提供相關證據,工作人員會在 5 個工作天內聯絡您,一經查實,本站將立刻刪除涉嫌侵權內容。

A Free Trial That Lets You Build Big!

Start building with 50+ products and up to 12 months usage for Elastic Compute Service

  • Sales Support

    1 on 1 presale consultation

  • After-Sales Support

    24/7 Technical Support 6 Free Tickets per Quarter Faster Response

  • Alibaba Cloud offers highly flexible support services tailored to meet your exact needs.