1、构造函数里没有对pMatrix初始化为0值，将导致后面有些地方错误。
2、拷贝构造函数和默认构造函数只会调用一个，你的拷贝构造函数中没有初始化pMatrix以及rows和columns。
3、Matrix::Mul中，if可以写在外面；由于构造函数中没有初始化0，c.pMatrix[i][y] +=这里肯定是错误的；另外，既然a.GetColumns() == b.GetRows()，就没有必要用j和x这2个变量了，一个就行了。
4、main函数里面的2个for循环，怎么i和j都不用初始化0的吗？
5、最后2行，是很不妥。可以考虑写成static，或写一个全局的operator*。

暂时只看出来这些。至于对错，这个最好自己调试。可以把算法转成自己看得懂的语言输出出来，比如：把矩阵a的第几行第几列和矩阵b的第几行第几列相乘，加到矩阵c的第几行第几列。这样的话，自己看得懂能排错就行了。  回复  更多评论
  

void Matrix::Mul(Matrix a,Matrix b)给谁用呢？
  回复  更多评论
  

看到了楼主的这篇文章，我试着改了一下，如下：希望大家一起加入讨论。

#include <iostream>
using namespace std;

class Matrix
{
private:
int rows,columns;
int **pMatrix;
public:
Matrix(int rows = 3,int columns = 2);
Matrix(const Matrix &M);
~Matrix();
Matrix& operator=(const Matrix& M);
int GetRows() const;
int GetColumns() const;
void SetValue();
friend Matrix operator*(const Matrix& a,const Matrix& b);
friend ostream& operator<<(ostream& os,const Matrix& M);
};

int Matrix::GetRows() const { return rows;}
int Matrix::GetColumns() const { return columns;}

// 构造函数
Matrix::Matrix(int x,int y)
{
rows = x;
columns = y;
//有的时候为了考虑创建对象的效率，在使用的时候分配存储空间，而不在构造函数中分配
pMatrix = new int* [x];
for (int i = 0 ; i < x; i++ )
{
pMatrix[i] = new int[y];
for(int j = 0;j < y;j++) //初始化每个值为0
pMatrix[i][j] = 0;
}
}
// 析构函数
Matrix::~Matrix()
{
for (int i = 0 ;i < rows;i ++ )
delete[] pMatrix[i];
delete[] pMatrix;
}

// 赋值函数
Matrix& Matrix::operator=(const Matrix& M)
{
if(this != &M)
{
for (int ii = 0 ;ii < rows;ii++ )
if(pMatrix[ii])
delete[] pMatrix[ii];
if(pMatrix)
delete[] pMatrix;
rows = M.rows;
columns = M.columns;
//分配存储空间
pMatrix = new int* [rows];
for (int k = 0 ; k < rows; k++ )
pMatrix[k] = new int[columns];

for ( int i = 0 ; i < rows; i ++ )
for ( int j = 0 ; j < columns; j ++ )
pMatrix[i][j] = M.pMatrix[i][j];
}
return *this;
}
void Matrix::SetValue()
{
int i,j,value;
for ( i = 0 ; i < rows; i ++ )
{
for ( j = 0 ; j < columns; j ++ )
{
cout << " 第 " << i << " 行 " ;
cout << " 第 " << j << " 列: " ;
cin >> value;
cout << endl;
pMatrix[i][j] = value;
}
}
}
// 拷贝构造函数
Matrix::Matrix(const Matrix& M)
{
rows = M.rows;
columns = M.columns;
//分配存储空间
pMatrix = new int* [rows];
for (int k = 0 ; k < rows; k++ )
pMatrix[k] = new int[columns];

for ( int i = 0 ; i < rows; i ++ )
for ( int j = 0 ; j < columns; j ++ )
pMatrix[i][j] = M.pMatrix[i][j];
}

Matrix operator*(const Matrix& a,const Matrix& b)
{
if (a.columns == b.rows)
{
Matrix c(a.rows,b.columns);
for ( int i = 0 ;i < a.rows;i ++ )
{
for ( int j = 0 ;j < b.columns;j ++ )
{
for ( int columnIndex= 0 ;columnIndex < a.columns;columnIndex++ )
c.pMatrix[i][j] += a.pMatrix[i][columnIndex] * b.pMatrix[columnIndex][j];
}
}
return c;
}
else
return Matrix();
}

ostream& operator<<(ostream& os,const Matrix& M)
{
for (int i = 0;i < M.rows;i++ )
{
for (int j = 0;j < M.columns;j++ )
os << M.pMatrix[i][j] << " ";
os << endl;
}
return (os << endl);
}


// 主函数
void main()
{
Matrix Ma(3,2),Mb(2,2);
Ma.SetValue();
Mb.SetValue();
cout << Ma << endl;
cout << Mb << endl;

Matrix Mc = Ma * Mb;//拷贝构造函数
cout << Mc << endl;
Mc = Mb; //=运算符，即赋值函数
cout << Mb << endl;
}   回复  更多评论
  

还是有错，编译出现7个错误！
ostream& operator<<(ostream& os,const Matrix& M) 中没有权限访问类中私有的rows和columns 这两个成员。
  回复  更多评论
  

不是呀，我这里编译，运行都没出错呀  回复  更多评论
  

@roa420
我在Visual C＋＋ 6.0中编译的时候也出现过问题，我怀疑它对友元支持的不好。
建议你换一个编译器试试看。  回复  更多评论