C ++ создание объекта с * новым

Question

C ++ создание объекта с * новым

0

У меня есть метод с моим пользовательским объектом в качестве параметра:

void processObject(Myobject instance)
{
 //some code using instance
}

Я вызываю метод выше из другого метода:

...

processObject(*new MyObject());

...

Может ли это вызвать утечку памяти? Должен ли я вызвать что-то вроде delete & instance в методе объекта процесса?

Большое спасибо за вашу помощь!

РЕДАКТИРОВАТЬ

теперь я знаю, что есть утечка памяти, что бы вы предложили как самый простой способ исправить это? Моя идея - изменить этот код:

void processObject(Myobject* instance)
{
 //some code using instance
    delete instance;
}

...

processObject(new MyObject());

...

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

David Broza 17 июль 2014, в 10:59

Источник

1

Почему вы используете new в первую очередь? Почему бы просто не processObject(MyObject()); ?
Wojtek Surowka 17 июль 2014, в 09:02
0

Вы звонили delete где-нибудь? Нет, потому что ты не можешь. Итак, у вас есть утечка памяти.
juanchopanza 17 июль 2014, в 09:07
0

удаление никогда не вызывалось - я попал в этот проект и нашел этот код во многих местах. Так что самый простой способ это исправить:
David Broza 18 июль 2014, в 08:52

Показать ещё 1 комментарий

Теги:

c++

pointers

memory-leaks

memory

delete-operator

4 ответа

Ещё вопросы

Почему вы используете new в первую очередь? Почему бы просто не processObject(MyObject()); ?
Вы звонили delete где-нибудь? Нет, потому что ты не можешь. Итак, у вас есть утечка памяти.
удаление никогда не вызывалось - я попал в этот проект и нашел этот код во многих местах. Так что самый простой способ это исправить:

maxdev · Answer 1 · 2014-07-17T06-15-00.000Z

Вы не можете удалить объект экземпляра, что processObject процедура получает, потому что это копия объекта, созданного с new. Представьте себе это так:

Вы создаете новый объект с помощью new, это создает объект в куче и дает вам MyObject*
Вы разыскиваете указатель, поэтому у вас есть объект типа MyObject. Здесь вы также теряете указатель, так как вы его нигде не храните
Вы передаете объект методу processObject, и по мере того как подпись этой функции заявляет, копия вашего объекта создается и передается функции

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

Решением не будет выделение объекта new и передача refrence методу процесса:

void processObject(MyObject& ref) {
    // do something with the reference
}

void test() {
    MyObject x;
    processObject(x);
    // do stuff with x
}

Или выделите его new и выполните следующее:

void processObject(MyObject* ptr) {
    // do something with the pointer
}

void test() {
    MyObject* x = new MyObject();
    processObject(x);
    // do stuff with x
    delete x;
}

И не забудьте потом удалить его, и подумать об использовании RAII или умного указателя
Лучше ... но не исключение безопасно, верно?

Spook · Answer 2 · 2014-07-17T08-51-00.000Z

Пытаться:

class MyObject
{
public:
    MyObject() { std::cout << "Reporting construction of object (ptr: " << (int)this << ")\n"; }
    MyObject(const MyObject & other) { std::cout << "Reporting copy-construction of object (ptr: " << (int)this << ")\n"; }
    MyObject(MyObject && other) { std::cout << "Reporting move-construction of object (ptr: " << (int)this << ")\n"; }
    virtual ~MyObject() { std::cout << "Reporting destruction of object (ptr: " << (int)this << ")\n"; }
};

void DoSomething(MyObject obj)
{
}

int main()
{
    DoSomething(*new MyObject());
}

Результат:

Reporting construction of object (ptr: 140173320)
Reporting copy-construction of object (ptr: -1078250292)
Reporting destruction of object (ptr: -1078250292)

Как вы видите, первый экземпляр не был уничтожен - таким образом, просочился.

doctorlove · Answer 3 · 2014-07-17T08-07-00.000Z

В его нынешнем виде происходит утечка памяти. Вы можете удалить объект внутри функции processObject адрес, однако подпись

void processObject(Myobject instance)

не мешает кому-либо передавать локальный объект variable/stack.

Myobject instance;
processObject(instance);

Если это произойдет, вы не сможете delete объект.

Избегайте использования необработанных указателей и избегайте запутывания пользователей вашей функции.

Ultimate Hawk · Answer 4 · 2014-07-17T06-38-00.000Z

Да, вы должны удалить его после нового, но вы не сможете удалить этот вновь созданный объект.

Всякий раз, когда вы пишете *new x; , вы разыскиваете указатель на фактическую ссылку (или объект).
В вашем случае Myobject передается по значению и, таким образом, копируется, оставляя указатель не в пространстве стека и тем самым вызывая утечку памяти.

Я предлагаю вам использовать следующую процедуру вызова с почти таким же эффектом: (без утечки памяти):
processObject(MyObject());

Я думаю, что вы, возможно, получили новый синтаксис с Java, где мы вынуждены его использовать. В C++ вы можете просто вызвать конструктор неявно, как это, и создать новый объект на месте, который затем передается как аргумент функции.
В этом случае объект строится на стеке и, следовательно, имеет автоматическую продолжительность хранения (без утечки памяти).