如何安全地將靜態類轉換爲C＃中的實例化類＃

Question

我最近（幾個月前）更改了作業並繼承了一個代碼庫，它違反了SOLID原則中的每一個原則，儘可能多次。看起來編寫這些代碼的人似乎決定詳細研究每一個好的編碼實踐，並儘可能經常地，最激進地違反它們。

我是該產品的唯一開發人員 - 組織中沒有人知道代碼和代碼庫太大又複雜，無法完全重寫。我正在研究可以使代碼庫靈活而強大的最高價值變化。放棄這種產品也不是一種選擇。

產品中所有問題的根源都來自於一組作爲核心業務邏輯數據結構的類。有很多的這些類的問題，但什麼我真正感興趣的是：

public static class NetCollection 
{ 
    private static Logger LogFile { get { return Logger.GetMethodLogger(2); } } 
    // Declare local variables. 
    private static Dictionary<string, NetObject> netObjectHashTable; 
    private static Dictionary<string, NetTitle> titlePropertyHashTable; 
    private static Dictionary<string, NetObject> referenceDataHashTable; 
    private static Dictionary<int, SortedDictionary<string, int>> picklistHashTable; 

    public static IEnumerable<NetObject> NetObjects 
    { 
     get 
     { 
      return netObjectHashTable.Values; 
     } 
    } 

    static NetCollection() 
    { 
     netObjectHashTable = new Dictionary<string, NetObject>(); 
     titlePropertyHashTable = new Dictionary<string, NetTitle>(); 
     referenceDataHashTable = new Dictionary<string, NetObject>(); 
     picklistHashTable = new Dictionary<int, SortedDictionary<string, int>>(); 
    } 

    public static void AddNetObject(NetObject newObject) 
    { 
     if (newObject == null) 
      return; 
     if (newObject.TitleType == "Reference Data") 
     { 
      // Check if hash table contains key 
      if (!referenceDataHashTable.ContainsKey(newObject.ID.ToString())) 
      { 
       referenceDataHashTable.Add(newObject.ID.ToString(), newObject); 
      } 
     } 
     else 
     { 
      // Check if hash table contains key 
      if (!netObjectHashTable.ContainsKey(newObject.ID.ToString())) 
      { 
       netObjectHashTable.Add(newObject.ID.ToString(), newObject); 
      } 
     } 
    } 
} 

我已經剪掉不少其他方法從這個類爲簡潔起見。如你所見，圍繞這個類有大量的問題（在靜態類中存儲狀態是一種巨大的代碼異味 - 在你的類中編寫你的整個應用程序只是瘋狂）。

我目前的意圖是將這個類重構成一個合適的單例類（最終成爲一個普通類，這樣我就可以讓用戶同時打開多個文檔）。

我應該這樣做嗎？

進行此更改最大的風險是什麼？我是否可以採取任何方法來降低進行此變更的風險？

Answer 1

如果您對應用程序中這種類型的流式無所知，這是一項危險的任務。但如果你真的需要做，沒有，可能，打破一切，我會這樣做：

知道我需要的是文件之間的明顯分裂，我知道（這是時間證明），這種類型爲單個文檔工作，讓我們添加文檔切片。

假設Document有Name屬性，我們可以想想類似的信息（例如）：

public static void AddNetObject(string documentName, NetObject newObject) 
{ 
    .... 
} 

使各領域非靜：

//NO STATIC 
    ... 
    private Logger LogFile { get { return Logger.GetMethodLogger(2); } } 
    private Dictionary<string, NetObject> netObjectHashTable; 
    private Dictionary<string, NetTitle> titlePropertyHashTable; 
    private Dictionary<string, NetObject> referenceDataHashTable; 
    private Dictionary<int, SortedDictionary<string, int>> picklistHashTable; 

將它們轉換爲內部

private class NetDocument { 
     public string DocumentName {get;set;} //DEFINE DOCUMENT IT RELATED TO ! 
     ... 
     private Logger LogFile { get { return Logger.GetMethodLogger(2); } } 
     private Dictionary<string, NetObject> netObjectHashTable; 
     .... 

    } 

因此，您可以在單個文檔之間創建具體的隔離並與其相關數據。

主類內部之後喲可以有：

public static class NetCollection 
{ 
    ... 

    //Key: document name 
    //Value: NetDocument data 
    private Dictionary<string, NetDocument> documents = new .... 
} 

這只是一個一般的想法（草圖），你肯定需要改變，以適應您的需求。

Answer 2

你能圍繞一個靜態工廠類來包裝它嗎？把你認爲你需要的所有東西都作爲一個單例來使用，並根據需要保持某些靜態的東西？它會解決一些問題，並給你一些靈活性。

Answer 3

是的，轉換爲單身人士似乎是一個很好的第一步。它仍然不是線程安全的，但它是一個開始。然後，您可以將其從真實單身更改爲一個允許單獨的實例被構造，但也具有與單身相同的方式的「默認」實例。 （當然，您可以將「實例持有者」分爲單獨的類）。這將允許您開始編寫每次都以新實例開始的可測試代碼。

之後，您可以開始引入依賴注入，以便每個需要訪問同一實例的類都可以獲取它並刪除「默認」實例。

當然，如果你可以減少需要訪問同一個實例的類的數量，那也會更好。

對於線程，您需要鎖定每種方法，或使用ConcurrentDictionary。

Answer 4

static類和單例之間沒有真正的區別。是的，它的實施方式不同，但你有相同的問題（只是爲了不需要採取行動）。

如果你能夠去「真實」的實例，那就去做吧。但只是重構辛格爾頓，因爲林特不抱怨是不是去恕我直言的方式。