Ошибки перегрузки операторов для простого матричного класса 3x3 - PullRequest
Новая
       14

Ошибки перегрузки операторов для простого матричного класса 3x3

0 голосов
/ 30 января 2019

Итак, у меня есть это как мой файл class.cpp 

#include "matrixType3.h"
#include <iostream>
#include <iomanip>
#include <cmath>

using namespace std;

void matrixType3::setElement(int row, int col, int newvalue)
{
    matrix[row][col] = newvalue;
}

double matrixType3::getElement(int row, int col)
{
    int j = matrix[row][col];
    return j;
}

void matrixType3::setMatrix(int x1, int x2, int x3, int x4, int x5, int x6, int x7, int x8, int x9)
{
    matrix[0][0] = x1;
    matrix[0][1] = x2;
    matrix[0][2] = x3;
    matrix[1][0] = x4;
    matrix[1][1] = x5;
    matrix[1][2] = x6;
    matrix[2][0] = x7;
    matrix[2][1] = x8;
    matrix[2][2] = x9;
}

void matrixType3::display()
{
    for (int i = 0; i < 3; i++)
    {
        for (int r = 0; r < 3; r++)
        {
            cout << matrix[i][r] << " ";
        }
        cout << endl;
    }
    cout << endl;
}

void matrixType3::displayRow(int row)
{
    for (int i = 0; i < 3; i++)
    {
        cout << matrix[row][i] << " ";
    }
    cout << endl;
}

void matrixType3::displayColumn(int col)
{
    for (int i = 0; i < 3; i++)
    {
        cout << matrix[i][col] << endl;
    }
}

void matrixType3::multipleMatrix(int c)
{
    for (int i = 0; i < 3; i++)
    {
        for (int r = 0; r < 3; r++)
        {
            int j = matrix[i][r];
            matrix[i][r] = j * c;
        }
    }
}

matrixType3 matrixType3::operator=(const matrixType3&mat) const
{
    matrixType3 tempMatrix;
    for (int i = 0; i < 3; i++)
    {
        for (int r = 0; r < 3; r++)
        {
            tempMatrix.matrix[i][r] = mat.matrix[i][r];
        }
    }
    return tempMatrix;
}

matrixType3 matrixType3::operator+(const matrixType3& mat) const
{
    matrixType3 tempMatrix;
    for (int i = 0; i < 3; i++)
    {
        for (int r = 0; r < 3; r++)
        {
            tempMatrix.matrix[i][r] = mat.matrix[i][r] + matrix[i][r];
        }
    }
    return tempMatrix;
}

matrixType3 matrixType3::operator-(const matrixType3& mat) const
{
    matrixType3 tempMatrix;
    for (int i = 0; i < 3; i++)
    {
        for (int r = 0; r < 3; r++)
        {
            tempMatrix.matrix[i][r] = matrix[i][r] - mat.matrix[i][r];
        }
    }
    return tempMatrix;
}

matrixType3 matrixType3::operator*(const matrixType3& mat) const
{
    matrixType3 tempMatrix;
    for (int i = 0; i < 3; i++)
    {
        for (int r = 0; r < 3; r++)
        {
            for (int k = 0; k < 3; k++)
            {
                tempMatrix.matrix[i][r] = tempMatrix.matrix[i][r] + matrix[i][k] * mat.matrix[k][r];
            }
        }
    }
    return tempMatrix;
}

matrixType3 matrixType3::operator*(const int&num) const
{
    matrixType3 tempMatrix;
    for (int i = 0; i < 3; i++)
    {
        for (int j = 0; j < 3; j++)
        {
            tempMatrix.matrix[i][j] = matrix[i][j] * num;
        }
    }
    return tempMatrix;
}

bool matrixType3::operator==(const matrixType3& mat) const
{
    for (int i = 0; i < 3; i++)
    {
        for (int r = 0; r < 3; r++)
        {
            if (matrix[i][r] != mat.matrix[i][r])
                return false;
        }
    }
    return true;
}

matrixType3::matrixType3()
{
    for (int i = 0; i < 3; i++)
    {
        for (int c = 0; c < 3; c++)
        {
            matrix[i][c] = 0;
        }
    }
}

matrixType3::matrixType3(int x1, int x2, int x3, int x4, int x5, int x6, int x7, int x8, int x9)
{
    matrix[0][0] = x1;
    matrix[0][1] = x2;
    matrix[0][2] = x3;
    matrix[1][0] = x4;
    matrix[1][1] = x5;
    matrix[1][2] = x6;
    matrix[2][0] = x7;
    matrix[2][1] = x8;
    matrix[2][2] = x9;
}

matrixType3::~matrixType3()
{

}

Однако моя проблема в том, что функции перегрузки оператора не работают.Файл source.cpp выполняется для всех других функций-членов, кроме всех функций перегрузки оператора.Мне интересно, если кто-нибудь может увидеть, что не так с этим кодом?Так как, когда я пытаюсь C = A * B, он просто возвращает построенную матрицу c из 0 в матрице три на три.Класс просто должен взять матрицу 3х3 и отредактировать ее.Перегрузки операторов предназначены для +, -, *, == и =.В качестве примечания, компилятор не показывает ошибок, но функции перегрузки не работают.

1 Ответ

0 голосов
/ 30 января 2019

К сожалению, OP не предоставил MCVE .

Следовательно, я копирую / вставляю части кода OP, которые были ИМХО в порядке (а не причина сообщенной проблемы)и сделал все остальное сам.

Вот образец, который я подготовил: 

#include <iostream>

class matrixType3 {

  private:
    int matrix[3][3];

  public:
    matrixType3(): matrix{ 0 } { }

    matrixType3(int x1, int x2, int x3, int x4, int x5, int x6, int x7, int x8, int x9):
      matrix{ { x1, x4, x7 }, { x2, x5, x8 }, { x3, x6, x9 } }
    { }

    ~matrixType3() = default;

    matrixType3(const matrixType3&) = default;

#if 0 // OP's version:
    matrixType3 operator=(const matrixType3&mat) const
    {
      std::cout << "matrixType3 matrixType3::operator=(const matrixType3 &mat)()\n";
      matrixType3 tempMatrix;
      for (int i = 0; i < 3; i++) {
        for (int r = 0; r < 3; r++) {
          tempMatrix.matrix[i][r] = mat.matrix[i][r];
        }
      }
      return tempMatrix;
    }
#else // My version:
    /* This would be actually sufficient:
    matrixType3& operator=(const matrixType3&) = default;
     * but for demonstration a custom implementation:
     */
    matrixType3& operator=(const matrixType3 &mat)
    {
      std::cout << "matrixType3& operator=(const matrixType3 &mat)()\n";
      std::copy(
        std::begin(mat.matrix[0]), std::end(mat.matrix[0]),
        std::begin(matrix[0]));
      std::copy(
        std::begin(mat.matrix[1]), std::end(mat.matrix[1]),
        std::begin(matrix[1]));
      std::copy(
        std::begin(mat.matrix[2]), std::end(mat.matrix[2]),
        std::begin(matrix[2]));
      return *this;
    }
#endif // 0

    // copy/pasted from OP:
    matrixType3 operator+(const matrixType3& mat) const
    {
      matrixType3 tempMatrix;
      for (int i = 0; i < 3; i++) {
        for (int r = 0; r < 3; r++) {
          tempMatrix.matrix[i][r] = mat.matrix[i][r] + matrix[i][r];
        }
      }
      return tempMatrix;
    }

    /* a "getter" to provide read-only access to the member variable
     *
     * (Returning a const reference to a 2d array makes this function
     *a bit strange looking...) ;-)
     */
    const int (&get() const)[3][3] { return matrix; }
};

ОП заявил, что

Однако моя проблема в том, что операторФункции перегрузки не работают.

Я твердо верю, что именно OP operator=() является проблемой, когда другие операторы выглядят не так уж плохо.Подпись, выбранная ОП для operator=, выглядит для меня очень необычно.Вместо этого должно быть: 

matrixType3& operator=(const matrixType3 &mat)

Оператор должен присвоить аргумент mat самому себе (то есть *this).Следовательно, он не может быть const и не должен возвращать ссылку const.(Последнее, вероятно, сработало бы, но без необходимости ограничивало бы применимость возвращаемого значения.)

Чтобы доказать, что я на правильном пути, я скопировал / вставил OP operator=() в мой пример кода и точно увидел, какой OPописано.

Кстати.Я поместил оператор вывода в оператор присваивания, чтобы показать, когда он вызывается.

Чтобы проверить приведенный выше код, остальная часть примера: 

std::ostream& operator << (std::ostream &out, const matrixType3 &mat)
{
  const int (&values)[3][3] = mat.get();
  return out
    << values[0][0] << '\t' << values[0][1] << '\t' << values[0][2] << '\n'
    << values[1][0] << '\t' << values[1][1] << '\t' << values[1][2] << '\n'
    << values[2][0] << '\t' << values[2][1] << '\t' << values[2][2] << '\n';
}

int main()
{
  matrixType3 mat1(11, 21, 31, 12, 22, 32, 13, 23, 33);
  std::cout << "mat1:\n" << mat1;
  matrixType3 mat2;
  std::cout << "mat2:\n" << mat2;
  std::cout << "mat2 = mat1;\n";
  mat2 = mat1;
  std::cout << "mat2:\n" << mat2;
  std::cout << "mat1 + mat2:\n" << mat1 + mat2;
  matrixType3 mat3;
  mat3 = mat1 + mat2;
  std::cout << "mat3 = mat1 + mat2:\n" << mat3;
  // This is no assigment:
  std::cout << "No assignment: matrixType3 mat4 = mat1;\n";
  matrixType3 mat4 = mat1;
  std::cout << "... but initialization/copy construction\n";
  std::cout << "mat4:\n" << mat4;  
  return 0;
}

Скомпилировано и протестировано (с моим operator=()): 

mat1:
11  12  13
21  22  23
31  32  33
mat2:
0   0   0
0   0   0
0   0   0
mat2 = mat1;
matrixType3& operator=(const matrixType3 &mat)()
mat2:
11  12  13
21  22  23
31  32  33
mat1 + mat2:
22  24  26
42  44  46
62  64  66
matrixType3& operator=(const matrixType3 &mat)()
mat3 = mat1 + mat2:
22  24  26
42  44  46
62  64  66
No assignment: matrixType3 mat4 = mat1;
... but initialization/copy construction
mat4:
11  12  13
21  22  23
31  32  33

Демонстрация в реальном времени на coliru

При подготовке образца кода я споткнулся: 

matrixType3 mat1;
matrixType3 mat2; mat2 = mat1;

против 

matrixType3 mat1;
matrixType3 mat2 = mat1;

, которые я также счел нужным уточнить.

Хотя оба образца выглядят очень похожими, есть большая разница:1 st использует назначение копирования, но в 2 nd = компилируется как инициализация (потому что следует непосредственно за объявлением).Следовательно, в случае 2 и нет никакой помощи.(Вместо этого вызывается конструктор копирования.)

...