Ошибка деструктора - C++

@Sasha760 · Регистрация: 03.05.2013

Имеется шаблон класса- динамического массива. После создания функции push_back начал ругаться конструктор. Объясните пожалуйста в чем дело?

C++

#include<iostream>
#include<time.h>
#include<stdlib.h>
 
using namespace std;
template<typename T> class myvector
{
private:
    int size;   
    T* p;
public:
    myvector (int s = 10)
{
    size=s;
    p = new T[size];
    srand(time(0));
    for(int i=0;i<size;i++)
    p[i]=rand()%100;
}
void show()
{ 
    for(int i = 0; i<size; ++i)        
    cout << p[i] << ' ';
    cout << endl;
}
void push_back();//добавляем элемент в конец
~myvector(){delete[] p;cout<<"memory is free"<<endl;}
};
template<class T> void myvector<T>::push_back()
{
    
 
    T *u=new T[size++];
    
    for(int i=0;i<size-1;i++)
    {
        u[i]=p[i];
    }
    delete []p;
    cout<<"what do you want to add?"<<endl;
    cin>>u[size-1];
    p=u;
    
}
int main()
{
    myvector<int> a1(2);
    a1.show();
    a1.push_back();
    a1.show();
    
}

@bgm123 · 05.05.2013, 21:31

C++

template<class T> void myvector<T>::push_back()
{
    int newsize = size + 1;
    T *u=new T[newsize];
    
    for(int i = 0; i < size; i++)
        u[i] = p[i];
    
    delete []p;
    cout<<"what do you want to add?"<<endl;
    cin >> u[newsize-1];
    p = u;
}

Добавлено через 38 секунд
В этой функции ошибка. В своем варианте вы освобождаете больше, чем надо.

@metaluga145 · 05.05.2013, 21:31

Sasha760,

@bgm123 · 05.05.2013, 21:32

Сообщение от metaluga145

Вы вызываете деструктор класса в классе. Как Вы думаете, в каком состоянии будет находится класс после такого?

В какой строке?

@metaluga145 · 05.05.2013, 21:36

bgm123, где Вы это видите?

Добавлено через 1 минуту

Сообщение от bgm123

for(int i = 0; i < size; i++)
* * * * u[i] = p[i];

Вместо присваивания поэлементно, быстрее будет использовать memcpy

@bgm123 · 05.05.2013, 21:36

Вообще лучше написать функцию вот так:

C++

void myvector::push_back()
{   
    int *buf = new int[size + 1];
        
    for(int i = 0; i < size; i++)
        buf[i] = p[i];
    
    delete []p;
 
    size++;
    
    std::cout << "what do you want to add?" << endl;
    std::cin >> buf[size - 1];
    
    p = buf;
}

@metaluga145 · 05.05.2013, 21:38

bgm123, вообще по канонам, функция push_back принимает значение, которое будет добавлено в конец, и спрашивать "что Вы хотите добавить?" как-то не логично

Добавлено через 19 секунд
ибо не всегда надо добавлять что-то из консоли

@abit · 05.05.2013, 21:45

Sasha760,
сложно сказать наверняка, надо в компиляторе смотреть, но так сходу нашёл у вас ошибку в листинге, не в конструкторе, а в самом методе push_back() :

вместо

C++

1	T *u=new T[size++];

следует писать

C++

1	T *u=new T[++size];

это принципиально, потому что запись cin<<[size-1] будет уже с другим size, отличным от выделенного массива

исправьте этот пункт и скажите результат

@Sasha760 · 05.05.2013, 21:55 **[ТС]**

abit, , да вы совершенно права, Деструктор не выдал ошибки

Добавлено через 3 минуты

Сообщение от abit

Sasha760,
сложно сказать наверняка, надо в компиляторе смотреть, но так сходу нашёл у вас ошибку в листинге, не в конструкторе, а в самом методе push_back() :

вместо

C++

1	T *u=new T[size++];

следует писать

C++

1	T *u=new T[++size];

это принципиально, потому что запись cin<<[size-1] будет уже с другим size, отличным от выделенного массива

исправьте этот пункт и скажите результат

Но я все равно не понимаю, в чем принципиальное отличие? С каким size будет если T *u=new T[size++]

@abit · 05.05.2013, 21:57

Sasha760,
в случае new T[size++] сначала произойдёт выделение памяти размером size, а потом инкрементирование, а в new T[++size] - сперва инкрементирование, а потом выделение памяти размером size (увеличенного)

@Sasha760 · 05.05.2013, 21:59 **[ТС]**

abit, Да да да, точно, спасибо большое!

@metaluga145 243 / 244 / 20 Регистрация: 08.04.2013 Сообщений: 927
	05.05.2013, 21:31	#3
	Sasha760, 0

@bgm123 40 / 40 / 16 Регистрация: 29.01.2013 Сообщений: 277
	05.05.2013, 21:32	#4
	Сообщение от metaluga145 Вы вызываете деструктор класса в классе. Как Вы думаете, в каком состоянии будет находится класс после такого? В какой строке? 0

@metaluga145 243 / 244 / 20 Регистрация: 08.04.2013 Сообщений: 927
	05.05.2013, 21:36	#5
	bgm123, где Вы это видите? Добавлено через 1 минуту Сообщение от bgm123 for(int i = 0; i < size; i++) * * * * u[i] = p[i]; Вместо присваивания поэлементно, быстрее будет использовать memcpy 0

@metaluga145 243 / 244 / 20 Регистрация: 08.04.2013 Сообщений: 927
	05.05.2013, 21:38	#7
	bgm123, вообще по канонам, функция push_back принимает значение, которое будет добавлено в конец, и спрашивать "что Вы хотите добавить?" как-то не логично Добавлено через 19 секунд ибо не всегда надо добавлять что-то из консоли 0

@abit 271 / 270 / 35 Регистрация: 03.02.2013 Сообщений: 770
	05.05.2013, 21:57	#10
	Sasha760, в случае new T[size++] сначала произойдёт выделение памяти размером size, а потом инкрементирование, а в new T[++size] - сперва инкрементирование, а потом выделение памяти размером size (увеличенного) 1

@Sasha760 1 / 1 / 0 Регистрация: 03.05.2013 Сообщений: 27
	05.05.2013, 21:59 [ТС]	#11
	abit, Да да да, точно, спасибо большое! 0

Опции темы

С++ для начинающих

Ошибка деструктора - C++