Dijkstra算法输出不正确

Dijkstra Algorithm not outputting correctly

本文关键字：不正确 输出 算法 Dijkstra      更新时间：2023-10-16

我在实现Dijkstra算法时遇到了问题。

我已经初始化了以下变量：

enum GraphLimit300 {MAX_NODES = 50};
typedef unsigned int Element300;
Element300 adjacencyMatrix[MAX_NODES][MAX_NODES];
Element300 distanceArray[MAX_NODES];
bool visitedSet[MAX_NODES];
int numberNodes;

以下是我目前的算法实现：int startNode；int访问；

unsigned int smallest = UINT_MAX;
do
{
    startNode = getStartNode300();
    setVisitedSet300();
    smallest = UINT_MAX;
    visitedSet[startNode] = true;
    for (int i = 0; i < numberNodes; i++)
    {
        distanceArray[i] = adjacencyMatrix[startNode][i];
    }
    for (int i = 0; i < numberNodes - 1; i++)   
    {
        for (int v = 0; v < numberNodes; v++)
        {
            if (visitedSet[v] == false)
            {
                if (distanceArray[v] < smallest)
                {
                    smallest = distanceArray[v];
                    visited = v;
                }
            }
        }
        visitedSet[visited] = true;
        for (int w = 0; w < numberNodes; w++)
        {
            if (visitedSet[w] == false)
            {
                distanceArray[w] = min(distanceArray[w], distanceArray[visited] + adjacencyMatrix[visited][w]);
            }
        }
    }

在这个特定的for循环中，它应该进行特定的数学运算，在节点为false的某个索引处找到值之间的最小值，并将最小值存储在该索引中。我发现，在它选择distaceArray中的最小值并将其设置为true之后（如果我使用下面的数据文件从1开始，则为3，这是正确的。）

    for (int w = 0; w < numberNodes; w++)
        {
            if (visitedSet[w] == false)
            {
                distanceArray[w] = min(distanceArray[w], distanceArray[visited] + adjacencyMatrix[visited][w]);
            }
        }

它使用节点"0"answers"2"，因为此特定迭代的节点为false。并计算时间，并将其存储在错误的数组中。

我正在使用这个数据集，它可以正确地存储adjacencyMatrix中的数字。

0           5          10          4294967295
4294967295  0          4294967295  3
4294967295  7          0           4294967295
4294967295  4294967295 4           0

正确的输出是：

Distance[0] =4294967295
Distance[1] =0 //Which is the node that I choose to start with
Distance[2] =7
Distance[3] =3

我得到的是：

Distance[0] =3
Distance[1] =0
Distance[2] =3
Distance[3] =3

我已经手工完成了这个过程，以确认我应该得到的正确输出是真的，它是

更新条件：

if (visitedSet[w] == false && adjacencyMatrix[visited][w] != UINT_MAX)
                {
                    distanceArray[w] = min(distanceArray[w], distanceArray[visited] + adjacencyMatrix[visited][w]);
                }

这里的问题再次出现，即您应该只考虑存在的边，即adjacencyMatrix[visited][w]!=UINT_MAX。

如果不排除这些边，distanceArray[visited] + adjacencyMatrix[visited][w]将溢出，而min()将不会返回您期望的结果。

您可以通过更改以下行来解决此问题：

if (visitedSet[w] == false && adjacencyMatrix[visited][w]!=UINT_MAX )

编辑：

嵌套的for循环中确实隐藏着另一个问题。第一个内部for循环每次都会查找要移植的最短子路径。不幸的是，您没有重置smallest，因此它从上一次迭代的最小值开始。

更新循环如下，你会得到你的预期结果：

for (int i = 0; i < numberNodes - 1; i++)   
{
    smallest = UINT_MAX; // !!!!!!!!!!!!!!!!!!!!!!!
    for (int v = 0; v < numberNodes; v++) 
    {
        ...
    }
    ...
}
相关文章：