Question

我遇到过一种情况，我有一堆＆＃34;系统＆＃34;需要按顺序初始化，如果所有进行中的系统都成功初始化，则只会初始化下一个系统。

这让我得到了一大堆嵌套的if - else语句。这里有一些用于可视化的伪代码。

bool mainInit () {
    if (!system1Init ()) {
        reportError ();  // some error reporting function
    }
    else {
        if (!system2Init ()) {
            reportError ();
        }
        else {
            if (!system3Init ()) {
            // ... and so on

我发现当你获得一些关卡时，这看起来就像一团糟。

现在我想到使用一个switch语句，从第一个案例开始，到成功的其他案例，只有在出现错误时才会中断。

bool mainInit () {

    switch (1) {
    case 1:
        if (!system1Init ()) {
            reportError ();
            break;
        }
    case 2:
        if (!system2Init ())
            reportError ();
            break;
        }
    // ....
}

现在，我更喜欢这个。我发现阅读起来要容易得多，特别是有一些不错的评论，但我对编程很新。

所以，我的问题是：看到这不是传统上如何使用switch语句（至少从我所见过的），这样的事情是可以接受的，还是会被视为不良形式？

作为编程的新手，我努力不养成太多可能会让其他程序员感到沮丧的坏习惯。

我进行了搜索，但我发现的大部分内容都与替换if - else if语句的链而不是替换嵌套的语句有关。

Answer 1

引用数组中的所有系统，例如std::vector<mySystem*>，并按顺序循环它们，在第一次失败时中断。这样，即使对于500多个系统，您的整个代码也减少到少于5行代码。

建议的交换机黑客是XY problem solving的一个邪恶的例子：你真正的问题是你没有系统阵列，并且正在使用命名变量，因此消除了所有选项以更灵活地使用所有系统，就像在循环中一样。

Answer 2

假设您的所有system#Init()调用在编译时都是已知的，您可以非常轻松地将它们放在表中，然后遍历该表。

typedef (*system_init)(void);

system_init initialization_functions[] =
{
    system1Init,
    system2Init,
    system3Init,
      ...
    systemNInit
};

bool mainInit()
{
    for(size_t idx(0); idx < sizeof(initialization_functions) / sizeof(initialization_functions[0]); ++idx)
    {
        if(!initialization_functions[idx]())
        {
            ReportError();
            return false;
        }
    }
    return true;
}

但是，您的现有代码看起来不正确，因为第一个mainInit()只调用system1Init()然后退出。可能不是你想要的第一个。

if(!system1Init())
{
    ReportError();
    return false;
}
// if you add an else, the system2Init() does not get called
// even if system1Init() succeeds
if(!system2Init())
{
    ReportError();
    return false;
}
[...]
return true;

开关会回答你的问题吗？不像它写的那样。也就是说，如果你想用计数器调用mainInit()函数，它可能很有用。 Drupal使用这种机制：

bool mainInit(int idx)
{
    bool r(true);
    switch(idx)
    {
    case 1:
        r = system1Init();
        break;

    case 2:
        r = system2Init();
        break;

    [...]
    }
    if(!r)
    {
        ReportError();
    }
    return r
}

请注意，表机制的工作方式与交换机相同。只要在systemNInit()函数中找到所有代码（它应该是），交换机就不会添加任何内容，所以你也可以这样做：

bool mainInit(int idx)
{
    if(idx < 0 || idx >= sizeof(initialization_functions) / sizeof(initialization_functions[0]))
    {
        throw std::range_error("index out of bounds");
    }
    if(!initialization_functions[idx]())
    {
        ReportError();
        return false;
    }
    return true;
}

使用索引调用mainInit()可能会有所帮助，以防您想要＆＃34;取消初始化＆＃34;正确：

int main()
{
    for(size_t idx(0); idx < ...; ++idx)
    {
        if(!mainInit(idx))
        {
            while(idx > 0)
            {
                --idx;
                mainDeinit(idx);
            }
            exit(1);
        }
    }
    ...app do something here...
}

Answer 3

使用带有明确错误消息的自定义异常，并在main()中的代码周围添加try-catch-report-die。例外情况是通过隐含“坏路径”来使你的案例看起来很好。

void initX() { ...; throw std::invalid_argument_exception("..."); }

int main() {
    try {
        init1(); init2(); ... run();
        return 0;
    } catch (std::exception const& e) {
        log(e.what()); exit 42;
    }
}

Answer 4

我这样做：

bool mainInit () {
  if (!system1Init ()) {
    return(false);
  }

  if (!system2Init ()) {
    return(false);
  }

  if (!system3Init ()) {
    return(false);
  }

  //...                 

  return(true);
}

//...

if(!mainInit()) {
  reportError();
}

切换语句而不是多个嵌套if - else？

4 个答案: