1

Всем привет! Сегодня пытался перегружать операторы, но ничего не вышло. Вот мой код:

    class VectorInt {
 public:
  VectorInt();
  VectorInt(size_t);
  VectorInt(int, int);
  virtual ~VectorInt();

  int &get(size_t);
  void set(size_t);
  void set(size_t, int);

  size_t getSize();

  static size_t getMaximumSize();

  VectorInt & operator=(VectorInt & obj);
  friend VectorInt operator+(VectorInt & obj1,VectorInt & obj2);

 private:
  void setAllTo(int);
  size_t size;
  int *data;
};

/* CONSTRUCTORS */

VectorInt::VectorInt() : size(1), data(new int[size]()) {}

VectorInt::VectorInt(size_t size) : size(size), data(new int[size]) 
{
    for (size_t i = 0; i < size; ++i) {
      data[i] = i;
    }
}

VectorInt::VectorInt(int size, int initializer) : size(size),data(new int[size]) 
{
    setAllTo(initializer);
}

VectorInt::~VectorInt() 
{
    delete [] data;
}

/* PUBLIC METHODS */

int &VectorInt::get(size_t position) 
{
    if (position >= size)
      throw std::out_of_range("index went out of range");

    return data[position];
}

void VectorInt::set(size_t position) 
{
    if (position >= size)
      throw std::out_of_range("index went out of range");

    data[position] = 0;
}

void VectorInt::set(size_t position, int value) 
{
    if (position >= size)
      throw std::out_of_range("index went out of range");

    data[position] = value;
}

size_t VectorInt::getSize() 
{
    return size; 
}

/* STATIC METHODS */

size_t VectorInt::getMaximumSize() 
{ 
    return static_cast<size_t>(-1); 
}

/* PRIVATE METHODS*/

void VectorInt::setAllTo(int value) 
{
    for (size_t i = 0; i < size; ++i)
    {
      data[i] = value;
    }
}

/* OPERATORS */

VectorInt operator+(VectorInt &obj1, VectorInt &obj2) 
{
    VectorInt newObj = VectorInt(obj1.getSize());

    for (size_t i = 0; i < obj1.getSize(); i++)
      newObj.set(i, obj1.get(i)+obj2.get(i));

    return newObj;
}

VectorInt & VectorInt::operator=(VectorInt & obj) 
{
    if (this != &obj) 
    {
      size = obj.size;
      delete [] data;
      data = new int[size];

      for (size_t i = 0; i < size; ++i)
        data[i] = obj.data[i];

    }

    return *this;
}

int main() 
{
  VectorInt v(10);
  v.set(9);
  v.set(3, 76);

  VectorInt v2(10);
  v2.set(9);
  v2.set(3, 24);
  VectorInt v3 = v+v2;

  cout << v3.get(5);

  return 0;
}

В итоге v3.get(5) выводит -57662...
Хотя должен выводить 10. Я так понял, что перегрузка не сработала, не сохранились числа. Если внутри перегрузки оператора плюс вывести newObj.get(5), то выведет 10. Помогите, в чём здесь проблема?(

10
  • 1
    У вашего VectorInt ну очень странная логика. Операции делают что угодно, но только не то, что от них ожидаешь. Соответственно, понять, что должно выводиться и почему — невозможно. Использование операторов для странных операций ("сложение" векторов — это добавление пяти ко всем значениям первого вектора без участия второго вектора) ну нисколько не помогает. Приведите код к виду, когда он несёт хоть какой-то смысл.
    – Kyubey
    13 мая 2015 в 11:55
  • Да, извините, я тестил просто, поправил
    – iproger
    13 мая 2015 в 11:56
  • 1
    @iproger У вас для возвращаемого объекта VectorInt в операторе + вызывается деструктор и делается delete [] data.
    – cybrex
    13 мая 2015 в 12:03
  • @cybrex, ого, действительно, убрал деструктор - заработало. а почему он там вызывается? и как это предотвратить?
    – iproger
    13 мая 2015 в 12:07
  • @iproger Уже лучше. Но всё равно много непонятного. 1) Почему при создании по размеру в массив засовываются последовательные числа, а при передаче конкретного значения — одинаковые? 2) Почему проверка индекса только сверху? 3) Почему set без значения устанавливает 0? 4) Почему при сложении не проверяется равенство размеров?
    – Kyubey
    13 мая 2015 в 12:08

1 ответ 1

1

Благодаря пользователю @cybrex я понял в чём проблема. Создаваемый newObj стирается из локальной области функции (вызывается деструктор) и поэтому возвращает функция пустой объект. Я подумал, что нужно создать конструктор, который будет копировать объект. И всё удалось.
Проблему решает это:

VectorInt::VectorInt(const VectorInt & other) 
{
    if (this != &other) 
    {
      size = other.size;
      data = new int[size];

      for (size_t i = 0; i < size; ++i) 
        data[i] = other.data[i];

    }
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.