Господа, прошу помочь начинающему программисту. Только учусь работать с потоками, ну и для закрепления материала решил написать программу для поиска дубликатов файлов. Алгоритм выглядит примерно так:
- В отдельном потоке запускается функция которая ищет все файлы по указанному пути (в подпапках тоже), эти файлы записываются в HashSet;
- В процессе поиска этих файлов, в label выводится текущая папка, где программа ищет файлы;
- После окончания поиска, при помощи linq в этом массиве ищутся дубликаты файлов;
- Эти дубликаты выводятся в listView через foreach, при этом высчитывается процент выполнения работы и он отображается в ProgressBar'e.
Всё бы ничего, да вот только удумал я замерить время выполнения поиска. Получилось порядка 1,05 минут. Ради эксперимента, взял и убрал вывод в label текущей папки, где идёт поиск. Вышло уже 44секунды. После убрал вывод в прогрессбар процента поиска, вышло (4 секунды!!!). Замеры производил при помощи StopWatch'a. Код представлен ниже:
class FileClass
{
public string Name { get; set; }
public double Size { get; set; }
public string FullPath { get; set; }
}
public partial class Form1 : Form
{
delegate void PositionDelegate(int pos);
HashSet<FileClass> files = new HashSet<FileClass>();
event PositionDelegate ChangePosition;
int current;
/// <summary>
/// текущий прогресс работы
/// </summary>
int CurrentPosition
{
get
{ return current; }
set
{
if (ChangePosition != null)
ChangePosition(current);
current = value;
}
}
public Form1()
{
InitializeComponent();
ChangePosition += delegate(int pos)
{
if (progressBar1.InvokeRequired)
progressBar1.Invoke(new MethodInvoker(delegate
{
progressBar1.Value = pos;
}));
else
progressBar1.Value = pos;
};
}
/// <summary>
/// Отображение в label текущей папки
/// </summary>
/// <param name="currentDir"></param>
void controlCallBack(string currentDir)
{
label1.Invoke(new MethodInvoker(delegate
{
label1.Text = currentDir;
}));
}
/// <summary>
/// Поиск файлов в папках(включая подпапки)
/// </summary>
/// <param name="dr"></param>
/// <param name="pattern"></param>
void getallfiles(DirectoryInfo dr, string pattern)
{
try
{
dr.GetFiles(pattern).ToList().ForEach(s =>
{
files.Add(new FileClass() { Name = s.Name, FullPath = s.FullName, Size = s.Length });
controlCallBack(s.FullName);
});
}
catch (UnauthorizedAccessException)
{
}
try
{
foreach (DirectoryInfo d in dr.GetDirectories())
getallfiles(d, pattern);
}
catch (UnauthorizedAccessException)
{ }
}
/// <summary>
/// создание потока для поиска дубликатов
/// </summary>
/// <param name="sender"></param>
/// <param name="e"></param>
private void button1_Click(object sender, EventArgs e)
{
new Thread(new ThreadStart(delegate
{
getallfiles(new DirectoryInfo(@"C:\Users\lionell"), "*.*");
var dublicates = from rep in files group rep by rep.Name into g where g.Skip(1).Any() from rep in g select rep;
double curr=0;
listView1.Invoke(new MethodInvoker(delegate
{
listView1.BeginUpdate();
}));
foreach (var f in dublicates)
{
listView1.Invoke(new MethodInvoker(delegate
{
ListViewItem it = new ListViewItem(f.Name);
it.SubItems.Add(f.FullPath);
listView1.Items.Add(it);
}));
curr += 1;
CurrentPosition = int.Parse(Math.Round((curr / dublicates.Count()) * 100, 0).ToString());
}
listView1.Invoke(new MethodInvoker(delegate
{
listView1.EndUpdate();
}));
})) { IsBackground = true }.Start();
}
Буду благодарен любой помощи