c++ - 有什么办法可以简化这段代码吗？

Question

我觉得这段代码可以简化并做得更干净一些。 (不，这不是作业，我不是学生。)

m_configured 是我正在使用的临时 bool 值，我不需要使用它。Load() 是一个函数，用于初始化我正在处理的板上的所有寄存器。Unload() 撤消 Load 所做的一切。

我不想使用循环，因为我只想让这个例程发生一次。即加载 -- 失败 --> 卸载 --> 加载 -- 再次失败 --> 停在那里，不要再试一次。

        bool m_configured = Load();

        if (!m_configured)
        {
            xlog_info("Failed to start board. Trying again..");
            Unload();
            m_configured = Load();
            if(!m_configured)
                xlog_error("Failed to start board.");
        }

        if (m_configured)
        {
            xlog_info("Board Started");
        }

更新/澄清:此代码在构造函数中。

Answer 1

如果你真的想简化它(你没有说明你想要的原因)——你可以这样写:

m_configured = Load() || (Unload() && Load());
if( !m_configured )
  xlog_error("Failed to start board.");

 xlog_info("Board Started");

假设您的 xlog_error() 函数停止程序，并且 Unload() 返回是否卸载成功。