Изменение форматирования DataGridView при выборе в ComboBox

@USTeam · Регистрация: 06.12.2015

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

Здравствуйте! Никак не могу понять в чем причина

Вот событие. Если перед запросом выбрать в комбобоксе "ячейка", он загрузит данные и покрасит только ячейку. Но если потом в комбобоксе выбрать "строка" то покрасит всю строку. Хитрость в том что если мы снова захотим покрасить только ячейку - ничего не срабатывает, приходится перезапрос делать.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
void DataGridView1CellFormatting(object sender, DataGridViewCellFormattingEventArgs e)
        {
            try
            {
                 for (int i =0; i < dataGridView1.Rows.Count; i++)
                 {
                    DataGridViewRow row = dataGridView1.Rows[i];
                    if (comboBox3.SelectedItem.ToString() == "Строка")
                    {
                      switch (Convert.ToInt32(row.Cells["perceived_severity"].Value))
                      {
                        case 0:
                        dataGridView1.Rows[i].DefaultCellStyle.BackColor = ColorTranslator.FromHtml("#8ca3f6");
                        dataGridView1.Rows[i].DefaultCellStyle.ForeColor = Color.Black;
                        break;
                        case 1:
                        dataGridView1.Rows[i].DefaultCellStyle.BackColor = ColorTranslator.FromHtml("#F24949");
                        CRITICAL_SMS++;
                        break;
                        case 2:
                        dataGridView1.Rows[i].DefaultCellStyle.BackColor = ColorTranslator.FromHtml("#F29544");
                        MAJOR_SMS++;
                        break;
                        case 3:
                        dataGridView1.Rows[i].DefaultCellStyle.BackColor = ColorTranslator.FromHtml("#F8F153");
                        MINOR_SMS++;
                        break;
                        case 4:
                        dataGridView1.Rows[i].DefaultCellStyle.BackColor = ColorTranslator.FromHtml("#5ad7f1");
                        WARNING_SMS++;
                        break;
                        case 5:
                        dataGridView1.Rows[i].DefaultCellStyle.BackColor = ColorTranslator.FromHtml("#8ca3f6");
                        dataGridView1.Rows[i].DefaultCellStyle.ForeColor = Color.Black;
                        INDETEMINATE_SMS++;
                        break;
                        case 6:
                        dataGridView1.Rows[i].DefaultCellStyle.BackColor = ColorTranslator.FromHtml("#61f55c");
                        dataGridView1.Rows[i].DefaultCellStyle.ForeColor = Color.Black;
                        CLEAR_SMS++;
                        break;
                        default: return;
                      }
                    } 
                    if (comboBox3.SelectedItem.ToString() == "Ячейка")
                    {
                    switch (Convert.ToInt32(row.Cells["perceived_severity"].Value))
                    {
                        case 0:
                        dataGridView1.Rows[i].Cells["perceived_severity"].Style.BackColor = ColorTranslator.FromHtml("#8ca3f6");
                        dataGridView1.Rows[i].DefaultCellStyle.ForeColor = Color.Black;
                        break;
                        case 1:
                            dataGridView1.Rows[i].Cells["perceived_severity"].Style.BackColor = ColorTranslator.FromHtml("#F24949");
                            break;
                        case 2:
                            dataGridView1.Rows[i].Cells["perceived_severity"].Style.BackColor = ColorTranslator.FromHtml("#F29544");
                            break;
                        case 3:
                            dataGridView1.Rows[i].Cells["perceived_severity"].Style.BackColor = ColorTranslator.FromHtml("#F8F153");
                            break;
                        case 4:
                            dataGridView1.Rows[i].Cells["perceived_severity"].Style.BackColor = ColorTranslator.FromHtml("#5ad7f1");
                            break;
                        case 5:
                            dataGridView1.Rows[i].Cells["perceived_severity"].Style.BackColor = ColorTranslator.FromHtml("#8ca3f6");
                            dataGridView1.Rows[i].DefaultCellStyle.ForeColor = Color.Black;
                            break;
                        case 6:
                            dataGridView1.Rows[i].Cells["perceived_severity"].Style.BackColor = ColorTranslator.FromHtml("#61f55c");
                            dataGridView1.Rows[i].DefaultCellStyle.ForeColor = Color.Black;
                            break;
                            default: return;
                       }
                     }
                 }
            }
            catch (InvalidDataException ex) {}
            catch (NullReferenceException ex) {}
            catch (ArgumentException ex) {}
        }

здесь все пусто, но почему то работает

C#
1
2
3
4
5
6
7
8
9
10
11
void ComboBox3SelectedIndexChanged(object sender, EventArgs e)
        {
            if (comboBox3.SelectedItem.ToString() == "Строка")
            {
                
            }
            if (comboBox3.SelectedItem.ToString() == "Ячейка")
            {
                
            }
        }

@OwenGlendower · 03.09.2016, 01:36

USTeam, наверное просто не срабатывает событие CellFormatting. Или происходит исключение которое ты не видишь из-за пустых catch.

Я здесь немного другое вижу - 1) неудачный выбор события для настройки вида всего грида 2) смешение действий - кроме изменения стиля идет и подсчет 3) неэффективный код.

Чуть подробнее о каждом пункте.

1) Событие CellFormatting предназначено для настройки формата одной конкретной ячейки. Поэтому непонятно почему ты выбрал именно его. Почему бы не перенести код в обработчик ComboBox3SelectedIndexChanged?

В документации на событие CellFormatting сказано:

The CellFormatting event occurs every time each cell is painted, so you should avoid lengthy processing when handling this event.

Перебор всех строк в гриде подпадает под определение длительной операции. Однозначно нужен другой подход.

2) Насчет смешения действий думаю пояснять не нужно. По идее подсчет можно сделать один раз после получения данных.

3) Первое что плохо это постоянные вызовы comboBox3.SelectedItem.ToString(). Для каждой строки это делается два раза. Если в comboBox хранятся именно строки, то накладные расходы небольшие. Но если там не строки, то на каждой итерации происходит два выделения памяти под временную строку. Плюс постоянное сравнение значения которое не меняется замедляет цикл. Это классическая оптимизации цикла. Вместо

C#
1
2
3
4
5
6
7
8
9
10
11
for (int i =0; i < dataGridView1.Rows.Count; i++)
{
    if (comboBox3.SelectedItem.ToString() == "Строка")
    {
        ...
    }
    else if (comboBox3.SelectedItem.ToString() == "Ячейка")
    {
        ...
    }
}

Лучше написать так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
if (comboBox3.SelectedItem.ToString() == "Строка")
{
    for (int i =0; i < dataGridView1.Rows.Count; i++)
    {
        ...
    }
}
else if (comboBox3.SelectedItem.ToString() == "Ячейка")
{
    for (int i =0; i < dataGridView1.Rows.Count; i++)
    {
        ...
    }
}

Еще один удар по производительности это постоянные вызовы ColorTranslator.FromHtml. Это можно сделать один раз записав их в массив и используя perceived_severity в качестве индекса.

Обращения вида dataGridView1.Rows[i] можно заменить на переменную row которая у тебя уже есть.

В итоге код можно переделать так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
void SomeMethod()
{
    Color[] colors = {
        ColorTranslator.FromHtml("#8ca3f6"), ColorTranslator.FromHtml("#F24949"),
        ColorTranslator.FromHtml("#F29544"), ColorTranslator.FromHtml("#F8F153"),
        ColorTranslator.FromHtml("#5ad7f1"), ColorTranslator.FromHtml("#8ca3f6"),
        ColorTranslator.FromHtml("#61f55c")
    };
    try
    {
        if (comboBox3.SelectedItem.ToString() == "Строка")
        {
            for (int i =0; i < dataGridView1.Rows.Count; i++)
            {
                DataGridViewRow row = dataGridView1.Rows[i];
 
                int perceived_severity = Convert.ToInt32(row.Cells["perceived_severity"].Value;
                if (perceived_severity >= colors.Length) return;
 
                row.DefaultCellStyle.BackColor = colors[perceived_severity];
 
                switch (perceived_severity)
                {
                    case 0:
                    row.DefaultCellStyle.ForeColor = Color.Black;
                    break;
                case 1:
                    CRITICAL_SMS++;
                    break;
                case 2:
                    MAJOR_SMS++;
                    break;
                case 3:
                    MINOR_SMS++;
                    break;
                case 4:
                    WARNING_SMS++;
                    break;
                case 5:
                    row.DefaultCellStyle.ForeColor = Color.Black;
                    INDETEMINATE_SMS++;
                    break;
                case 6:
                    row.DefaultCellStyle.ForeColor = Color.Black;
                    CLEAR_SMS++;
                    break;
                }
            } 
        }
        else if (comboBox3.SelectedItem.ToString() == "Ячейка")
        {
            for (int i =0; i < dataGridView1.Rows.Count; i++)
            {
                DataGridViewRow row = dataGridView1.Rows[i];
 
                int perceived_severity = Convert.ToInt32(row.Cells["perceived_severity"].Value;
                if (perceived_severity >= colors.Length) return;
 
                row.Cells["perceived_severity"].Style.BackColor = colors[perceived_severity];
                switch (perceived_severity)
                {
                    case 0:
                        row.DefaultCellStyle.ForeColor = Color.Black;
                        break;
                    //case 1:
                    //    break;
                    //case 2:
                    //    break;
                    //case 3:
                    //    break;
                    //case 4:
                    //    break;
                    case 5:
                        row.DefaultCellStyle.ForeColor = Color.Black;
                        break;
                    case 6:
                        row.DefaultCellStyle.ForeColor = Color.Black;
                        break;
                }
            }
 
        }
    }
    catch (InvalidDataException ex) {}
    catch (NullReferenceException ex) {}
    catch (ArgumentException ex) {}
}

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .