Question

到目前为止，我的代码在文本文件中找到了最大的数字，但它找不到最小的数字，当它运行时，最小的数字仍为0，而最大的数字为9997.

    integers = open('numbers.txt', 'r') # opens numbers.txt

largestInt = 0 # making variables to store the largest/smallest int
smallestInt = 0

# loop where we check every line for largest/smallest int
for line in integers:
    while largestInt <= line.strip():
        largestInt = line
    while smallestInt >= line.strip():
        smallestInt = line

# print results
print "Smallest = ", smallestInt
print "Largest = ", largestInt

numbers.txt看起来像：

等等。

这里有什么问题？如果我做错了，或者逻辑错误，请纠正我。

EDIT

我要感谢@Martijn Pieters和@Gexos解释我做错了什么。我理解为什么我的代码现在有用了！

最终守则：

integers = open('numbers.txt', 'r') # opens numbers.txt

largestInt = 0 # making variables to store the largest/smallest int
smallestInt = 0

# loop where we check every line for largest/smallest int
for line in integers:
    if largestInt <= int(line.strip()): # converted a string into an int
        largestInt = int(line.strip()) # made that int into largestInt
    if smallestInt >= int(line.strip()): # converted a string into an int
        smallestInt = int(line.strip()) # made that int into smallestInt

integers.close() # closes the file

# print results
print "Smallest = ", smallestInt
print "Largest = ", largestInt

结果

Smallest =  -9993
Largest =  9997

Answer 1

您正在比较字符串，而不是整数;在比较前将您的行转换为整数：

largestInt = float('-inf')
smallestInt = float('inf')

for line in integers:
    number = int(line)
    if largestInt < number:
        largestInt = number
    if smallestInt > number:
        smallestInt = number

请注意，您要在此使用if，而不是while;后者创造了一个循环。

我分别使用largestInt和smallestInt启动了float('-inf')和float('inf')，保证数字小于等于其他任何数字。这使得第一次测试largestInt < number 总是为真，无论第一行是什么数字。

比较字符串是按字典顺序完成的，其中字符逐一进行比较; 10小于2，因为1在2之前排序。

您可以轻松使用max()和min()内置函数，但效率会低一些，因为内部这些函数也会执行循环：

numbers = {int(line) for line in integers}
largestInt = max(numbers)
smallestInt = min(numbers)

Answer 2

虽然我通常喜欢使用min和max的解决方案，但在这种情况下需要两次线性传递，并且会产生大量内存开销。这是一个需要一个线性传递和常量存储器的方法：

with open('numbers.txt') as infile:
    smallest, largest = '', None
    for line in infile:
        n = int(line)
        smallest = min(n, smallest)
        largest = max(n, largest)
    print "the smallest number is", smallest
    print "the largest number is", largest

Answer 3

正如其他人所说，您需要先将行转换为整数。此外，如果该数字大于零，您的脚本将不会输出正确的最小数字。要解决此问题，请将最大数量和最小数量都设置为文件中的第一个条目。然后检查所有其他数字，看看它们是否比当前数字更大/更小。

with open('numbers.txt', 'r') as data_file:
    num = int(next(data_file))
    min_num, max_num = num, num
    for line in data_file:
        num = int(line)
        if num > max_num:
            max_num = num
        elif num < min_num:
            min_num = num

print 'Smallest number: {}'.format(min_num)
print 'Largest number: {}'.format(max_num)

这也可以通过列表推导来解决：

nums = [int(line) for line in open('numbers.txt', 'r')]
min_num, max_num = min(nums), max(nums)

Answer 4

with open('number.txt') as f:
    s = [int(n.strip()) for n in f]
    print min(s), max(s)

Answer 5

一些事情。您需要将line转换为字符串中的int：int(line.strip())目前您正在将int与字符串进行比较。您还应该在largestInt = int(line.strip())中投射作业，并为smallestInt投射相同的作品。

您不应该使用while。 while用于循环，而不是用于比较。您应该使用if。

最后但并非最不重要的是确保最后关闭文件。 integers.close()

Answer 6

integers = open('numbers.txt', 'r')
intList = [int(x) for x in integers.readlines()]
print max(intList), min(intList)

Answer 7

您正在比较字符串，而不是整数。你需要在最有可能将它们转换为数字的某个阶段调用int函数。

这是一个不同方法的例子：

with open('numbers.txt', 'r') as f:
    integers = [int(n) for n in f.readlines()]

smallest = min(integers)
biggest = max(integers)

使用with可确保文件在列表理解后自动关闭，这是一种很好的做法。列表理解结果如下：

[6037, -2228, 8712, -5951, 9485, 8354, 1467, 8089, 559, 9439, -4274, 9278, -813, 1156, -7528, 1843, -9329, 574]

然后在该列表中调用min和max。

Answer 8

将tinyInt设置为0时需要注意的一件事。如果文本文档中包含的最小数字为1，那么您仍然会以0作为答案。

如何在文本文件中找到最小的数字（Python 2.5）

8 个答案: