Поле класса - динамический массив

@OdessaNA · Регистрация: 10.01.2011

Author24 — интернет-сервис помощи студентам

Здравствуйте, друзья.
Проверьте, пожалуйста, всё ли правильно написано (отсутствие сообщений об ошибках не исключает их присутствия).
Цель - создать класс с указателем типа int. Этот поле использовать как динамический массив.
Вот код:

C++

#include <iostream>
#include <conio.h>
using namespace std;
 
const int SIZE = 3;
 
class DinArr
{
    int *array;
    int length;
public:
    DinArr() { array = new int [SIZE]; length = 0; }
    DinArr(const DinArr&);
    ~DinArr () { delete [] array; }
    void Verific( int );
    void Add( int );
    void Show();
 
};
DinArr :: DinArr(const DinArr& obj)
{
    array = obj.array;
    length = obj.length;
}
 
void DinArr :: Verific( int x )
{
    if( x >= SIZE)
    {
         int *temp = new int [x + 1];
         for(int i = 0; i < x; ++i)
         {
             temp[i] = array[i];
         }
         array = temp;
    }
}
 
void DinArr :: Add( int x )
{
    Verific( length );
    
    array[length++] = x;
}
 
void DinArr :: Show()
{
    for(int i = 0; i < length; ++i)
    {
        cout << array[i] << ' ';
    }
    cout << endl;
}
 
void main()
{
    DinArr obj;
 
    int z=0;
 
    while(z < 5)
    {
        obj.Add(7);
        z++;
    }
 
    obj.Show();
 
    _getch();
}

Интересует вопрос - очищается ли вся выделенная память?

@Avazart · 31.12.2012, 01:13

C++

DinArr :: DinArr(const DinArr& obj)
{
    array = obj.array;
    length = obj.length;
}

Это не правильно, и еще нехватает перегрузки оператора присвоения

@Uvi · 31.12.2012, 04:22

C++

DinArr :: DinArr(const DinArr& obj)
{
    this->arr=new int[obj.length];
    for(int i=0;i<obj.length;i++)
    {
        arr[i]=obj.arr[i];
    }
    this->length=obj.length;
}

во первых нужно выделить память для твоего поинтера, а потом делать глубокое копирование .
То что ты сделал, если выделить память, тоже будет работать. Я тоже студент, далеко не специалист. Нас учили, что правильнее переписывать глубоко все переменные. С интеджерами это работает, но есле у тебя кроме них еще куча всего будет внутри, то не факт.
кроме этого вместо array нужно поставить другое название, например arr (как я поставил) в копи констракторе,
array - это уже готовый массив в библиотеке.

@UserAK · 31.12.2012, 05:57

если есть new, то должен быть и delete
например

C++

1	int *temp = new int [x + 1];

...
надо бы сначала

C++

1	delete[] array;

а уж потом

C++

1	array = temp;

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

можно вместо void DinArr::Show() перегрузить оператор <<

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

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

array - это уже готовый массив в библиотеке.

впринципе это не особо страшно, просто придётся указывать пространство имён, но конечно для удобства лучше избегать таких ситуаций.
многие используют в именах переменных членов класса приставку m_
получится m_array

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

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

правильнее переписывать глубоко все переменные

почти у всех классов имеются закрытые члены, и поэтому в особых случаях надо использовать специальные функции вроде assign.
а структуры и так прекрасно копируются. вообще за правильное копирование отвечает разработчик класса.

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

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

DinArr() { array = new int [SIZE]; length = 0; }

получается размер массива и length разные

@OdessaNA · 31.12.2012, 17:41 **[ТС]**

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

C++

DinArr :: DinArr(const DinArr& obj)
{
    array = obj.array;
    length = obj.length;
}

Это не правильно, и еще нехватает перегрузки оператора присвоения

Ёлку установил, можно и исправить:

C++

DinArr :: DinArr(const DinArr& obj)
{
    length = obj.length;
 
    array = new int [length + 1];
    for (int i = 0; i < length; i++)
    {
        array[i] = obj.array[i];
    }
}

...

перегрузка будет "между" салатами и отбивными.

@Avazart · 31.12.2012, 17:46

А зачем length + 1 ?

@OdessaNA · 31.12.2012, 20:43 **[ТС]**

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

А зачем length + 1 ?

Т.к. в моём классе length - индекс элементов. Соответственно размер массива length+1 .

@Avazart · 31.12.2012, 20:46

Т.е. ? length обычно обозначают размер массива ?
Зачем выделять на один элемент больше ?

go · 31.12.2012, 21:34

OdessaNA, на каждый new должен приходиться delete. (new[] / delete[]). Вот и считайте.

@Avazart · 31.12.2012, 21:36

C++

void DinArr :: Verific( int x )
{
    if( x >= SIZE)
    {
         int *temp = new int [x + 1];
         for(int i = 0; i < x; ++i)
         {
             temp[i] = array[i];
         }
         array = temp;
    }
}

Эту ф-цию надо переписать с учетом того что сказал go

@OdessaNA · 01.01.2013, 18:47 **[ТС]**

Добавлено через 9 минут
Спасибо, друзья, за помощь!
Учел Ваши подсказки и замечания, и переписал код:

C++

#include <iostream>
#include <conio.h>
using namespace std;
 
class DinArr
{
    int *array;
    int size;
public:
    DinArr();
    ~DinArr() { delete [] array; }
    DinArr(const DinArr&); //
    DinArr operator = (const DinArr&); //
    DinArr& operator ++ (); //
    DinArr& operator -- (); //
    void Add( int );
    void Show();
};
 
DinArr :: DinArr()
{
    size = 0;
    array = new int [size + 1];
}
 
DinArr :: DinArr(const DinArr& obj)
{
    size = obj.size;
 
    array = new int [size];
    for(int i = 0; i < size; ++i)
        array[i] = obj.array[i];
}
 
DinArr DinArr :: operator = (const DinArr& obj)
{
    size = obj.size;
 
    delete [] array;
 
    array = new int [size];
    for(int i = 0; i < size; ++i)
        array[i] = obj.array[i];
 
    return *this;
}
 
DinArr& DinArr :: operator ++ ()
{
 
    int *temp = new int[size + 1];
    for(int i = 0; i < size; ++i)
        temp[i] = array[i];
 
    delete [] array;
    array = temp;
 
    ++size;
    array[size-1] = 0;
 
    return *this;
}
 
DinArr& DinArr :: operator -- ()
{
    --size;
 
    int *temp = new int [size];
    for(int i = 0; i < size; ++i)
        temp[i] = array[i];
 
    delete [] array;
    array = temp;
 
    return *this;
}
 
void DinArr :: Add( int x )
{
        int *temp = new int [size + 1];
        for(int i = 0; i < size; ++i)
            temp[i] = array[i];
        
        delete [] array;
        array = temp;
 
        array[size] = x;
        ++size;
}
 
void DinArr :: Show()
{
    for(int i = 0; i < size; ++i)
        cout << array[i] << ' ';
    cout << endl;
}
 
void main()
{
    DinArr obj, obj2;
    
    int i = 0;
 
    while(i < 10)
        obj.Add(++i);
 
    obj.Show();
 
    obj2 = obj;
    ++obj2;
    --obj2;
 
    obj2.Show();
    
    _getch();
}

У меня остались некоторые сомнения по конструктору копирования. Подскажите, пожалуйста, он правильно реализован, или в нём есть ошибка?

@OhMyGodSoLong · 01.01.2013, 19:02

Всё окей. (Чтоб всё точно было окей, стоит перенести инициализацию размеров после оператора new. Если new сломается, то у вас будет хотя бы пустой массив, а не чёрти что.)

@MrCold · 01.01.2013, 19:58

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

сомнения по конструктору копирования

delete пропустили , без него будет утечка памяти

C++

DinArr :: DinArr(const DinArr& obj)
{   
       delete [] array;
 
    array = new int [obj.size];
      size = obj.size;
    for(int i = 0; i < size; ++i)
        array[i] = obj.array[i];
}

@OhMyGodSoLong · 01.01.2013, 20:41

Зачем? Это конструктор, 1) он вызывается один раз, 2) значение array не определено. А в operator=() delete[] есть.

@MrCold · 01.01.2013, 20:45

Сообщение от ~OhMyGodSoLong~

Это конструктор, 1) он вызывается один раз

Точно

@rangerx · 01.01.2013, 21:13

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

void Show();

Должна быть const.

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

DinArr :: DinArr()
{
* * size = 0;
* * array = new int [size + 1];
}

Непонятен этот size + 1. Почему не NULL?

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

DinArr DinArr :: operator = (const DinArr& obj)
{
* * size = obj.size;
delete [] array;
array = new int [size];
* * for(int i = 0; i < size; ++i)
* * * * array[i] = obj.array[i];
return *this;
}

Желательно возвращать ссылку(по аналогии со встроенными типами). И где проверка на присваивание объекту его собственной копии?
P.S. Касательно

C++

1 2	delete [] array; array = new int [size];

При возникновении исключения в случае ошибки выделения памяти, после выполнения delete [] array, array так и останется "неинициализированным" указателем. При этом, во время обработки исключения декструктор данного(уже сконструированного) объекта всеравно будет вызван. Для того чтобы избежать повторного вызова delete [] array в деструкторе, после освобождения памяти данный указатель следует обнулить.

@OdessaNA · 01.01.2013, 21:41 **[ТС]**

rangerx, ~OhMyGodSoLong~, спасибо за поправки!

Добавлено через 15 минут
Внес изменения с учетом Ваших замечаний. Теперь "читабельнее".
Ещё раз спасибо!

@OhMyGodSoLong ~ Эврика! ~ 1256 / 1005 / 74 Регистрация: 24.07.2012 Сообщений: 2,002
	01.01.2013, 20:41	14
	Зачем? Это конструктор, 1) он вызывается один раз, 2) значение array не определено. А в operator=() delete[] есть. 2

@Avazart 8385 / 6147 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	31.12.2012, 17:46	6
	А зачем length + 1 ? 0

@OdessaNA 19 / 19 / 6 Регистрация: 10.01.2011 Сообщений: 241
	31.12.2012, 20:43 [ТС]	7
	Сообщение от Avazart А зачем length + 1 ? Т.к. в моём классе length - индекс элементов. Соответственно размер массива length+1 . 0

@Avazart 8385 / 6147 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	31.12.2012, 20:46	8
	Т.е. ? length обычно обозначают размер массива ? Зачем выделять на один элемент больше ? 0

go 3646 / 1378 / 243 Регистрация: 16.04.2009 Сообщений: 4,526
	31.12.2012, 21:34	9
	OdessaNA, на каждый new должен приходиться delete. (new[] / delete[]). Вот и считайте. 1

@OhMyGodSoLong ~ Эврика! ~ 1256 / 1005 / 74 Регистрация: 24.07.2012 Сообщений: 2,002
	01.01.2013, 19:02	12
	Всё окей. (Чтоб всё точно было окей, стоит перенести инициализацию размеров после оператора new. Если new сломается, то у вас будет хотя бы пустой массив, а не чёрти что.) 1

@MrCold 873 / 771 / 173 Регистрация: 11.01.2012 Сообщений: 1,942
	01.01.2013, 20:45	15
	Сообщение от ~OhMyGodSoLong~ Это конструктор, 1) он вызывается один раз Точно 0

@OdessaNA 19 / 19 / 6 Регистрация: 10.01.2011 Сообщений: 241
	01.01.2013, 21:41 [ТС]	17
	rangerx, ~OhMyGodSoLong~, спасибо за поправки! Добавлено через 15 минут Внес изменения с учетом Ваших замечаний. Теперь "читабельнее". Ещё раз спасибо! 0