24

我有这段代码,由于某种原因,当我尝试返回路径时,我得到None了:

def get_path(dictionary, rqfile, prefix=[]):        
    for filename in dictionary.keys():
        path = prefix + [filename]
        if not isinstance(dictionary[filename], dict):          
            if rqfile in str(os.path.join(*path)):
                return str(os.path.join(*path))
        else:
            get_path(directory[filename], rqfile, path)

有没有办法解决这个问题?

4

2 回答 2

45

您需要返回递归结果:

else:
   return get_path(directory[filename], rqfile, path)

否则,函数在执行该语句后简单地结束,导致None返回。

你可能想放弃else:总是在最后返回:

for filename in dictionary.keys():
    path = prefix+[filename]
    if not isinstance(dictionary[filename], dict):

        if rqfile in str(os.path.join(*path)):
            return str(os.path.join(*path))

    return get_path(directory[filename], rqfile, path)

因为如果rqfile in str(os.path.join(*path))False那么你也结束了你的功能没有 a return。如果在这种情况下递归不是正确的选择,但返回None不是,您也需要处理该边缘情况。

于 2013-10-06T23:26:31.050 回答
4

虽然我认为 Martijn Pieters 的回答解决了他回答中的主要问题(您需要从递归案例中返回),但我认为他建议的代码不会正常工作。

rqfile您正在尝试对嵌套dictionary字典中的值进行深度优先搜索。但是您当前的代码没有正确处理递归情况。如果在其递归调用之一中找到结果,或者如果递归调用未能找到目标,它需要做出适当的响应。

这是我认为您需要的,为了清楚起见,对某些内容进行了重命名或重新排列:

def get_path(directory, rqfile, prefix=[]):
    for filename, value in directory.items():
        path_list = prefix + [filename]
        if not isinstance(value, dict): # base case
            path = os.path.join(*path_list)
            if rqfile in path:   # Found the file. Do you want to do something
                return path      # with the value here, or is it junk?

        else: # recursive case
            try:
                return get_path(value, rqfile, path_list) # this only returns if 
            except ValueError:                     # the recursion doesn't raise
                pass

    raise ValueError("Requested file not found") # not found here or in children

示例用法:

>>> directory = {"a": "a info",
                 "b": {"c": "b/c info", "d": "b/d info"},
                 "e": {"f": "e/f info", "g": {"h": "e/g/h info"}}}
>>> print(get_path(directory, "h"))
e\g\h
>>> print(get_path(directory, r'g\h'))
e\g\h

如果您不想在找不到文件时引发异常,您还可以返回一个哨兵值None代替最后一行,并在递归情况下检查哨兵值而不是try/ except

 result = get_path(value, rqfile, path)
 if result is not None:
     return result
于 2013-10-07T06:42:42.570 回答