所以我将此作为我的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时,它只是在三乘三的矩阵中返回构造的0的c矩阵。 该类仅应采用3x3矩阵进行编辑。 运算符重载用于+,-,*,==和=。 顺便提一句,编译器未显示任何错误,但过载功能不起作用。
答案 0 :(得分:0)
很遗憾,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声称
但是,我的问题是运算符重载功能不起作用。
我坚信OP的operator=()
正是其他运算符看上去还不错的问题。 OP为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
在准备示例代码时,我偶然发现:
matrixType3 mat1;
matrixType3 mat2; mat2 = mat1;
vs。
matrixType3 mat1;
matrixType3 mat2 = mat1;
我发现也值得详细阐述。
尽管这两个样本看起来非常相似,但仍存在很大差异:
1 st 使用副本分配,但是在2 nd 中,=
被编译为初始化(因为直接跟随声明)。因此,在第二个 nd 情况下,没有分配。 (而是调用复制构造函数。)