0

what i'm trying to do here is to delete the longest line from a txt file. Code does it's job, but i also need it to delete multiple "longest lines" and blank lines as well. Any ideas on how to do it?

Code is in C#

    namespace _5_2
    {
    //------------------------------------------------------------
    class Program
    {
         const string CFd = "..\\..\\U1.txt";
         const string CFr = "..\\..\\Results.txt";
         static void Main(string[] args)
         {
             int nr;
             Read(CFd, out nr);
             Print(CFd, CFr, nr);
             Console.WriteLine("Longest line nr. {0, 4:d}", nr + 1);
             Console.WriteLine("Program done");
         }
         //------------------------------------------------------------
         /** Finds number of the longest line.
         @param fv - text file name
         @param nr - number of the longest line */
         //------------------------------------------------------------
         static void Read(string fv, out int nr)
         {
             string[] lines = File.ReadAllLines(fv, Encoding.GetEncoding(1257));
             int ilgis = 0;
             nr = 0;
             int nreil = 0;
         foreach (string line in lines)
         {
            if (line.Length > ilgis)
               {
                  ilgis = line.Length;
                   nr = nreil;
               }
              nreil++;
          }
        }
         static void Print(string fv, string fvr, int nr)
         {
             string[] lines = File.ReadAllLines(fv, Encoding.GetEncoding(1257));
             int nreil = 0;
             using (var fr = File.CreateText(fvr))
             {
                 foreach (string line in lines)
                 {
                     if (nr != nreil)
                     {
                         fr.WriteLine(line);
                     }
                     nreil++;
                 }
             }
         }
      }
  }
Ron Beyer
  • 11,003
  • 1
  • 19
  • 37
Deividas
  • 79
  • 1
  • 9
  • 1
    You could identify the longest line, and then loop through the list, deleting all of that length. To also delete empty ones, you could test against String.IsNullOrWhiteSpace. – B. Clay Shannon-B. Crow Raven Nov 11 '15 at 19:05
  • I'm going to try that looping though the list to delete multiples, but i still don't understand what to do with the empty ones. In my Print method i changed line if ( nr != nreil) to if (line != "") . This works until it's just a blank line, but as soon as i "write" a space in that line, line stays. – Deividas Nov 11 '15 at 19:19
  • If you add a space, you've added a line. Just because you can't see air, that doesn't mean it's not there / has no weight. – B. Clay Shannon-B. Crow Raven Nov 11 '15 at 19:45

3 Answers3

0

I would suggest using LINQ. Take advantage of the .Max extension method and iterate over the string array.

string[] lines = { "1", "two", "three" };
var longestLine = lines.Max(line => line.Length);
var revised = lines.Where(line => line.Length < longestLine).ToArray();

The revised variable will contain a string array that excludes the lines with the longest line count.

David Pine
  • 23,787
  • 10
  • 79
  • 107
0

Read lines, filter out empty lines and the 10 longest lines, write lines:

     string[] lines = File.ReadAllLines(inputFile, Encoding.GetEncoding(1257));
     var filtered = lines
         .Where(line => line.Length > 0) // remove all empty lines
         .Except(lines.OrderByDescending(line => line.Length).Take(10)); // remove 10 longest lines
     File.WriteAllLines(outputFile, filtered);
Mud
  • 28,277
  • 11
  • 59
  • 92
-1

You could identify the longest line, and then loop through the list, deleting all of that length. To also delete empty ones, you could test against String.IsNullOrWhiteSpace.

Something like (pseudocode):

     foreach (string line in lines)
     {
        if (String.IsNullOrWhiteSpace(line)) 
        {
            lines.Delete(line);
            Continue;
        }
        if (line.Length >= longestLine) // ">=" instead of "==" just to be on the safe side
        {
           lines.Delete(line);
        }
    }
B. Clay Shannon-B. Crow Raven
  • 8,547
  • 144
  • 472
  • 862
  • Lines.Delete thingy didn't worked for me too. But i found a way. : If (!String.IsNullOrWhiteSpace(line)) and then just simple fr.Writeline(line) – Deividas Nov 11 '15 at 19:51