带有中间变量的构造函数链接

Question

我遇到以下带有重载构造函数的情况，我正在努力寻找一个很好的解决方案。我不知道如何使用带有构造函数链接的中间赋值。

以下内容无效，但显示了我想要做的事情

public MyThing(IServiceLocator services, int? userId)
{
    // blah....
}

public MyThing(IServiceLocator services, string userName)
{
    User user = services.UserService.GetUserByName(userName);
    int userId = user == null ? null : (int?)user.Id;
    // call the other constructor   
    this(services, userId);
}

我知道用有效代码编写上述内容的唯一方法是

public MyThing(IServiceLocator services, string userName)
    : this(services,
           services.UserService.GetUserByName(userName) == null ?
              null : (int?)services.UserService.GetUserByName(userName).Id)

，这不仅是丑陋的代码，而且还需要数据库调用两次（除非编译器足够聪明来工作）出来，我对此表示怀疑）。

上面的内容有更好的写法吗？

Answer 1

怎么样：

public MyThing(IServiceLocator services, string userName)
{
    User user = services.UserService.GetUserByName(userName);
    int? userId = user == null ? null : (int?)user.Id;

    Initialize(services, userId);
}


public MyThing(IServiceLocator services, int? userId)
{
    Initialize(services, userId);
}

private void Initialize(IServiceLocator services, int? userId)
{
    // initialization logic
}

编辑

如果我是你，我会用工厂方法替换构造函数，如下所示：

private MyThing(IServiceLocator services, int? userId)
{
    // blah....
} 

public static Create(IServiceLocator services, int? userId)
{
    return new MyThing(services, userId);
}

public static Create(IServiceLocator services, string userName)
{
    User user = services.UserService.GetUserByName(userName);
    int userId = user == null ? null : (int?)user.Id;

    return new MyThing(services, userId);
}

用法：

var myThing = MyThing.Create(services, 123);
var myOtherThing = MyThing.Create(services, "userName");

替换带工厂方法的构造函数（refactoring.com）

Answer 2

有，是的。我知道 示例 是用 Java 编写的，但它是这样的对于您的问题来说，这是一个很好的解决方案，移植到 C# 的一些努力确实有意义。

Answer 3

您可以使用静态辅助方法：

public MyThing(IServiceLocator services, int? userId)
{
    // blah....
}

public MyThing(IServiceLocator services, string userName)
    : this(services, GetUserId(services, userName))
{
}

private static int? GetUserId(IServiceLocator services, string userName)
{
    User user = services.UserService.GetUserByName(userName);
    return (user == null) ? (int?)null : user.Id;
}