如何编写有异常的构造函数

Question

我想知道如何编写一个带有异常的构造函数

我有一个 class Character和另一个 class Matrix<T>

template<class T>
 class Matrix
 {
     T** data;
     int col;
     int row;
     public:
//ctor of matrix
 template<class T>
Matrix(const Matrix<T>& mat):data(new T*[mat.row]), col(mat.col), row(mat.row)
{
    
}
};

然后我有 class 游戏：

class Game
{ 
    private:
    Matrix<std::shared_ptr<Character>> data;
    int height;
    int width;
    
    
    
public:
//ctor of Game, and the one I want to know how to write
    Game(int height, int width) :height(height),width(width)
    {
        try
        {
            data(Matrix<std::shared_ptr<Character>>(height,width));
        }
        catch(const std::badalloc&)
        {
           throw IllegalArgument();
        }
        
    }  

我为上面的Game编写 ctor 的方式不正确，有谁知道我怎样才能正确编写它并且仍然检查它是否正确分配，

所以为了正确分配height和width必须大于零，但我找不到任何方法来检查它们然后初始化数据，这就是我尝试使用try和catch的原因，但是这是不正确的，所以有人知道编写构造函数的方法，并确保分配成功???????????????(((使用异常)))

`

Answer 1

如何在初始化需要分配的数据之前检查height>0&&width>0

使用 function：

class Game
{ 
    // ...
    static Matrix<std::shared_ptr<Character>>
    init_data(int height, int width)
    {
        if (height <= 0 || width <= 0)
            throw std::runtime_error("naughty dimensions");
        return {height, width};
    }
    
    
public:
    Game(int height, int width) :
        data(init_data(height, width)),
        height(height),
        width(width)
    {}

这样，除非提供的维度有效，否则成员永远不会被初始化。

---

其他注意事项：

• 诸如Matrix中的那些裸露的拥有指针是不好的。 你应该避免它们。 使用它们的一个后果是Matrix的构造函数不是异常安全的。 如果创建T s 抛出，它将泄漏 memory。
• 在Matrix和Game中存储维度似乎是多余的。

Answer 2

对于任何需要它的人，解决方案是：

 Game(int height, int width):data((height<=0)||(width<=0) ? throw IllegalArgument() : Dimensions(height,width))
    ,height(height),width(width)
    {

    }