Refactoring & Test Kata

存在一份處理 Bookmark 文字檔的 console 程式，但現行功能已經無法滿足使用者，需要在保持原本功能正常的前提下，不停的增加/修改功能。 請在增加需求功能的同時, 補上測試，並進行重構。

現有功能如下

• 待處理的檔案的內容為 url | web site title 的資料行組成。
• 移除重覆相同的 Url bookmark
• 移除 utm, fblid 的 tracking code
• 處理結果，會儲存於 csv 檔

已知後續的需求變更順序如下

• 在執行 console 程式時，可傳入指定的檔案位置，若無傳入參數，則維持讀取原本的檔案位置。
• 將處理後的資料改存至資料庫。(友情提示: 在 kata 之中，可使用 List、Dictionary 等方式模擬資料庫。)
• 增加移除 Url 中，特定 tracking code 的功能。
• 將 console 程式改為 webapi 的方式，提供外部使用。
• 新增的資料，若在現存資料已存在相同 url，不進行增加。

需求變更的延伸思考

• 在執行 console 程式時，可傳入指定的檔案位置，若無傳入參數，則維持讀取原本的檔案位置。
  • 延伸思考:
    • 在完全沒有測試的情況下，要如何進行功能調整。
    • 業務邏輯與應用程式是否需要分層？有何意義？
• 將處理後的資料改存至資料庫。
  • 延伸思考:
    • 如何收斂儲存資料的邏輯？
    • 是否需要切分儲存功能至其他類別？為什麼？
    • 測試保護的重點為何？
• 增加移除 Url 中，特定 tracking code ( 的功能。
  • 延伸思考:
    • 業務邏輯的職責是否足夠專一、足夠明確？
    • 能否使用測試來保護這次的修改？
• 將 console 程式改為 webapi 的方式，提供外部使用。
  • 延伸思考:
    • 物件類別的職責是否專一？
    • 分層架構的背後的意義？
    • 原有的測試案例是否還有意義？為什麼？
• 新增的資料，若在現存資料已存在相同 url，不進行增加。
  • 延伸思考:
    • 重覆資料驗證與判斷的職責歸屬？

測試可能會使用到的方法

public string GetFileHash(string filename)
{
    var hash = new SHA1Managed();
    var clearBytes = File.ReadAllBytes(filename);
    var hashedBytes = hash.ComputeHash(clearBytes);
    return ConvertBytesToHex(hashedBytes);
}

public string ConvertBytesToHex(byte[] bytes)
{
    var sb = new StringBuilder();
    for (var i = 0; i < bytes.Length; i++)
    {
        sb.Append(bytes[i].ToString("x"));
    }
    return sb.ToString();
}