2

サードパーティの XML ファイルを処理しています。XMLTextReader.Read()これらのファイルには、例外をスローする原因となる無効な文字が含まれていることがあります。

現在、次の関数でこれを処理しています。

XmlTextReader GetCharSafeXMLTextReader(string fileName)
{
    try
    {
        MemoryStream ms = new MemoryStream();
        StreamReader sr = new StreamReader(fileName);
        StreamWriter sw = new StreamWriter(ms);
        string temp;
        while ((temp = sr.ReadLine()) != null)
            sw.WriteLine(temp.Replace(((char)4).ToString(), "").Replace(((char)0x14).ToString(), ""));

        sw.Flush();
        sr.Close();
        ms.Seek(0, SeekOrigin.Begin);
        return new XmlTextReader(ms);
    }
    catch (Exception exp)
    {
        throw new Exception("Error parsing file: " + fileName + " " + exp.Message, exp.InnerException);
    }
}

私の腸は、これを行うためのより良い/より速い方法があるべきだと言っています。(そうです、サードパーティに XML を修正してもらうのは素晴らしいことですが、現時点では実現していません。)

編集: cfeduke の回答に基づく最終的な解決策は次のとおりです。


    public class SanitizedStreamReader : StreamReader
    {
        public SanitizedStreamReader(string filename) : base(filename) { }
        /* other ctors as needed */
        // this is the only one that XmlTextReader appears to use but
        // it is unclear from the documentation which methods call each other
        // so best bet is to override all of the Read* methods and Peek
        public override string ReadLine()
        {
            return Sanitize(base.ReadLine());
        }

        public override int Read()
        {
            int temp = base.Read();
            while (temp == 0x4 || temp == 0x14)
                temp = base.Read();
            return temp;
        }

        public override int Peek()
        {
            int temp = base.Peek();
            while (temp == 0x4 || temp == 0x14)
            {
                temp = base.Read();
                temp = base.Peek();
            }
            return temp;
        }

        public override int Read(char[] buffer, int index, int count)
        {
            int temp = base.Read(buffer, index, count);
            for (int x = index; x < buffer.Length; x++)
            {
                if (buffer[x] == 0x4 || buffer[x] == 0x14)
                {
                    for (int a = x; a < buffer.Length - 1; a++)
                        buffer[a] = buffer[a + 1];
                    temp--; //decrement the number of characters read
                }  
            }
            return temp;
        }

        private static string Sanitize(string unclean)
        {
            if (unclean == null)
                return null;
            if (String.IsNullOrEmpty(unclean))
                return "";
            return unclean.Replace(((char)4).ToString(), "").Replace(((char)0x14).ToString(), "");
        }
    }
4

2 に答える 2

6

データのサニタイズは重要です。場合によっては、特殊なケース (「XML」の無効な文字) が発生することがあります。あなたの解決策は正しいです。ストリーミングに関して .NET フレームワークに適合するソリューションが必要な場合は、独自のストリームに適合するようにコードを再構築します。

public class SanitizedStreamReader : StreamReader {
  public SanitizedStreamReader(string filename) : base(filename) { }
  /* other ctors as needed */

  // it is unclear from the documentation which methods call each other
  // so best bet is to override all of the Read* methods and Peak
  public override string ReadLine() {
    return Sanitize(base.ReadLine());
  }

  // TODO override Read*, Peak with a similar logic as this.ReadLine()
  // remember Read(Char[], Int32, Int32) to modify the return value by
  // the number of removed characters

  private static string Sanitize(string unclean) {
    if (String.IsNullOrEmpty(unclean)
      return "";
    return unclean.Replace(((char)4).ToString(), "").Replace(((char)0x14);
  }
}

この新しいSanitizedStreamReader機能を使用すると、必要に応じてストリームを処理するようにチェーンすることができます。マジック メソッドに頼って物事をクリーンアップし、XmlTextReader を提示する必要はありません。

return new XmlTextReader(new SanitizedStreamReader("filename.xml"));

確かに、これは必要以上の作業になる可能性がありますが、このアプローチにより柔軟性が得られます。

于 2013-01-09T17:10:09.423 に答える
1

XML に関する懸念はさておき、ファイルが十分に大きくないために順次処理を行うことができない場合は、次のようにコードを簡略化します。

var xml = File.ReadAllText(pathName);
var fixedXml = xml.Replace(((char)4).ToString(), "").Replace(((char)0x14).ToString(), "");
File.WriteAllText(pathName, fixedXml);
于 2013-01-09T17:12:22.077 に答える