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

Перегрузка оператора ++ - C++

Восстановить пароль Регистрация
 
lest_
0 / 0 / 0
Регистрация: 28.11.2013
Сообщений: 94
13.07.2014, 12:10     Перегрузка оператора ++ #1
выводит не то значение

привет. нужна Ваша подсказка. листинг из учебника: (тема: перегрузка оператора ++)
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
#include <iostream>
using namespace std;
 
class Counter
{
public:
    Counter();  
    ~Counter() {} 
    int GetItsVal() const { return itsVal; }
    void SetItsVal(int x) { itsVal = x; }
    void Increment()      { ++itsVal; }
    const Counter& operator++();
 
private:
    int itsVal;
};
 
Counter::Counter():
itsVal(0)
{}
 
const Counter& Counter::operator++()
{
    ++itsVal;
    return *this;
}
 
int main()
{
    Counter i;
    cout << "The value of i is " << i.GetItsVal() << endl;
    i.Increment();
    cout << "The value of i is " << i.GetItsVal() << endl;
    ++i;
    cout << "The value of i is " << i.GetItsVal() << endl;
    Counter a = ++i;
    cout << "The value of a: " << a.GetItsVal();
    cout << " and i: " << i.GetItsVal() << endl;
    
    char response; cin >> response; // для задержки экрана
    return 0;     
}
Результат этой программы:
The value of i is 0
The value of i is 1
The value of i is 2
The value of a: 3 and i: 3


я решил немного листинг видоизменить. сделал: реализация конструктора внутри класса, написал в явном виде конструктор копий, для конструкторов и деструктора добавил вывод на экран сообщения об их работе, функцию operator++() сделал встроенной, и вроде бы всё. но после этого программа перестала работать. то есть она работает, но неправильно. выводит какое-то непонятное значение для объекта а. код:
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
#include <iostream>
using namespace std;
 
class Counter
{
public:
    Counter():
      itsVal(0)
      { cout << "In the Constructor" << endl; }
      
    Counter(const Counter&)
      { cout << "COPY Constructor" << endl; }
 
     ~Counter()
      { cout << "DESTRUCTOR" << endl; }
 
    int GetItsVal() const { return itsVal; }
    void SetItsVal(int x) { itsVal = x; }
    void Increment()      { ++itsVal; }
 
    const Counter& operator++()
    {
        ++itsVal;
        return *this;
    }
    
private:
    int itsVal;
};
 
int main()
{
    {
    Counter i;
    cout << "(1) The value of i is " << i.GetItsVal() << endl;
    i.Increment();
    cout << "(2) The value of i is " << i.GetItsVal() << endl;
    ++i;
    cout << "(3) The value of i is " << i.GetItsVal() << endl;
    Counter a = ++i;
    cout << "(4) The value of a: " << a.GetItsVal();
    cout << " (5) and i: " << i.GetItsVal() << endl;
    }
    system("pause");        
    return 0;     
}
Результат 2:
In the Constructor
(1) The value of i is 0
(2) The value of i is 1
(3) The value of i is 2
COPY Constructor
(4) The value of a: -858993460 (5) and i: 3
DESTRUCTOR
DESTRUCTOR
Для продолжения нажмите любую клавишу...


Почему значение а стало -858993460 ? должно же быть 3 вроде как. на несколько раз пересмотрел код, так и не пойму, где я ошибся. и компилятор молчит
Лучшие ответы (1)
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
13.07.2014, 12:10     Перегрузка оператора ++
Посмотрите здесь:

C++ Перегрузка оператора +=
Перегрузка оператора = C++
Перегрузка оператора * C++
Перегрузка оператора ~ C++
C++ Перегрузка оператора <<
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
cudi
0 / 0 / 2
Регистрация: 12.07.2014
Сообщений: 9
13.07.2014, 12:35     Перегрузка оператора ++ #2
Сообщение было отмечено автором темы, экспертом или модератором как ответ
Все дело в конструкторе копирования, если ты посмотришь на свой вывод, то увидишь:
COPY Constructor
Чтобы работало нужно либо удалить твой конструктор копирования, либо реализовать его правильно:
C++
1
2
3
4
5
Counter(const Counter& o)
{
    cout << "COPY Constructor" << endl; 
    this->itsVal = o.itsVal;
}
lest_
0 / 0 / 0
Регистрация: 28.11.2013
Сообщений: 94
13.07.2014, 12:44  [ТС]     Перегрузка оператора ++ #3
Цитата Сообщение от cudi Посмотреть сообщение
нужно либо удалить твой конструктор копирования, либо реализовать его правильно
спасибо. заработало
IGPIGP
Комп_Оратор)
 Аватар для IGPIGP
6176 / 2905 / 284
Регистрация: 04.12.2011
Сообщений: 7,731
Записей в блоге: 3
13.07.2014, 13:12     Перегрузка оператора ++ #4
Да пустой copy ctor. И ещё. Лучше объявить тип возвращаемого значения оператора преф++ неконстантной ссылкой, тогда можно инициализировать и переменные и ссылки и константные ссылки.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <iostream>
using namespace std;
struct Counter{
int i;
Counter(int a = 0 ) :i(a){}
//const Counter& operator++(){ ++i; return *this; }
Counter& operator++(){ ++i; return *this; }
Counter(Counter& rhs){i = rhs.i;}
};
 
int main(){
Counter a;
cout<<a.i<<endl;
cout<<(++a).i<<endl;
cout<<(++a).i<<endl;
cout<<(++a).i<<endl;
Counter c = ++a;//
const Counter& b = ++a;//
cout<<b.i<<endl;
cout << endl;
cin.get();
return 0 ;
}
Yandex
Объявления
13.07.2014, 13:12     Перегрузка оператора ++
Ответ Создать тему
Опции темы

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