如何从文件中删除条目

Question

try to delete some entry from csv file, if i know only part of string code

 //open File
        FileStream fs = new FileStream(filePath, FileMode.Open, 
            FileAccess.ReadWrite);
        StreamReader sr = new StreamReader(fs);            
 //divide file entry for separate lines
        string []line = sr.ReadToEnd().Split(new string[] { Environment.NewLine }, StringSplitOptions.RemoveEmptyEntries);
 // find required entry and create new List without it
        List<string> finalData = new List<string>();
        foreach (var l in line)
        {
            if (!line.Contains((dateTimePicker1.Text.ToString().Trim() + ','+ eventNameDeleteTextBox.Text.ToString().Trim()+',')))
                finalData.Add(l);
        }
  // convert to array
        string toCsvoutput = string.Join(Environment.NewLine, finalData.ToArray());
      //  viewTextBox1.Text=toCsvoutput; 

        //updateFile
        sr.Close();
        FileStream fs1 = new FileStream(filePath, FileMode.Open,FileAccess.Write, FileShare.None);
        StreamWriter sw = new StreamWriter(fs1);
        sw.WriteLine(toCsvoutput);
        sw.Close();

As result - nothing heppend, where i'm wrong?

Answer 1

Replace line with l:

  if (!l.Contains((dateTimePicker1.Text.ToString().Trim() + ','+ 
    eventNameDeleteTextBox.Text.ToString().Trim()+',')))
     finalData.Add(l);

Answer 2

You can clean up a lot:

string skipText = (dateTimePicker1.Text.ToString().Trim() + ','+
      eventNameDeleteTextBox.Text.ToString().Trim()+',')

var linesIn = System.IO.File.ReadLines(filePath);
var linesOut = linesIn.Where(line => ! line.Contains(skipText));
linesOut = linesOut.ToList();  // make sure the file is closed again
System.IO.File.WriteAllLines(filePath, linesOut);

Answer 3

the condition in the foreach statement probably always pass. you should use debug to check the condition itself, make sure it's what you though.

also, you should do

string condition = dateTimePicker1.Text.ToString().Trim() + ','+ eventNameDeleteTextBox.Text.ToString().Trim()+',';

outside the foreach statement, so you won't waste time for nothing.