不使用goto来重构代码

Refactor code without goto

本文关键字：重构 代码 goto      更新时间：2023-10-16

我有点糊涂了。如果没有goto，如何重构这个语句?

if(first_expression)
{
    // ....
    if(second_expression)
    {
        // ....
    }
    else
    {
        goto jump_out;
    }
}
else
{
    jump_out:
    // ....
}

这样好吗?

bool expressionFailed = false;
if(first_expression)
{
    if(second_expression)
    {
        // ....
    }
    else
        expressionsFailed = true;
}
else
    expressionsFailed = true;
if(expressionsFailed)
{
    // ....
}

我不能使用&&，因为second_expression = false

需要执行if(first_expression)之后的代码

我对你的代码一无所知，但你可以这样做:

void on_error ()
{
  //error stuff
}
int main ()
{
    if(first_expression)
    {
        if(second_expression)
        {
            // ....
        }
        else
            on_error();
    }
    else
        on_error();
}

或者更好:

void job()
{
    if(first_expression)
    {
        if(second_expression)
        {
            // ....
        }
        else
            throw;
    }
    else
        throw;
}
int main () {
  try
  {
    job();
  }
  catch ()
  {
    // err stuff 
  }
  return 0;
}

你做对了。我将把它简化为:

bool succeeded = false;
if(first_expression)
{
    if(second_expression)
    {
       succeeded = true;
       // ....
    }
}
if(!succeeded)
{
   // ....
}

你可以这样重写:

if(first_expression)
{
    // ....
    if(second_expression)
    {
        // ....
    }
}
if(!first_expression || (first_expression && !second_expression))
{
}
相关文章：