如何处理从不执行的代码

Question

我有一些看起来像这样的代码，我不确定如何处理永远不会执行的部分，因为该代码的一部分在等待连接时无限循环运行，当我终止程序时，它会从只有那里。

main(){

// do some stuff....

    while(1) {
        int newFD =
            accept(sockFD, (struct sockaddr *)&client_addr, &client_addr_size);
        if(newFD == -1) {
            std::cerr << "Error while Accepting on socket" << std::endl;
            continue;
        }

        if(!fork()) {

            close(sockFD); // close child's sockfd - not needed here

            // lalala do stuff send message here                

            close(newFD);  // finally close its newFD - message sent, no use 
            return 0;
        }
        close(newFD);  // close parent's newFD - no use here
    }

    // now execution never reaches here
    close(sockFD);      // so how to handle this?
    freeaddrinfo(res);  // and this?

    return 0;
}

Answer 1

如果您的代码要被其他人使用或者您自己只是希望它更干净，您可以并且可能应该添加退出处理程序。在您的退出处理程序中，您可以切换一个使while() 循环终止的标志。以下代码可以 100% 正常工作，并且可靠且跨平台，但如果您想做更复杂的事情，您应该使用适当的线程安全操作系统特定函数或类似 Boost 或 C++11 的东西

首先声明两个全局变量，将它们设为 volatile，这样编译器将始终强制我们读取或写入其实际内存值。如果我们不将其声明为 volatile，那么编译器可能会将其值放入寄存器中，这将使其不起作用。使用 volatile 设置，它会在每个循环中读取内存位置并正常工作，即使使用多个线程也是如此。

volatile bool bRunning=true;
volatile bool bFinished=false;

而不是您的 while(1) {} 循环，将其更改为此

while(bRunning)
{
    dostuff
}
bFinished=true;

在您的退出处理程序中只需设置bRunning=false;

void ExitHandler()
{
    bRunning=false;
    while(bFinished==false) { Sleep(1); }
}

您没有指定操作系统，但看起来您是基于 Linux 的，要在 Linux 上设置处理程序，您需要这个。

void ExitHandler(int s)
{
    bRunning=false;
}

int main()
{
    struct sigaction sigIntHandler;
    sigIntHandler.sa_handler = ExitHandler;
    sigemptyset(&sigIntHandler.sa_mask);
    sigIntHandler.sa_flags = 0;
    sigaction(SIGINT, &sigIntHandler, NULL);
    while(bRunning)
    {
        dostuff
    }
    ...error_handling...
}

在 Windows 上，当您是控制台应用程序时，它如下所示。

BOOL WINAPI ConsoleHandler(DWORD CEvent)
{
    switch (CEvent)
    {
        case CTRL_C_EVENT:
        case CTRL_BREAK_EVENT:
        case CTRL_CLOSE_EVENT:
        case CTRL_LOGOFF_EVENT:
        case CTRL_SHUTDOWN_EVENT:
            bRunning = false;
            while (bFinished == false) Sleep(1);
            break;
    }
    return TRUE;
}

int main()
{
    SetConsoleCtrlHandler(ConsoleHandler, TRUE);
    while(bRunning()
    {
        dostuff
    }
    ...error_handling...
}

请注意这里需要测试并等待bFinished。如果您不在 Windows 上执行此操作，您的应用程序可能没有足够的时间关闭，因为退出处理程序由单独的操作系统特定线程调用。在 Linux 上，这不是必需的，您需要退出处理程序才能继续主线程。

另外需要注意的是，默认情况下，Windows 在终止您之前只给您大约 5 秒的时间来关闭它。这在许多情况下是不幸的，如果需要更多时间，您将需要更改注册表设置（坏主意）或实现一个具有更好挂钩的服务。对于您的简单情况，它会很好。

Answer 2

对于这些事情，操作系统会在关机时妥善释放资源。但是，更一般地，您仍然需要确保分配的资源在程序执行期间不会堆积，即使它们被操作系统自动回收，因为这样的资源泄漏仍然会影响程序的行为和性能。

现在，关于手头的资源，没有理由不将它们视为 C++ 中的所有资源。公认的规则是将它们绑定到将在析构函数中释放它们的对象，另请参见 RAII 习语。这样，即使在稍后的某个阶段有人添加了 break 语句，代码仍然会正常运行。

顺便说一句：我在这里看到的更严重的问题是缺乏适当的错误处理。