Jak szybkie jest tworzenie instancji klas za pomocą metod, ale bez pól i właściwości?

Question

Generalnie tworzenie klas metodami, ale bez pól i właściwości, ma dużo narzutów?

Pracuję nad aplikacją ASP.NET MVC, która intensywnie korzysta z wtrysku konstruktora, a niektóre sterowniki mają do 10 zależności. Ale ze względu na dużą liczbę zależności, że ucieka się do IMyAppServiceProvider interfejs i klasy udostępniającej rodzajowy dostęp do wszystkich zależności, przez DependencyResolver w MVC 3.

ja wyrwane całe specyficzny kod aplikacji i stworzył z Gist moja podstawowa konfiguracja (nie dotyczy to jednak poniższej konfiguracji BaseController).

Utworzyłem również klasę BaseController, która akceptuje IMyAppServiceProvider. Wszystkie kontrolery dziedziczą z tej klasy bazowej. Klasa podstawowa pobiera obiekt IMyAppServiceProvider i ma zmienne chronione dla wszystkich różnych usług. Kod wygląda mniej więcej tak:

public class BaseController 
{ 
    protected IService1 _service1; 
    protected IService2 _service2; 
    protected IService3 _service3; 
    // ... 

    public BaseController(IMyAppServiceProvider serviceProvider) 
    { 
     _service1 = serviceProvider.GetService<IService1>; 
     _service2 = serviceProvider.GetService<IService2>; 
     _service3 = serviceProvider.GetService<IService3>; 
     // ... 
    } 
} 

To sprawia, że ​​kod dla kontrolerów jest "skrzypiący w czystości". Brak zmiennych prywatnych/chronionych, żadnych przydziałów w konstruktorze i usług odwołują się do zmiennych chronionych klasy podstawowej. Jednak każde żądanie będzie tworzyło każdą usługę, z której korzysta moja aplikacja, niezależnie od tego, czy dany kontroler używa ich wszystkich.

Moje usługi są proste i zawierają tylko wywołania metod z pewną logiką biznesową i interakcją z bazą danych. Są bezpaństwowcami i nie mają pól ani właściwości klasy. Dlatego tworzenie instancji powinno być szybkie, ale zastanawiam się, czy jest to najlepsza praktyka (wiem, że jest to termin obciążony).

Answer 1

każdy wniosek będzie instancji każdą usługę, że moja aplikacja używa, czy specyficzny kontroler wykorzystuje wszystkie im.

Uważam, że sam odpowiedziałeś na swoje pytanie, to nie jest dobre podejście. Ponadto używanie tego rodzaju rozwiązywania zależności (wtrysku serwisowego) jest złym zwyczajem, ponieważ interfejs API kontrolera staje się bałaganiarski. Klienci kontrolerów nie są świadomi, jakie usługi są naprawdę potrzebne dla konkretnego kontrolera, więc możesz skończyć się nieoczekiwanymi błędami czasu wykonywania, testy jednostkowe również będą kłopotliwe.

BTW jeszcze jedna sugestia - zaznacz wszystkie klasy, które są uważane za klasę podstawową przez słowo kluczowe abstract, w ten sposób możesz uniknąć używania go jako konkretnej klasy. Projektowanie i wdrażanie klasy bazowej jest konkretną decyzją projektową, dzięki czemu twoje zamiary projektowe są jasne.

odniesieniu do kosztów instancji, w twoim przypadku to nie duża różnica, ale w ogóle do zmniejszenia kosztów ciężkich przedmiotów instancji można:

• Zastosowanie Prototype pattern do „uniknąć nieuniknionego koszt tworzenia nowego obiektu w standardowy sposób (np. za pomocą słowa kluczowego "nowy"), gdy jest on zbyt kosztowny dla danej aplikacji " (c) Wikipedia
• Użyj usługi Lazy Initialization dla usług, które nie są potrzebne od początku czas życia obiektu właściciela, aby zostały one zainicjowane na żądanie. Od .NET Framework 4.0 yo można użyć wbudowanego Lazy(T) class

Answer 2

Myślę, że rozwiązaniem, którego szukasz, jest użycie fabryki niestandardowych kontrolerów. W ten sposób każdy utworzony kontroler ma dokładnie te zależności, których potrzebuje. Oto jeden dla StructureMap, od weblogs.asp.net:

using StructureMap; 
public class StructureMapControllerFactory : DefaultControllerFactory { 

    protected override IController GetControllerInstance(Type controllerType) { 
     try { 
      return ObjectFactory.GetInstance(controllerType) as Controller; 
     } 
     catch (StructureMapException) { 
      System.Diagnostics.Debug.WriteLine(ObjectFactory.WhatDoIHave()); 
      throw; 
     } 
    } 
} 

protected void Application_Start() { 
    RegisterRoutes(RouteTable.Routes); 

    //Configure StructureMapConfiguration 
    // TODO: config structuremap   

    //Set current Controller factory as StructureMapControllerFactory 
    ControllerBuilder.Current.SetControllerFactory(new StructureMapControllerFactory()); 
}