설치

Question

나는 개체가 이미 null의 경우 사용하여 문을 초기화하는 하나의 방법이 있어야합니다 같은 느낌이 코드가 좀 덜 중복이 될 수 있도록 노력하고 있습니다

public static CMSContent GetContent(string title, ContextDb db = null) 
    { 
     if (db == null) 
     { 
      using (db = new ContextDb()) 
      { 
       return db.CMSContents.FirstOrDefault(c => c.Title == title && !c.Archived); 
      } 
     } 
     else 
     { 
      return db.CMSContents.FirstOrDefault(c => c.Title == title && !c.Archived); 
     } 

    } 

호출 문이 반환 된 개체에 대해 추가 db 요청을 실행하려고하거나 여러 호출이있을 수 있으며 연결이 여러 연결을 완화하기 위해 미리 설정되어 있기 때문입니다.

return 문을 복제 할 수 없으면 db가 null이고 then을 사용하면 더 좋을 것입니다.

Answer 1

using의 설명서를 보면 finally 절에 Dispose()을 호출하여 동일한 동작을 얻을 수 있습니다.

: Nkosi의 의견과 관련하여 - 초기에 db이 Null인지 기억해야합니다.

public static CMSContent GetContent(string title, ContextDb db = null) 
    { 
     var _db = db ?? new ContextDb(); 

     try 
     { 
      return _db.CMSContents.FirstOrDefault(c => c.Title == title && !c.Archived); 
     } 
     finally 
     { 
      if (db == null) _db.Dispose(); 
     } 
    } 

: 결과 코드 내가 편집으로, 게오르그의 답변을했다, 그리고 약간 그것을 손질

public static CMSContent GetContent(string title, ContextDb db = null) { 
    var dbWasNull = (db == null); 
    try { 
     if (dbWasNull) { 
      db = new ContextDb(); 
     } 
     return db.CMSContents.FirstOrDefault(c => c.Title == title && !c.Archived); 
    } 
    finally { 
     if (dbWasNull) { 
      db.Dispose(); 
     } 
    } 
} 

Answer 2

return 문이 더 이상 중복되지 않은 적어도 개선이 ... 정말 확실하지

더 나은 답변을 찾기 위해 많은 사냥을했는데 찾을 수 없습니다. 이것을 제외하고 싱글 톤으로 관리되는 연결 풀을 가질 수도 있습니다. 그러나 여기에는 실용적이지 않으며 희망적으로 이것은 다른 누군가를 돕는 역할을 할 것입니다.

나는 컨텍스트를 처리하지 않고 GC가 그렇게하도록하는 아이디어를 보았지만, 그렇게했다면 밤에 잠을 잘 수 있다고 생각하지 않았습니다.

public static CMSContent GetContent(string title, ContextDb db = null) 
{ 
    return (db ?? new ContextDb()).CMSContents.FirstOrDefault(c => c.Title == title && !c.Archived); 
} 

네 - 그렇게하지 마십시오. 분명히 EF5는 자동으로 연결을 닫지 만, EF6은 그렇지 않으며 일회용이면 폐기해야합니다.