c – 使用goto干净地退出循环

我有一个关于在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语句 – 因此我请求专家帮助.我还缺少哪些其他基本想法可以使这些代码更清晰？

谢谢.

解决方法

有了你的那个,奇怪的条件导致循环早破,我会简单地使用休息.不需要goto那是什么突破.

但是,如果这些函数调用中的任何一个都可以抛出异常,或者如果你最终需要多次中断,我宁愿使用RAII样式的容器,这就是析构函数的确切类型.你总是执行对DeInitializeHardware的调用,所以……

// todo: add error checking if needed
class HardwareWrapper {
public:
    HardwareWrapper(Hardware *pH) 
      : _pHardware(pH) { 
        InitializeAndCheckHardware(_pHardware);
    }

    ~HardwareWrapper() {
        DeInitializeHardware(_pHardware);
    }

    const Hardware *getHardware() const {
        return _pHardware;
    }

    const Hardware *operator->() const {
        return _pHardware;
    }

    const Hardware& operator*() const {
        return *_pHardware;
    }

private:
    Hardware *_pHardware;
    // if you don't want to allow copies...
    HardwareWrapper(const HardwareWrapper &other);
    HardwareWrapper& operator=(const HardwareWrapper &other);
}

// ...

void ControlLoop::main_loop()
{
    HardwareWrapper hw(pHardware);
    // code
}

现在,无论发生什么,当该函数返回时,您将始终调用DeInitializeHardware.

c – 使用goto干净地退出循环

解决方法

相关文章