Куда деваются объекты в std::vector??

Question

[pixik]

Доброго времени!
Есть класс singleton, в котором находится

// Файл singleton.h
class singleton{
... // вся кухня с инициализацией singleton

private:
... 
std::vector<someclass *> vect;
std::vector<someclass * >  initVector();
};

В конструкторе класса я заполняю vect объектами:

// Файл singleton.cpp
singleton::singleton()
{
	this->vect = singleton::initVector();
}


std::vector<someclass * >  singleton::initVector(){
    std::vector<someclass * > _vect;
    for (int i = 0; i < 4; i++){ _vect.push_back( new someclass()); }
    return _vect;
}

Во время инициализации вектор наполняется, но после того, как объект singleton сформировался (после выхода из конструктора), в vect остается только пустой вектор vect (size = 0).
не понимаю, куда делись объекты, добавленные в конструкторе singleton ?

У класса someclass конструктор по умолчанию определен. Использую MSVS 2013 в mfc проекте.
Спасибо!

Answer 1

[pavel null]

Вы создали vector в стеке и пытаетесь объект вернуть из метода по значению.
Возвращая данные по значению они копируются, не так ли ? Походу создается новый пустой вектор,
который присваивается вашей переменной. А _vect благополучно умирает не выходя из области видимости
метода. Стоит почитать www.e-reading.link/book.php?book=1002058

Comments

[pavel null]

Ну а решение, vecor создавать через new и возвращать указатель из метода

[Армянское Радио]

one pavel: по вашим словам получается, что и строки нужно возвращать при помощи new. Все не правильно:
вектор у автора создается не в статической памяти (нет модификатора static), а в локальной памяти метода. После чего делается возврат значения и присваивание его полю в классе. Это означает копирование всех элементов из первого вектора во второй.

[pavel null]

согласен, конечно же на стеке создаются, утро неожиданно рано началось =)

Answer 2

[pixik]

one pavel: Вы создали vector в статической памяти и пытаетесь объект вернуть из метода по значению.
Возвращая данные по значению они копируются, не так ли ? Походу создается новый пустой вектор,
который присваивается вашей переменной. А _vect благополучно умирает не выходя из области видимости
метода.

Возвращая данные по значению они копируются, не так ли ?

Копируются конечно. оператор = перегружен для этих целей.

На самом деле вопрос немного меняется. Я вызывал конструктор из конструктора:

singleton::singleton(){
	singleton::singleton(600, 400);
}

А инициализация vect была тут:

singleton::singleton(int , int)
{
  this->vect = singleton::initVector();
}

Я думал, что это не существенно, но вот так получилось то, что нужно:

// Файл singleton.h
class singleton{
... // вся кухня с инициализацией singleton

private:
... 
std::vector<someclass *> * vect;
std::vector<someclass * >  *initVector();
};

// Файл singleton.cpp
singleton::singleton(){
	this->vect = *this->initVector();
}

std::vector<someclass* > * singleton::initVector(){
	std::vector<someclass* >  * vect = new std::vector<someclass* >();
	for (int i = 0; i < 4; {	_vect->push_back(new someclass()); }
	return _vect;
}

Спасибо за книгу! Я уже не один десяток раз на неё натыкался. В ближайшее время изучу:)

Comments

[Армянское Радио]

Вызов конструктора из конструктора ради того, чтобы задать параметры? Про возможность задания параметров по умолчанию вы знаете?
singleton::singleton(int a=600,int b=600);
Вы сделали костыль.

[pixik]

Армянское Радио: про костыль я знаю) и про "возможность задания параметров по умолчанию вы знаете" тоже знаю.
сейчас по быстрому набрасываю код, по этому костыли. Нужно быстро показать мокетик. Я другого не понял, почему при вызове конструктора из конструктора поля обнулились?
так не работает
singleton::singleton(){
singleton::singleton(600, 400);
}
Но зато
singleton::singleton() :singleton::singleton(600, 400){
...
}
все работает.

[Армянское Радио]

Потому, что ваш вызов конструктора из конструктора делает не то, что вы от него ожидаете. Он создает временный экземпляр вашего класса, который тут же убивается, а тот экземпляр, который вызывал конструктор, остается не инициализированным.
Объяснение

Нужный вам функционал есть в C++11.

Answer 3

[yagorik]

Вызов конструктора из конструктора доступен только С++11/14
И в вашем случае он должен выглядеть как-то так:

singleton::singleton() : singleton(600, 400)
{}

На самом деле вопрос немного меняется. Я вызывал конструктор из конструктора:
singleton::singleton(){
  singleton::singleton(600, 400);
}

То что вы написали работает иначе, в этой строке singleton::singleton(600, 400); создаётся безымянный локальный объект singleton, который будет удалён после выхода из тела вызывающего конструктора.

Answer 4

[Армянское Радио]

У вашего singleton потомки есть? Потому что пока что я не вижу, что вам ногу отстреливает, нужно больше кода.

Comments

[pavel null]

Армянское Радио все нормально =) pixik идет правильным путем, методом проб и ошибок. В отличие от большого кол-ва пользователей с вопросами "кем быть, как сделать и тд"

[xseven]

Армянское Радио: Так и хочется увидеть ваш ответ начинающийся с фразы: "Нас часто спрашивают...."

[pavel null]

отличная идея =) ребята в основном линки на вопросы на тосторе бросают, с пинанием в сторону поиска

[Армянское Радио]

xseven: у меня есть похожий ответ

[xseven]

Армянское Радио: Так держать. Каждому аппендиксу по скальпелю в умелых руках!