Перечисление обьектов image.упрощение кода

@KoFeMaH · Регистрация: 21.02.2009

В общем такая ситуация есть событие при нажатии кнопки где расставляются 30 картинок по позициям.попытался упростить код и сделать чтоб вызывалась процедура в которой можно просто задать допустим с 1 по 5 нужно по такому х и у.но при преобразовании в процедуру выдает такую ошибку:
[BCC32 Error] Unit1.cpp(23): E2297 'this' can only be used within a member function
на эту строчку

C++

1	TImage Image = dynamic_cast <TImage > (this->FindComponent(name));

далее приложение того как было и как надо сделать:
было:

C++

void __fastcall TForm1::N1Click(TObject *Sender)
{
AnsiString name;
int s,e,x=25,y=344;
y=264;
    for (int j=6; j <= 10; j++) {
        name="Image"+AnsiString(j);
        TImage *Image = dynamic_cast <TImage *> (this->FindComponent(name));
        if (Image) {
            Image->Left=x;
            Image->Top=y;
            y=y+20;
        } else ShowMessage("tu 4e tvorish"); }
         y=264;
    for (int j=11; j <= 15; j++) {
        name="Image"+AnsiString(j);
        TImage *Image = dynamic_cast <TImage *> (this->FindComponent(name));
        if (Image) {
            Image->Left=x;
            Image->Top=y;
            y=y+20;
        } else ShowMessage("tu 4e tvorish");
    }
    y=264;x=32;
    for (int z = 1; z <= 5; z++) {
        name="Image"+AnsiString(z);
        TImage *Image = dynamic_cast <TImage *> (this->FindComponent(name));
        if (Image) {
            Image->Left=x;
            Image->Top=y;
            y=y+20;
        } else ShowMessage("tu 4e tvorish");
    }
x=490;y=20;
    for (int z = 16; z <= 20; z++) {
        name="Image"+AnsiString(z);
        TImage *Image = dynamic_cast <TImage *> (this->FindComponent(name));
        if (Image) {
            Image->Left=x;
            Image->Top=y;
            y=y+20;
        } else ShowMessage("tu 4e tvorish");
    }
x=497;y=20;
    for (int z = 21; z <= 25; z++) {
        name="Image"+AnsiString(z);
        TImage *Image = dynamic_cast <TImage *> (this->FindComponent(name));
        if (Image) {
            Image->Left=x;
            Image->Top=y;
            y=y+20;
        } else ShowMessage("tu 4e tvorish");
    }
x=497;y=20; s=26; e=30;
    for (int z = s; z <= e; z++) {
        name="Image"+AnsiString(z);
        TImage *Image = dynamic_cast <TImage *> (this->FindComponent(name));
        if (Image) {
            Image->Left=x;
            Image->Top=y;
            y=y+20;
        } else ShowMessage("tu 4e tvorish");
    }
}

вот как хочу и где выдает ошибку:

C++

void posimg (int x,int y,int s,int e){
AnsiString name;
int k=y;
for (int j=s; j <= e; j++) {
        name="Image"+AnsiString(j);
        TImage *Image = dynamic_cast <TImage *> (this->FindComponent(name));
        if (Image) {
            Image->Left=x;
            Image->Top=k;
            k=k+20;
        } else ShowMessage("tu 4e tvorish"); }
}

Добавлено через 3 минуты
з.ы. еще как сделать спойлер в теме?))а то не смог сообразить как

@MikeSoft · 17.12.2010, 21:18

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

[BCC32 Error] Unit1.cpp(23): E2297 'this' can only be used within a member function

Выхода минимум два:
1. Явно указать форму:

C++

1
2
3

int i = 1;
String Name="Image" + IntToStr(i);
TImage *Image = dynamic_cast <TImage *> (Form1->FindComponent(Name));

2. Объявить функцию в public-области класса.
В заголовочном файле (*.h) объявите прототип в public-области:

C++

1	void PosImg (int x, int y, int s, int e);

А в файле кода наполните функцию содержанием:

C++

void TForm1::PosImg (int x, int y, int s, int e)
{
  // ваши действия
}

Спойлер делается с помощью такого кода:
[SPOILER="Спойлер"]Текст, который нужно скрыть[/SPOILER]

@KoFeMaH · 18.12.2010, 03:44 **[ТС]**

так как прошлую мою тему назвали дубликатом этой пишу тут.как можно сделать общее событие онмаусап для нескольких image причем в событии будет менятся координата image т.е. мы будем брать координату на которой ее отпустили и сравнивать с диапозоном координат.если она туда попала то поставить ее на заранее заданную позицию.
П.с. с уважением к участникам форума=)

@MikeSoft · 18.12.2010, 13:22

KoFeMaH, так как вы не захотели прочитать решения, которые предлагались вам в предыдущих темах, продублирую часть личного сообщения вам в эту тему:

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

В ней тоже обсуждается вопрос "как повесить одну функцию-обработчик на несколько одинаковых объектов".
То, что вы здесь чуть-чуть изменили задание - не оправдание. Суть осталась той же.
Мы уже неоднократно предлагали вам решение: выделите идентичные объекты, перейдите к событиям и напишите один единственный обработчик, в котором вместо явного указания объекта используйте приведение к типу параметр Sender.

@KoFeMaH · 18.12.2010, 13:47 **[ТС]**

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

@KoFeMaH 38 / 24 / 4 Регистрация: 21.02.2009 Сообщений: 249
	18.12.2010, 03:44 [ТС]	3
	так как прошлую мою тему назвали дубликатом этой пишу тут.как можно сделать общее событие онмаусап для нескольких image причем в событии будет менятся координата image т.е. мы будем брать координату на которой ее отпустили и сравнивать с диапозоном координат.если она туда попала то поставить ее на заранее заданную позицию. П.с. с уважением к участникам форума=) 0

@MikeSoft 3950 / 1805 / 184 Регистрация: 21.11.2009 Сообщений: 2,540
	18.12.2010, 13:22	4
	KoFeMaH, так как вы не захотели прочитать решения, которые предлагались вам в предыдущих темах, продублирую часть личного сообщения вам в эту тему: Сообщение от MikeSoft В ней тоже обсуждается вопрос "как повесить одну функцию-обработчик на несколько одинаковых объектов". То, что вы здесь чуть-чуть изменили задание - не оправдание. Суть осталась той же. Мы уже неоднократно предлагали вам решение: выделите идентичные объекты, перейдите к событиям и напишите один единственный обработчик, в котором вместо явного указания объекта используйте приведение к типу параметр Sender. 0

@KoFeMaH 38 / 24 / 4 Регистрация: 21.02.2009 Сообщений: 249
	18.12.2010, 13:47 [ТС]	5
	оффтоп ето еще до личного было) по времени гляньте я сообразил уже как.почистите тут сообщения если можно.ну чтоб лишнего не было.т.е. только то что кому то может пригодится))) 0

Опции темы