имеет статический метод, возвращающий экземпляр класса (например, id) плохой конструкции неэлементного класса?

Question

Я сказал коллеге, что следующий - плохой дизайн, но в конце концов я не мог по-настоящему объяснить причину (кроме того, в этом случае уже был один сингл-класс, который использовался как «коллекция» для всех эти). Но в общем случае есть что-то не так со следующим (только «статический метод, возвращающий экземпляр класса», - а не каким способом обрабатывается interop.) Это просто пример статического метода, а не реального кода.

public class MyClass 
{ 
    public int Id { get; private set; } 
    /... other properties, constructors & methods 

    // this is only static method 
    public static GetMyClass(int id) 
    { 
     // get it in this case from unmanaged code 
     return new MyClass(InteropThing.GetMyClassStruct(id)); 
    } 
} 

Answer 1

Из примера кода он выглядит как вы прячетесь зависимость с помощью вызова статического класса, который будет делает вещи неудобно издеваться и модульного тестирования.

Возможно, лучше было бы:

public class MyClass 
{ 
    private readonly IInteropThing _interop; 

    public int Id { get; set; } 

    public MyClass (IInteropThing interop) 
    { 
     _interop = interop; 
    } 

    public GetMyClass(int id) 
    { 
     return new MyClass(_interop.GetMyClassStruct(id)); 
    } 
} 

Это также выглядит, как вы могли бы нарушить единый принцип ответственности в зависимости от того, что еще делает MyClass.

Возможно, отдельный класс, который является единственной функцией, заключается в создании объектов MyClass, будет лучшим подходом к продвижению свободной связи и тестируемости.