关于C#中构造函数的问题

时间:2011-03-25 00:33:12

标签: c# constructor

您好
关于名为Square的类中的部分代码:

public Square( int i_RowIndex, eColumn i_ColIndex) 
{
    m_RowIndex = i_RowIndex;
    m_ColIndex = i_ColIndex;
    **new Square(i_RowIndex, i_ColIndex, eCoinType.NoCoin);**
}

public Square(int i_RowIndex, eColumn i_ColIndex, eCoinType i_CoinType) 
{
    m_RowIndex = i_RowIndex;
    m_ColIndex = i_ColIndex;
    m_Coin = i_CoinType;
}

在其他C'tor中调用过载的C'tor以及用粗体看到的“新”声明是不是很好? 我认为这是错误的,每次我们调用new时我们都会分配一个新实例,从C'tor分配2个重复实例是不对的,这意味着从一开始就分配一个实例。

我错了吗?

由于

4 个答案:

答案 0 :(得分:9)

您不应该在构造函数中调用重载构造函数,而是创建一个新实例。

应该更像是:

public Square( int i_RowIndex, eColumn i_ColIndex)
    : this(i_RowIndex, i_ColIndex, eCoinType.NoCoin)
{
}

public Square(int i_RowIndex, eColumn i_ColIndex, eCoinType i_CoinType) 
{
    m_RowIndex = i_RowIndex;
    m_ColIndex = i_ColIndex;
    m_Coin = i_CoinType;
}    

答案 1 :(得分:3)

这不对。实际上,这句话:

new Square(i_RowIndex, i_ColIndex, eCoinType.NoCoin);

构造函数绝对没有(有用)。

答案 2 :(得分:0)

我认为你想要的是更像这样的东西:

public Square( int i_RowIndex, eColumn i_ColIndex) 
    : this(i_RowIndex, i_ColIndex, eCoinType.NoCoin) 
{}

public Square(int i_RowIndex, eColumn i_ColIndex, eCoinType i_CoinType) 
{
    m_RowIndex = i_RowIndex;
    m_ColIndex = i_ColIndex;
    m_Coin = i_CoinType;
} 

调用new将创建另一个只会被垃圾收集的实例,因为您没有存储对它的引用。此外,原始实例中的m_Coin将不会被设置(或者更准确地说将设置为default(eCointType)

答案 3 :(得分:0)

public Square( int i_RowIndex, eColumn i_ColIndex) : this(i_RowIndex, i_ColIndex, eCoinType.NoCoin)
{

}

public Square(int i_RowIndex, eColumn i_ColIndex, eCoinType i_CoinType) 
{
            m_RowIndex = i_RowIndex;
            m_ColIndex = i_ColIndex;
            m_Coin = i_CoinType;
}