Winfprm Application Hangs on exit.

[Learning.Net]

hi ,

I have a application which reads files, directory,and its version and
version information is written to text file.Its working fine if files
in directory are less but problem arises when no of file increases.
Its hangs and on pressing exit button it showing application not
responding..
Plz help me out
Here is da sample code::::::

using System;
using System.Drawing;
using System.Collecti ons;
using System.Componen tModel;
using System.Windows. Forms;
using System.Data;
using System.Diagnost ics;
using System.IO;
using System.Collecti ons.Specialized ;
using System.Threadin g;

namespace FileVersionUiti lity
{
/// <summary>
/// Summary description for Form1.
/// </summary>
public class Form1 : System.Windows. Forms.Form
{
private System.Windows. Forms.Label lblSourceName;
private System.Windows. Forms.TextBox txtBFileInfo;
private System.Windows. Forms.Button btnVersion;
private System.Windows. Forms.Button button1;
/// <summary>
/// Required designer variable.
/// </summary>
public string DirSourcePath ,DirDestination Path;
// FileInfo[] files;
public static StreamWriter writer;
private System.Windows. Forms.GroupBox groupBox1;
private System.Windows. Forms.Label lblDestinationP ath;
private System.Windows. Forms.TextBox txtBDestPath;
private System.Windows. Forms.Button btn2Browse;
private System.Windows. Forms.SaveFileD ialog saveFileDialog1 ;
private System.Windows. Forms.Button button2;
private System.Componen tModel.Containe r components = null;

public Form1()
{
//
// Required for Windows Form Designer support
//
InitializeCompo nent();

//
// TODO: Add any constructor code after InitializeCompo nent call
//

}

/// <summary>
/// Clean up any resources being used.
/// </summary>
protected override void Dispose( bool disposing )
{
if( disposing )
{
if (components != null)
{
components.Disp ose();
}
}
base.Dispose( disposing );
}

#region Windows Form Designer generated code
/// <summary>
/// Required method for Designer support - do not modify
/// the contents of this method with the code editor.
/// </summary>
private void InitializeCompo nent()
{
this.lblSourceN ame = new System.Windows. Forms.Label();
this.txtBFileIn fo = new System.Windows. Forms.TextBox() ;
this.btnVersion = new System.Windows. Forms.Button();
this.button1 = new System.Windows. Forms.Button();
this.groupBox1 = new System.Windows. Forms.GroupBox( );
this.button2 = new System.Windows. Forms.Button();
this.btn2Browse = new System.Windows. Forms.Button();
this.txtBDestPa th = new System.Windows. Forms.TextBox() ;
this.lblDestina tionPath = new System.Windows. Forms.Label();
this.saveFileDi alog1 = new System.Windows. Forms.SaveFileD ialog();
this.groupBox1. SuspendLayout() ;
this.SuspendLay out();
//
// lblSourceName
//
this.lblSourceN ame.Location = new System.Drawing. Point(16, 48);
this.lblSourceN ame.Name = "lblSourceName" ;
this.lblSourceN ame.Size = new System.Drawing. Size(104, 23);
this.lblSourceN ame.TabIndex = 0;
this.lblSourceN ame.Text = "Source Path:";
this.lblSourceN ame.TextAlign =
System.Drawing. ContentAlignmen t.MiddleRight;
//
// txtBFileInfo
//
this.txtBFileIn fo.AutoSize = false;
this.txtBFileIn fo.BorderStyle =
System.Windows. Forms.BorderSty le.FixedSingle;
this.txtBFileIn fo.ForeColor = System.Drawing. Color.Wheat;
this.txtBFileIn fo.Location = new System.Drawing. Point(120, 48);
this.txtBFileIn fo.Name = "txtBFileIn fo";
this.txtBFileIn fo.ReadOnly = true;
this.txtBFileIn fo.Size = new System.Drawing. Size(224, 24);
this.txtBFileIn fo.TabIndex = 1;
this.txtBFileIn fo.Text = "";
//
// btnVersion
//
this.btnVersion .FlatStyle = System.Windows. Forms.FlatStyle .Popup;
this.btnVersion .Location = new System.Drawing. Point(16, 144);
this.btnVersion .Name = "btnVersion ";
this.btnVersion .Size = new System.Drawing. Size(104, 23);
this.btnVersion .TabIndex = 2;
this.btnVersion .Text = "Get File Version";
this.btnVersion .Click += new
System.EventHan dler(this.btnVe rsion_Click);
//
// button1
//
this.button1.Ba ckColor = System.Drawing. Color.OldLace;
this.button1.Fl atStyle = System.Windows. Forms.FlatStyle .Popup;
this.button1.Lo cation = new System.Drawing. Point(360, 48);
this.button1.Na me = "button1";
this.button1.Ta bIndex = 3;
this.button1.Te xt = "Browse";
this.button1.Cl ick += new System.EventHan dler(this.butto n1_Click);
//
// groupBox1
//
this.groupBox1. BackColor = System.Drawing. Color.OldLace;
this.groupBox1. Controls.Add(th is.button2);
this.groupBox1. Controls.Add(th is.btn2Browse);
this.groupBox1. Controls.Add(th is.txtBDestPath );
this.groupBox1. Controls.Add(th is.lblDestinati onPath);
this.groupBox1. Controls.Add(th is.lblSourceNam e);
this.groupBox1. Controls.Add(th is.txtBFileInfo );
this.groupBox1. Controls.Add(th is.button1);
this.groupBox1. Controls.Add(th is.btnVersion);
this.groupBox1. Font = new System.Drawing. Font("Microsoft Sans
Serif", 8.25F, System.Drawing. FontStyle.Bold,
System.Drawing. GraphicsUnit.Po int, ((System.Byte)( 0)));
this.groupBox1. Location = new System.Drawing. Point(24, 48);
this.groupBox1. Name = "groupBox1" ;
this.groupBox1. Size = new System.Drawing. Size(472, 192);
this.groupBox1. TabIndex = 4;
this.groupBox1. TabStop = false;
this.groupBox1. Text = "File Version Information";
//
// button2
//
this.button2.Fl atStyle = System.Windows. Forms.FlatStyle .Popup;
this.button2.Lo cation = new System.Drawing. Point(360, 144);
this.button2.Na me = "button2";
this.button2.Si ze = new System.Drawing. Size(72, 23);
this.button2.Ta bIndex = 7;
this.button2.Te xt = "Exit";
this.button2.Cl ick += new System.EventHan dler(this.butto n2_Click);
//
// btn2Browse
//
this.btn2Browse .FlatStyle = System.Windows. Forms.FlatStyle .Popup;
this.btn2Browse .Location = new System.Drawing. Point(360, 96);
this.btn2Browse .Name = "btn2Browse ";
this.btn2Browse .TabIndex = 6;
this.btn2Browse .Text = "Browse";
this.btn2Browse .Click += new
System.EventHan dler(this.btn2B rowse_Click);
//
// txtBDestPath
//
this.txtBDestPa th.AutoSize = false;
this.txtBDestPa th.BorderStyle =
System.Windows. Forms.BorderSty le.FixedSingle;
this.txtBDestPa th.Location = new System.Drawing. Point(120, 96);
this.txtBDestPa th.Name = "txtBDestPa th";
this.txtBDestPa th.ReadOnly = true;
this.txtBDestPa th.Size = new System.Drawing. Size(224, 24);
this.txtBDestPa th.TabIndex = 5;
this.txtBDestPa th.Text = "";
//
// lblDestinationP ath
//
this.lblDestina tionPath.Locati on = new System.Drawing. Point(16,
96);
this.lblDestina tionPath.Name = "lblDestination Path";
this.lblDestina tionPath.Size = new System.Drawing. Size(104, 23);
this.lblDestina tionPath.TabInd ex = 4;
this.lblDestina tionPath.Text = "Destinatio n Path:";
this.lblDestina tionPath.TextAl ign =
System.Drawing. ContentAlignmen t.MiddleRight;
//
// Form1
//
this.AutoScaleB aseSize = new System.Drawing. Size(5, 13);
this.BackColor = System.Drawing. Color.OldLace;
this.ClientSize = new System.Drawing. Size(528, 278);
this.Controls.A dd(this.groupBo x1);
this.Name = "Form1";
this.Text = "Form1";
this.groupBox1. ResumeLayout(fa lse);
this.ResumeLayo ut(false);

}
#endregion

/// <summary>
/// The main entry point for the application.
/// </summary>
[STAThread]
static void Main()
{
Application.Run (new Form1());
}
private void btnVersion_Clic k(object sender, System.EventArg s e)
{

if(txtBFileInfo .Text=="")
{
MessageBox.Show ("Source Path Missing!! Please Select The Directory
Path ","Error report",Message BoxButtons.OK,M essageBoxIcon.E rror);
}
else if(txtBDestPath .Text=="")
{
MessageBox.Show ("Destinatio n Path Missing!! Please Select The
Destination Path ","Error
report",Message BoxButtons.OK,M essageBoxIcon.E rror);
}

// FileVersionInfo myFileVersionIn fo;
// DirectoryInfo root = new DirectoryInfo(D irSourcePath);
// DirectoryInfo[] dirs = root.GetDirecto ries();
// files = root.GetFiles() ;
// string fileName;
// foreach (FileInfo file in files)
// {
// fileName = file.FullName;
//// MessageBox.Show (file.FullName) ;
// myFileVersionIn fo =
FileVersionInfo .GetVersionInfo (fileName);
//
// MessageBox.Show ("File Name: " + file.Name + " Version
Info : "+ myFileVersionIn fo.FileVersion) ;
// writer.WriteLin e("File Name: " + file.Name + " Version
Info : "+ myFileVersionIn fo.FileVersion) ;
//
// }
else
{
writer = new StreamWriter(Di rDestinationPat h);
//Create a Directory object using DirectoryInfo
// writer.WriteLin e("File Name :" + " Version " + "Location") ;
DirectoryInfo dir =
new DirectoryInfo(D irSourcePath);
//Pass the Directory for displaying the contents
getDirsFiles(di r);
writer.Close();
MessageBox.Show ("File is Created......." );
}

}

private void button1_Click(o bject sender, System.EventArg s e)
{
FolderBrowserDi alog folderBrowserDi alog = new
FolderBrowserDi alog();
folderBrowserDi alog.ShowDialog ();
txtBFileInfo.Te xt = folderBrowserDi alog.SelectedPa th;
DirSourcePath =txtBFileInfo.T ext;
}

public static void getDirsFiles(Di rectoryInfo d)
{
//create an arrakmly of files using FileInfo object
FileInfo [] files;
//get all files for the current directory
files = d.GetFiles("*.* ");
ArrayList arr = new ArrayList();
//iterate through the directory and print the files
foreach (FileInfo file in files)
{
//get details of each file using file object
String fileFullName = file.FullName;
String fileName = file.Name;
String fileSize = file.Length.ToS tring();
String fileExtension =file.Extension ;
String fileCreated = file.LastWriteT ime.ToString();
FileVersionInfo fileVersion =
FileVersionInfo .GetVersionInfo (fileFullName);
if(fileVersion. FileVersion!="" )
{
arr.Add(fileNam e + " Version Details:: " +
fileVersion.Fil eVersion);
}
else
{
arr.Add(fileNam e + "Last Modified date :" +
file.LastWriteT ime);
}
// arr.Add(fileNam e);
// arr.Add(fileVer sion.FileVersio n);
arr.Sort();

//
// Console.WriteLi ne(fileName + " " + fileSize +
// " " + fileExtension + " " + fileCreated);

// writer.WriteLin e(fileName + " " + fileSize + " " +
fileExtension + " "
// + fileCreated + "Version :" + fileVersion.Fil eVersion);
// writer.WriteLin e(fileName + " " +
fileVersion.Fil eVersion + " " + fileFullName);

}

foreach( string s in arr)
{
writer.WriteLin e(s);
}
//get sub-folders for the current directory
DirectoryInfo [] dirs = d.GetDirectorie s("*.*");

//This is the code that calls
//the getDirsFiles (calls itself recursively)
//This is also the stopping point
//(End Condition) for this recursion function
//as it loops through until
//reaches the child folder and then stops.
foreach (DirectoryInfo dir in dirs)
{
// Console.WriteLi ne("--------->{0} ", dir.Name);
writer.WriteLin e(dir.FullName) ;
writer.WriteLin e();
getDirsFiles(di r);
}

}

private void btn2Browse_Clic k(object sender, System.EventArg s e)
{
saveFileDialog1 .Filter = "Text files(*.txt)|*. txt";
saveFileDialog1 .ShowDialog();
DirDestinationP ath = saveFileDialog1 .FileName;
txtBDestPath.Te xt = DirDestinationP ath;
}

private void button2_Click(o bject sender, System.EventArg s e)
{
Application.Exi t();
this.Close();
}

private void button3_Click(o bject sender, System.EventArg s e)
{

}
}
}

[Moty Michaely]

Hi,

I would firstly suggest doing all the time consuming work in a
background thread.
You can easily use the BackgroundWorke r component:
http://msdn2.microsoft.com/en-us/library/8xs8549b.aspx

If you don't want to do that, you should be able to pump the message
queue in the middle of the time consuming job. Insert an
Application.DoE vents() call in your foreach loop to pump the messages
(such as the button Click).

Feel free to ask any further questions.
Cheers,
Moty.

[Learning.Net]

On Jun 6, 7:44 pm, Moty Michaely <Moty...@gmail. comwrote:

Hi,

I would firstly suggest doing all the time consuming work in a
background thread.
You can easily use the BackgroundWorke r component:http://msdn2.microsoft.com/en-us/library/8xs8549b.aspx

If you don't want to do that, you should be able to pump the message
queue in the middle of the time consuming job. Insert an
Application.DoE vents() call in your foreach loop to pump the messages
(such as the button Click).

Feel free to ask any further questions.
Cheers,
Moty.

Thanx for da help,If possible can u Suggest with some code.