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

@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-а, а не прибивать гвоздями что-то определенное. Мало ли, во втором гриде захочется точно так же раскрашивать ячейки...

Новые блоги и статьи Все статьи Все блоги /
Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. 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/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .

@volvo Супер-модератор 33197 / 21493 / 8233 Регистрация: 22.10.2011 Сообщений: 36,886 Записей в блоге: 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