Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
~GiPeRPrOgEr~
0 / 0 / 0
Регистрация: 21.12.2010
Сообщений: 85
#1

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

29.05.2011, 21:08. Просмотров 1006. Ответов 15
Метки нет (Все метки)

Все здравствуйте. У меня такая проблема, перегрузил основные операции для работы с одномернымм массивом, если массив статический все работает, но если динамический прога, вылетает. Мне кажется дело в конструкторе копирования, но его я не могу сообразить, помогите пожалуйста кто может
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();
}
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.05.2011, 21:08
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Перегрузка операторов и конструктор копироваия (C++):

Класс "Матрица": конструктор, деструктор и перегрузка операторов - C++
Матрица – KMatrix Методы: конструкторы, деструктор; операции: , =, ==, !=,...

Что такое "перегрузка операторов"? Каковы принципы работы перегруженных операторов и назначение указателя this - C++
Добрый день . Помогите понять принцип работы перегрузки операторов. объясните пожалуйста в зависимости от чего зависит агрумент при...

Перегрузка операторов - C++
#include &lt;stdio.h&gt; #include &lt;stdlib.h&gt; class matrix { protected: int **mass; int n; public: ...

Перегрузка операторов - C++
В программе нужно сделать перегрузку оператора умножения, не могу понять что и как делать... Помогите разобраться :) #include...

Перегрузка операторов - C++
Как правильно перегружать подобные операторы #include &quot;stdafx.h&quot; #include &lt;conio.h&gt; using namespace std; class Point{ public: ...

C++ перегрузка операторов - C++
Здраствуйте! Программа писать надо чтобы работало a=b=c и (a=b)=c. с уважением

15
lemegeton
2925 / 1354 / 135
Регистрация: 29.11.2010
Сообщений: 2,725
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;
        }

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

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

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

Не по теме:

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

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

0
kravam
быдлокодер
1700 / 887 / 45
Регистрация: 04.06.2008
Сообщений: 5,498
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
И всё, вот и поэтому память вылетает, понял? Это первая ошибка, её надо устранить, а потом смотреть чё дальше. А ты про конструктора пишешь...
1
Kastaneda
Jesus loves me
Эксперт С++
4688 / 2892 / 236
Регистрация: 12.12.2009
Сообщений: 7,353
Записей в блоге: 2
Завершенные тесты: 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;
...
1
kravam
быдлокодер
1700 / 887 / 45
Регистрация: 04.06.2008
Сообщений: 5,498
26.06.2011, 19:30 #13
Да ясно дело, вот я попытался сказать, как я бы искал ошибку. Вот пусть роет в направлении изменения n
0
~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 теперь ясно почему вылет
0
pito211
186 / 173 / 8
Регистрация: 22.03.2010
Сообщений: 612
27.06.2011, 09:09 #15
странно, под А память выделяется, а деструктор пустой
0
27.06.2011, 09:09
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
27.06.2011, 09:09
Привет! Вот еще темы с ответами:

Перегрузка операторов - C++
Здравствуйте уважаемые форумчане :) У меня имеется задание написать самодельный класс строки. Если с потоковым выводом я разобрался, то с...

Перегрузка операторов - C++
Добрый вечер. Не подскажите, в чём дело? #include &quot;stdafx.h&quot; #include &lt;iostream&gt; struct programTime { programTime() { ...

Перегрузка операторов - C++
Нужно для класса комплексных чисел перегрузить операторы. Односимвольные определить как дружественные, а двусимвольные, присваивание и...

Перегрузка операторов - C++
Помогите проверить и сделать надо 1) Базовый класс Строка Описать базовый класс СТРОКА. Обязательные члены класса: * указатель...


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

Или воспользуйтесь поиском по форуму:
15
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru