Проблема с конструктором копирования в C++

среда, 08 октября 2014

14:18

Проблема с конструктором копирования в C++

все записи пользователя в сообществе Mik94

Добрый день. У меня проблема с конструктором копирования. При работе программы происходит ошибка: сегментация памяти
Не знаю в чем проблема. Пожалуйста помогите!
#include "iostream"
using namespace std;

class Matrix {
    int n;
    int m;
    public:int**mas;
    public:
    Matrix();
    Matrix(int a, int b);
    Matrix(const Matrix &A);
    friend istream & operator >> (istream &in, Matrix &A);
    friend ostream & operator << (ostream &out, Matrix &A);
    Matrix & operator + (Matrix &A);
};
//Файл Matrix.cpp
#include "Matrix.h"
#include "iostream"
using namespace std;

Matrix::Matrix() {
    cout << "Введите кол-во строк: ";
    cin >> n;
    cout << "\n Введите кол-во столбцов: ";
    cin >> m;
    mas = new int*[n];
    for(int i=0; i<n; i++) 
	mas[i] = new int[m];
}
Matrix::Matrix(const Matrix &A) {
    //n = A.n;
    //m = A.m;
    mas = new int*[A.n];
    for(int i=0; i<A.n; i++) 
	mas[i] = new int[A.m];
	for(int i=0; i<A.n; i++)
	    for(int j=0; j<A.m; j++)
		mas[i][j] = A.mas[i][j];
}
Matrix::Matrix(int a, int b) {
    n = a;
    m = b;
    mas = new int*[n];
    for(int i=0; i<n; i++)
	mas[i] = new int[m];
}
istream & operator >> (istream &in, Matrix &A) {
    for(int i=0; i<A.n; i++)
	for(int j=0; j<A.m; j++) {
	    cout << "Введите " << i << " " << j << " элемент: ";
	    cin >> A.mas[i][j];
	}
}
ostream & operator << (ostream &out, Matrix &A) {
    for(int i=0; i<A.n; i++) {
	for(int j=0; j<A.m; j++) {
	    cout << A.mas[i][j];
	}
	cout << endl;
    }
}
Matrix &Matrix::operator + (Matrix &A) {
    Matrix V(n, m);
    for(int i=0; i<n; i++)
	for(int j=0; j<m; j++)
	    V.mas[i][j] = mas[i][j] + A.mas[i][j];
    return V;
}
int main() {
    Matrix A;
    cin >> A;
    Matrix B = A+A;
    cout << A;
    cout << B;
    return 0;
}

@темы: C++

URL

Сегодня самый ужасный в мире день... Может быть... ... http://www.synergetic.ru/sections/i...pitza/index.htm Кто-нибудь может мне подсказать, как установить пятый дел...

Когда ты понимаешь, ты живешь среди родни. Когда ты не... Сегодня на одной из улиц увидела здание с забором метра 4... http://www.d-kitchen.com/ Current music: Никольский,...

Комментарии

08.10.2014 в 14:46

> mas = new int*[n];
Маловато будет.
Матрица n*m, а памяти только на строки.
А если хочется, чтобы кто-то читал больше, чем отдельные куски текста - есть тег CODE.

URL

08.10.2014 в 16:11

Mik94

Извиняюсь, некорректно выложил код

URL

08.10.2014 в 18:17

reinar➜~/tmp» g++ -Wall test.cpp -o test
test.cpp: In function ‘std::istream& operator>>(std::istream&, Matrix&)’:
test.cpp:50:5: warning: no return statement in function returning non-void [-Wreturn-type]
     }
     ^
test.cpp: In function ‘std::ostream& operator<<(std::ostream&, Matrix&)’:
test.cpp:58:5: warning: no return statement in function returning non-void [-Wreturn-type]
     }
     ^
test.cpp: In member function ‘Matrix& Matrix::operator+(Matrix&)’:
test.cpp:60:12: warning: reference to local variable ‘V’ returned [-Wreturn-local-addr]
     Matrix V(n, m);
            ^

Все ответы тут. А сегфолтится из-за третьего варнинга.

URL

10.10.2014 в 03:53

Mik94

Спасибо.

URL


Запомнить

Проблема с конструктором копирования в C++

ru_programming