无需 goto 即可编写重试逻辑的更好方法 [重复]

Question

有没有更好的方法来编写这段代码而不使用goto？这似乎很尴尬，但我想不出更好的方法。我需要能够执行一次重试尝试，但我不想复制任何代码。

public void Write(string body)
{
    bool retry = false;
RetryPoint:
    try
    {
        m_Outputfile.Write(body);
        m_Outputfile.Flush();
    }
    catch (Exception)
    {
        if( retry )
            throw; 
        // try to re-open the file...
        m_Outputfile = new StreamWriter(m_Filepath, true);
        retry = true;
        goto RetryPoint;
    }
}

Answer 1

这是我将使用的基本逻辑，而不是 goto 语句：

bool succeeded = false;
int tries = 2;

do
{
    try
    {
        m_Outputfile = new StreamWriter(m_Filepath, true);
        m_Outputfile.Write(body); 
        m_Outputfile.Flush(); 
        succeeded = true;
    }
    catch(Exception)
    {
        tries--;
    }
}
while (!succeeded && tries > 0);

我刚刚添加了尝试次数逻辑，即使原始问题没有。

Answer 2

@Michael's answer（带有正确实现的 out catch 块）在您的情况下可能是最容易使用的，也是最简单的。但为了展示替代方案，这里有一个版本，它将“重试”流控制纳入一个单独的方法：

// define a flow control method that performs an action, with an optional retry
public static void WithRetry( Action action, Action recovery )
{
    try {
        action(); 
    }
    catch (Exception) {
        recovery();
        action();
    }
}

public void Send(string body)
{
    WithRetry(() =>
    // action logic:
    {
       m_Outputfile.Write(body);
       m_Outputfile.Flush();
    },
    // retry logic:
    () =>
    {
       m_Outputfile = new StreamWriter(m_Filepath, true);
    });
}

当然，您可以通过重试计数、更好的错误传播等来改进这一点。

Answer 3

Michael 的方案不太符合要求，就是重试固定次数，抛出最后一次失败。

为此，我会推荐一个简单的 for 循环，倒计时。如果成功，请使用 break 退出（或者，如果方便，请返回）。否则，让 catch 检查索引是否降至 0。如果是，请重新抛出，而不是记录或忽略。

public void Write(string body, bool retryOnError)
{
    for (int tries = MaxRetries; tries >= 0; tries--)
    {
        try
        {
            _outputfile.Write(body);
            _outputfile.Flush();
            break;
        }
        catch (Exception)
        {
            if (tries == 0)
                throw; 

            _outputfile.Close();
            _outputfile = new StreamWriter(_filepath, true);
        }
    }
}

在上面的示例中，返回就可以了，但我想展示一般情况。

Answer 4

如果你把它放在一个循环中呢？可能是类似的东西。

while(tryToOpenFile)
{
    try
    {
        //some code
    }
    catch
    {
    }
    finally
    {
        //set tryToOpenFile to false when you need to break
    }
}

Answer 5

public void Write(string body, bool retryOnError)
{
    try
    {
        m_Outputfile.Write(body);
        m_Outputfile.Flush();
    }
    catch (Exception)
    {
        if(!retryOnError)
            throw; 
        // try to re-open the file...
        m_Outputfile = new StreamWriter(m_Filepath, true);
        Write(body, false);
    }
}

Answer 6

尝试以下方法：

int tryCount = 0;
bool succeeded = false;

while(!succeeded && tryCount<2){
    tryCount++;
    try{
        //interesting stuff here that may fail.

        succeeded=true;
    } catch {
    }
}

Answer 7

带有布尔值

public void Write(string body)
{
        bool NotFailedOnce = true;            
        while (true)
        {
            try
            {
                 _outputfile.Write(body);
                 _outputfile.Flush();           
                 return;                    
            }
            catch (Exception)
            {
                NotFailedOnce = !NotFailedOnce;
                if (NotFailedOnce)
                {
                    throw;
                }
                else
                {
                     m_Outputfile = new StreamWriter(m_Filepath, true);
                }
            }
      }        
}