Форум программистов, компьютерный форум CyberForum.ru

Перегрузка operator+ - C++

Восстановить пароль Регистрация
 
Рейтинг: Рейтинг темы: голосов - 13, средняя оценка - 4.85
Sagamor
0 / 0 / 0
Регистрация: 11.02.2009
Сообщений: 57
24.12.2009, 16:58     Перегрузка operator+ #1
при сложени myVector2 = myVector0 + myVector1; выодит сообщения "конструктоор 0" ...1,...2,....2(копирования)
а потом деструктор 2,2,1,2 по идеи последний должен быть 0 вектор
грешу на строчку
C++
1
2
3
MyClass MyClass::operator + (const MyClass &c)
{
col=2;....
как нужно сделать чтобы параметры не изменялись. Я так понял чтобы можно было сделать operator + (const MyClass &c) const
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
//---------------------------------------------------------------------------
 
#ifndef Unit2H
#define Unit2H
//---------------------------------------------------------------------------
#endif
  class MyClass
{ private:
double *vec; 
int rows, col; 
public:
MyClass (int gridCol, int fill); 
~MyClass ();
MyClass operator + (const MyClass &c) ;  
   double& operator () (int i);
    MyClass& operator = (const MyClass& c);
    MyClass (const MyClass& c);
};
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
//---------------------------------------------------------------------------
#pragma hdrstop
#include "Unit1.h"
#include "Unit2.h"
#include <vcl/dstring.h>
#include <Math.hpp>
#include <math.h>
#include <Dialogs.hpp>
//---------------------------------------------------------------------------
 
#pragma package(smart_init)
MyClass::MyClass(int gridCol, int fill)
{
col = gridCol;
  int r = StrToInt(Form1->Edit1->Text);
  rows = r;
  vec = new double [rows];
  if (fill == 1)MyFill();
  else if (fill == -1) MyGridToVector();
  ShowMessage("конструктор MyClass Vector " + IntToStr(gridCol));
}
MyClass::MyClass (const MyClass& c)
{
 col=c.col;
 ShowMessage("конструктор MyClass Vector " + IntToStr(col));
 rows = c.rows;
 vec = new double [rows];
 for (int i=0; i<rows; ++i) vec [i] = c.vec [i];
}
 
MyClass::~MyClass()
{
  ShowMessage("äåñòðóêòîð ГЄГ«Г*Г±Г±Г* MyClass Vector " + IntToStr(col));
  delete [] vec;
}
MyClass MyClass::operator + (const MyClass &c)
{
col=2;
if(rows != c.rows) throw 1;
for (int i=0; i<rows; ++i)
vec [i] =pow(vec [i]+c.vec [i],2);
return *this; }
 
 
double& MyClass::operator () (int i)
{if(i<=rows)  return vec[i];
 else
 throw 1;
 }
 
MyClass& MyClass:: operator = (const MyClass& c)
 
{ if (this == &c) return *this;
if(rows != c.rows) throw 2;
for (int i=0; i<rows; ++i)
{
(*this) (i) = c.vec [i];
}
return *this;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
24.12.2009, 16:58     Перегрузка operator+
Посмотрите здесь:

C++ перегрузка operator<<
Перегрузка operator= C++
C++ перегрузка operator[][][]
C++ Перегрузка operator new
C++ Перегрузка operator->()
Перегрузка operator+ C++
Перегрузка operator-> () C++
Перегрузка operator[]= C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Nick Alte
Эксперт С++
1590 / 982 / 115
Регистрация: 27.09.2009
Сообщений: 1,897
Завершенные тесты: 1
24.12.2009, 18:51     Перегрузка operator+ #2
Оператор + должен возвращать новое значение, а не первый операнд. Соответственно, сам первый операнд изменять нельзя. Так что надо в теле оператора описать новый локальный объект того же типа MyClass и вернуть его.
C++
1
2
3
4
5
6
7
8
MyClass MyClass::operator + (const MyClass &c) const
{
MyClass r(*this);
col=2;
if(r.rows != c.rows) throw 1;
for (int i=0; i<r.rows; ++i)
r.vec [i] =pow(r.vec [i]+c.vec [i],2);
return r; }
Ну а с точки зрения идеологии правильнее реализовать оператор += и потом писать так:
C++
1
2
3
4
5
6
MyClass MyClass::operator + (const MyClass &c)
{
    MyClass r(*this);
    r += c;
    return r;
}
Sagamor
0 / 0 / 0
Регистрация: 11.02.2009
Сообщений: 57
24.12.2009, 19:07  [ТС]     Перегрузка operator+ #3
теперь вообще непонятно пишет
констр 0,1,2,0,0
дестр 0,0,2,1,2 опять последний не тот который дожен быть..т.е не 0...

Добавлено через 3 минуты
col=2; если вычеркнуть то все вароде работает.
Yandex
Объявления
24.12.2009, 19:07     Перегрузка operator+
Ответ Создать тему
Опции темы

Текущее время: 15:04. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru