c++ - 我把这个 if 语句放在哪里会有所不同吗？

Question

所以我在一个带有链接列表的实验室工作，我遇到了一个部分，如果在遍历列表后没有找到元素，我必须输出一条错误消息。我的问题是，将这个 if 语句放在 while 循环之后还是放在 else 语句之后，难道不应该有所不同吗？如果我把它放在第二个 else 之后，函数不应该在 else 执行完后立即结束吗？我试着运行它，但我仍然得到相同的预期输出，但我试图弄清楚把它放在哪里最有意义，因为实验室是根据效率和风格进行分级的。

void AnyList::deleteNode(int deleteData)
{
    if (count == 0)
    {
        cerr << "Cannot delete from an empty list.\n";
    }
    else
    {
        Node *current = ptrToFirst; 
        bool found = false; 

        if (current->getData() == deleteData)
        {
            ptrToFirst = current->getPtrToNext();
            delete current;
            current = NULL;
            count--; 
            found = true; 
        }
        else
        {

            Node *trailCurrent = current;
            current = current->getPtrToNext(); 

            while (current != NULL && !found)
            {
                if (current->getData() == deleteData)
                {
                    trailCurrent->setPtrToNext(current->getPtrToNext());
                    delete current;
                    current = NULL;
                    trailCurrent = NULL;
                    count--;
                    found = true;
                }
                else
                {
                    trailCurrent = current; 
                    current = current->getPtrToNext();
                }
            }
            //if (!found)             <-- Does it make more sense to put it here?
            //{
            //  cerr << "Item to be deleted is not in the list.\n";
            //}

        }
        //if (!found)                 <-- Or here?
        //{
        //  cerr << "Item to be deleted is not in the list.\n";
        //}
    }

}

Answer 1

将 if (!found) block 放在它可能实际运行的更有限的范围内，有助于读者理解和维护代码。

没有功能差异(即程序以两种方式做同样的事情)。

应该没有性能差异(优化构建)。当 current->getData() == deleteData 为 true 时，它将设置 found true，所以有'检查 if (!found) 毫无意义。您的编译器应该足够聪明，可以从 if (current->getData() == deleteData) 控制的语句 block 末尾跳到 if ( !found) { ... } block ，在这种情况下，性能将与拥有 `if (!found) 是更本地化的范围相同。