2016-05-15 14 views
-1

Ich habe die Aufgabe, Klassen für Dijkstra-Algorithmus zu schreiben. Obwohl ich bin nicht der Dijkstra-Klasse zu bearbeiten erlaubt:NoneType-Objekt ist nicht iterierbar Fehler

class Dijkstra(): 
# initialize with a string containing the root and a 
# weighted edge list 
def __init__(self, in_string): 
    self.root, self.nnodes, self.adj_list = self.convert_to_adj_list(in_string) 
    self.nodes = [Node(i) for i in range(self.nnodes)] 
    self.nodes[self.root].key = 0 
    self.heap = MinHeap(self.nodes) 
# the input is expected to be a string 
# consisting of the number of nodes 
# and a root followed by 
# vertex pairs with a non-negative weight 
def convert_to_adj_list(self, in_string): 
    nnodes, root, edges = in_string.split(';') 
    root = int(root) 
    nnodes = int(nnodes) 
    adj_list = {} 
    edges = [ map(int,wedge.split()) for wedge in edges.split(',')] 
    for u,v,w in edges: 
     (adj_list.setdefault(u,[])).append((v,w)) 
    for u in range(nnodes): 
     adj_list.setdefault(u,[]) 

Das ist mein Problem:

string = '3; 0; 1 2 8, 2 0 5, 1 0 8, 2 1 3' 
print(Dijkstra(string)) 
Traceback (most recent call last): 
    File "<pyshell#321>", line 1, in <module> 
    print(Dijkstra(string)) 
    File "C:\Users\TheDude\Downloads\dijkstra.py", line 71, in __init__ 
    self.root, self.nnodes, self.adj_list = self.convert_to_adj_list(in_string) 
TypeError: 'NoneType' object is not iterable 

Muss ich append ‚s Rückgabewert zuweisen? Und wie kann ich es ohne Bearbeitung beheben class Djikstra() Tanks zum Lesen.

+0

'convert_to_adj_list' kehrt' None', da man es nicht mit einer return-Anweisung liefern hat. – miradulo

+0

Sie sollten einen Rückgabewert von 'convert_to_adj_list' haben, also fügen Sie' return adj_list' am Ende der 'convert_to_adj_list' Definition hinzu. –

+0

Wenn ich also keinen Rückgabewert festlege, gibt jede Funktion None zurück? Ich muss die Dijkstra-Klasse verwenden, also muss ich einen Lehrer kontaktieren, um das zu beheben. – TheDude

Antwort

2

Damit die Zuordnung

self.root, self.nnodes, self.adj_list = self.convert_to_adj_list(in_string) 

zu arbeiten, hat convert_to_adj_list ein Tupel von drei Werten zurück in diese drei Variablen ausgepackt werden. Ihre Methode gibt jedoch nichts zurück (und gibt implizit None zurück). Ändern Sie Ihre convert_to_adj_list Methode, um so etwas wie dieses, dann sollte es funktionieren:

def convert_to_adj_list(self, in_string): 
    ... your code ... 
    return root, nnodes, adj_list 
+0

Danke Tobias! – TheDude