どのように無限ループを置き換えることができますか？

Question

私は、空白行を削除したい、かなり非効率的なC＃コードに取り組んでいます。

 
      string b; 
... 
      while (b.IndexOf("\n\n") >= 0) 
       b = b.Replace ("\n\n", "\n"); 

入力に1つの置換が（たとえば）\ n \ n \ nに対応しないため、ループが必要です。私はそれが働くべきだと思います、そして、それは通常行います。

しかし、それは無限ループに陥ることがありますが、私はどのように理解できません。すべての反復で、\ nの数は減らされるはずなので、最終的に終了する必要があります。

アイデア？

Answer 1

うこの作品：

String c = Regex.Replace(b, "\n\n+", "\n"); 

Answer 2

あなたは、これが無限ループに入っているため、文字列の例を与えることができますか？また、プログラムをデバッグする場合は、次のように置き換えてみてください。

出力内容を確認してください。

Answer 3

説明できない無限ループについては説明がありません（文字列が変更されているかどうか確認しましたか？）が、正規表現ではこれをはるかに簡単かつ迅速に行うことができます：

b = System.Text.RegularExpressions.Regex.Replace(b, "\n+", "\n") 

Answer 4

他の誰かが来た場合に、この点を明確にするために、この回答をここに載せて、bが空文字列の場合、上に掲載されたコードが無限ループすることを示唆しています。それは正しくありません。それに渡された値パラメータが空の場合、IndexOfは、文字列自体（この場合b）が空でない場合は、0を返すこと

String b = String.Empty; 

Console.WriteLine(b.IndexOf("\n\n")); 

// output: -1 

documentation状態。

Answer 5

私はファイルを読むことによって（以下の完全なコード）、厄介な文字列に問題を固定しました。ここではFF FE 00 0D 0A 00 42 00

41 00 0D 0A 00 0D 0Aは私のプログラムのデバッグ出力です：

だから、

b.Length=8 loop n=1, i=3, b=A?? 
?? B 
stuck at i=3, b(i)=10 2573 3328... 
done n=1, i=3, b=A?? 
?? B 

ファイルs.tabは、これらの18進のバイトが含まれていますそれは無効なユニコードと関係しています。私は文字列bの文字の小数点を、i = 3 = IndexOf（ "\ n \ n"）で始まります。 IndexOfは10を改行（OK）と見なし、2573（0D 0A）を別のもの（OKではない）と見なします。次に、Replaceは同意しません。

明らかにファイル内のデータに問題があります。しかし、私はまだこれが起こるとは思わない。 IndexOfとReplaceは同意する必要があります。

私はmsaeedのソリューションを実装しています。どうもありがとう。

デバッグコード：

 { 
      System.IO.StreamReader aFile = System.IO.File.OpenText(@"c:\xfer\s.tab"); 
      string a = aFile.ReadToEnd(); 
      aFile.Close(); 

      int nn=0, ii; 
      Console.WriteLine ("a.Length={0}", a.Length); 
      while ((ii=a.IndexOf("\n\n")) >= 0) 
      { 
       nn++; 
       Console.WriteLine("loop n={0}, i={1}, a={2}" 
        , nn 
        , ii 
        , a); 
       if (ii == a.IndexOf("\n\n")) 
       { 
        Console.WriteLine ("stuck at i={0}, a(i)={1} {2} {3}..." 
         , ii 
         , (int)(a.ToCharArray()[ii]) 
         , (int)(a.ToCharArray()[ii+1]) 
         , (int)(a.ToCharArray()[ii+2]) 
         ); 
        break; 
       } 
       a = a.Replace ("\n\n", "\n"); 
      } 
      Console.WriteLine("done n={0}, i={1}, a={2}", nn, ii, a); 
     }