Неверно работает закрашивание ячеек

@DeadHipo · Регистрация: 21.09.2013

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
int max2 = NewGird->RowCount;
for (int ii = 1; ii < 3; ii++)
{
for (int jj = 1; jj < max2; jj++)
    {
    if (NewGird->Cells[jj][ii]== "")
        {
        NewGird->Canvas->Brush->Color=clRed;
        NewGird->Canvas->FillRect(NewGird->CellRect(ii, jj));
        }
    }
}

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

@Нитонисе · 14.10.2013, 20:22

Вот здесь кажется должно быть наоборот. Сначала идет индекс колонки, затем строки.

C++
1
2
// if (NewGird->Cells[jj][ii]== "")
if (NewGird->Cells[ii][jj]== "")

@volvo · 14.10.2013, 21:20

А почему, собственно, закрашивание делается в цикле, а не в OnDrawCell грида, для конкретной ячейки по ее содержимому?

@DeadHipo · 14.10.2013, 23:58 **[ТС]**

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

А почему, собственно, закрашивание делается в цикле, а не в OnDrawCell грида, для конкретной ячейки по ее содержимому?

Спасибо. запихнул всё в OnDrawCell , устроит)
ещё раз спасибо

@gumi250 · 15.10.2013, 00:05

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

Спасибо. запихнул всё в OnDrawCell

Все не надо запихивать, циклы надеюсь не запихнули

@DeadHipo · 15.10.2013, 00:07 **[ТС]**

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

Все не надо запихивать, циклы надеюсь не запихнули

и их тоже

прошу прощения, но как лучше сделать?

@gumi250 · 15.10.2013, 00:11

Не запихивать циклы. В OnDrawCell красить не все, а только 1 ячейку индексы которой приходят в параметрах ф-ции OnDrawCell

@DeadHipo · 15.10.2013, 00:17 **[ТС]**

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

Не запихивать циклы. В OnDrawCell красить не все, а только 1 ячейку индексы которой приходят в параметрах ф-ции OnDrawCell

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void __fastcall TForm5::NewGirdDrawCell(TObject *Sender, int ACol, int ARow, TRect &Rect,
          TGridDrawState State)
{
//int max2 = NewGird->RowCount;
//for (int ii = 1; ii < 3; ii++)
    {
    //for (int jj = 1; jj < max2; jj++)
        {
        if (NewGird->Cells[ACol][ARow]== "")
            {
            NewGird->Canvas->Brush->Color=clRed;
            NewGird->Canvas->FillRect(NewGird->CellRect(ACol, ARow));
            }
        }
    }
}

Примерно так?

@gumi250 · 15.10.2013, 00:18

Да так

@DeadHipo · 15.10.2013, 00:20 **[ТС]**

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

Да так

благодарю.

@volvo · 15.10.2013, 00:42

C++
1
2
3
4
5
        if (NewGird->Cells[ACol][ARow]== "")
            {
            NewGird->Canvas->Brush->Color=clRed;
            NewGird->Canvas->FillRect(Rect); // Лишние вызовы-то зачем?
            }

А вообще, лучше бы получать указатель на грид из Sender-а, а не прибивать гвоздями что-то определенное. Мало ли, во втором гриде захочется точно так же раскрашивать ячейки...

Новые блоги и статьи Все статьи Все блоги /
Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .	Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .
Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .	Использование значений реквизитов справочника в документе, с определенными условиями и правами Maks 07.04.2026 1. Контроль срока действия договора Алгоритм из решения ниже реализован на примере нетипового документа "ЗаявкаНаРаботу", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если. . .

@volvo Супер-модератор 33407 / 21517 / 8236 Регистрация: 22.10.2011 Сообщений: 36,917 Записей в блоге: 12
	14.10.2013, 21:20
	А почему, собственно, закрашивание делается в цикле, а не в OnDrawCell грида, для конкретной ячейки по ее содержимому? 1

@gumi250 435 / 402 / 57 Регистрация: 06.02.2012 Сообщений: 1,384
	15.10.2013, 00:11
	Не запихивать циклы. В OnDrawCell красить не все, а только 1 ячейку индексы которой приходят в параметрах ф-ции OnDrawCell 1

@gumi250 435 / 402 / 57 Регистрация: 06.02.2012 Сообщений: 1,384
	15.10.2013, 00:18
	Да так 1