小编Hun*_*ter的帖子

我有一个关于在C++中使用goto语句的问题.我理解这个话题是有争议的,并且对任何广泛的建议或论点都不感兴趣(我通常偏离使用goto).相反,我有一个特定的情况,并想要了解我的解决方案,它使用goto语句,是否是一个好的.我不会称自己是C++的新手,但也不会将自己归类为专业级程序员.生成我的问题的代码部分一旦启动就会在无限循环中旋转.伪代码中线程的一般流程如下:

void ControlLoop::main_loop()
{
    InitializeAndCheckHardware(pHardware) //pHardware is a pointer given from outside
    //The main loop
    while (m_bIsRunning)
    {
        simulated_time += time_increment; //this will probably be += 0.001 seconds
        ReadSensorData();
        if (data_is_bad) {
            m_bIsRunning = false;
            goto loop_end;
        }    
        ApplyFilterToData();
        ComputeControllerOutput();
        SendOutputToHardware();
        ProcessPendingEvents();

        while ( GetWallClockTime() < simulated_time ) {}
        if ( end_condition_is_satisified ) m_bIsRunning = false;
    }
    loop_end:
    DeInitializeHardware(pHardware);
}

pHardware指针从ControlLoop对象外部传入并具有多态类型,因此对我来说使用RAII并在main_loop中创建和破坏硬件接口本身没有多大意义.我想我可以让pHardware创建一个临时对象,代表硬件的"会话"或"使用",可以在main_loop退出时自动清理,但我不确定这个想法是否会让某些人更清楚否则我的意图是什么.循环中只有三种方式:第一种方法是从外部硬件读取坏数据; 第二个是如果ProcessPendingEvents()指示用户启动的中止,这只会导致m_bIsRunning变为false; 最后一个是如果在循环的底部满足结束条件.我还应该注意,main_loop可以在ControlLoop对象的生命周期内多次启动和完成,因此它应该在m_bIsRunning = false之后干净利落地退出.

另外,我意识到我可以在这里使用break关键字,但是main_loop中的大多数伪代码函数调用并没有真正封装为函数,只是因为它们需要有很多参数,或者它们都需要访问成员变量.在我看来,这两种情况都比让简单地将main_loop作为一个更长的函数更容易混淆,而且由于big while循环的长度,一个类似的声明goto loop_end似乎对我来说更清晰.

现在提出一个问题:如果您在自己的代码中编写它,这个解决方案会让您感到不舒服吗？它对我来说确实有些不对劲,但之后我从未在C++代码中使用过goto语句 - 因此我请求专家提供帮助.我还缺少哪些其他基本想法可以使这些代码更清晰？

谢谢.