Új hozzászólás Aktív témák

• #9917 Peter Kiss őstag PazsitZ #9912

  Új Válasz 2012-05-27 13:12:13 #9917

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Peter Kiss

  őstag

  válasz PazsitZ #9912 üzenetére

  A Singleton anti-pattern, tesztelhetetlen alkalmazást eredményez, illetve fogalmad sem lesz arról, hogyan működik az alkalmazás.

  Elég csak arra gondolni, mi van akkor, ha kapsz egy egyébként működő kódhalmazt, amit használni szeretnél, de rejtett dependency-k vannak benne a singletonok miatt. Sosem szabad ilyet csinálni.

  Példa:

  <?php
  /* ... */
  $controllerFactory = $this->_controllerBuilder->GetControllerFactory($this->HttpContext);
  
  $controller = $controllerFactory->CreateController($this->HttpContext);
  
  if (!$controller->GetType()->ImplementsInterface("\\System\\Web\\Mvc\\IHttpHandler") || !$controller->IsStateLess()) {
  $sdsf = new SessionProviderFactory();
  
  $this->SessionManager = new SessionManager($this->HttpContext, new UsersAndGroupsDataContext(), $sdsf);
  $this->SessionManager->Validate();
  
  $this->HttpContext->Session = $this->SessionManager->GetCurrentSession();
  }
  
  if (!$controller->Execute($this->HttpContext, $this->_actionInvokers->GetActionInvoker(typeof($controller)))) {
  throw new \Exception("Failed to execute action: " . $this->HttpContext->Route());
  }
  /* ... */

  Itt mondhatnám én is a $this->HttpContext átadása helyett, hogy akkor HttpContext::Instance() az egyes helyeken (kihagyva a metódusok paraméterlistájából), mert itt ugyanazt jelenti, de azt eredményezné, hogy még számomra se lenne világos, minek mire van szüksége.

  A példában szereplő if block egyébként még javításra szorul.

Új hozzászólás Aktív témák