이 XDocument 코드는 안전한가요?

Question

이라는 클래스를 작성 중이며이 클래스에는 외부 리소스에서 XML 데이터를 가져 오는 두 가지 정적 메서드가 있습니다. 아래의 예에서는 매우 유사하기 때문에 하나만 보여 드리겠습니다.

내가 알아 내려고하는 것은이 코드가 유효하지 않은 URL, 유효하지 않은 데이터 등의 측면에서 "안전"한 것인지 여부입니다. 기본적으로 더 강력합니다. 여기 TEH 코드

private static string XmlUri 
    { 
     get { return "path-to-xml-file"; } 
    } 
private static XDocument XmlFile { get; set; } 
public int ID { get; set; } 
public string Name { get; set; } 
public int Parent { get; set; } 

/// <summary> 
/// Gets a specific category 
/// </summary> 
/// <param name="id"></param> 
/// <returns>A Category with the specified ID</returns> 
public static Category Get(int id) 
{ 
    try 
    { 
     if (XmlFile == null) 
      XmlFile = XDocument.Load(XmlUri); 
    } 
    // Invalid URL or data 
    catch (Exception ex) 
    { 

     // TODO: Log exception 
     Console.WriteLine(ex.Message); 
    } 

    if (XmlFile == null) 
     return null; 

    var cat = from category in XmlFile.Descendants("Category") 
       where category.Attribute("id").Value.ParseSafe() == id 
       select new Category 
       { 
        ID = category.Attribute("id").Value.ParseSafe(), 
        Parent = category.Attribute("parent").Value.ParseSafe(), 
        Name = category.Value 
       }; 

    return cat.SingleOrDefault(); 
} 

Answer 1

정의 '안전'입니다. 무언가 잘못되었을 때 코드에서 null을 생성합니다. 나는 XDocument.Load() 후에 catch 블록에 (다시) 던지기를 고려할 것이다. 에 있지 않으려면이 아니라 잘못된 URL을 무시하십시오.

그리고 ParseSafe() 및 SingleOrDefault도 문제가됩니다. id이 누락되었거나 잘못 형성된 경우 어떻게 하시겠습니까?

작은 개선 : XmlFile의 getter에 Load-on-Demand 로직을 넣을 수 있습니다. 카테고리 옆에있는 다른 요소를 원할 경우 쉽게 만듭니다.

Answer 2

언급 된 ChaosPandion만큼 안전하지 않습니다.

혼란스러운 성능 동작 - Get (int)은 간단한 빠른 방법처럼 보이지만 실제로는 중요한 작업을 포함하지 않습니다. 게으른 초기화를 사용하고 사전에 범주 onece의 전체 모음을 읽습니다.

예외를 잡아서 삼키지 않아야합니다. 특정 예외를 사용하거나 (이 경우 IOException 및 XMLExcepetion이 있다고 생각하거나) 적어도 치명적인 예외는 정상적으로 throw 될 수 있습니다.

분명히 XML 파일을 수정하지 않으면 대단히 큰 파일을로드하는 동안 slownes/hangs가 발생할 수도 있습니다. 독자의 행동과 XML의 복잡성 (악의적 인 사용자에 의해 주어질 경우)에 따라 StackOverflow와 같은 일부 예외가 프로세스를 중지시킬 수 있습니다.