foreachを使用してtxtで文字列を置換する

Question

テキストファイルの文字列を他のファイルにあるすべてのものに置き換えようとしています。私のHTMLの電子メール変数。 しかし、私はforeachを実行しようとするたびに、それは文字列に文字を変換することができますエラーを私に与える。これを別のやり方でどうやって行うのですか？

 StreamReader myreader = new StreamReader("VUCresult.txt"); 
     StreamReader myreaderhtml = new StreamReader("htmlemail.html"); 
     string lines = myreader.ReadToEnd(); 
     string htmlmailbody = myreaderhtml.ReadToEnd(); 


     if (lines == "Der er ikke nogen udmeldinger idag") 
     { 
      htmlmailbody.Replace("ingen", lines); 
     } 
     else 
     { 
      foreach (string s in lines) 
      { 
       htmlmailbody = htmlmailbody.Replace("Row2", s); 
      } 
     htmlmailbody = htmlmailbody.Replace("Row1", lines); 
     htmlmailbody = htmlmailbody.Replace("Row3", DateTime.Now.ToString()); 
     } 

Answer 1

非常に長い文字列（改行を含むことがあります）にはforeachが使用されています。それぞれの個性をあなたに返します。ただFile.ReadAllLinesを使用します（コレクションとして）ファイル内のすべての行を取得するには

：

string htmlmailbody;   
    using StreamReader myreaderhtml = new StreamReader("htmlemail.html")) 
    { 
     htmlmailbody = myreaderhtml.ReadToEnd(); 
    } 
    string[] lines = File.ReadAllLines("VUCresult.txt"); 

    foreach (string s in lines) 
    { 
     ... 
    } 

あなたがラインのコレクションの代わりに、ファイル全体を持っているので、あなたのオリジナルifチェックはここに意味することはありませんelseの2番目のreplace文も意味をなさないでしょう。あなたは本当に見たいものを決める必要があります。

Answer 2

ここでforeachはすでに文字列であり、foreachを呼び出すと変数 's'がcharになるため、foreachは必要ありません。さらに、文s.ToString（）は文字列を返すので何もしないので、s自体は変更されません。

さらに、 'using'ステートメントでリーダーを囲むことを検討してください。