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 that you have read and understand our privacy policy and code of conduct.

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