Sorting A Text File

To get rid of the compiler warning, change this line:

   private Container components;

to this:

   private Container components = null;

 The reason why you're getting the 'file is not a valid image' exception is because you're using the literal string "fileName" rather than the variable fileName as the argument to the File.ReadAllLines() method. However, the code still won't work as it stands unless you permanently alter the ordering of the file by writing the sorted lines back to disk, like this:

     string[] lines = File.ReadAllLines(fileName); // should be variable here
     LineComparer comparer = new LineComparer();
     Array.Sort(lines, comparer);
     File.WriteAllLines(fileName, lines); // overwrites original file on disk

     ReadFileInfo(fileName);

If you don't want to do that, then you'll need to alter the two methods concerned to something like the following as per my original suggestion:

            private void onFileOpen (object sender, EventArgs e)
            {
                if( pictureBox1 !=null)
                    pictureBox1.Hide();

                fileLoadArea.Text ="";
                mainPanel.Show();
                fileLoadArea.Show();

                OpenFileDialog openFileDialog1 = new OpenFileDialog();
                openFileDialog1.InitialDirectory = "c:\\" ; // changed from d drive
                openFileDialog1.RestoreDirectory = true ;
                openFileDialog1.Filter =
                    "Text Files (*.txt)|*.txt|Rich Text Format (*.rtf)|*.rtf)|"+
                    " All Files (*.*)|*.*";

                if (openFileDialog1.ShowDialog () == DialogResult.OK)
                {
                    String fileName = openFileDialog1.FileName;
                   
                    if (fileName.Length != 0)
                    {
                        try
                        {                        
                            ReadFileInfo(fileName, true); // true to sort file in memory
                        }
                        catch
                        {
                            MessageBox.Show (String.Format ("{0} is not " +
                            "a valid image file", fileName), "Error",
                            MessageBoxButtons.OK , MessageBoxIcon.Error);
                        }
                    }
                }
            }

           private void ReadFileInfo(String filename, bool sort)
           {
                try
                {
                    FileStream fs = new FileStream(filename, FileMode.Open, FileAccess.Read);
                    FileInfo fInfo = new FileInfo(filename);
                    string fext = fInfo.Extension.ToUpper();
                    if (fext.Equals(".RTF"))
                        fileLoadArea.LoadFile(fs, RichTextBoxStreamType.RichText);
                    else if (!sort)
                        fileLoadArea.LoadFile(fs, RichTextBoxStreamType.PlainText);
                    else
                    {
                        string[] lines = File.ReadAllLines(filename); 
                        LineComparer comparer = new LineComparer();
                        Array.Sort(lines, comparer);

                        fileLoadArea.Text = String.Join("\r\n", lines);
                    }
                    fs.Close();
                }
                catch(Exception e)
                {
                    MessageBox.Show("Exception"+e.StackTrace);
                }
            }

Another approach would be to first show the unsorted file in the RTB and then have a 'Sort' button which the user can press to sort the RTB's contents.

I am so close its not funny........

ive done what you suggested and when i build the application and try to open a text file it says this is not a valid image.the only message i get in express is;

Field 'FileDisplay.windows_forms.components' is never assigned to, and will always have its default value null

Ill inlcude the code to show you what i have

using System; using System.Drawing; using System.ComponentModel; using System.Windows.Forms; using System.IO; using System.Resources; using System.Collections.Generic; namespace FileDisplay {     // create a class to compare lines by date     public class LineComparer : IComparer     {         public int Compare(string s1, string s2)         {             string dateFormat = "MM.dd.yyyy HH:mm:ss";             DateTime dt1 = DateTime.ParseExact(s1.Substring(1, 19), dateFormat, null);             DateTime dt2 = DateTime.ParseExact(s2.Substring(1, 19), dateFormat, null);             if (dt1 == dt2)                 return 0;             else if (dt1 < dt2)                 return -1;             else                 return 1;         }     }     //windows_forms inherits from the Form class     public class windows_forms : Form //class in System.Windows.Forms     {         // Container is the class in System.ComponentModel namespace         private Container components;         //MenuItem is the class in System.Windows.Forms namespace         private MenuItem file;         private MenuItem openfile;         private MenuItem openTextfile;         private MenuItem openImagefile;         private MenuItem closefile;         private MenuItem exit;         private MenuItem help;         private MenuItem abouthelp;                  //MainMenu is the class in System.Windows.Forms namespace         private MainMenu mainMenu1;         //RichTextBox, PictureBox, Panel are the classes in System.Windows.Forms namespace         private RichTextBox fileLoadArea;         private PictureBox pictureBox1 ;         private Panel mainPanel;         public windows_forms() //constructor         {             InitializeComponent();                     }         private void InitializeComponent()         {             //initializing the classes             this.mainMenu1 = new MainMenu();             this.file = new MenuItem();             this.openfile = new MenuItem();             this.openTextfile = new MenuItem();             this.openImagefile = new MenuItem();             this.closefile = new MenuItem();             this.exit = new MenuItem();             this.help = new MenuItem();             this.abouthelp= new MenuItem();                         this.fileLoadArea = new RichTextBox();             this.pictureBox1 = new PictureBox();             this.mainPanel = new Panel();                         this.SuspendLayout();                                 // mainMenu1                 this.mainMenu1.MenuItems.AddRange(new MenuItem[]                                                         {                                                               this.file,                                                               this.help                                                         });                 // file                 this.file.Index = 0;                 this.file.MenuItems.AddRange(new MenuItem[]                                                         {                                                             this.openfile,                                                             this.closefile,                                                             this.exit                                                         });                 this.file.Text = "File";                                 // openfile                     this.openfile.Index = 0;                     this.openfile.MenuItems.AddRange(new MenuItem[]                                                         {                                                           this.openTextfile,                                                          this.openImagefile                                                         });                         this.openfile.Text = "OpenFile";                                     // openTextfile                         this.openTextfile.Index = 0;                         this.openTextfile.Text = "OpenTextFile...";                         this.openTextfile.Click += new System.EventHandler(this.onFileOpen);                                       // openImagefile                         this.openImagefile.Index = 1;                         this.openImagefile.Text = "&OpenImageFile...";                         this.openImagefile.Click += new System.EventHandler(this.onImageOpen);                                 // closefile                     this.closefile.Index = 1;                     this.closefile.Text = "CloseFile";                     this.closefile.Click += new System.EventHandler(this.onFileClose);                                 // exit                     this.exit.Index = 2;                     this.exit.Text = "exit";                     this.exit.Click += new System.EventHandler(this.onWindowClose);                             // help                 this.help.Index = 1;                 this.help.MenuItems.AddRange(new MenuItem[]                                                         {                                                              this.abouthelp                                                         });                 this.help.Text = "Help";                     // abouthelp                     this.abouthelp.Index = 0;                     this.abouthelp.Text = "Learning .NET";                                   // fileLoadArea                  this.fileLoadArea.Dock = DockStyle.Fill;                 this.fileLoadArea.Name = "fileLoadArea";                 this.fileLoadArea.Size = new Size(600, 400);                 this.fileLoadArea.Text = "";                               // pictureBox1             this.pictureBox1.Location = new Point(32, 40);             this.pictureBox1.Name = "pictureBox1";             this.pictureBox1.Size = new Size(600,400);                                           // mainPanel             //Control class is in System.Windows.Forms namespace             this.mainPanel.Controls.AddRange(new Control[]                                                             {                                                                 this.fileLoadArea,                                                                 this.pictureBox1                                                             });             this.mainPanel.Dock = DockStyle.Fill;             this.mainPanel.Name = "mainPanel";             this.mainPanel.Size = new Size(600, 400);                                                   // windows_forms                 this.ClientSize = new System.Drawing.Size(600, 500);                 this.Controls.AddRange(new Control[]                                                 {                                                         this.mainPanel                                                 });                 this.Menu = this.mainMenu1;                 this.Name = "windows_forms";                 this.Text = "Learning Windows Forms";                 mainPanel.Hide();                 this.ResumeLayout();         }                    // Handler for the TextFileOpen command             private void onFileOpen (object sender, EventArgs e)             {                 if( pictureBox1 !=null)                     pictureBox1.Hide();                 fileLoadArea.Text ="";                 mainPanel.Show();                 fileLoadArea.Show();                 OpenFileDialog openFileDialog1 = new OpenFileDialog();                 openFileDialog1.InitialDirectory = "d:\\" ;                 openFileDialog1.RestoreDirectory = true ;                 openFileDialog1.Filter =                     "Text Files (*.txt)|*.txt|Rich Text Format (*.rtf)|*.rtf)|"+                     " All Files (*.*)|*.*";                 if (openFileDialog1.ShowDialog () == DialogResult.OK)                 {                     String fileName = openFileDialog1.FileName;                     if (fileName.Length != 0)                     {                         try                         {                             string[] lines = File.ReadAllLines("fileName");                             LineComparer comparer = new LineComparer();                             Array.Sort(lines, comparer);                             //ReadFileInfo(fileName);                         }                         catch                         {                             MessageBox.Show (String.Format ("{0} is not " +                             "a valid image file", fileName), "Error",                             MessageBoxButtons.OK , MessageBoxIcon.Error);                         }                     }                 }             }         private void ReadFileInfo(String filename)             {                 try                 {                     FileStream fs = new FileStream(filename, FileMode.Open, FileAccess.Read);                     FileInfo fInfo = new FileInfo(filename);                     string fext = fInfo.Extension.ToUpper();                     if (fext.Equals(".RTF"))                         fileLoadArea.LoadFile(fs, RichTextBoxStreamType.RichText);                     else                         fileLoadArea.LoadFile(fs, RichTextBoxStreamType.PlainText);                     fs.Close();                 }                 catch(Exception e)                 {                     Console.WriteLine("Exception"+e.StackTrace);                 }             }             // Handler for the ImageFileOpen command             private void onImageOpen (object sender, EventArgs e)             {                 if (fileLoadArea !=null)                 {                     fileLoadArea.Hide();                 }                 mainPanel.Show();                                                 OpenFileDialog ofd = new OpenFileDialog ();                 ofd.Filter = "Image Files (*.bmp)|*.bmp|JPEG Files (*.jpeg)|*.jpeg|"+                             " All Files (*.*)|*.*";                 if (ofd.ShowDialog () == DialogResult.OK)                 {                     String fileName = ofd.FileName;                     if (fileName.Length != 0)                     {                         try                         {                             pictureBox1.BackgroundImage = new Bitmap(fileName);                             pictureBox1.Show();                         }                         catch                         {                             MessageBox.Show (String.Format ("{0} is not " +                             "a valid image file", fileName), "Error",                             MessageBoxButtons.OK , MessageBoxIcon.Error);                         }                     }                 }             }                     // method to drive the File/Close button             private void onFileClose (object sender, System.EventArgs e)             {                     mainPanel.Hide();             }                             // method to drive the Window/Close button         private void onWindowClose (object sender, System.EventArgs e)         {             // Handler for the Close command                 Close ();         }         public static void Main(string[] args)         {             Application.Run(new windows_forms());         }         protected override void Dispose( bool disposing )         {             if( disposing )             {                 if (components != null)                 {                     components.Dispose();                 }                             }             base.Dispose( disposing );         }     } }

Thanks again..

ill have a look tomorrow it way to late now....but thankyou very much

Yes, Visual C# 205 Express uses .NET 2.0 so it will be compatible.

The program will run as it stands as a console application.

However, you could integrate it into the tutorial which is a Windows Forms application.

Basically, you'd need to add this line to the 'using' directives:

           using System.Collections.Generic;

and then add the whole of the code for the LineComparer class.

The only other thing is to integrate these three lines (or something similar) into the ReadFileInfo() method:

          string[] lines = File.ReadAllLines("filetosort.txt");
          LineComparer comparer = new LineComparer();
          Array.Sort(lines, comparer);

As I haven't downloaded the code, perhaps I could leave that to you.

The filename will, of course, be a parameter to the ReadFileInfo() method and it might be a good idea to add a second parameter (bool sort) to determine whether the file is to be sorted or not. Needless to say, the LineComparer class will only work with text files which are in the format you specified.

I am using visual c# 2005 express edition will that be compatible....

and do i add this code into the tutorial program.

Thanks

Try this (.NET 2.0 or later):

using System;
using System.IO;
using System.Collections.Generic;

class Program
{
     static void Main()
     {
          string[] lines = File.ReadAllLines("filetosort.txt");
          LineComparer comparer = new LineComparer();
          Array.Sort(lines, comparer);

          // check the lines are now in date order
          Console.Clear();

          foreach(string s in lines)

          {
             Console.WriteLine(s);
          }

          Console.ReadKey();
     }
}

// create a class to compare lines by date
public class LineComparer : IComparer
{

     public int Compare(string s1, string s2)
     {
        string dateFormat = "MM.dd.yyyy HH:mm:ss";
        DateTime dt1 = DateTime.ParseExact(s1.Substring(1, 19), dateFormat, null);
        DateTime dt2 = DateTime.ParseExact(s2.Substring(1, 19), dateFormat, null);
        if (dt1 == dt2)
           return 0;
        else if (dt1 < dt2)
           return -1;
        else
           return 1;
     }
}