TImage в сочетании с new и std::pair

[Гром]

Доброго времени суток!
Пишу сейчас игру (условно говоря - РПГ), решил сделать в ней буфер картинок для централизованного хранения и рисования объектов карты. Картинки храню в ассоциативном массиве, проиндексированном по AnsiString (ID картинки). Сами картинки хранятся в виде битмапов в папке с экзешником. При загрузке картинок возникают проблемы с TImage.

ImageBuffer::ImageBuffer()
   {
   TStringList* ImageFiles = new TStringList;
   ImageFiles -> LoadFromFile("images.txt");   //Информация о расположении картинок в виде ID=Image_path
   AnsiString s;
   int id, pos;
   AnsiString filename;
   TImage* LoadingImage = new TImage;
   for (int i = 0; i < ImageFiles -> Count; i++)
      {
      s = ImageFiles -> Strings[i];
      pos = s.Pos("=");
      id = s.SubString(0, pos - 1).ToInt();   //Выделяем ID
      filename = s.SubString(pos + 1, s.Length() - pos);   //Выделяем путь
      LoadingImage -> Picture -> LoadFromFile(filename);
      std::pair<int, TImage> IndexedImage(id, LoadingImage);
      ImageList.insert(IndexedImage);   //Добавляем в map новую картинку
      }
   delete LoadingImage;
   delete ImageFiles;
   }

Собственно проблемы две. Первая - в строке

TImage* LoadingImage = new TImage;

TImage не имеет конструктора по умолчанию, необходимо в качестве аргумента передавать указатель на TComponent. Сакральный смысл этого TComponent я не очень понимаю - для чего его надо указывать? Эксперименты показали, что если вместо такой строки написать

TImage* LoadingImage = new TImage(Form1);

то ошибка пропадает. Однако, хотелось бы узнать, что стоит за этим?)
Другая проблема - в строке

std::pair<int, TImage> IndexedImage(id, LoadingImage);

Выдает ошибку VCL style classes must be constructed using operator new.

[Руслантус]

Привет Гром!

В данном случаи:

std::pair<int, TImage> IndexedImage(id, LoadingImage);

Нужно использовать указатель на объект, а ты просто используеш объект.
Тоесть нужно так:

std::pair<int, TImage*> IndexedImage(id, LoadingImage);

По поводу твоего 1 вопроса, то там устанавливаецо некий владелец данного компонента, чтобы потом из владельца влиять на данный компонент. Не путай с родителем, его нужно будет самому устанавливать в LoadingImage->Parent.

[Гром]

Руслантус, спасибо большое!)
Вроде бы разобрался!)
Только еще вопрос - нужно ли устанавливать Parent для картинки, которая сама по себе не будет видна, и единственное назначение которой - копироваться через CopyRect в PaintBox? (других действий с ней производить пока не планирую)

Все-таки никак не могу подружить pair и TImage. Билдер выкидывает ошибку
VCL style classes must be constructed using operator new
причем посылает меня в _pair.h, а конкретно в строку

pair(const _T1& __a, const _T2& __b) : first(__a), second(__b) {}

Не знаю уж, как организовать пару с имэйджем...

[Руслантус]

Гром
Тогда ставить Parent думаю ненужно.
Если тебе просто нужно хранить изображение, то используй спец. классы.
Лучший вариант это:

Graphics::TBitmap *NewImage = new Graphics::TBitmap()

У него также есть много полезных методов.

Насчёт ошибки с pair, то покажи код.

[Гром]

Насколько я понимаю, проблема в том, что в pair для обоих элементов вызывается конструктор, а дельфовские объекты на этапе исполнения должны быть созданы при помощи new, а pair об этом ни сном ни духом, в результате чего и возникает взаимонепонимание...

В варианте с указателями я просто переписал код, добавив где надо звездочек:

ImageBuffer::ImageBuffer()
   {
   TStringList* ImageFiles = new TStringList;
   ImageFiles -> LoadFromFile("images.txt");
   AnsiString s;
   int pos;
   AnsiString id, filename;
   TImage* LoadingImage = new TImage(Form1);
   for (int i = 0; i < ImageFiles -> Count; i++)
      {
      s = ImageFiles -> Strings[i];
      pos = s.Pos("=");
      id = s.SubString(0, pos - 1);
      filename = s.SubString(pos + 1, s.Length() - pos);
      LoadingImage -> Picture -> LoadFromFile(filename);
      std::pair<AnsiString, TImage*>* IndexedImage = new std::pair<AnsiString, TImage*>(id, LoadingImage);
      ImageList.insert(*IndexedImage);
      }
   delete LoadingImage;
   delete ImageFiles;
   }

Соответственно, создаю пару с адресом нужного имэйджа, добавляю пару в map, удаляю имэйдж... Соответственно, получаем полную потенциально опасную ерунду... Может быть, просто не надо удалять указатель на Image?.. Что-то я и в указателях запутался...

[Гром]

Кажется, и впрямь... Перенес

  TImage* LoadingImage = new TImage(Form1);

в цикл for, закомментировал

delete LoadingImage;

и как будто бы заработало!!)))

[netrino]

Всё верно, delete ведь не указатель удаляет, а освобождает память, на которую он указывает, то есть Вы грузили картинку в память, а затем сразу же удаляли )

 std::pair<AnsiString, TImage*>* IndexedImage = new std::pair<AnsiString, TImage*>(id, LoadingImage);

Не забываем освобождать память после того, как положили пару в мэп, или, создавайте её в стеке, а не в куче. Элементы pair копируются в мэп, потому, после копирования, pair можно спокойно удалять.