5

そのため、長さが不明な xml ファイルを読み込んで、各要素をリスト構造に読み込んでいます。現在、読み続けているファイルの最後に到達すると、例外が発生します。今、私はこの例外をキャッチして自分の人生を続けていますが、これを行うためのよりクリーンな方法はありますか?

try
{
    while(!textReader.EOF)
    {
        // Used to store info from each command as they are read from the xml file
        ATAPassThroughCommands command = new ATAPassThroughCommands ();                      
        // the following is just commands being read and their contents being saved
        XmlNodeType node = textReader.NodeType;                                             

        textReader.ReadStartElement( "Command" );
        node = textReader.NodeType;
        name = textReader.ReadElementString( "Name" );
        node = textReader.NodeType;
        CommandListContext.Add(name);
        command.m_Name = name;
        command.m_CMD = Convert .ToByte(textReader.ReadElementString("CMD" ),16);
        command.m_Feature = Convert .ToByte(textReader.ReadElementString("Feature" ),16);

        textReader.ReadEndElement(); //</command>
        m_ATACommands.Add(command);
    }
}
catch ( Exception ex)
{
    //</ATAPassThrough>   TODO: this is an ugly fix come up with something better later
    textReader.ReadEndElement(); 
    //cUtils.DisplayError(ex.Message);
}

xml ファイル:

<ATAPassThrough>
  <Command>
    <Name>Smart</Name>
    <CMD>B0</CMD>
    <Feature>D0</Feature>
  </Command>
  <Command>
    <Name>Identify</Name>
    <CMD>B1</CMD>
    <Feature>D0</Feature>
  </Command>
    .
    .
    .
    .
</ATAPassThrough>
4

2 に答える 2

11

XMLデータを読み取るためにXDocumentを使用することをお勧めします...たとえば、あなたの場合、XML用のTextReaderがすでにあるので、それをXDocument.Loadメソッドに渡すだけです...上記の関数全体は次のようになります..

var doc = XDocument.Load(textReader);
foreach (var commandXml in doc.Descendants("Command"))
{
    var command = new ATAPassThroughCommands();
    var name = commandXml.Descendants("Name").Single().Value;
    // I'm not sure what this does but it looks important...
    CommandListContext.Add(name); 
    command.m_Name = name;
    command.m_CMD = 
         Convert.ToByte(commandXml.Descendants("CMD").Single().Value, 16);
    command.m_Feature = 
         Convert.ToByte(commandXml.Descendants("Feature").Single().Value, 16);
    m_ATACommands.Add(command);
}

大幅に簡単。面倒な作業はフレームワークに任せましょう。

于 2013-07-30T22:17:56.043 に答える
6

通常の一貫した XML がある場合、おそらく最も簡単な方法は、XML Serializerを使用することです。

最初に XML に一致するオブジェクトを作成します

[Serializable()]
public class Command
{
  [System.Xml.Serialization.XmlElement("Name")]
  public string Name { get; set; }

  [System.Xml.Serialization.XmlElement("CMD")]
  public string Cmd { get; set; }

  [System.Xml.Serialization.XmlElement("Feature")]
  public string Feature { get; set; }
}

[Serializable()]
[System.Xml.Serialization.XmlRoot("ATAPassthrough")]
public class CommandCollection
{
  [XmlArrayItem("Command", typeof(Command))]
  public Command[] Command { get; set; }
}

を返す a メソッドCommandCollection

public class CommandSerializer
{
  public commands Deserialize(string path)
  {
    CommandCollection commands = null;

    XmlSerializer serializer = new XmlSerializer(typeof(CommandCollection ));

    StreamReader reader = new StreamReader(path);
    reader.ReadToEnd();
    commands = (CommandCollection)serializer.Deserialize(reader);
    reader.Close();

    return commands ;
  }
}

これが正確かどうかはわかりません。テストする手段はありませんが、実際には近いはずです。

于 2013-07-30T22:20:42.987 に答える