Сравнение ячеек в столбце если значение не совпадает поменять цвет

@vadca · Регистрация: 07.04.2016

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

Привет. Нужно сравнить 2 столбика из DataGridView и если значения в столбиках различается, в столбике котором меняются значения, покрасить другой цвет. Вот мой код, но работает не корректно:

C#

 for (int i = 0; i < advancedDataGridView2.RowCount; i++)
            for (int j = advancedDataGridView2.RowCount+1; j > i; j--)
                if (advancedDataGridView2[4, i].Value.ToString().Trim() == advancedDataGridView2[5, j].Value.ToString().Trim())
                    advancedDataGridView2[5, j].Style.BackColor = Color.LemonChiffon;
                else advancedDataGridView2[5, j].Style.BackColor = Color.Red;

В чём моя ошибка ?

Добавлено через 54 секунды

C#

1	for (int j = advancedDataGridView2.RowCount -1; j > i; j--)

указав -1 тоже не работает

@RuS_984 · 29.06.2017, 15:05

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

Нужно сравнить 2 столбика из DataGridView

advancedDataGridView2[4, i] - неверные индексы
сначала задается строка, а потом столбец

C#

1	advancedDataGridView2[i, 4]

А если сверяются данные одной строки, то достаточно одного цикла с переменной i

@vadca · 30.06.2017, 12:56 **[ТС]**

Решил проблему, вот код если кому нибудь пригодится:

C#

for (int i = 0; i < advancedDataGridView2.RowCount; i++)
            {
                advancedDataGridView2[5, i].Style.BackColor = Color.White;
                if (advancedDataGridView2[4, i].Value.ToString().Trim() == advancedDataGridView2[5, i].Value.ToString().Trim())
                    advancedDataGridView2[5, i].Style.BackColor = Color.LemonChiffon;
                else advancedDataGridView2[5, i].Style.BackColor = Color.Red;
            }

@hoolygan · 30.06.2017, 14:01

Знаете, я бы не делал так, слишком некрасиво. Я бы добавил столбик в таблицу, не выводя ее в грид, и при кадом изменении строки, во время валидации, сравнивал бы значения в таблице, а не ячейках. Далее, по результату вычмсления/сравнения, прописывал бы 0 или 1 в это поле. И в обработчике закрашки грида - смотрел бы на это скрытое поле. Без гемора, и с чистым кодом.
Сегодня сравниваете 1 значение, завтра другое, через месяц 5 значений, и через пол года забудете что нужно было делать, а Ваш приемник, что будет дописывать.... я ему сочуствую.

@vadca · 30.06.2017, 14:05 **[ТС]**

Тут система проще, заполняется недельный план отгрузки, сравнивается план и факт, и чтобы не сделать ошибку, нужно закрасить другим цветом. Ваш вариант тоже очень логичен, спасибо

@hoolygan 360 / 287 / 76 Регистрация: 21.06.2016 Сообщений: 1,115
	30.06.2017, 14:01	4
	Знаете, я бы не делал так, слишком некрасиво. Я бы добавил столбик в таблицу, не выводя ее в грид, и при кадом изменении строки, во время валидации, сравнивал бы значения в таблице, а не ячейках. Далее, по результату вычмсления/сравнения, прописывал бы 0 или 1 в это поле. И в обработчике закрашки грида - смотрел бы на это скрытое поле. Без гемора, и с чистым кодом. Сегодня сравниваете 1 значение, завтра другое, через месяц 5 значений, и через пол года забудете что нужно было делать, а Ваш приемник, что будет дописывать.... я ему сочуствую. 0

@vadca 0 / 0 / 0 Регистрация: 07.04.2016 Сообщений: 109
	30.06.2017, 14:05 [ТС]	5
	Тут система проще, заполняется недельный план отгрузки, сравнивается план и факт, и чтобы не сделать ошибку, нужно закрасить другим цветом. Ваш вариант тоже очень логичен, спасибо 0