Đây có phải là một tùy chọn thiết kế tốt để gọi phương thức khởi tạo của lớp trong một nhà máy trước khi tiêm nó

Question

Tôi đang trong quá trình tái cấu trúc một phần lớn mã spaghetti. Tóm lại, nó là một lớp học "Thiên Chúa" giống như các chi nhánh thành hai quá trình khác nhau tùy thuộc vào một số điều kiện. Cả hai quy trình đều dài và có nhiều mã trùng lặp.

Vì vậy, nỗ lực đầu tiên của tôi là trích xuất hai quy trình đó thành các lớp của riêng chúng và đặt mã chung vào một cấp độ gốc mà cả hai đều kế thừa.

Nó trông giống như sau:

public class ExportProcess 
{ 
    public ExportClass(IExportDataProvider dataProvider, IExporterFactory exporterFactory) 
    { 
     _dataProvider = dataProvider; 
     _exporterFactory = exporterFactory; 
    } 

    public void DoExport(SomeDataStructure someDataStructure) 
    { 
     _dataProvider.Load(someDataStructure.Id); 

     var exporter = _exporterFactory.Create(_dataProvider, someDataStructure); 

     exporter.Export(); 
    } 
} 

Tôi là một độc giả cuồng nhiệt của blog của Mark SEEMANN và trong this entry ông giải thích rằng mã này có một khớp nối mùi thời gian kể từ khi nó là cần thiết để gọi phương thức tải về nhà cung cấp dữ liệu trước khi nó ở trạng thái có thể sử dụng.

Trên cơ sở đó, và kể từ khi đối tượng đang được tiêm cho những người được trả về bởi các nhà máy dù sao, tôi đang nghĩ đến việc thay đổi nhà máy để làm điều này:

public IExporter Create(IExportDataProvider dataProvider, SomeDataStructure someDataStructure) 
{ 
    dataProvider.Load(someDataStructure.Id); 

    if(dataProvider.IsNewExport) 
    { 
     return new NewExportExporter(dataProvider, someDataStructure); 
    } 
    return new UpdateExportExporter(dataProvider, someDataStructure); 
} 

Bởi vì tên "DataProvider" bạn có thể đoán rằng phương thức Load thực sự đang thực hiện truy cập cơ sở dữ liệu.

Điều gì đó cho tôi biết một đối tượng đang thực hiện truy cập cơ sở dữ liệu bên trong phương thức tạo của nhà máy trừu tượng không phải là thiết kế tốt.

Có bất kỳ nguyên tắc, phương pháp hay nhất hoặc điều gì đó cho biết đây có phải là một ý tưởng tồi không?

Cảm ơn sự giúp đỡ của bạn.

Answer 1

Thông thường, một nhà máy được sử dụng để giải quyết các loại bê tông của giao diện được yêu cầu hoặc loại tóm tắt, vì vậy bạn có thể tách riêng người tiêu dùng khỏi triển khai. Vì vậy, thường là một nhà máy chỉ là sẽ khám phá hoặc xác định loại cụ thể, giúp giải quyết phụ thuộc, và nhanh chóng loại bê tông và trả lại nó. Tuy nhiên, không có quy tắc cứng hoặc nhanh về những gì nó có thể hoặc không thể làm, nhưng nó là hợp lý để cung cấp cho nó đủ quyền truy cập vào các tài nguyên mà nó cần phải giải quyết và khởi tạo các loại cụ thể.

Một cách sử dụng tốt khác của nhà máy là che giấu người tiêu dùng các loại phụ thuộc không liên quan đến người tiêu dùng. Ví dụ: có vẻ như IExportDataProvider chỉ liên quan đến nội bộ và có thể được trừu tượng hóa khỏi người tiêu dùng (chẳng hạn như ExportProcess).

Mùi mã trong ví dụ của bạn, tuy nhiên, cách sử dụng IExportDataProvider. Cách nó hiện có vẻ hoạt động, bạn sẽ có được một ví dụ của nó một lần, nhưng nó có thể thay đổi trạng thái của nó trong các tập quán tiếp theo (bằng cách gọi Load). Điều này có thể dẫn đến các vấn đề với trạng thái đồng thời và bị hỏng. Vì tôi không biết loại đó thực sự là gì hoặc nó được sử dụng như thế nào bởi IExporter của bạn, thật khó để đưa ra đề xuất. Trong ví dụ dưới đây, tôi thực hiện điều chỉnh để chúng tôi có thể giả định rằng nhà cung cấp là không quốc tịch và thay vào đó, Load trả về một số loại đối tượng trạng thái mà nhà máy có thể sử dụng để giải quyết loại xuất bản cụ thể và sau đó cung cấp dữ liệu cho nhà xuất bản. Bạn có thể điều chỉnh như bạn thấy phù hợp. Mặt khác, nếu nhà cung cấp phải là nhà nước, bạn sẽ muốn tạo một IExportDataProviderFactory, sử dụng nó trong nhà máy xuất khẩu của bạn và tạo một trường hợp mới của nhà cung cấp từ nhà máy cho mỗi cuộc gọi đến nhà máy xuất khẩu Create.

public interface IExporterFactory 
{ 
    IExporter Create(SomeDataStructure someData); 
} 

public class MyConcreteExporterFactory : IExporterFactory 
{ 
    public MyConcreteExporterFactory(IExportDataProvider provider) 
    { 
      if (provider == null) throw new ArgumentNullException(); 

      Provider = provider; 
    } 

    public IExportDataProvider Provider { get; private set; }  

    public IExporter Create(SomeDataStructure someData) 
    { 
     var providerData = Provider.Load(someData.Id); 

     // do whatever. for example... 
     return providerData.IsNewExport ? new NewExportExporter(providerData, someData) : new UpdateExportExporter(providerData, someData); 
    } 
} 

Và sau đó tiêu thụ:

public class ExportProcess 
{ 
    public ExportProcess(IExporterFactory exporterFactory) 
    { 
     if (exporterFactory == null) throw new ArgumentNullException(); 

     _exporterFactory = factory; 
    } 

    private IExporterFactory _exporterFactory; 

    public void DoExport(SomeDataStructure someData) 
    { 
     var exporter = _exporterFactory.Create(someData); 
     // etc. 
    } 
}