DatagridView тормозит при загрузке данных

@Vla00 · Регистрация: 06.10.2012

Author24 — интернет-сервис помощи студентам

C#

var dataAdapter = new SqlDataAdapter(command, _connection);
            var dataSet = new DataSet();
            dataAdapter.Fill(dataSet);
            _bindingSource = new BindingSource
            {
                DataSource = dataSet.Tables[0]
            };

Таким методом получаю данные из БД.
Запрос получает данные быстро. а вот забрасывает данные в таблицу долго:

C#

1	dataGridView1.DataSource = _bindingSource;

300 записей выводит примерно 10 секунд. Как ускорить?

@kolorotur · 27.12.2016, 18:07

Сообщение от Vla00

Как ускорить?

Используйте DGV в виртуальном режиме.
Заполнять ячейки придется вручную по запросу, но работать будет во много раз быстрее, т.к. отрисовываться будут только видимые записи.

@Vla00 · 28.12.2016, 14:06 **[ТС]**

Сообщение от kolorotur

Используйте DGV в виртуальном режиме.
Заполнять ячейки придется вручную по запросу, но работать будет во много раз быстрее, т.к. отрисовываться будут только видимые записи.

Немогу понять как он реализовывается. Мне кажется что он не подойдет. При каждом запросе приходят новые данные. Запрос динамический. т.е. каждый раз разное количество столбцов и разные данные. Насколько я понял он один раз загрузит и потом при определенных параметрах он будет отбирать и отображать которые подходят под условие?

@MVK · 28.12.2016, 14:33

Заполнять ячейки вручную не стоит. Почему DataSource DGV устанавливается в другом месте? Покажите весь код связывания.

@Vla00 · 28.12.2016, 15:02 **[ТС]**

Метод вызывается при нажатии на кнопку "Отобразить"

C#

private void UpdateDateGrid(string request)
        {
            Cursor = Cursors.AppStarting;
            _connection.Open();
            if (string.IsNullOrEmpty(request))
                request = _requestGroup;
            string command;
            string commandReq = request;
 
            if (!_flagDetail)
            {
                commandReq = commandReq.Replace(@", category.category as [категория]", @"/*, category.category as [категория]*/");
                if (radioButton1.Checked)
                    command = commandReq + Where + @" and protivopojar.ApiDate is not null" + Group +
                              @" order by alone.fio";
                else
                {
                    if (radioButton2.Checked)
                        command = commandReq + Where + @" and protivopojar.SzuDate is not null" + Group +
                                  @"order by alone.fio";
                    else
                        command = request + Where + Group + @" order by alone.fio";
                }
            }
            else
            {
                commandReq = commandReq.Replace("max(", "");
                commandReq = commandReq.Replace(")", "");
                commandReq = commandReq.Replace(@"/*, category.category as [категория]*/", ", category.category as [категория]");
                if (radioButton1.Checked)
                    command = commandReq + Where + @" and protivopojar.ApiDate is not null order by alone.fio";
                else
                {
                    if (radioButton2.Checked)
                        command = commandReq + Where + @" and protivopojar.SzuDate is not null order by alone.fio";
                    else
                        command = commandReq + Where + @" order by alone.fio";
                }
            }
 
            var dataAdapter = new SqlDataAdapter(command, _connection);
            var dataSet = new DataSet();
            dataAdapter.Fill(dataSet);
            _bindingSource = new BindingSource
            {
                DataSource = dataSet.Tables[0]
            };
 
            try
            {
                if (string.IsNullOrEmpty(comboBox2.Text))
                {
                    if (!string.IsNullOrEmpty(comboBox1.Text))
                    {
                        _bindingSource = new BindingSource
                        {
                            DataSource =
                                dataSet.Tables[0].Select("[selsovet] = '" + comboBox1.Text + "'").CopyToDataTable()
                        };
                    }
                }
                else
                {
                    _bindingSource = new BindingSource
                    {
                        DataSource = dataSet.Tables[0].Select("[country] = '" + comboBox2.Text + "'").CopyToDataTable()
                    };
                }
 
 
                dataGridView1.DataSource = _bindingSource;
                if (dataGridView1.Columns.Count > 0)
                {
                    dataGridView1.Columns[0].Visible = false;
                    dataGridView1.Columns[1].Visible = false;
                    dataGridView1.Columns[2].Visible = false;
                }
            }
            catch (Exception)
            {
                dataGridView1.DataSource = null;
            }
            
                
            _connection.Close();
            toolStripStatusLabel1.Text = @"Записей: " + dataGridView1.Rows.Count;
            Cursor = Cursors.Arrow;
        }

@MVK · 28.12.2016, 15:41

Дело в строках:

C#

1	DataSource = dataSet.Tables[0].Select("[country] = '" + comboBox2.Text + "'").CopyToDataTable()

Вы фильтруете и создаете новую таблицу. И зачем??
Вы хотите вставить фильтр, так вставляйте его либо в BindingSource или в DataTable.
У Вас связка: DataTable - BindingSource - DataGridView. DataGridView - только для отображения нужного. DataTable - для хранения всех данных. BindingSource - посредник между DataTable и DataGridView - в нем (или DataTable) фильтруем то, что будет отображаться.

@Vla00 · 28.12.2016, 16:10 **[ТС]**

всё быстро отрабатывает. В дебаге проверил что тормозит из-за dataGridView1.DataSource = _bindingSource;
Как я понимаю в этом дело, или я ошибаюсь?

Добавлено через 9 минут
даже когда я убрал фильтрацию - время выполнения не изменилось, т.е. не в этом дело. и если вы посмотрите внимательно код, то видно что сдесь 3 случая: когда не требуется фильтровать, когда фильтруется по selsovet и по country.
Заметил что при первом запросе, время выполнения долгое, но при следующих, время выполнения заметно сокращается.

@aquaMakc · 28.12.2016, 16:38

Vla00, а на DGV случайно не навешено обработчиков событий изменений? Просто я ни разу не встречал тормозов при биндинге данных в таблицу.

@MVK · 28.12.2016, 16:46

Верно, тормозит из-за dataGridView1.DataSource = _bindingSource;
Итак, все нужные Вам данные лежать в dataSet.Tables[0] и поэтому запишем для связывания:

C#

1
2
3

_bindingSource = new BindingSource();
_bindingSource.DataSource = dataSet.Tables[0];
dataGridView1.DataSource = _bindingSource;

У Вас в зависимости от условий разные фильтры.Приведу один пример - строки 54-60 заменить на:

C#

1	BindingSource.Filter = string.Format("selsovet = '{0}'", comboBox1.Text);

Пробуйте.

@Vla00 · 29.12.2016, 08:22 **[ТС]**

Сообщение от aquaMakc

а на DGV случайно не навешено обработчиков событий изменений?

На нем повешен только один обработчик, при нажатии на строку открывается форма для редактирования данных. Но это не относится к этому.

Добавлено через 14 минут

Сообщение от MVK

поэтому запишем для связывания

Написал вот так, на полторы секунды работает быстрее, но всё равно для 300 записей, 8-8,5 сек. долго я считаю.

C#

_bindingSource = new BindingSource();
            _bindingSource.DataSource = dataSet.Tables[0];
            dataGridView1.DataSource = _bindingSource;
 
            try
            {
                if (string.IsNullOrEmpty(comboBox2.Text))
                {
                    if (!string.IsNullOrEmpty(comboBox1.Text))
                    {
                        _bindingSource.Filter = string.Format("selsovet = '{0}'", comboBox1.Text);
                    }
                }
                else
                {
                    _bindingSource.Filter = string.Format("country = '{0}'", comboBox2.Text);
                }
 
                if (dataGridView1.Columns.Count > 0)
                {
                    dataGridView1.Columns[0].Visible = false;
                    dataGridView1.Columns[1].Visible = false;
                    dataGridView1.Columns[2].Visible = false;
                }
            }

@aquaMakc · 29.12.2016, 08:47

Vla00, ты с таблицей что-то делаешь из DGV?
Можно попробовать биндить не Table[0], а коллекцию классов-обёртёк. Например, для таблицы с полями id, Name, Phone создаётся класс, содержащий свойства string Id, Name, Phone. Коллекция заполняется данными из таблицы и биндится к контролу.

@MVK · 29.12.2016, 09:20

Покажите, 1) какое значение request Вы передаете в метод UpdateDateGrid(string request),
2) какое значение command Вы передаете в SqlDataAdapter(command, _connection);

@Vla00 · 03.01.2017, 09:21 **[ТС]**

Сообщение от MVK

какое значение request Вы передаете в метод UpdateDateGrid(string request)

SQL

SELECT MAX(alone.key_alone), MAX(selsovet.selsovet) AS selsovet, MAX(country.country) AS country, 
    MAX(alone.fio) AS [ФИО], MAX(alone.date_ro) AS [дата рождения], 
    MAX(country.country) + ' ' + MAX(alone.street) AS [адрес],
    MAX(protivopojar.ApiDate) AS [АПИ], MAX(protivopojar.SzuDate) AS [СЗУ] /*, category.category as [категория]*/
        FROM alone INNER JOIN category ON alone.key_alone = category.fk_alone 
            INNER JOIN country ON alone.fk_country = country.key_country 
            INNER JOIN selsovet ON selsovet.key_selsovet = country.fk_selsovet 
            INNER JOIN protivopojar ON protivopojar.fk_alone = alone.key_alone JOIN category c1 ON alone.key_alone = c1.fk_alone AND c1.category = 'Одинокие пожилые граждане пенсионного возраста'

Сообщение от MVK

какое значение command Вы передаете в SqlDataAdapter(command, _connection);

Такое же только частичка дополняется.

Добавлено через 1 час 6 минут

Сообщение от aquaMakc

Коллекция заполняется данными из таблицы и биндится к контролу.

Хотел сделать так как вы сказали, но не получилось. Не могли бы вы скинуть пример или примерно набросать код? Буду очень благодарен...

@Igr_ok · 03.01.2017, 13:18

Сообщение от Vla00

order by alone.fio

Сортировка по неиндексированному полю может существенно увеличить время выполнения запроса.

@MVK · 03.01.2017, 17:08

Посоветовать могу только одно: оптимизируйте запрос к БД (или ее таблицы). Для проверки сделайте простой запрос с выборкой и закачайте данные в DataSet с воспроизведением в DataGridView. Все будет быстро.

@Vla00 · 04.01.2017, 15:20 **[ТС]**

Сообщение от Igr_ok

Сортировка по неиндексированному полю может существенно увеличить время выполнения запроса.

Пробовал создавать индекс и убирал сортировку, время выполнения одинаковое.

Добавлено через 1 минуту

Сообщение от MVK

Посоветовать могу только одно: оптимизируйте запрос к БД (или ее таблицы). Для проверки сделайте простой запрос с выборкой и закачайте данные в DataSet с воспроизведением в DataGridView. Все будет быстро.

Дело в том что если первый раз обращаться по данному запросу, то запрос выполняется долго. При дальнейших обращениях - быстро. Если даже закрыть форму и открыть заново и выполнить запрос, всё же долго....

Добавлено через 1 минуту

Сообщение от Igr_ok

Сортировка по неиндексированному полю может существенно увеличить время выполнения запроса.

Сообщение от MVK

Посоветовать могу только одно: оптимизируйте запрос к БД (или ее таблицы). Для проверки сделайте простой запрос с выборкой и закачайте данные в DataSet с воспроизведением в DataGridView. Все будет быстро.

Как я писал выше время выполнения занимает долго из-за занесения данных в DataSource.

Добавлено через 4 часа 4 минуты
Разобрался в чем дело.... Оказалось что нужно было просто убрать:

C#

1
2
3

dataGridView1.DefaultCellStyle.WrapMode = DataGridViewTriState.True;
            dataGridView1.AutoSizeColumnsMode = DataGridViewAutoSizeColumnsMode.Fill;
            dataGridView1.AutoSizeRowsMode = DataGridViewAutoSizeRowsMode.AllCells;

И авторазмер поставил на необходимые столбцы. Всем спасибо, кто помогал.

Добавлено через 19 секунд
Разобрался в чем дело.... Оказалось что нужно было просто убрать:

C#

1
2
3

dataGridView1.DefaultCellStyle.WrapMode = DataGridViewTriState.True;
            dataGridView1.AutoSizeColumnsMode = DataGridViewAutoSizeColumnsMode.Fill;
            dataGridView1.AutoSizeRowsMode = DataGridViewAutoSizeRowsMode.AllCells;

И авторазмер поставил на необходимые столбцы. Всем спасибо, кто помогал.

@aquaMakc 484 / 397 / 68 Регистрация: 14.02.2014 Сообщений: 1,930
	29.12.2016, 08:47	11
	Vla00, ты с таблицей что-то делаешь из DGV? Можно попробовать биндить не Table[0], а коллекцию классов-обёртёк. Например, для таблицы с полями id, Name, Phone создаётся класс, содержащий свойства string Id, Name, Phone. Коллекция заполняется данными из таблицы и биндится к контролу. 1

@kolorotur 17685 / 12871 / 3365 Регистрация: 17.09.2011 Сообщений: 21,136
	27.12.2016, 18:07	2
	Сообщение от Vla00 Как ускорить? Используйте DGV в виртуальном режиме. Заполнять ячейки придется вручную по запросу, но работать будет во много раз быстрее, т.к. отрисовываться будут только видимые записи. 0

@Vla00 11 / 11 / 10 Регистрация: 06.10.2012 Сообщений: 667
	28.12.2016, 14:06 [ТС]	3
	Сообщение от kolorotur Используйте DGV в виртуальном режиме. Заполнять ячейки придется вручную по запросу, но работать будет во много раз быстрее, т.к. отрисовываться будут только видимые записи. Немогу понять как он реализовывается. Мне кажется что он не подойдет. При каждом запросе приходят новые данные. Запрос динамический. т.е. каждый раз разное количество столбцов и разные данные. Насколько я понял он один раз загрузит и потом при определенных параметрах он будет отбирать и отображать которые подходят под условие? 0

@MVK 160 / 148 / 92 Регистрация: 30.03.2011 Сообщений: 569
	28.12.2016, 14:33	4
	Заполнять ячейки вручную не стоит. Почему DataSource DGV устанавливается в другом месте? Покажите весь код связывания. 0

@Vla00 11 / 11 / 10 Регистрация: 06.10.2012 Сообщений: 667
	28.12.2016, 16:10 [ТС]	7
	всё быстро отрабатывает. В дебаге проверил что тормозит из-за dataGridView1.DataSource = _bindingSource; Как я понимаю в этом дело, или я ошибаюсь? Добавлено через 9 минут даже когда я убрал фильтрацию - время выполнения не изменилось, т.е. не в этом дело. и если вы посмотрите внимательно код, то видно что сдесь 3 случая: когда не требуется фильтровать, когда фильтруется по selsovet и по country. Заметил что при первом запросе, время выполнения долгое, но при следующих, время выполнения заметно сокращается. 0

@aquaMakc 484 / 397 / 68 Регистрация: 14.02.2014 Сообщений: 1,930
	28.12.2016, 16:38	8
	Vla00, а на DGV случайно не навешено обработчиков событий изменений? Просто я ни разу не встречал тормозов при биндинге данных в таблицу. 1

@MVK 160 / 148 / 92 Регистрация: 30.03.2011 Сообщений: 569
	29.12.2016, 09:20	12
	Покажите, 1) какое значение request Вы передаете в метод UpdateDateGrid(string request), 2) какое значение command Вы передаете в SqlDataAdapter(command, _connection); 0

@Igr_ok 784 / 615 / 273 Регистрация: 04.08.2015 Сообщений: 1,707
	03.01.2017, 13:18	14
	Сообщение от Vla00 order by alone.fio Сортировка по неиндексированному полю может существенно увеличить время выполнения запроса. 0

@MVK 160 / 148 / 92 Регистрация: 30.03.2011 Сообщений: 569
	03.01.2017, 17:08	15
	Посоветовать могу только одно: оптимизируйте запрос к БД (или ее таблицы). Для проверки сделайте простой запрос с выборкой и закачайте данные в DataSet с воспроизведением в DataGridView. Все будет быстро. 0