2013-03-02 58 views
0

好吧,我正试图实现一个简单的2D矩阵类。这是什么样子至今:实现动态矩阵类的错误

template <typename Type> 
class dyMatrix { 
    private: 
     Type *mat; 

     int width, height; 
     int length; 

    public: 
     dyMatrix (int _width, int _height) 
      : width(_width), height(_height), mat(0) 
     { 
      length = width * height; 
      mat = new Type[length]; 
     }; 

     // --- 

     int getWidth() { 
      return width; 
     }; 

     int getHeight() { 
      return height; 
     }; 

     int getLength() { 
      return length; 
     } 

     // --- 

     Type& operator() (int i, int j) { 
      return mat[j * width + i]; 
     }; 

     Type& operator() (int i) { 
      return mat[i]; 
     }; 

     // --- 

     ~dyMatrix() { 
      delete[] mat; 
     }; 
}; 

为了测试它,并与静态多维数组比较,我写了下面的代码片段:

#include <iostream> 
using namespace std; 

/* matrix class goes here */ 

struct Coord { 
    int x, y; 

    Coord() 
     : x(0), y(0) 
    {}; 

    Coord (int _x, int _y) 
     : x(_x), y(_y) 
    {}; 

    void print() { 
     cout << x << ", " << y; 
    }; 
}; 

int main() { 
    dyMatrix<Coord> adabo(5, 7); 
    Coord inakos[5][7]; 

    int i = 5, j = 0; 

    adabo(i, j) = *(new Coord(i, j)); 
    inakos[i][j] = *(new Coord(i, j)); 

    inakos[i][j].print(); 
    adabo(i, j).print(); 

    return 0; 
} 

“Adabo”和“Inakos “是任意选择的名字。执行后,inakos将打印其内容,但程序在adabo可以执行任何操作之前崩溃。另一个有趣的事情是,如果我分别给出ij以外的值5和0,分别如5和1,它可以正常工作。

我不知道确切的数字是什么工作,哪些使得程序出现故障,我只知道这里存在不规则性。我可能做错了什么?我是C++的业余爱好者,所以我可能会或不会在任何结构中滥用某些东西。

如果有人也有时间,我非常想知道在我的矩阵类中是否有任何其他通知错误。任何不可能与问题有关的事情,但却是一个谬论。

我也用下面的main()来测试它,但是它在inakos打印[5] [1]中的内容后仍然崩溃。也许它不必与dyMatrix,但松散实施的Coord?

int main() { 
    dyMatrix<Coord> adabo(5, 7); 
    Coord inakos[5][7]; 

    for (int i = 0; i < adabo.getHeight(); i++) { 
     for (int j = 0; j < adabo.getWidth(); j++) { 
      adabo(i, j) = *(new Coord(i, j)); 
      inakos[i][j] = *(new Coord(i, j)); 

      inakos[i][j].print(); 
      cout << "; "; 
     } 
     cout << "\n\n"; 
    } 
    cout << "\n\n\n"; 

    Coord temp; 
    for (int i = 0; i < 7; i++) { 
     for (int j = 0; j < 5; j++) { 
      temp = adabo(i, j); 

      temp.print(); 
      cout << "; "; 
     } 
     cout << "\n\n"; 
    } 

    return 0; 
} 

编辑:它与Coord没有关系。刚刚用一个int型的dyMatrix和一个int型静态矩阵进行了测试,然后在[5] [0]之后崩溃了。

+0

我鼓励你花时间学习使用调试器。这将为自己付出很多。 – NPE 2013-03-02 15:18:01

+0

@NPE是的,我可以试试。 Code :: Blocks的调试器不是很直观,但是,嘿它在那里。 – Mutoh 2013-03-02 15:21:57

回答

1

在第一个示例中,您声明inakos[5][7],因此索引范围为0到4,0到6之间。因此,inakos[5][0]inakos[5][1]可能会崩溃。

在你的第二个例子中,你再次声明inakos[5][7],但你让第一个索引循环从0到6.再次inakos[i][j]可以崩溃。一种修复方法是切换索引(即更改为i<adabo.getWidth()j<adabo.getHeight())。

+0

嗯,的确如此。第一,列第二。 – Mutoh 2013-03-02 15:23:55