2013-04-07 49 views
1

这里是代码,它应该从文件中读取,然后产生一个字典...类型错误:遍历所有非序列...在python

def get_songs(setlists): 
    f = file(setlists) 
    d = {} 
    next_line = f.readline() 
    while (next_line != ""): 
     if next_line[0].isdigit(): 
      concert1 = concert(int(next_line.split("-")[0]) , int(next_line.split("-")[1]) , \ 
             int(next_line.split("-")[2]) , next_line.split("-")[3].strip("\n")) 
      for i in f.readline().split('/'): 
       d[i.strip(" ").strip("\n").split("(")[0]] = (concert1 if not d.has_key(i.strip(" ").strip("\n").split("(")[0]) 
                  else ([d[i.strip(" ").strip("\n").split("(")[0]] , concert1] if 
                    isinstance(d[i.strip(" ").strip("\n").split("(")[0]] , concert) 
                    else d[i.strip(" ").strip("\n").split("(")[0]] + [concert1])) 
     next_line = f.readline()  
    return d 
+2

请出示完整的错误消息。 – BrenBarn 2013-04-07 00:14:42

+0

你应该至少重写你的程序,以避免多次执行相同的事情,即:'next_line.split(“ - ”)'可以完成一次并分配给变量 – soulcheck 2013-04-07 00:31:04

+1

这段代码充满了东西,看起来不对,你应该使用'open()'不是'file()',并且在文件中使用'for'循环而不是'file.readline()'和'with'。 – 2013-04-07 00:43:16

回答

0

目前尚不清楚错误所在。一件事我可以在代码中注意的是,在for i in ...代码之后,d [..]它可以是一个单一的值或数组:

d[..] = (concert1 
    if cond1 else 
    ([d[..], concert1] if cond2 
     else d[..]+concert1) 

我会避免把这样的表达式内联,并且如果做一个/否则,如:

tmp = None 
if cond1: 
    tmp = concert1 
elif cond2: 
    tmp = [d[d[i.strip(" ").strip("\n").split("(")[0]] , concert1] 
else: 
    tmp = d[i.strip(" ").strip("\n").split("(")[0]] + [concert1] 


d[i.strip(" ").strip("\n").split("(")[0]] = tmp 

我可以看到,最后else是有点不正确STR +阵列将是海峡恕我直言(?)?

一点题外话,而不是所有的带/修剪你可能要使用正规...