1、構(gòu)造函數(shù)里沒有對pMatrix初始化為0值，將導(dǎo)致后面有些地方錯誤。
2、拷貝構(gòu)造函數(shù)和默認構(gòu)造函數(shù)只會調(diào)用一個，你的拷貝構(gòu)造函數(shù)中沒有初始化pMatrix以及rows和columns。
3、Matrix::Mul中，if可以寫在外面；由于構(gòu)造函數(shù)中沒有初始化0，c.pMatrix[i][y] +=這里肯定是錯誤的；另外，既然a.GetColumns() == b.GetRows()，就沒有必要用j和x這2個變量了，一個就行了。
4、main函數(shù)里面的2個for循環(huán)，怎么i和j都不用初始化0的嗎？
5、最后2行，是很不妥。可以考慮寫成static，或?qū)懸粋€全局的operator*。

暫時只看出來這些。至于對錯，這個最好自己調(diào)試。可以把算法轉(zhuǎn)成自己看得懂的語言輸出出來，比如：把矩陣a的第幾行第幾列和矩陣b的第幾行第幾列相乘，加到矩陣c的第幾行第幾列。這樣的話，自己看得懂能排錯就行了。  回復(fù)  更多評論
  

void Matrix::Mul(Matrix a,Matrix b)給誰用呢？
  回復(fù)  更多評論
  

看到了樓主的這篇文章，我試著改了一下，如下：希望大家一起加入討論。

#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;}

// 構(gòu)造函數(shù)
Matrix::Matrix(int x,int y)
{
rows = x;
columns = y;
//有的時候為了考慮創(chuàng)建對象的效率，在使用的時候分配存儲空間，而不在構(gòu)造函數(shù)中分配
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;
}
}
// 析構(gòu)函數(shù)
Matrix::~Matrix()
{
for (int i = 0 ;i < rows;i ++ )
delete[] pMatrix[i];
delete[] pMatrix;
}

// 賦值函數(shù)
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;
}
}
}
// 拷貝構(gòu)造函數(shù)
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);
}


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

Matrix Mc = Ma * Mb;//拷貝構(gòu)造函數(shù)
cout << Mc << endl;
Mc = Mb; //=運算符，即賦值函數(shù)
cout << Mb << endl;
}   回復(fù)  更多評論
  

還是有錯，編譯出現(xiàn)7個錯誤！
ostream& operator<<(ostream& os,const Matrix& M) 中沒有權(quán)限訪問類中私有的rows和columns 這兩個成員。
  回復(fù)  更多評論
  

不是呀，我這里編譯，運行都沒出錯呀  回復(fù)  更多評論
  

@roa420
我在Visual C＋＋ 6.0中編譯的時候也出現(xiàn)過問題，我懷疑它對友元支持的不好。
建議你換一個編譯器試試看。  回復(fù)  更多評論