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

Перегрузка операторов и конструктор копироваия - C++

Восстановить пароль Регистрация
 
~GiPeRPrOgEr~
0 / 0 / 0
Регистрация: 21.12.2010
Сообщений: 85
29.05.2011, 21:08     Перегрузка операторов и конструктор копироваия #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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
#include <iostream.h>
#include<windows.h>
#include<vector>
 
using namespace std;
 
char Rustext[256];                                                        
 char *Rus(const char*text)                                               
 {
    CharToOem(text, Rustext);
    return Rustext;
 }
 
class Overload
{
private:
    int x,A[100],S,i,j,n;
    vector <int>  Vec;
public:
    void print(){
        for(i=0;i<n;i++)
            cout<<A[i]<<'\t';
        
        cout<<endl<<endl;
    }
 
    int get(){
        return *A;
    }
 
 
    Overload (){
    }
    Overload (int a){
        n=a;
 
    for(i=0;i<n;i++)
            A[i]=i;
 
    }
    ~Overload (){
    }
 
    Overload operator +(Overload obj){
        cout<<Rus("Сработала перегруженная операция сложения для двух матриц.")<<endl; 
        Overload B;
        for(i=0;i<n;i++)
        B.A[i]=A[i]+obj.A[i];
        return B;
    }
    Overload operator *(Overload obj){
        cout<<Rus("Сработала перегруженная операция умножения.")<<endl; 
        Overload B;
 
        for(int i=0; i<n; i++)
               B.A[i]=A[i]*obj.A[i];
           
       
 
        return B;
    }
    Overload operator -(Overload obj){
        cout<<Rus("Сработала перегруженная операция вычитания.")<<endl; 
        Overload B;
        for(i=0;i<n;i++)
        B.A[i]=A[i]-obj.A[i];
        return B;
    }
    Overload operator /(Overload obj){
        cout<<Rus("Сработала перегруженная операция деления.")<<endl; 
        Overload B;
        for(i=0;i<n;i++)
        B.A[i]=A[i]-obj.A[i];
        return B;
    }
};
 
void main (){
    char d;
    int a,b;
    cout<<Rus("На сколько эдементов вы хотите создать вектор?")<<endl;
    cin>>a;
    Overload A(a), B(a),C;
    cout<<Rus("Вектор №1 имеет следующий вид")<<endl;
    A.print();
    cout<<Rus("Вектор №2 имеет следующий вид")<<endl;
    B.print();
 
    cout<<Rus("Какую операцию вы хотите совершить")<<endl;
    cout<<Rus("Для выхода введите любой символ, кроме основных арифметических операций")<<endl;
    cin>>d;
    switch (d){
    case '+':
         C=A+B; break;
    case '-': 
        C=A-B; break;
    case '*':
         C=A*B; break;
    case '/':
     C=A/B; break;
    }
    C.print();
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.05.2011, 21:08     Перегрузка операторов и конструктор копироваия
Посмотрите здесь:

C++ перегрузка операторов C++
C++ Перегрузка операторов
перегрузка операторов C++
Перегрузка операторов. C++ C++
Перегрузка операторов C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
lemegeton
 Аватар для lemegeton
2909 / 1338 / 133
Регистрация: 29.11.2010
Сообщений: 2,720
29.05.2011, 23:26     Перегрузка операторов и конструктор копироваия #2
Эмм... Не вижу конструктора копирования, не вижу, где бы он мог понадобится в коде.
Покажите, что-ли, в каком виде оно не работает и какие и где выдает ошибки.

Не по теме:

И пожалуйста-пожалуйста, заканчивайте с неговорящими идентификаторами. Вырвиглазный же код.

C++
1
2
3
4
5
6
7
Overload operator +(Overload obj){
                cout<<Rus("Сработала перегруженная операция сложения для двух матриц.")<<endl; 
                Overload B;
                for(i=0;i<n;i++)
                B.A[i]=A[i]+obj.A[i];
                return B;
        }

~GiPeRPrOgEr~
0 / 0 / 0
Регистрация: 21.12.2010
Сообщений: 85
31.05.2011, 06:16  [ТС]     Перегрузка операторов и конструктор копироваия #3
В данном примере, массив у меня статичесикий на 100 элементов A[100], но если я делаю его динамическим, *A, и выделяю память A=new int [n], то как только срабатывает одна из перегруженных операций, программа вылетает.Вот в этом проблема.
no0ker
100 / 87 / 4
Регистрация: 17.12.2010
Сообщений: 416
31.05.2011, 08:24     Перегрузка операторов и конструктор копироваия #4
у вас в классе много полей.
неужели при создании объекта их не нужно чем то заполнять?

C++
1
2
        Overload (){
        }
очень странный конструктор.
Kastaneda
Модератор
Эксперт С++
 Аватар для Kastaneda
4236 / 2769 / 218
Регистрация: 12.12.2009
Сообщений: 7,104
Записей в блоге: 1
Завершенные тесты: 1
31.05.2011, 09:44     Перегрузка операторов и конструктор копироваия #5
Цитата Сообщение от lemegeton Посмотреть сообщение
Не вижу конструктора копирования, не вижу, где бы он мог понадобится в коде.
Все перегруженные операторы принимают аргумент по значению, а не по ссылке. Поэтому вызывается конструктор копирования, при завершении работы оператора объект разрушается, отсюда и проблема с динамической памятью.
~GiPeRPrOgEr~
0 / 0 / 0
Регистрация: 21.12.2010
Сообщений: 85
26.06.2011, 16:30  [ТС]     Перегрузка операторов и конструктор копироваия #6
Kastaneda- ну и какое решение вы посоветуете???
Jupiter
Каратель
Эксперт C++
6542 / 3962 / 226
Регистрация: 26.03.2010
Сообщений: 9,273
Записей в блоге: 1
Завершенные тесты: 2
26.06.2011, 16:37     Перегрузка операторов и конструктор копироваия #7
~GiPeRPrOgEr~, для начала выложите код в котором у вас вместо статического массива, динамический int* A
Kastaneda
Модератор
Эксперт С++
 Аватар для Kastaneda
4236 / 2769 / 218
Регистрация: 12.12.2009
Сообщений: 7,104
Записей в блоге: 1
Завершенные тесты: 1
26.06.2011, 17:01     Перегрузка операторов и конструктор копироваия #8
Цитата Сообщение от ~GiPeRPrOgEr~ Посмотреть сообщение
Kastaneda- ну и какое решение вы посоветуете???
Так вот же:

Цитата Сообщение от Kastaneda Посмотреть сообщение
Все перегруженные операторы принимают аргумент по значению, а не по ссылке. Поэтому вызывается конструктор копирования, при завершении работы оператора объект разрушается, отсюда и проблема с динамической памятью.
Jupiter
Каратель
Эксперт C++
6542 / 3962 / 226
Регистрация: 26.03.2010
Сообщений: 9,273
Записей в блоге: 1
Завершенные тесты: 2
26.06.2011, 17:03     Перегрузка операторов и конструктор копироваия #9
Kastaneda, это похоже на перегрузку +=
Kastaneda
26.06.2011, 17:05
  #10

Не по теме:

Maxwe11, похоже ты увидел мой код до редактирования поста... я только что проснулся - туплю, код удалил сразу же)

Цитата Сообщение от Maxwe11 Посмотреть сообщение
это похоже на перегрузку +=
ага, оно и было)

kravam
быдлокодер
 Аватар для kravam
1512 / 872 / 44
Регистрация: 04.06.2008
Сообщений: 5,267
26.06.2011, 19:19     Перегрузка операторов и конструктор копироваия #11
ТС, всё это время я занимался только тем, что сокращал твой код. Вот как должен выглядет код, который ты выкладываешь в первом приближении:
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
#include <iostream>
#include<windows.h>
#include<vector>
 
using namespace std;
 
char Rustext[256];                                                        
 char *Rus(const char*text)                                               
 {
        CharToOem(text, Rustext);
        return Rustext;
 }
 
class Overload
{
private:
        int x,A[100],S,i,j,n;
        vector <int>  Vec;
public:
        void print(){
                for(i=0;i<n;i++)
                        cout<<A[i]<<'\t';
                
                cout<<endl<<endl;
        }
 
        Overload (){
        }
        
        Overload (int a){
                n=a;
 
        for(i=0;i<n;i++)
                        A[i]=i;
 
        }
        ~Overload (){
        }
 
        Overload operator +(Overload obj){
                cout<<Rus("Сработала перегруженная операция сложения для двух матриц.")<<endl; 
                Overload B;
                for(i=0;i<n;i++)
                B.A[i]=A[i]+obj.A[i];
                return B;
        }
};
 
int main (){
        int a,b;
        Overload A(2), B(2),C;
        C=A+B; 
        C.print();
}
Понял? Мне абсолютно по барабану перегрузки * и /, если они не используются; мне неохота о них спотыкаться, мне также незачем спотыкаться о ввод переменной a. С этим кодом уже можно работать. Уважай блин других.

Добавлено через 12 минут
Так, а теперь смотри, когда ты так сделал, легко можно и самому работать. Ага, видим, что print работет чёрт знает сколько раз. Легко и просто в этом коде увидеть, что это количество зависит от переменной n. Чё делаем? Правильно, смотрим, чё из себя представляет перменная n при выводе. Пишем:
C++
1
2
3
4
5
                for(i=0;i<n;i++) {
                        cout<<A[i]<<'\t';
                        printf ("n= %d\n", n);
                        getchar ();
                }
Я, например, любуюсь
Bash
1
2
Сработала перегруженная операция сложения для двух матриц.
0       n= 2147319808
И всё, вот и поэтому память вылетает, понял? Это первая ошибка, её надо устранить, а потом смотреть чё дальше. А ты про конструктора пишешь...
Kastaneda
Модератор
Эксперт С++
 Аватар для Kastaneda
4236 / 2769 / 218
Регистрация: 12.12.2009
Сообщений: 7,104
Записей в блоге: 1
Завершенные тесты: 1
26.06.2011, 19:27     Перегрузка операторов и конструктор копироваия #12
Цитата Сообщение от kravam Посмотреть сообщение
Я, например, любуюсь
Bash
1
2
Сработала перегруженная операция сложения для двух матриц.
 n= 2147319808
2ТС
Что естественно, при:
C++
1
2
3
4
5
6
7
8
9
Overload (){
        }
 
 
 
 Overload operator +(Overload obj){
                cout<<Rus("Сработала перегруженная операция сложения для двух матриц.")<<endl; 
                Overload B;
...
kravam
быдлокодер
 Аватар для kravam
1512 / 872 / 44
Регистрация: 04.06.2008
Сообщений: 5,267
26.06.2011, 19:30     Перегрузка операторов и конструктор копироваия #13
Да ясно дело, вот я попытался сказать, как я бы искал ошибку. Вот пусть роет в направлении изменения n
~GiPeRPrOgEr~
0 / 0 / 0
Регистрация: 21.12.2010
Сообщений: 85
27.06.2011, 08:55  [ТС]     Перегрузка операторов и конструктор копироваия #14
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
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
#include <iostream.h>
#include<windows.h>
#include<vector>
 
using namespace std;
 
char Rustext[256];                                                        
 char *Rus(const char*text)                                               
 {
        CharToOem(text, Rustext);
        return Rustext;
 }
 
class Overload
{
private:
        int x,*A,S,i,j,n;
        vector <int>  Vec;
public:
        void print(){
                for(i=0;i<n;i++)
                        cout<<A[i]<<'\t';
                
                cout<<endl<<endl;
        }
 
        int get(){
                return *A;
        }
 
 
        Overload (){
        }
        Overload (int a){
            n=a;
                A = new int [n];
 
        for(i=0;i<n;i++)
                        A[i]=i;
 
        }
        ~Overload (){
        }
 
        Overload operator +(Overload obj){
                cout<<Rus("ÑðГ*áîòГ*Г«Г* ïåðåãðóæåГ*Г*Г*Гї îïåðГ*öèÿ ñëîæåГ*ГЁГї äëÿ äâóõ Г¬Г*òðèö.")<<endl; 
                Overload B;
                for(i=0;i<n;i++)
                B.A[i]=A[i]+obj.A[i];
                return B;
        }
        Overload operator *(Overload obj){
                cout<<Rus("ÑðГ*áîòГ*Г«Г* ïåðåãðóæåГ*Г*Г*Гї îïåðГ*öèÿ ГіГ¬Г*îæåГ*ГЁГї.")<<endl; 
                Overload B;
 
                for(int i=0; i<n; i++)
                           B.A[i]=A[i]*obj.A[i];
                   
           
 
                return B;
        }
        Overload operator -(Overload obj){
                cout<<Rus("ÑðГ*áîòГ*Г«Г* ïåðåãðóæåГ*Г*Г*Гї îïåðГ*öèÿ âû÷èòГ*Г*ГЁГї.")<<endl; 
                Overload B;
                for(i=0;i<n;i++)
                B.A[i]=A[i]-obj.A[i];
                return B;
        }
        Overload operator /(Overload obj){
                cout<<Rus("ÑðГ*áîòГ*Г«Г* ïåðåãðóæåГ*Г*Г*Гї îïåðГ*öèÿ äåëåГ*ГЁГї.")<<endl; 
                Overload B;
                for(i=0;i<n;i++)
                B.A[i]=A[i]-obj.A[i];
                return B;
        }
};
 
void main (){
        char d;
        int a,b;
        cout<<Rus("ГЌГ* ñêîëüêî ýäåìåГ*òîâ ГўГ» õîòèòå ñîçäГ*ГІГј âåêòîð?")<<endl;
        cin>>a;
        Overload A(a), B(a),C;
        cout<<Rus("Âåêòîð В№1 èìååò ñëåäóþùèé ГўГЁГ¤")<<endl;
        A.print();
        cout<<Rus("Âåêòîð В№2 èìååò ñëåäóþùèé ГўГЁГ¤")<<endl;
        B.print();
 
    cout<<Rus("ГЉГ*ГЄГіГѕ îïåðГ*öèþ ГўГ» õîòèòå ñîâåðøèòü")<<endl;
        cout<<Rus("Äëÿ âûõîäГ* ââåäèòå ëþáîé ñèìâîë, êðîìå îñГ*îâГ*ûõ Г*ðèôìåòè÷åñêèõ îïåðГ*öèé")<<endl;
        cin>>d;
        switch (d){
        case '+':
                 C=A+B; break;
        case '-': 
                C=A-B; break;
        case '*':
                 C=A*B; break;
        case '/':
         C=A/B; break;
        }
        C.print();
}
Вот он код с динамикой

Добавлено через 2 минуты
Вот оно что, спасибо большое Kastaneda теперь ясно почему вылет
pito211
 Аватар для pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
27.06.2011, 09:09     Перегрузка операторов и конструктор копироваия #15
странно, под А память выделяется, а деструктор пустой
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
27.06.2011, 13:18     Перегрузка операторов и конструктор копироваия
Еще ссылки по теме:

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

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

Или воспользуйтесь поиском по форуму:
Jupiter
Каратель
Эксперт C++
6542 / 3962 / 226
Регистрация: 26.03.2010
Сообщений: 9,273
Записей в блоге: 1
Завершенные тесты: 2
27.06.2011, 13:18     Перегрузка операторов и конструктор копироваия #16
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
61
62
63
64
65
66
67
68
69
class Overload {
    int* A;
    int size;
public:
    explicit Overload (int Size = 0)
    {
        size = Size;
        A = new int [size];
        for(int i = 0; i < size; i++)
        A[i] = i;
    }
    
    Overload(const Overload & ob)
    {
        size = ob.size;
        A = new int [size];
        for(int i = 0; i < size; i++)
        A[i] = ob.A[i];
    }
 
    ~Overload ()
    {
        delete [] A;
    }
    
    friend std::ostream & operator << (std::ostream & out, const Overload & ob)
    {
        for(int i = 0; i < ob.size; i++)
         out << ob.A[i] << ' ';
        return out;
    }
    
    int get()
    {
        return *A;
    }
 
    Overload operator + (const Overload & obj) const
    {
        Overload B;
        for(int i = 0; i < size; i++)
            B.A[i] = A[i] + obj.A[i];
        return B;
    }
    
    Overload operator * (const Overload & obj) const
    {
        Overload B;
        for(int i = 0; i < size; i++)
               B.A[i] = A[i] * obj.A[i];
        return B;
    }
    
    Overload operator - (const Overload & obj) const
    {
        Overload B;
        for(int i = 0; i < size; i++)
    B.A[i] = A[i] - obj.A[i];
        return B;
    }
 
    Overload operator / (const Overload & obj) const
    {
        Overload B;
        for(int i = 0; i < size; i++)
    B.A[i] = A[i] - obj.A[i];
        return B;
    }
};
Yandex
Объявления
27.06.2011, 13:18     Перегрузка операторов и конструктор копироваия
Ответ Создать тему
Опции темы

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