你能在没有重载方法的情况下调用第二个构造函数吗

Question

所以我有一个查询。这并不是一个真正的问题，因为代码仍然有效，但看起来很草率。

我需要两个建设者。

public class JsonQuery
{
    const string currentWorkSheet = "GLHTanneryData_CurrentWeek";
    const string unitTestingWorkSheet = "GLHTanneryData_UnitTesting";
    private const string authorisation = "XXXX";
    JObject jObject;
    public JsonQuery()
    {
        SmartsheetQuery smartsheetQuery = new SmartsheetQuery();
        jObject = JObject.Parse(smartsheetQuery.getJsonAsString(currentWorkSheet));
    }
    public JsonQuery(bool testing)
    {
        SmartsheetQuery smartsheetQuery = new SmartsheetQuery();
        jObject = JObject.Parse(smartsheetQuery.getJsonAsString(unitTestingWorkSheet));
    }
}

每个从智能工作表上的不同位置获取一个json文件。所以为了调用第二个构造函数，我只给它一个布尔值true。问题是，正如你从代码中看到的，我实际上不需要在方法中使用任何布尔值，我只需要知道我想调用的是这个构造函数。

有更好的方法吗？

Answer 1

您可能会考虑设置用于测试的编译器指令

public JsonQuery()
{
    SmartsheetQuery smartsheetQuery = new SmartsheetQuery();
#if TEST
    jObject = JObject.Parse(smartsheetQuery.getJsonAsString(unitTestingWorkSheet));
#else
    jObject = JObject.Parse(smartsheetQuery.getJsonAsString(currentWorkSheet));
#endif
}

Answer 2

看起来你的类违反了单一责任原则，它既确定数据又解析数据。

在您的情况下，这个问题使测试解析机制变得非常困难，因为它们在类中交织在一起。

另一种选择是让解析类依赖于某种数据提供程序接口。

public interface ISmartSheetProvider
{
    string GetJsonData();
}
public class ProductionSmartSheetProvider : ISmartSheetProvider
{
    public string GetJsonData()
    {
        return new SmartsheetQuery().getJsonAsString("GLHTanneryData_CurrentWeek"));
    }
}
public class MockSmartSheetProvider : ISmartSheetProvider
{
    public string GetJsonData()
    {    
        return "..."; // whatever test data.
    }
}
public class JsonQuery(ISmartSheetProvider smartSheetProvider)
{
    jObject = jObject.Parse(smartSheetProvider.GetJsonData());
}

尽管所有的工作都是在构造函数中完成的，但最好将其作为字符串提供，或者将整个类变成一个静态方法。我假设这里省略了一些代码，并假设该类所做的工作比所提供的要多。

Answer 3

您实际上并不需要两个构造函数-看起来您将从传递工作表中受益：

 public JsonQuery(string worksheet)
 {
    SmartsheetQuery smartsheetQuery = new SmartsheetQuery();
    jObject = JObject.Parse(smartsheetQuery.getJsonAsString(worksheet));    
 }

只需输入不同的工作表进行测试。

Answer 4

像这样做吗：

public JsonQuery(bool testing = false)
{
    SmartsheetQuery smartsheetQuery = new SmartsheetQuery();
    if (testing)
         jObject = JObject.Parse(smartsheetQuery.getJsonAsString(unitTestingWorkSheet));
    else
         jObject = JObject.Parse(smartsheetQuery.getJsonAsString(currentWorkSheet));
}

然后，对于非测试，您只需要使用new JsonQuery() 进行初始化

Answer 5

它被称为构造函数链接或嵌套：

public JsonQuery() : this(false) // or true
{
}
public JsonQuery(bool testing) // or optional parameter value
{
    // use 'testing' here
}