Изменить объект в std::vevtor на другой тип

Question

[Славка]

В чем суть, есть вектор указателей на базовый класс std::vector Mas;
вот так он заполняется

void __fastcall TForm2::Button1Click(TObject *Sender)
{
	String name=Edit2->Text;

if (RadioGroup2->ItemIndex==0) {
	CPointIndic *tmp =new CPointIndic;
	tmp->setName(name);

	Mas.push_back(tmp);
	Form1->updateList(Form1->List);
}

if (RadioGroup2->ItemIndex==1) {
	CMenuIndic *tmp =new CMenuIndic;
	tmp->setName(name);

	Mas.push_back(tmp);
	Form1->updateList(Form1->List);
}

if (RadioGroup2->ItemIndex==2) {
	CCircleIndic *tmp =new CCircleIndic;
	tmp->setName(name);

	Mas.push_back(tmp);
	Form1->updateList(Form1->List);
}


Close();
}

теперь нужно, изменять типы объектов

вот так я его меняю, но он не хочет компилироваться, помогите разобраться с этим механизмом

int index = List->ItemIndex;

if (index>=0) {

	String name = Mas[index]->getName();
	int n = StrToInt(InputBox("","",""));

	switch(n)
	{
		case 1 :
		delete Mas[index];
		CPointIndic *tmp = new CPointIndic;
		Mas.insert(index+1,tmp);
		break;
	}


}

}

Comments

[tsarevfs]

что пишет?

[Славка]

[bcc32 Error] Unit1.cpp(96): E2285 Could not find a match for 'std::vector >::insert(int,CPointIndic *)'
Full parser context
Unit1.cpp(83): parsing: void _fastcall TForm1::N4Click(TObject *)

Answer 1

[tsarevfs]

Insert, например, не умеет вставлять по номеру позиции.

typedef std::vector<int>::iterator vector_iter;
vector_iter iter = Mas.begin() + (index + 1);
Mas.insert(iter, tmp);

Comments

[Славка]

вообще я правильно по логике то делаю?

[tsarevfs]

@Slavka_online Я бы копипасту убрал:

void __fastcall TForm2::Button1Click(TObject *Sender)
{
	CBaseIndic * baseIndic = NULL;
	switch (RadioGroup2->ItemIndex)
	{
	case 0:
		baseIndic = new CPointIndic;
		break;
	case 1:
		baseIndic = new CMenuIndic;
		break;
	case 2:
		baseIndic = new CCircleIndic;
		break;
	}
	
	String name = Edit2->Text;
	baseIndic->setName(name);
	Mas.push_back(baseIndic);
	
	Form1->updateList(Form1->List);

	Close();
}

По смыслу не знаю что ты делаешь.

[Славка]

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

[Славка]

в С# или Java я бы просто написал Mas[i] = new тип()

[tsarevfs]

@Slavka_online стоп, тогда у тебя еще косяк. Ты сейчас удаляешь класс, оставляешь сломанный указатель в массиве и вставляешь после него новый. Все проще:

delete Mas[idx];
Mas[idx] = new CPointIndic();

или, что тоже самое:

CbaseIndic * oldValue = Mas[idx];
Mas[idx] = new CPointIndic();
delete oldValue; //java сделала бы это сама, когда поняла что на этот объект никто не ссылается

Answer 2

[XO8964]

Всё намного проще

CPointIndic *tmp = Mas[index];
delete tmp;
tmp = new CPointIndic;
Mas[index] = tmp;