Question

我一直致力于为用户输入机场名称的机场制定成本最低的算法，然后我运行此算法来吐出您可以去的所有目的地，并且成本最低，包括转机航班。我使用列表迭代器来迭代来自源位置的可到达目的地，但是在一次迭代之后，代码中断并且出现一条消息告诉我迭代器不可解除引用。这是我的代码

//Finds minimum cost
void findPaths(std::string source)
{
    std::list<int> Reachable;
    int min = INTMAX_MAX;
    int lowestIndex = -1;
    bool existsInList = true;
    std::stack<std::string> connectingFlights;
    //Make arrays
    //Initialize costs to a high value so any value will be smaller
    int costs[MAX]{INTMAX_MAX};

    //Initialize paths to negative one so that we know there is no location
    int path[MAX]{ -1 };

    //Find where the source is
    int srcIndex = findOrInsert(source);

    //Put the costs into the array, leaving the high number for where there    is no path
    for (int i = 0; i < MAX; i++)
    {
        costs[i] = priceEdges[srcIndex][i];
    }

    //Put the source index in places that have a path
    for (int i = 0; i < MAX; i++)
    {
        if (priceEdges[srcIndex][i] == 0)
        {
            path[i] = -1;
        }
        else
        {
            path[i] = srcIndex;
            Reachable.push_back(i);
        }
    }

    //If the list is empty, we are done;
    while (!Reachable.empty())
    {
        //Find the index that has the lowest value in costs
        for (std::list<int>::iterator it = Reachable.begin(); *it < Reachable.size(); it)
        {
            if (costs[*it] < min)
            {
                min = costs[*it];
                int lowestIndex = *it;
            }

            //Remove the index with the lowest value in costs
            Reachable.erase(it);

            //Save the previous cost to compare after a change may occur
            int prevCost = costs[lowestIndex];

            //Assign the value to the lowest cost it can find
            costs[lowestIndex] = FindMin(costs[lowestIndex], costs[srcIndex] + priceEdges[srcIndex][lowestIndex]);

            //If the price has changed
            if (prevCost != costs[lowestIndex])
            {
                path[lowestIndex] = srcIndex;
            }
            existsInList = std::find(Reachable.begin(), Reachable.end(), lowestIndex) != Reachable.end();
            if (!existsInList)
            {
                Reachable.push_back(lowestIndex);
            }
        }
    }

Answer 1

你的for循环是完全错误的。您正在取消引用迭代器而不验证它是否有效，并且您正在比较迭代器引用的目标值与向量的 size ，这没有任何意义它们是两个完全不同的东西。

您需要使用此替换循环：

for (std::list<int>::iterator it = Reachable.begin(); it != Reachable.end(); )

甚至这个：

std::list<int>::iterator it = Reachable.begin();
while (it != Reachable.end())

然后，为了满足循环的停止条件，您需要更改此行：

Reachable.erase(it);

改为：

it = Reachable.erase(it);

您正在从list中删除一个使迭代器无效的项，但是您永远不会更新迭代器以指向下一个项，因此代码在尝试再次取消引用迭代器时会遇到问题。 erase()将一个迭代器返回到正被删除的项后面的列表中的下一个项目。

另外，在这一行：

int lowestIndex = *it;

您正在声明一个新的临时变量，此后会立即超出范围，因此永远不会使用它。您在函数开头声明了一个先前的lowestIndex变量，您在初始化之后永远不会为其赋值，因此它始终为-1。您需要从作业中删除int：

lowestIndex = *it;

Answer 2

//Remove the index with the lowest value in costs
Reachable.erase(it);

这使迭代器无效，但for循环执行*it < Reachable.size()，它取消引用无效的迭代器。相反，应该这样做。

it = Reachable.erase(it);

此外，*it < Reachable.size()可能应该替换为it != Reachable.end()。

最后，for循环的增量部分应该是空的，因为它没有做任何事情。你也可以使用while循环。

auto it = Reachable.begin();
while (it != Reachable.end())
{
    // ...

    it = Reachable.erase(it);

    // ...
}

列表迭代器不可解除引用

2 个答案: