Question

I am trying to figure out the best way to keep my application responsive. Below shows the code that I am currently working with. What i have found is that the Background worker thread is the way to go.

    private void cleanFiles()
    {
            if (listView1.CheckedItems.Count != 0)
            {

                // If so, loop through all checked files and delete.

                foreach (ListViewItem item in listView1.CheckedItems)
                {

                    string fileName = item.Text;
                    string filePath = Path.Combine(tFile + fileName);

                    try
                    {

                        File.Delete(filePath);
                    }
                    catch (Exception)
                    {
                        //ignore files being in use
                    }

                    MessageBox.Show("Files Cleaned");
                }


            }
            else
            {
                 MessageBox.Show("Please put a check by the files you want to delete");
            }

        }


    }

    }

Answer 1

The easiest way to keep your program responsive is to use the BackgroundWorker.

List<string listWithFilenames = new List<string>();
BackgroundWorker bw = new BackgroundWorker();
bw.DoWork += new DoWorkEventHandler(bw_DoWork);
bw.RunWorkerCompleted += new RunWorkerCompletedEventHandler(bw_RunWorkerCompleted);
bw.RunWorkerAsync(listWithFilenames);

See the documentation.