Я пытаюсь реализовать алгоритм Warshall в Python 3, чтобы создать матрицу с кратчайшим расстоянием между каждой точкой.

Это должна быть простая реализация, я делаю матрицу и заполняю ее расстоянием между каждой точкой.

Тем не менее, я получаю неправильный результат, и я не знаю, в чем проблема с моей реализацией.

#number of vertex (N), number of connections(M)
N, M = 4,4;
#my matrix [A,B,C] where A and B indicates a connection
#from A to B with a distance C 
A = [[0,1,2],[0,2,4],[1,3,1],[2,3,5]];
#matrix alocation
inf = float("inf");
dist = [[inf for x in range(N)] for y in range(M)];

#set distances from/to the same vertex as 0
for vertex in range(N):
    dist[vertex][vertex] = 0;
#set the distances from each vertex to the other
#they are bidirectional. 
for vertex in A:
    dist[vertex[0]][vertex[1]] = vertex[2];
    dist[vertex[1]][vertex[0]] = vertex[2];

#floyd warshall algorithm
for k in range(N):
    for i in range(N):
        for j in range(N): 
            if dist[i][j] > dist[i][k] + dist[k][j]:
                dist[1][j] = dist[i][k] + dist[k][j];
print(dist);

Ожидаемая матрица по первому индексу (dist [0]):

[0, 2, 4, 3]

Фактический результат:

[0, 2, 4, инф.]

По некоторым причинам я продолжаю получать inf вместо 3 на dist [0] [3]. Что мне не хватает?

1
Helio Ramos 18 Окт 2019 в 22:45

1 ответ

Лучший ответ

Это немного сложно определить, но простая трассировка изменений вашей программы выявляет проблему:

        if dist[i][j] > dist[i][k] + dist[k][j]:
            dist[1][j] = dist[i][k] + dist[k][j];
                 ^  This should be i, not 1

Вы меняете расстояние от узла 1 до целевого узла; а не из исходного узла. Ваша полученная матрица расстояний

[0, 2, 4, 3]
[2, 0, 6, 1]
[4, 6, 0, 5]
[3, 1, 5, 0]

За помощью обратитесь к этому прекрасному блогу debug.

4
Prune 18 Окт 2019 в 20:01