【问题标题】:Python algorithm error when trying to find the next largest value尝试找到下一个最大值时的Python算法错误
【发布时间】:2015-06-20 17:37:29
【问题描述】:

我编写了一个算法,它扫描“ID”文件并将该值与整数 i 的值进行比较(我已将整数转换为字符串以进行比较,并且我已经修剪了“\ n" 前缀)。该算法比较文件中每一行(每个 ID)的这些值。如果它们相等,则算法将 i 增加 1,并使用新的 i 值进行递归。如果该值不相等,则将其与文件中的下一行进行比较。它会执行此操作,直到 i 的值不在文件中,然后返回该值以用作下一条记录的 ID。

我的问题是我有一个 ID 文件,其中列出了 1、3、2,因为我删除了 ID 为 2 的记录,然后创建了一个新记录。这表明该算法正常工作,因为它为新记录提供了 ID 2,该 ID 之前已被删除。但是,当我创建新记录时,下一个 ID 为 3,导致我的 ID 列表显示为:1,3,2,3 而不是 1,3,2,4。 Bellow 是我的算法,带有print() 命令的结果。我可以看到哪里出了问题,但不知道为什么。有什么想法吗?

算法:

def _getAvailableID(iD):
        i = iD
        f = open(IDFileName,"r")
        lines = f.readlines()
        for line in lines:
            print("%s,%s,%s"%("i=" + str(i), "ID=" + line[:-1], (str(i) == line[:-1])))
            if str(i) == line[:-1]:
                i += 1
                f.close()
                _getAvailableID(i)
        return str(i)

输出: (运行算法以查找 ID 为 4 的记录的适当 ID 时的输出):

i=1,ID=1,True
i=2,ID=1,False
i=2,ID=3,False
i=2,ID=2,True
i=3,ID=1,False
i=3,ID=3,True
i=4,ID=1,False
i=4,ID=3,False
i=4,ID=2,False
i=4,ID=2,False
i=2,ID=3,False
i=2,ID=2,True
i=3,ID=1,False
i=3,ID=3,True
i=4,ID=1,False
i=4,ID=3,False
i=4,ID=2,False
i=4,ID=2,False

【问题讨论】:

  • 您是否有理由必须使用递归函数,或者您可以使用列表(或列表)或字典?

标签: python algorithm file output


【解决方案1】:

我认为你的程序失败了,因为你需要改变:

_getAvailableID(i)

 return _getAvailableID(i)

(此时递归函数找到正确答案被丢弃。)

但是,最好将您看到的所有 id 简单地放入一个集合中以使程序更高效。

例如在伪代码中:

S = set()
loop over all items and S.add(int(line.rstrip()))
i = 0
while i in S:
   i += 1
return i

【讨论】:

  • 如果我放return _getAvailableID(i),它会返回到_getAvailableID(i)的第一次调用,这违背了递归的目的吗?
  • 我很抱歉,你是对的。现在可以了,非常感谢:)
【解决方案2】:

如果您只是在文件中查找最大 ID,然后想要返回下一个可用值:

def _getAvailableID(IDFileName):
    iD = '0'
    with open(IDFileName,"r") as f:
        for line in f:
            print("ID=%s, line=%s" % (iD, line))
            if line > iD:
                iD = line

    return str(int(iD)+1)

print(_getAvailableID("IDs.txt"))

输入文件包含

1
3
2

输出

ID=1, line=1

ID=1
, line=3

ID=3
, line=2

4

但是,我们可以用更 Pythonic 的方式来解决它:

def _getAvailableID(IDFileName):
    with open(IDFileName,"r") as f:
        mx_id = max(f, key=int)
    return int(mx_id)+1

【讨论】:

  • 是的,这是正确的。最小 ID 始终为 1
猜你喜欢
  • 2015-10-13
  • 1970-01-01
  • 2023-03-27
  • 1970-01-01
  • 2016-04-04
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2022-11-12
相关资源
最近更新 更多