new->auto_ptr

[SilverFish]

В коде, представленном на рисунке, память выделяется оператором new. Затем идет присвоение конструкции auto_ptr в другом блоке кода. При завершении блока, auto_ptr корректно удалить блок выделенный переменной _return?
Спасибо.

[S0mbre]

Привет. Не понял вопроса. Эти два блока (функции) - насколько можно судить - совершенно не связаны между собой. В первом память выделяется и освобождается вручную, а во втором - автоматически. Переменная _return относится только к 1-му блоку... Пример оформления первого блока в 2 вариантах (ручное и автоматическое управление памятью):

Код

// ВАРИАНТ 1 (РУЧНОЕ УПРАВЛЕНИЕ ПАМЯТЬЮ)
TMemoryStream* _return = new TMemoryStream();
try {
  Image1->Bitmap->SaveToStream(_return);
}
__finally {
  delete _return;
}
 
// ВАРИАНТ 2 (АВТОМАТИЧЕСКОЕ УПРАВЛЕНИЕ ПАМЯТЬЮ) - ПРЕДПОЧТИТЕЛЬНЫЙ ВАРИАНТ
std::auto_ptrTMemoryStream> _return1 (new TMemoryStream());
Image1->Bitmap->SaveToStream(_return1.get());
 

[SilverFish]

Как не связаны? При нажатии на кнопку вызывается функция через this и результат выполнения возвращается и учавствует в инициализации lStrem.
Поэтому возник вопрос ведь оператор new выделил память в одном блоке кода, а затем используется в другом блоке кода.

[S0mbre]

Да, сразу не увидел 
Нет, здесь ничего страшного - SaveImage возвращает либо указатель на созданный объект, либо в случае ошибки - нуль. auto_ptr в своем конструкторе забирает владение этим объектом и при выходе за область видимости (в своем деструкторе) освобождает память, так что с этой стороны все ок. Только функцию сохранения надо немного изменить - после создания auto_ptr необходимо дополнительно проверить на нулевой указатель:

Код

if (!lStream.get()) return;
 

[SilverFish]

auto_ptr в своем конструкторе забирает владение этим объектом и при выходе за область видимости (в своем деструкторе) освобождает память, так что с этой стороны все ок.

Спасибо.

std::auto_ptr _return1 (new TMemoryStream());
Image1->Bitmap->SaveToStream(_return1.get());

А функция get() вообще что делает? Она всегда должна возвращать кусок памяти куда указывает auto_ptr? Правильно?

Только функцию сохранения надо немного изменить - после создания auto_ptr необходимо дополнительно проверить на нулевой указатель

Эмбаркодеровцы не доработали пример.
Этот пример был мною замечен прямо в RADXE7 на стартовой странице студии (см.рис.).

[S0mbre]

А функция get() вообще что делает? Она всегда должна возвращать кусок памяти куда указывает auto_ptr? Правильно?

get() возвращает указатель, хранящийся внутри "умного" указателя (смарт-пойнтера) типа auto_ptr, unique_ptr, shared_ptr и т.п. Эти классы из стандартной библиотеки - всего лишь шаблонные обертки вокруг обычных указателей. Реализация у auto_ptr примерно такая:

Код

template class T>
class auto_ptr
{
private:
  T* ptr;
public:
  auto_ptr(T* AObj): ptr(AObj) { }    // конструктор с присваиванием (забирает владение объектом)
  auto_ptr() : ptr(0) { }                    // конструктор по умолчанию
  ~auto_ptr() { delete ptr; }             // деструктор - удаление объекта из памяти
 
  T* get() { return ptr; }                  // возвращает внутренний объект
  void reset(T* AObj)                       // замена объекта 
  {
     delete ptr; ptr = AObj;
  }
};
 

Эмбаркодеровцы не доработали пример.
Этот пример был мною замечен прямо в RADXE7 на стартовой странице студии (см.рис.).

  

[SilverFish]

Большое спасибо.