중첩 된 Try..catch 블록을 사용하는 것이 좋지 않습니까?

Question

이것은 나쁜 생각입니까? 동일한 효과를 얻을 수있는 더 좋은 방법이 있습니까?

// assume that "name" is a string passed as a parameter to this code block 
try 
{ 
    MainsDataContext dx = new MainsDataContext(); 
    try 
    { 
     Main m = dx.Main.Single(s => s.Name == name); 
     return m.ID; 
    } 
    catch (InvalidOperationException) 
    { 
     Guid g = Guid.NewGuid(); 

     Main s = new Main 
     { 
      Name = name, 
      ID = g 
     }; 

     dx.Mains.InsertOnSubmit(s); 
     dx.SubmitChanges(); 

     return g; 
    } 
} 
catch (Exception ex) 
{ 
    // handle this 
} 

여기서 목표는 레코드의 ID를 얻는 것입니다. 그렇지 않으면 레코드를 생성하고 ID를 반환합니다.

Answer 1

레코드가 존재하지 않으면 null 인 클래스의 기본값을 반환하는 SingleOrDefault를 사용해야합니다. 유형 Main이 클래스가 또는 경우의 DataContext

using (MainsDataContext dx = new MainsDataContext()) 
{ 
     Main m = dx.Main.SingleOrDefault(s => s.Name == name); 

     if (m == null) 
     { 
      Guid g = Guid.NewGuid(); 

      m = new Main 
      { 
       Name = name, 
       ID = g 
      }; 

     dx.Mains.InsertOnSubmit(m); 
     dx.SubmitChanges(); 

     } 

     return m.ID; 
} 

그것은 질문에서 명확하지 않다

Answer 2

Main m = dx.Main.SingleOrDefault(s => s.Name == name); 

if (m == default(Main)) 
{ 
    // it does not exist 
} 
else 
{ 
    // it does exist 
} 

를 사용하는 경우

MainsDataContext dx = null;  
try 
    { 
     dx = new MainsDataContext(); 

     Main m = dx.Main.SingleOrDefault(s => s.Name == name); 

     if (m == null) 
     { 
      Guid g = Guid.NewGuid(); 

      m = new Main 
      { 
       Name = name, 
       ID = g 
      }; 

     dx.Mains.InsertOnSubmit(m); 
     dx.SubmitChanges(); 

     } 

     return m.ID; 
    } 
    catch (Exception ex) 
    { 
     // handle this 
    } 
    finally 
    { 
     if(dx != null) 
      dx.Dispose(); 
    } 

IT는 가 키워드를 사용하여 사용하는 것이 좋습니다 struct (편집 : 방금 실제로 클래스가 있어야한다는 것을 깨달 았기 때문에) null 대신에 default()을 사용했습니다.

Answer 3

아니요.하지만 내부 블록을 외부 방법으로 리파리 할 수 ​​있습니다. 당신은 순서가 두 개 이상의 요소를 포함하거나 비어 있기 때문에 싱글() InvalidOperationException이 던진 것을 알고

// handle this 

을 첫 번째 catch 블록으로 :

Answer 4

내 질문은 여기에 넣어하려는 어떤 코드가 될 것입니다 .

둘째, 모든 종류의 오류가 발생할 수 있습니다. 널 참조, 데이터 액세스 등 어떻게 처리 할 것인가?

처리 방법을 알고있는 것을 잡아 내고 가능하면 예외 유형으로 한정하십시오.

Answer 5

어쨌든 중첩 된 Try Catch 블록은 형제처럼 Catch 블록이 각각 다른 문제를 포착하는 것처럼 좋습니다. 오류에 대해 구체적으로 설명하는 것이 좋습니다. 캐치 올은 생산을위한 안전망이어야합니다.