구성된 오브젝트에서 Null을 플래그로 사용하는 것이 좋은가요?

Question

신청서에 문제가 있습니다. 서비스에서 응답을 얻고 있습니다. 때로는 데이터베이스 또는 일부 다른 미디어에서 때로는 캐시에서 응답을 변환하고 저장해야합니다. 응답을 유지할 필요가 없지만 응답 만 변환하고 객체를 반환 할 필요가있는 시나리오도있을 수 있습니다. 나는 다음과 같은 설계 구현 : 난 단지 객체 (NO persiatance)을 변환하는 데 필요한 시나리오에서는, 위의 구현에서

public class Response { } 

class MyData { } 

interface IPersistance 
{ 
    public void PersistData(MyData data); 
} 

public class DBPersitance : IPersistance 
{ 
    public void PersistData(MyData data) { } 
} 

public class CachePersitance : IPersistance 
{ 
    public void PersistData(MyData data) { } 
} 

public class MySeviceData 
{ 
    private IPersistance persistence; 

    public MySeviceData(IPersistance persistance) 
    { 
     _persistance = persistance; 
    } 

    public MyData GetServiceData(Response reponseXML) 
    { 
     MyData output = new MyData(); 

     // Here output object is filled with data from responseXML 

     // Object persistance as defined in constructor 
     if(_persistance!=null) 
      _persistance.PersistData(output); 

     return output; 
    } 
} 

class Program 
{ 
    static void Main(string[] args) 
    { 
     MyData output = new MyData(); 
     Response responseXML = new Response() ; 

     //Case 1: Cache the response 
     CachePersitance cache = new CachePersitance(); 
     MySeviceData srvData = new MySeviceData(cache); 
     output = srvData.GetServiceData(responseXML); 

     //Case 2: Write the response in Database 
     DBPersitance db = new DBPersitance(); 
     MySeviceData srvData = new MySeviceData(db); 
     output = srvData.GetServiceData(responseXML); 

     //Case 3: No need to persist. Only Convert and return the response 
     MySeviceData srvData = new MySeviceData(null); 
     output = srvData.GetServiceData(responseXML); 
    } 
} 

을, 나는 null 지속성 객체를 전달했습니다. 이 접근법이 맞습니까? 아무도 이런 종류의 요구 사항에 대해 더 나은 디자인을 제안 할 수 있습니까?

Answer 1

가능하면 언제든지 if에서 null을 확인하는 것이 좋습니다. 우리는 동의할지 모르겠다. (그리고 이것은 매우 근거가있다. IMO 좋다. 그러나 granal salis, null은 정보을 가지고있을 때 항상 악의가있는 것은 아니다. 당신이 코드를 읽을 때

이 명백한 의도를 만들 것 대신 null의 빈 클래스를 사용하려면 ( 지속성가 null? 무슨 뜻인지? 기본 한 다음이 기본 무슨 경우? 하나 아무도 기본?).

비어있는 IPersistance (예 : NoPersistance) 구현이 작동합니다. 가장 일반적인 경우 파라미터가없는 생성자 오버로드 (예 : 다음 예와 같이)를 제공 할 수도 있습니다 (명시 적으로 만들 필요가 없으며이 경우 제거 만하면됩니다).

MySeviceData srvData = new MySeviceData(); 
output = srvData.GetServiceData(responseXML); 

또는 같은

가 :

MySeviceData srvData = new MySeviceData(new NoPersistance()); 
output = srvData.GetServiceData(responseXML);