执行多个功能，但发生错误时停止

Question

我想一个接一个地使用多个函数，但如果第一个函数出现问题，则不应执行其他函数。目前我正在使用带开关的while循环。有没有办法用其他东西跳过整个 while/switch 部分？可能是活动之类的？

while (!ErrorActive && iStep != 3)
{
    switch (iStep)
    {
        case 0:
            DoSomething(); // this can trigger ErrorActive
            iStep = 1;
            break;

        case 1:
            DoSomething2(); // this can trigger ErrorActive
            iStep = 2;
            break;

        case 2:
            DoSomething3(); // this can trigger ErrorActive
            iStep = 3;
            break;
    }
}

DoSomething 函数有这样的东西：

public void DoSomething()
{
    try
    {
        //calculate something
    }
    catch
    {
        ErrorActive = true;
    }
}

有没有办法跳过整个 while/switch 部分并用其他东西替换它（比如可能是一个事件？）或者我应该总是在每个函数之间保留一些东西来检查一切是否正常？

Answer 1

只需将渔获物上移一级：

// true if all steps executed, false otherwise
bool DoSteps()
{
   int lastExecutedStep = 0;
   try{
     DoSomething();
     lastExecutedStep = 1;
     DoSomething1();
     lastExecutedStep = 2;
     DoSomething2();
     lastExecutedStep = 3;
   }
   catch( IOException ioex )
   {
      // log IO Exception
   }
   // ... catch more expected exception types

   return (lastExecutedStep == 3);
}

void DoSomething(){
   // NO try/catch here
}

即使没有计步器也可以：

// true if all steps executed, false otherwise
bool DoSteps()
{
   try{
     DoSomething();
     DoSomething1();
     DoSomething2();
     return true;
   }
   catch( IOException ioex )
   {
      // log IO Exception
   }
   // ... catch more expected exception types

   return false;
}

要了解更多学术方法，您可能想探索Chain of responsibility pattern

Answer 2

你可以这样做，为什么在解决方案很简单的情况下使用这么马赫的代码。

if(!ErrorActive){
 DoSomething(); // this can trigger ErrorActive
}

if(!ErrorActive){
 DoSomething1(); // this can trigger ErrorActive
}

if(!ErrorActive){
 DoSomething2(); // this can trigger ErrorActive
}

Answer 3

您可以将while 循环简化为for 循环，并使用新的switch 表达式语法稍微简化一下。另外，让函数返回成功状态而不是设置一些共享变量：

void Run()
{
    var keepRunning = true;

    for(int i = 0; keepRunning; i++)
    {
        keepRunning = i switch
        {
            0 => DoSomething(),
            1 => DoSomething2(),
            2 => DoSomething3(),
            _ => false
        };
    }
}

bool DoSomething()
{
    try
    {
        return true;
    }
    catch
    {
        return false;
    }
}

bool DoSomething2()
{
    try
    {
        return true;
    }
    catch
    {
        return false;
    }
}

bool DoSomething3()
{
    try
    {
        return true;
    }
    catch
    {
        return false;
    }
}