Question

我有一个包含另一个对象的二维数组的类。它有一个构造函数，但在该数组内部始终用零初始化。因此，Others未初始化为公平：

public class FirstClass
{
    public OtherClass[,] Others { get; set; }

    ...

}

public class OtherClass
{
    public int Id { get; set; }
}

在运行时期间填充此数组Others。现在，我想编写一个测试，它会在填充Others时测试一些操作。所以我需要将样本数组传递给Test方法。我不想创建OtherClass数组，因为我有很多这样的示例数组，我将不得不写：

        OtherClass[][,] samples = new[]
        {
            new OtherClass[,]
            {
                { new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };},
                { new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };,new OtherClass() { Id = 2 };},

etc..

难看！

所以在我的Tests项目中，我创建了一些int（Id s）数组：

    int[][,] samples = new[]
    {
        new int[,]
        {
            {1,0,0,0,0,0,0},
            {0,2,0,0,0,0,0},
            {0,0,3,0,0,0,0},
            {0,0,0,4,0,0,0}
        },
        new int[,]
        {
            {0,0,0,0,0,0,0},
            {0,0,0,0,0,0,0},
            {1,2,3,4,5,6,7},
            {0,0,0,0,0,0,0}
        }
    };

更具可读性......但现在我需要为FirstClass创建一个构造函数，它将int[,]作为参数，并使用参数中的Ids创建OtherClass [，]。

理论上我应该很好，因为测试看起来像：

[TestFixture]
class BoardTests
{
    [Test]
    [TestCaseSource("samples")]
    public void FirstTest(int[,] board)
    {
        FirstClass aClass = new FirstClass(board);
        //Test an operation on aClass 
    }
}

所以，我的问题是： 为测试创建额外的构造函数 ONLY 是一种好习惯吗？我不会在生产代码中使用此构造函数。或者你有更好的解决方案吗？

Answer 1

现在我需要为FirstClass创建一个构造函数，将int[,]作为参数，并使用参数中的OtherClass[,]创建Id。

虽然这肯定是一种选择，但如果您愿意，您当然不会必须这样做。保持构造函数不变的解决方案是在测试类中创建一个私有方法，以将int[,]转换为OtherClass[,]：

private static ToOtherClass(int[,] ids) {
    var OtherClass[,] res = ...
    // Do the conversion here
    return res;
}

现在您可以使用此方法生成易于阅读的代码，该代码不使用特殊的构造函数：

OtherClass[][,] samples = new[]
{
    ToOtherClass( new int[,]
    {
        {1,0,0,0,0,0,0},
        {0,2,0,0,0,0,0},
        {0,0,3,0,0,0,0},
        {0,0,0,4,0,0,0}
    }),
    ToOtherClass( new int[,]
    {
        {0,0,0,0,0,0,0},
        {0,0,0,0,0,0,0},
        {1,2,3,4,5,6,7},
        {0,0,0,0,0,0,0}
    })
};

我应该仅为测试目的创建新的构造函数吗？

1 个答案: