调用层次结构问题

Question

我遇到一种情况，我以级联方式调用函数/方法。请参阅以下示例以了解说明和问题。我希望我知道一些关于这种情况的技术术语。人们会更容易理解我在说什么。

public static class test
{
    private void button1_Click(object sender, RoutedEventArgs e)
    {
        if (Login("johndoe","password")) 
        {
            if(checkForSomething("johndoe"))
            {
                DoOpenDashboard();

                // Now it opens dashboard, it has several buttons. 
                // Each button does several different things
                // On this example I am just giving you two level of hierarchy
                // but in my actual program, there are 7 levels.
            }
        }
    }

    public static bool Login(string userid, string password)
    {
        //valid user
        return true;
    }

    public static bool checkForSomething(string userid) 
    {
        return true;
    }

如果子方法成功运行，如何避免进程返回到先前的调用方法/函数？

例如，登录方法正在调用 checkForSomething("johndoe")。如果 checkForSomething("johndoe") 被传递，那么它将通过调用 DoOpenDashboard 打开 Dashboard 窗口。此时，我的进程不应该返回检查某件事，然后登录。我希望这是有道理的。

Answer 1

不太清楚你在这里问什么。您的伪代码显示了在类的构造函数中调用的 Login () 方法。如果这确实是您的代码的工作方式，那么为了防止再次调用 Login，您需要避免创建此类的新实例。

但是，我认为您确实在询问 Arrow 反模式：

http: //codinghorror.com/blog/2006/01/flattening-arrow-code.html

编辑

我试图避免复制和复制粘贴，但由于原始帖子似乎不够清楚，以下是上面链接的《编码恐怖》中的节选内容：

在适当的情况下，我通过执行以下操作来展平该箭头代码：

1. 用保护子句替换条件。这段代码..

   如果（一些必要条件）{
   // 函数体代码
   }

   .. 作为保护子句效果更好：

   if (!SomeNecessaryCondition)
   {
   抛出新的RequiredConditionMissingException；
   }
   // 函数体代码

（请注意，还列出了其他技术，但我认为第一个现在就足够了）

这样，每个附加检查都不会导致另一个嵌套 if - 一旦任何检查失败，方法调用失败。这也可以在不引发异常的情况下完成，方法是让 button1_Click 调用一个返回 bool 值的函数（true 表示成功，false 表示失败）并在失败条件下立即返回 false：

private void button1_Click(object sender, RoutedEventArgs e)
{
    if (AllSystemsGo())
    {
        DoOpenDashboard();
    }
}

private bool AllSystemsGo()
{
    if (!Login("johndoe","password"))
        return false;

    if (checkForSomethingEvil("johndoe"))
        return false;

    if (!checkForSomethingImportant())
        return false;

    return true;
}

Answer 2

您是否想确保您的方法仅检查一次？也许您需要多次查询某些属性，但只测试一次。

private bool? canLogin;
private bool? somethingOk;

private bool CanLogin
{
    get
    {
        if (canLogin == null)
            canLogin = Login("johndoe","password");
        return canLogin.Value;
    }
}

private bool SomethingOk
{
    get
    {
        if (somethingOk == null)
            somethingOk = checkForSomething("johndoe");
        return somethingOk .Value;
    }
}

private void button1_Click(object sender, RoutedEventArgs e)
{
    if (this.CanLogin && this.SomethingOk && // other checks) 
    {
        DoOpenDashboard();            
    }
}