Приложение фризит после подключения DataTable к DataGridView

Question

Мой код ищет файлы по названию в путях и и в таблицу добавляет результат. Для этого я использую Task, но после подключения DataTable к DataGridView, приложение фризит даже в Task'е и перестаёт отвечать хотя до подключения DataTable такого не было.

DataTable dt = new DataTable();

public Main()
{
    InitializeComponent();
    found_datagridview.DataSource = dt;
    dt.Columns.Add("Type");
    dt.Columns.Add("Date");
    dt.Columns.Add("Size");
    dt.Columns.Add("Location");
    found_datagridview.Columns[0].Width = 60;
    found_datagridview.Columns[1].Width = 141;
    found_datagridview.Columns[2].Width = 130;
    found_datagridview.Columns[3].Width = 418;
}

async Task FindFiles(CancellationToken ct)
{
    await Task.Run(() =>
    {
        path = path_textbox.Text;
        name = name_textbox.Text;

        dt.Rows.Clear();
        try
        {
            ct.ThrowIfCancellationRequested();
            if (path == "\\\\")
            {
                int any = 0;

                timer.Start();
                foreach (DriveInfo drive in DriveInfo.GetDrives())
                {
                    IEnumerable<string> dirs = Directory.EnumerateDirectories(drive.RootDirectory.ToString(), $"*{name}*", new EnumerationOptions { IgnoreInaccessible = true, RecurseSubdirectories = true });
                    IEnumerable<string> files = Directory.EnumerateFiles(drive.RootDirectory.ToString(), $"*{name}*", new EnumerationOptions { IgnoreInaccessible = true, RecurseSubdirectories = true });
                    if (dirs.Any() || files.Any())
                    {
                        if (dirs.Any())
                        {
                            foreach (string dir in dirs)
                            {
                                DirectoryInfo info = new DirectoryInfo(dir);
                                string time = info.CreationTime.ToString("dd.MM.yyyy HH:mm:ss");
                                string size = Directory.EnumerateFiles(dir, $"*", new EnumerationOptions { IgnoreInaccessible = true, RecurseSubdirectories = true }).Sum(file => new FileInfo(file).Length).ToString("N0");

                                ct.ThrowIfCancellationRequested();
                                dt.Rows.Add(new object[] { "<DIR>", time, $"{size} bytes", info.FullName });
                            }
                        }
                        if (files.Any())
                        {
                            foreach (string file in files)
                            {
                                FileInfo info = new FileInfo(file);
                                string time = info.CreationTime.ToString("dd.MM.yyyy HH:mm:ss");
                                string size = info.Length.ToString("N0");

                                ct.ThrowIfCancellationRequested();
                                dt.Rows.Add(new object[] { "<FILE>", time, $"{size} bytes", info.FullName });
                            }
                        }
                        any++;
                    }
                    else
                    {
                        if (any == 0)
                        {
                            MessageBox.Show($"No matches found", "Nothing found", MessageBoxButtons.OK, MessageBoxIcon.Information);
                            break;
                        }
                    }
                }
                timer.Stop();
            }
            else
            {
                timer.Start();
                IEnumerable<string> dirs = Directory.EnumerateDirectories(path, $"*{name}*", new EnumerationOptions { IgnoreInaccessible = true, RecurseSubdirectories = true });
                IEnumerable<string> files = Directory.EnumerateFiles(path, $"*{name}*", new EnumerationOptions { IgnoreInaccessible = true, RecurseSubdirectories = true });
                if (dirs.Any() || files.Any())
                {
                    if (dirs.Any())
                    {
                        foreach (string dir in dirs)
                        {
                            DirectoryInfo info = new DirectoryInfo(dir);
                            string time = info.CreationTime.ToString("dd.MM.yyyy HH:mm:ss");
                            string size = Directory.EnumerateFiles(dir, $"*", new EnumerationOptions { IgnoreInaccessible = true, RecurseSubdirectories = true }).Sum(file => new FileInfo(file).Length).ToString("N0");

                            ct.ThrowIfCancellationRequested();
                            dt.Rows.Add(new object[] { "<DIR>", time, $"{size} bytes", info.FullName });
                        }
                    }
                    if (files.Any())
                    {
                        foreach (string file in files)
                        {
                            FileInfo info = new FileInfo(file);
                            string time = info.CreationTime.ToString("dd.MM.yyyy HH:mm:ss");
                            string size = info.Length.ToString("N0");

                            ct.ThrowIfCancellationRequested();
                            dt.Rows.Add(new object[] { "<FILE>", time, $"{size} bytes", info.FullName });
                        }
                    }
                }
                else
                {
                    MessageBox.Show("No matches found", "Nothing found", MessageBoxButtons.OK, MessageBoxIcon.Information);
                }
                timer.Stop();
            }
        }
        catch (OperationCanceledException) { }
        catch (Exception ex)
        {
            MessageBox.Show(ex.Message, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
        }
    }, ct);
}

Answer 1

Некоторые участки кода у вас дублируются - их можно вынести в отдельные методы.
Также можно вынести в отдельные методы логически связанные части кода.

Внутри таски оставляем только код перебора директорий и файлов.
Саму задачу оборачиваем в try-catch.

Также выносим лишний код из конструктора формы в событие Load.

Предлагаю посмотреть на такой вариант:

private void Form1_Load(object sender, EventArgs e)
{
    dt.Columns.Add("Type");
    dt.Columns.Add("Date");
    dt.Columns.Add("Size");
    dt.Columns.Add("Location");

    found_datagridview.DataSource = dt;

    SetColumnsSize();
}

private void SetColumnsSize()
{
    found_datagridview.Columns[0].Width = 60;
    found_datagridview.Columns[1].Width = 141;
    found_datagridview.Columns[2].Width = 130;
    found_datagridview.Columns[3].Width = 418;

    // вместо ручного задания размеров:
    //found_datagridview.AutoResizeColumns(DataGridViewAutoSizeColumnsMode.AllCells);
}

async Task FindFiles(CancellationToken ct = default)
{
    found_datagridview.DataSource = null;

    try
    {
        await Task.Run(() =>
        {
            string path = path_textbox.Text;
            string name = name_textbox.Text;
            dt.Rows.Clear();

            if (path == "\\\\")
            {
                foreach (DriveInfo drive in DriveInfo.GetDrives())
                {
                    FindFiles(drive.RootDirectory.Name, name, ct);
                }
            }
            else
            {
                FindFiles(path, name, ct);
            }
        }, ct);
    }
    catch (OperationCanceledException) { }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
    }

    if (dt.Rows.Count == 0)
    {
        MessageBox.Show("No matches found", "Nothing found", MessageBoxButtons.OK, MessageBoxIcon.Information);
    }

    found_datagridview.DataSource = dt;
    SetColumnsSize();
}

private void FindFiles(string path, string name, CancellationToken ct = default)
{
    var options = new EnumerationOptions { IgnoreInaccessible = true, RecurseSubdirectories = true };
    var dirs = Directory.EnumerateDirectories(path, $"*{name}*", options);
    var files = Directory.EnumerateFiles(path, $"*{name}*", options);

    foreach (string dir in dirs)
    {
        ct.ThrowIfCancellationRequested();

        DirectoryInfo info = new DirectoryInfo(dir);
        string time = info.CreationTime.ToString("dd.MM.yyyy HH:mm:ss");
        string size = Directory.EnumerateFiles(dir, $"*", options).Sum(file => new FileInfo(file).Length).ToString("N0");
        dt.Rows.Add(new object[] { "<DIR>", time, $"{size} bytes", info.FullName });
    }

    foreach (string file in files)
    {
        ct.ThrowIfCancellationRequested();

        FileInfo info = new FileInfo(file);
        string time = info.CreationTime.ToString("dd.MM.yyyy HH:mm:ss");
        string size = info.Length.ToString("N0");
        dt.Rows.Add(new object[] { "<FILE>", time, $"{size} bytes", info.FullName });
    }
}

Проверки dirs.Any(), files.Any() не нужны, т. к. цикл foreach не выполнит ни одной итерации в случае пустых коллекций.
От переменной any тоже можно избавиться, проверяя свойство dt.Rows.Count.

Я явно указываю дефолтный параметр CancellationToken ct = default. Таково требование Framework Design Guidelines.

Вы стартуете и останавливаете какой-то таймер. Я не знаю, что он делает, поэтому просто убрал эти строки. Верните их на место, если необходимо.

БЛОГ НА HUSL

Приложение фризит после подключения DataTable к DataGridView

Ответы (1 шт):