Перегрузка операторов при работе с матрицей

@rura · Регистрация: 08.01.2014

Author24 — интернет-сервис помощи студентам

Ребята, помогите пожалуйста исправить ошибку. У меня в программе вроде не работает два оператор-функции.
С ними работаю первый раз. За грубейшие ошибки извиняюсь.

Вот моя программа:

C++

#include "stdafx.h"
#include <iostream>
 
using namespace std;
 
class Matrix
{
    int long** mas;
    int size_x;
    int size_y;
 
    Matrix operator+ (const Matrix&);
    Matrix operator- (const Matrix&);
    void Proizvedenie(Matrix, Matrix);
    void Multiply_num(int);
 
    void init()
    {
        mas = new int long*[size_y];
        for (int i = 0; i < size_y; i++)
        {
            mas[i] = new int long[size_x];
        }
    }
 
    void fill()
    {
        for (int i = 0; i < size_y; i++)
        {
            for (int j = 0; j < size_x; j++)
            {
                mas[i][j] = rand() % 10;
            }
        }
    }
 
public:
 
    Matrix()
    {
        size_x = 0;
        size_y = 0;
    }
 
    Matrix(int x, int y)
    {
        size_x = x;
        size_y = y;
        init();
        fill();
    };
 
    ~Matrix()
    {
        for (int i = 0; i < size_y; i++)
        {
            delete[] mas[i];
        }
        delete[] mas;
 
    }
 
    void print()
    {
        for (int i = 0; i < size_y; i++)
        {
            for (int j = 0; j < size_x; j++) {
                cout << mas[i][j] << ' ';
            }
            cout << endl;
        }
    }
};
 
Matrix Matrix::operator+(const Matrix &rhs)
{
    Matrix result(*this);
 
    for (size_t i = 0; i < 3; ++i)
    for (size_t j = 0; j < 3; ++j)
        result.mas[i][j] += rhs.mas[i][j];
 
    return result;
}
 
Matrix Matrix::operator-(const Matrix &rhs)
{
    Matrix result(*this);
 
    for (size_t i = 0; i < 3; ++i)
    for (size_t j = 0; j < 3; ++j)
        result.mas[i][j] -= rhs.mas[i][j];
 
    return result;
}
 
/*coord coord::operator + (coord ob2)
{
    coord temp;
    temp.x = x + ob2.x;
    temp.у = у + оb2.у;
    return temp;
}
*/
 
 
int main(){
 
    setlocale(LC_ALL, "Russian");
    int x, y;
    cout << "Введите количество строк: ";
    cin >> y;
    cout << "Введите количество столбцов: ";
    cin >> x;
 
    Matrix ob3;
    Matrix ob1(x, y);
    Matrix ob2(x, y);
 
    ob1.print();
    cout << endl;
    ob2.print();
 
    ob3 = ob1 + ob2;
    ob3.print();
 
    ob3 = ob1 - ob2;
    ob3.print();
 
 
    system("pause");
    return 0;
}

@rura · 18.06.2014, 10:38 **[ТС]**

Нет

C++

#include "stdafx.h"
#include <iostream>
 
using namespace std;
 
class Matrix
{
    int long** mas;
    int size_x;
    int size_y;
 
 
 
    void init()
    {
        mas = new int long*[size_y];
        for (int i = 0; i < size_y; i++)
        {
            mas[i] = new int long[size_x];
        }
    }
 
    void fill()
    {
        for (int i = 0; i < size_y; i++)
        {
            for (int j = 0; j < size_x; j++)
            {
                mas[i][j] = rand() % 10;
            }
        }
    }
 
public:
 
    Matrix() :mas(NULL)
    {
        size_x = 0;
        size_y = 0;
    }
 
    Matrix(int x, int y)
    {
        size_x = x;
        size_y = y;
        init();
        fill();
    }
 
    Matrix(const Matrix& rhs)
    {
        size_x = rhs.size_x;
        size_y = rhs.size_y;
        init();
        for (int i = 0; i < size_y; i++)
        {
            for (int j = 0; j < size_x; j++)
            {
                mas[i][j] = rhs.mas[i][j];
            }
        }
    }
 
    ~Matrix()
    {
        for (int i = 0; i < size_y; i++)
        {
            delete[] mas[i];
        }
        delete[] mas;
 
    }
 
    void print()
    {
        for (int i = 0; i < size_y; i++)
        {
            for (int j = 0; j < size_x; j++) {
                cout << mas[i][j] << ' ';
            }
            cout << endl;
        }
    }
    const Matrix operator+ (const Matrix&);
    const Matrix operator- (const Matrix&);
    const Matrix & operator=(const Matrix&);
    const Matrix operator*(int f);
    void Multiply_num(int);
    const Matrix operator*(const Matrix &rhs)
};
 
const Matrix Matrix::operator+(const Matrix &rhs)
{
    Matrix result(rhs.size_x, rhs.size_y);
 
    for (int i = 0; i < size_y; ++i)
    for (int j = 0; j < size_x; ++j)
        result.mas[i][j] = mas[i][j] + rhs.mas[i][j];
 
    return result;
}
 
const Matrix Matrix::operator-(const Matrix &rhs)
{
    Matrix result(rhs.size_x, rhs.size_y);
 
    for (int i = 0; i < size_y; ++i)
    for (int j = 0; j < size_x; ++j)
        result.mas[i][j] = mas[i][j] - rhs.mas[i][j];
 
    return result;
}
const Matrix & Matrix::operator=(const Matrix &rhs)
{
    if (this == &rhs) return *this;
 
 
    if (!mas)
    {
 
        init();
    }
    else
    {
        for (int i = 0; i < size_y; i++)
        {
            delete[] mas[i];
        }
        delete[] mas;
        size_x = rhs.size_x;
        size_y = rhs.size_y;
        init();
    }
    for (int i = 0; i < size_y; ++i)
    for (int j = 0; j < size_x; ++j)
        mas[i][j] = rhs.mas[i][j];
 
    return *this;
}
 
const Matrix Matrix::operator*(int f)
{
    Matrix result(size_x, size_y);
 
    for (int i = 0; i < size_y; ++i)
    for (int j = 0; j < size_x; ++j)
        result.mas[i][j] = mas[i][j] * f;
 
    return result;
}
 
const Matrix Matrix::operator*(const Matrix &rhs)
{
    Matrix result(rhs.size_x, size_y);
 
    for (int i = 0; i < result.size_y; ++i)
    for (int j = 0; j < result.size_x; ++j)
    {
        result.mas[i][j] = 0;
        for (int k = 0; k < size_x; ++k)
            result.mas[i][j] += mas[i][k] * rhs.mas[k][j];
    }
 
    return result;
}
 
int main(){
 
    setlocale(LC_ALL, "Russian");
    int x, y;
    cout << "Введите количество строк: ";
    cin >> y;
    cout << "Введите количество столбцов: ";
    cin >> x;
 
    Matrix ob3;
    Matrix ob1(x, y);
    Matrix ob2(x, y);
 
    cout << "Первая матрица А:"<< endl;
    ob1.print();
    cout << endl;
    cout << "Вторая матрица В:" << endl;
    ob2.print();
    cout << endl;
 
    cout << "Матрица А + матрица В:" << endl;
    ob3 = ob1 + ob2;
    ob3.print();
    cout << endl;
        
    cout << "Матрица А - матрица В:" << endl;
    ob3 = ob1 - ob2;
    ob3.print();
    cout << endl;
 
    cout << "Матрица А умножиется на число 3:" << endl;
    ob3 = ob1 * 3;
    ob3.print();
    cout << endl;
 
    cout << "Матрица А умножиется на матрица В:" << endl;
    ob3 = ob1 * ob2;
    ob3.print();
    cout << endl;
 
    system("pause");
    return 0;
}

@rura · 18.06.2014, 10:40 **[ТС]**

Вот наглядно..

@aLarman · 18.06.2014, 10:42

Сообщение от rura

if (!mas)
{
size_x = rhs.size_x;
size_y = rhs.size_y;
init();
}

в операторе =

@rura · 18.06.2014, 10:45 **[ТС]**

Ого!)) Спасибо!)))
А не подскажите пожалуйста для чего этот оператор :

C++

1	const Matrix & Matrix::operator=(const Matrix &rhs)

?

@aLarman · 18.06.2014, 10:53

оператор присваивающего копирования, необХодим тут

C++

1	ob3 = ob1 + ob2;

после вычисления суммы копирует значение суммарного объекта в объект ob3

Добавлено через 1 минуту
если его не написать он также будет реализован компилятором как и конструктор копирования, и опять же в случае работы с указателями, он будет некорректно работать, поэтому его необходимо руками описать

@rura · 18.06.2014, 10:56 **[ТС]**

Поняла, спасибо.

А если у меня в задании написано: "Конструктор без параметров выделяет место для
одного элемента и инициализирует его в ноль. Конструктор с одним параметром, - размер массива, -
выделяет место и инициализирует номером в массиве, конструктор с двумя параметрами выделяет
место (первый аргумент) и инициализирует вторым аргументом."
Конструкторы нужно изменить и перегрузить?

@aLarman · 18.06.2014, 10:58

Сообщение от rura

А если у меня в задании написано:

то у Вас не матрица а вектор

@rura · 18.06.2014, 11:00 **[ТС]**

Где? В задании?

@aLarman · 18.06.2014, 11:01

Сообщение от rura

размер массива

таки да, массив, вектор,...но не матрица

@rura · 18.06.2014, 11:05 **[ТС]**

Блин

А что мне делать, если задание такое?

Завтра сдавать, а препод меня съест...

@aLarman · 18.06.2014, 11:10

убрать одну размерность

Добавлено через 18 секунд

Не по теме:

Сообщение от rura

а препод меня съест...

жесть хД

Добавлено через 4 минуты

C++

class Vector
{
    int long* mas;
    int size_x;
public:
 
    Vector():mas(NULL)
    {
        mas = new long[1];
        mas[0] = 0;
    }
 
    Vector(int x):mas(NULL)
    {
        size_x = x;
        mas = new long[size_x];
        for(size_t i=0; i < size_x; i++)
        {
            mas[i] = i;
        }
    }
    Vector(int x, int g):mas(NULL)
    {
        size_x = x;
        mas = new long[size_x];
        for(size_t i=0; i < size_x; i++)
        {
            mas[i] = g;
        }
    }
};

какие метода еще надо реализовать?

@taras atavin · 18.06.2014, 11:15

Нда. Это надо додуматься для матрицы общего вида перегружать сложение матриц строго 3x3. Такую кашу не всякий повар осилит.

@rura · 18.06.2014, 12:25 **[ТС]**

Вот, я только не пойму откуда последняя матрица появляется...

@aLarman · 18.06.2014, 12:56

умножение матриц, Вы что не знаете как оно делается?

@rura · 18.06.2014, 12:59 **[ТС]**

Знаю, но по коду сначала не поняла, что это за функция..

@rura 0 / 0 / 0 Регистрация: 08.01.2014 Сообщений: 153
	18.06.2014, 10:40 [ТС]	22
	Вот наглядно.. Миниатюры 0

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	18.06.2014, 10:42	23
	Сообщение было отмечено rura как решение Решение Сообщение от rura if (!mas) { size_x = rhs.size_x; size_y = rhs.size_y; init(); } в операторе = 1

@rura 0 / 0 / 0 Регистрация: 08.01.2014 Сообщений: 153
	18.06.2014, 10:56 [ТС]	26
	Поняла, спасибо. А если у меня в задании написано: "Конструктор без параметров выделяет место для одного элемента и инициализирует его в ноль. Конструктор с одним параметром, - размер массива, - выделяет место и инициализирует номером в массиве, конструктор с двумя параметрами выделяет место (первый аргумент) и инициализирует вторым аргументом." Конструкторы нужно изменить и перегрузить? 0

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	18.06.2014, 10:58	27
	Сообщение от rura А если у меня в задании написано: то у Вас не матрица а вектор 0

@rura 0 / 0 / 0 Регистрация: 08.01.2014 Сообщений: 153
	18.06.2014, 11:00 [ТС]	28
	Где? В задании? 0

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	18.06.2014, 11:01	29
	Сообщение от rura размер массива таки да, массив, вектор,...но не матрица 0

@rura 0 / 0 / 0 Регистрация: 08.01.2014 Сообщений: 153
	18.06.2014, 11:05 [ТС]	30
	Блин А что мне делать, если задание такое? Завтра сдавать, а препод меня съест... 0

@taras atavin 4226 / 1795 / 211 Регистрация: 24.11.2009 Сообщений: 27,562
	18.06.2014, 11:15	32
	Нда. Это надо додуматься для матрицы общего вида перегружать сложение матриц строго 3x3. Такую кашу не всякий повар осилит. 0

@rura 0 / 0 / 0 Регистрация: 08.01.2014 Сообщений: 153
	18.06.2014, 12:25 [ТС]	33
	Вот, я только не пойму откуда последняя матрица появляется... Миниатюры 0

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	18.06.2014, 12:56	34
	умножение матриц, Вы что не знаете как оно делается? 0

@rura 0 / 0 / 0 Регистрация: 08.01.2014 Сообщений: 153
	18.06.2014, 12:59 [ТС]	35
	Знаю, но по коду сначала не поняла, что это за функция.. 0

Перегрузка операторов при работе с матрицей

Решение