스파게티 코드, 예외 처리 및 오류 처리?

Question

온라인 번역 서비스 용 래퍼를 쓰고 있습니다. 현재 OnlineTranslator 클래스는 스파게티 코드처럼 저에게보기 흉하게 보입니다. 왜냐하면 밑줄 서비스 (Bing, 실제로)에서 발생할 수있는 많은 오류에 대한 예외 생성 때문입니다.

예외 및 오류를 어떻게 처리하는지 잘 이해하지 못합니다. 내 코드를 리팩토링하는 모든 변경 (누가 패턴을 말 했나요?)? 코드에서 이렇게 많은 if을 제거하는 방법은 무엇입니까?

<?php 
namespace DL\AdminBundle\Service; 

class OnlineTranslator 
{ 

    private $_service; 
    private $_languages; 
    private $_from; 
    private $_to; 

    private $_ERRORS = array(
     'UNSUPPORTED_DETECTION' => "'%s' service doesn't support language 
      detection. Manually call 'setFrom' to set the source language.", 
     'UNSUPPORTED_FROM_LANGUAGE' => "Source language code '%s' unrecognized 
      or not supported by this service.", 
     'UNSUPPORTED_TO_LANGUAGE' => "Destination language code '%s' 
      unrecognized or not supported by this service.", 
     'MISSING_TO_LANGUAGE' => "Destination language code is missing.", 
     'GENERIC_SERVICE_ERROR' => "'%s' service returned an error: %s." 
    ); 

    function __construct(IOnlineTranslator $service) 
    { 

     // Imposta il servizio e la lingua sorgente su auto 
     $this->_service = $service; 
     $this->_from = 'auto'; 

     $response = $this->_service->getLanguages(); 

     if ($response->error) 
      throw new Exception(sprintf(
       $this->_ERRORS['GENERIC_SERVICE_ERROR'], 
       $this->_service->getName(), $response->data)); 

     $this->_languages = $response->data; 

    } 

    function setFrom($languageCode) 
    { 

     // Controlla se la lingua è supportata 
     if (!in_array($languageCode, $this->_languages)) 
      throw new Exception(sprintf(
       $this->_ERRORS['UNSUPPORTED_FROM_LANGUAGE'], 
       $languageCode)); 

     // Imposta la lingua sorgente 
     $this->_from = $languageCode; 

    } 

    function setTo($languageCode) 
    { 

     // Controlla se la lingua è supportata 
     if (!in_array($languageCode, $this->_languages)) 
      throw new Exception(sprintf(
       $this->_ERRORS['UNSUPPORTED_TO_LANGUAGE'], 
       $languageCode)); 

     // Imposta la lingua destinazione 
     $this->_to = $languageCode; 

    } 

    function translate($text) 
    { 

     // Controlla che sia impostata la lingua di destinazione 
     if (!isset($this->_to)) 
      throw new Exception($this->_ERRORS['MISSING_TO_LANGUAGE']); 

     // Se detect è auto controlla che il servizio lo supporti 
     if ('auto' == $this->_from && !$this->_service->isDetectAware()) 
      throw new Exception(sprintf(
       $this->_ERRORS['UNSUPPORTED_DETECTION'], 
       $this->_service->getName())); 

     // Imposta la lingua sorgente chiamando il metodo detect 
     $response = $this->_service->detect($text); 

     if ($response->error) 
      throw new Exception(sprintf(
       $this->_ERRORS['GENERIC_SERVICE_ERROR'], 
       $this->_service->getName(), $response->data)); 

     $this->_from = $response->data; 

     // Traduci il testo chiamando il metodo translate 
     $response = $this->_service->translate($text, $this->_from, 
      $this->_to); 

     if ($response->error) 
      throw new Exception(sprintf(
       $this->_ERRORS['GENERIC_SERVICE_ERROR'], 
       $this->_service->getName(), $response->data)); 

     return $response->data; 

    } 

} 

?> 

Answer 1

다른 사람들의 말처럼, 스파게티 코드는 아닙니다. 문제는 다소 자세한 검증과 예외가 모든 함수에서 발생한다는 것입니다. 유효성 검사는 cross-cutting concern이므로 모든 곳에서 흩어져서 가독성을 저해 할 수 있습니다.

이 문제를 처리하는 한 가지 방법은 유효성 검사 코드를 별도의 함수로 마무리하고 자세한 정보 및 중복을 줄이는 것입니다. 시스템 문제 일 경우 PHP AOP을 볼 수도 있습니다. 필자는 AOP에 대한 경험이 없지만 종종 로깅/오류 처리를 사용 예제로 제시합니다.