Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Центр помощи > Перегрузка оператора + для вектора


Автор: Foxxx 12.5.2008, 15:16
У меня есть класс Динамический Массив (Вектор)!
Надо реализовать перегрузку оператора + чтобы возможно было такое C = A + B;
Я пробовал, но у меня какието ошибки (невозможно прочесть память)!
Помогите пожалуйста реализовать это!
Вот исходник:
Код

// libs

#include <iostream>
using namespace std;

// class (interface)

class CVector
{
protected:
    int *V, Size;
public:
    CVector();
    CVector(int);
    CVector(const CVector &);
    ~CVector();
    void Set();
    void Get();
    CVector operator + (const CVector &);
//    const CVector &operator = (const CVector &);
};

// implements

CVector::CVector()
{
    cout << "Def_Construct";
}

CVector::CVector(int Size)
{
    cout << "Param_Construct\n";
    this->Size = Size;
    V = new int[Size];
    for (int I = 0; I < Size; I++)
        *(V + I) = 0;
}

CVector::CVector(const CVector &Vec)
{
    cout << "Cpy_Construct\n";
    V = new int[Size];
    for (int I = 0; I < Size; I++)
        *(V + I) = *(Vec.V + I);
    Size = Vec.Size;
}

CVector::~CVector()
{
    cout << "Destruct\n";
    delete[] V;
}

void CVector::Set()
{
    for (int I = 0; I < Size; I++)
        cin >> *(V + I);
}

void CVector::Get()
{
    for (int I = 0; I < Size; I++)
        cout << *(V + I) << " ";
    cout << "\n";
}

/*
const CVector &CVector::operator = (const CVector &Right)
{
    if (&Right == this) return *this;
    delete[] V;
    V = new int[Size = Right.Size];
    for (int I = 0; I < Size; I++)
        *(V + I) = *(Right.V + I);
    return *this;
}
*/

CVector CVector::operator + (const CVector &Right)
{
    CVector T(Right.Size);
    for (int I = 0; I < Right.Size; I++)
        *(T.V + I) += *(Right.V + I);
    return T;
}

// main function

int main()
{
    int Size;

    // set arrays

    cout << "Enter size: ";
    cin >> Size;
    CVector A(Size), B(Size), C(Size);
    cout << "Enter values for A-array: ";
    A.Set();
    cout << "Enter values for B-array: ";
    B.Set();

    // get arrays

    cout << "A-array before: ";
    A.Get();
    cout << "B-array before: ";
    B.Get();
    cout << "C-array before: ";
    C.Get();
    C = A + B;
    cout << "A-array after: ";
    A.Get();
    cout << "B-array after: ";
    B.Get();
    cout << "C-array after: ";
    C.Get();

    // end application

    return 0;
}

Автор: Rodman 12.5.2008, 16:04

M
Rodman
Модератор: Название темы должно содержать язык написания!

Автор: anatox91 12.5.2008, 16:11
1)давай сюда дословно ошибки компилятора
2)бегло просмотрев заметил пока что вот эту ошибку:
Код

CVector(int);

надо:
Код

CVector(int Size);

Автор: Foxxx 12.5.2008, 16:14
Блин! Забыл написать язык=) С++
Вот текст ошибки:
Invalid Allocation Size: 4294967295 bytes

Добавлено через 2 минуты и 58 секунд
Цитата(anatox91 @ 12.5.2008,  16:11)
2)бегло просмотрев заметил пока что вот эту ошибку:
Код

CVector(int);

надо:
Код

CVector(int Size);

В описании класса помоему не обязательно писать имя параметра, т.е. CVector(int) - это должно быть верно!

Автор: Sartorius 12.5.2008, 16:20
 
Foxxx, советую имя параметра в конструкторе сменить  smile 

Автор: Foxxx 12.5.2008, 17:12
Sartorius, в каком именно конструкторе?
НЕ понимаю тебя, объясни подробнее ПЛЗ!

Автор: Foxxx 12.5.2008, 17:28
 smile 

Автор: creatorcode 12.5.2008, 17:42
1. Раскомментируй код перегруженного присваивания.
2. Измени конструктор копирования
Код

CVector::CVector(const CVector &Vec)
{
    cout << "Cpy_Construct\n";
    Size=Vec.Size;
    V = new int[Size];
    for (int I = 0; I < Size; I++)
        *(V + I) = *(Vec.V + I);
}

3. Измени код перегруженного сложения
Код

CVector CVector::operator + (const CVector &Right)
{
    CVector T(Right.Size);
    for (int I = 0; I < Right.Size; I++)
        *(T.V + I) = *(Right.V + I)+*(V+I);
    return T;
}

Автор: Rififi 12.5.2008, 17:50
Foxxx, 
в конструкторе копии в этой строчке
V = new int[Size];
весело выделяется память случайного размера в пределах -2gb .. 2Gb
мусор в переменной Size заменяется на правильное значение только в конце выполнения функции.

Автор: Foxxx 12.5.2008, 17:54
creatorcode, спасибо тебе огромное!
Работает!!!!
Ток объясни плз, в чем именно главная проблема была?=)

Автор: creatorcode 12.5.2008, 18:02
Цитата(Foxxx @  12.5.2008,  17:54 Найти цитируемый пост)
Ток объясни плз, в чем именно главная проблема была?=) 

У тебя было 2 основных проблемы: первая заключается в том, что дефолтное присваивание работает только для примитивных типов, для любых динамических объектов необходимо написать свою версию оператора присваивания (у тебя она была написана, только почему-то закомментирована); вторую указал Rififi.
Ну и третья проблема - логическая: в твоем коде в операторе сложения собственно сложения не происходит, а происходит присваивание одного вектора другому.

Автор: Foxxx 12.5.2008, 18:08
Ясно! Спасибо огромное тебе еще раз!!!

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)