Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > Проблема с классом

Автор: fippo 26.5.2008, 06:23

Мне дали задание по инфе:

Код


class Vector
{
  double* Body; unsigned Size;
 public:
  Vector(unsigned size);
  ~Vector();
  double  operator[](unsigned i) const;
  double& operator[](unsigned i);
  unsigned size() const;
};

реализовать все член-функции и продемонстрировать на примере их использование.
Указание. 1). В конструкторе выделяется динамическая память под массив из size элементов типа double (в котором и будут храниться элементы вектора). На эту память ссылается указатель Body. 2). В деструкторе освобождается память из-под Body. 3). Операторы [ ] обеспечивают доступ к элементам вектора для считывания и изменения. В них производится проверка индексации. 4). Член-функция size возвращает размер вектора. 5). Программа должна состоять из двух модулей: ‘main.cpp’ и ‘vect.cpp’. В ‘vect.cpp’ реализуются член-функции класса Vector, а в ‘main.cpp’ этот класс используется. Описание класса Vector вынести в файл заголовка ‘vect.h’, который подключается в оба модуля программы.

Вот то что я написал:

Vect.h

Код


class Vector
{
  double* Body; unsigned Size;
  public:
  Vector(unsigned size);
  ~Vector();
  double  operator[](unsigned i) const;
  double& operator[](unsigned i);
  unsigned size() const;
};

Vect.cpp

Код


#include "stdafx.h"
#include <iostream>
#include "vect.h"

Vector::Vector(unsigned int size)
{
    Vector::Body = new double[size];
    Vector::Size = size;
}

Vector::~Vector()
{
    delete[] Vector::Body;
    Vector::Size = 0;
}

double& Vector::operator [](unsigned i)
  {
      if(i<0 || Vector::Size<=i){std::cout << "индекс выходит за границы";}
      else
      {return Body[i];}
  } 

unsigned Vector::size() const
{
return Size;
}

main.cpp

Код


#include "stdafx.h"
#include <iostream>
#include "vect.h"
#include <conio.h>

int _tmain(int argc, _TCHAR* argv[])
{
    Vector v1(100);
    v1[10]=5;
    v1[20]=v1[10];
    std::cout << v1[10] << "\n";
    std::cout << v1.size() << "\n";
    getch();
    return 0;
}

Программа компилируется и работает, но препод говорит что это не правильно, что у меня не реализовано копирование. Подскажите плиз, что тут не так, и как это исправить. И ещё вопрос, зачем даны две строки:
double operator[](unsigned i) const;
double& operator[](unsigned i);

Автор: rrrFer 26.5.2008, 06:38

fippo,

Цитата

не реализовано копирование

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

Код


Vector::Vector(const Vector &V){
    Body = V.Body;
    Size  = V.size  ;
}

-думаю он примерно так должен выглядеть

Автор: bronislav 26.5.2008, 07:20

Цитата(fippo @ 26.5.2008, 05:23

)

double operator[](unsigned i) const;
double& operator[](unsigned i);

Оператор индексации.

Чтобы можно было применять индексацию к объекту класса, не вызывая метод.

Код


Vector MyVector;
MyVector[2] = 1;

Автор: math64 26.5.2008, 08:18

Код


// Конструктор копирования
Vector::Vector(const Vector& v) {
Body = NULL;
*this = v;
}
// Оператор копирования
Vector& Vector::operator=(const Vector& v) {
if (Body != NULL)
  delete[] Body;
Size = v.size();
Body = new double[Size];
for (int i = 0; i < Size; i++)
  (*this)[i] = // используется Vector::operator[](unsigned i)
  v[i]; // используется Vector::operator[](unsigned i) const
return *this;
}