Làm cách nào để áp dụng việc tiêm phụ thuộc vào một nhà máy trừu tượng

Question

Tôi vừa hoàn thành cuốn sách của Mark Seemann Injection Dependency in .NET và bây giờ tôi đang cố gắng refactor một số mã cũ. (Tôi không, ở giai đoạn này, dựa vào bất kỳ thùng chứa DI cụ thể nào, thay vì chỉ cố di chuyển tất cả các phụ thuộc vào một nơi).

Tôi nhìn vào lớp nhà máy sau đó xác định ArchiveType bằng cách đọc vài byte đầu tiên của các kho lưu trữ với archiveReader.GetArchiveType() và sau đó trả về một thể hiện của một ArchiveRestorer dựa trên ArchiveType enum.

public class ArchiveRestorerFactory : IArchiveRestorerFactory 
{ 
    public ArchiveRestorer Create(ArchiveReader archiveReader) 
    { 
     ArchiveType type = archiveReader.GetArchiveType(); 
     switch (type) 
     { 
      case ArchiveType.CurrentData: 
       return new CurrentDataArchiveRestorer(archiveReader); 
       break; 
      case ArchiveType.HistoricalData: 
       return new HistoricalDataArchiveRestorer(archiveReader); 
       break; 
      case ArchiveType.AuditTrail: 
       return new AuditTrailArchiveRestorer(archiveReader); 
       break; 
      default: 
       throw new Exception("ArchiveRestorerFactory error: Unknown value for ArchiveType."); 
     } 
    } 
} 

Làm thế nào để cấu trúc lại này để các lớp không phụ thuộc vào các loại bê tông CurrentDataArchiveRestorer, HistoricalDataArchiveRestorer và AuditTrailArchiveRestorer?

Tôi có nên di chuyển ba người khôi phục bê tông vào nhà xây dựng của nhà máy không?

public ArchiveRestorer Create(ArchiveReader archiveReader, 
    ArchiveRestorer currentDataArchiveRestorer, 
    ArchiveRestorer historicalDataArchiveRestorer, 
    ArchiveRestorer auditTrailDataArchiveRestorer) 
{ 
    // guard clauses... 
    // assign to readonly fields 
} 

Đó dường như là cách tiếp cận đề nghị here, nhưng sau đó nó sẽ nhanh chóng cả ba phục chế khi chỉ có một là cần thiết? Điều gì xảy ra nếu tôi có 20 cách triển khai cụ thể khác nhau có thể thay thế?

Tôi cảm thấy như tôi nên triển khai một nhà máy cụ thể cho từng loại nhà phục chế và trả lại thay vào đó nhưng sau đó tôi chỉ thay thế new bằng một loại khác.

Cách tốt nhất để cấu trúc lại cái này là gì?

Answer 1

Cách tôi làm điều này, với mã bạn đã có, sẽ tạo một nhà máy cho từng đối tượng có phương pháp Create().

Tôi cũng có giao diện cho các nhà máy này và để họ kế thừa từ giao diện nhà máy chung.

Sau đó, bạn có thể sử dụng giao diện làm điểm để tiêm vào hàm tạo của mình.

nào sẽ được gọi là tương tự như sau:

case ArchiveType.CurrentData: 
       return _currentDateArchiveRestorerFactory.Create(archiveReader); 
       break; 

Ngoài ra, nó có thể là tốt hơn để có một nhà máy duy nhất tạo ra một thể hiện của một loại nhất định. Vì tất cả các đối tượng này là những người khôi phục bạn chỉ có thể tạo cá thể dựa trên số enum thay vì một số switch.

_restorerFactory.Create(ArchiveType.CurrentData); 

Answer 2

Thực hiện một giao diện với một phương pháp với lợi nhuận kiểu giao diện đó và để cho ba lớp Archiver thực hiện giao diện đó và sau đó trong phương pháp tạo ra các loại tham số sẽ chỉ là giao diện và nó sẽ trả lại đối tượng được yêu cầu bằng cách gọi phương thức giao diện bạn vừa tạo. Vì vậy, bạn không cần loại bê tông trong phương thức tạo.

Answer 3

interface ILogger 
{ 
    void Log(string data); 
} 

class Logger : ILogger 
{ 
    . 
    . 
    . 
} 

Tại thời điểm này, bạn sử dụng một đối tượng trung gian nhà máy để trả lại logger để được sử dụng trong các thành phần:

class MyComponent 
{ 
    void DoSomeWork() 
    { 
    // Get an instance of the logger 
    ILogger logger = Helpers.GetLogger(); 
    // Get data to log 
    string data = GetData(); 

    // Log 
    logger.Log(data); 
    } 
} 

class Helpers 
{ 
    public static ILogger GetLogger() 
    { 
    // Here, use any sophisticated logic you like 
    // to determine the right logger to instantiate. 

    ILogger logger = null; 
    if (UseDatabaseLogger) 
    { 
     logger = new DatabaseLogger(); 
    } 
    else 
    { 
     logger = new FileLogger(); 
    } 
    return logger; 
    } 
} 
class FileLogger : ILogger 
{ 
    . 
    . 
    . 
} 

class DatabaseLogger : ILogger 
{ 
    . 
    . 
    . 
} 

Answer 4

Tại sao không làm cho ArchiveReader chịu trách nhiệm tạo ArchiveRestorer thích hợp?Sau đó, phiên đầu tiên của mã sẽ trông như thế này:

public class ArchiveRestorerFactory : IArchiveRestorerFactory 
{ 
    public ArchiveRestorer Create(ArchiveReader archiveReader) 
    { 
     ArchiveRestorer restorer = archiveReader.GetArchiveRestorer(); 
     return restorer; 
    } 
} 

Đến lúc đó, nó nên được khá rõ ràng rằng các nhà máy là không cần thiết, vì vậy trong lần lặp thứ hai của mã bạn có thể vứt nó đi để cho người tiêu dùng gọi trực tiếp ArchiveReader.

Answer 5

Tôi sẽ giải quyết vấn đề này bằng cách đồng ý về quy ước đặt tên và sử dụng khả năng của Unity để đặt tên cho đăng ký. Ví dụ về điều này ở đây: https://dannyvanderkraan.wordpress.com/2015/06/29/real-world-example-of-dependency-injection-based-on-run-time-values/