Какое исключение я должен использовать для неудачного анализа файлов?

Question

У меня есть метод разбора файла. Однако этот синтаксический анализ может быть неудачным в любое время, в зависимости от различных условий (например, не очень осторожный пользователь, играющий с файлом).

public string ParseDatFile(string datFile) 
{ 
    string[] deezLines = File.ReadAllLines(datFile); 

    // We're searching for an essential data inside the file. 
    bool daEssentialDataFound = false; 
    foreach (string datLine in deezLines) 
    { 
     if (datLine.Contains("daEssentialData")) 
     { 
      daEssentialDataFound = true; 
      break; 
     } 
    } 

    if (!daEssentialDataFound) 
     throw new WhatShouldIThrowException("yo dood where's da essential data in " + datFile + "?"); 

    DoStuffWith(deezLines); 
} 

Есть ли исключение, которое я мог бы использовать в таком случае? я думал о:

• FormatException: не совсем понятно, о проблеме,
• Пользовательские исключения: У меня нет никакого специального лечения для исключения я выбрасываю, так что я бы предпочел избегайте использования специального исключения, хотя это всегда способ уладить все.

Answer 1

FileFormatException должно быть прекрасно:

Исключение, которое генерируется, когда входной файл или поток данных, который должен соответствовать определенной спецификации формата файла уродливы.

Возможна установка uri и описательного сообщения об ошибке.

---

Если вы не хотите, чтобы ссылаться на WindowsBase, то вы можете создать свое собственное исключение, специфичное для вашего формата. Исходя из того, что есть XmlException, брошенный XmlReader.Read.

Answer 2

Я хотел бы бросить специальное исключение, так что повышает читабельность и позволяет уловить, что, специфические исключения:

public class InvalidFileFormatException : System.FormatException 
{ 
    public InvalidFileFormatException(string exText) : base(exText) { } 
} 

// you could even provide an exception if a single line has an invalid format 
public class SpecificLineErrorException : InvalidFileFormatException 
{ 
    public string Line { get; set; } 
    public SpecificLineErrorException(string exText, string line) : base(exText) 
    { 
     this.Line = line; 
    } 
} 

Теперь ваш метод может выглядеть (также linqified его немного:

public string ParseDatFile(string datFile) 
{ 
    string[] deezLines = File.ReadAllLines(datFile); 
    if(!deezLines.Any(l => l.Contains("daEssentialData"))) 
     throw new InvalidFileFormatException("yo dood where's da essential data in " + datFile + "?"); 
    DoStuffWith(deezLines); 
}