通用回调机制重载成功处理程序作为流程控制的手段 - 代码味道？

Question

我对一个项目相当陌生，对于我们对数据库进行的一些异步调用（更改了变量和函数名称），遇到了一个有趣的设计范例：

private void OnLogin(object selectedInitialState,
                     AsyncEventCompletedCallback<EmptyAsyncEventArgs> userCallback,
                     object userState)

示例用法：

OnLogin(
    null,
    args =>
    {
        if (args.IsSuccess)
            DetermineNextStep(); //When done, continue to this step
        else
            //NOTE: This probably means we couldn't connect to the DB
            // Handle this case
    },
    null);

OnLogin(
    newInitialState,
    args =>
    {
         ReLoginUser(); //Was logged in; re-logging in user with different initial state
    },
    null);

示例用法显示了两种不同情况下对此函数的两种不同调用 -初始登录和重新登录（技术上不是重新登录，而是以不同的初始状态为当前登录的用户重新启动应用程序）。

令我困扰的是，这两种情况下的回调函数是不同的。我习惯于看到函数接受回调以允许函数的用户在被调用的函数的权限内提供自定义实现。

但在上述情况下，回调函数会更改控制流。根据提供的回调函数的不同，异步调用返回后后续的调用函数也不同。这是代码味道还是只是回调的创造性用途？

Answer 1

我将 OnLogin 函数解释为启动一个操作，该操作将在完成后调用一次回调。

在这种情况下，代码实际上是很正常的。在异步编写程序时，这种情况并不罕见。考虑到替代方案是在回调函数中使用（小）状态机保持“状态”，我认为不同的回调实际上是更优雅的解决方案。它隐式而不是显式地处理异步“状态”。