c++ - goto语句在C++中的用法

Question

我有一个 funcA 我每毫秒调用一次。由另一个 funcB。我想使用 goto 语句。但是当我查看流程时(当m_tempdata 不是NULL 时)，在打印"stage 2" 之后，它也在打印 “清理开始”。通常，我希望在为下一回合打印 "stage 2" 后返回。我错了吗？

void ClassA::funcA()
{
    m_tempdata = m_freedata;

    printf("stage 1 \n");

    if (NULL == m_tempdata)
    {
        printf("going cleaning \n" );
        goto cleanup;
    }
    m_freedata = m_tempdata->next;

    printf("stage 2 \n");

cleanup: printf("cleanup starts \n");
    // ... some additional work todo
}

Answer 1

Normally, I expect to return after printing "stage 2" for the next turn. Am I wrong?

是的，你错了。

在没有跳转到标签的情况下，代码将简单地跳转到它并继续。标签不是说明。它不会导致程序神奇地返回或跳转到其他地方。

这里有一个更好的写法:

void ClassA::funcA()
{
  m_tempdata = m_freedata;

  printf("stage 1 \n");

  if (NULL == m_tempdata)
  {
    printf("going cleaning \n" );
    Cleanup();
    return;
  }

  m_freedata = m_tempdata->next;
}

void ClassA::Cleanup()
{
  printf("cleanup starts \n");
  //      ... some additional work todo
}

请不要使用goto。它使代码更难理解、调试和维护。另外，您最终会犯下像您在这里所做的愚蠢错误，因为您之前做出了糟糕的设计决策。请改用现代流量控制结构。