Python中max()与min()函数获取日志数据最值异常问题

The issue here is that you're comparing raw strings directly, not the numeric values tied to each operation.

In Python, when you use max() or min() on strings, it relies on lexicographical (dictionary) order to determine "larger" or "smaller" values:

• For example, "operation6,38999" is considered larger than "operation5,54546" because the character "6" comes after "5" in ASCII order—even though 38999 is way smaller than 54546.
• Similarly, "operation1,12298" is the smallest string lexicographically, but its numeric value isn't actually the smallest in your dataset.

Also, a quick note: your readlines() call reads the file as a list of lines. Since your log data is all on one line separated by spaces, data ends up being a list with just one string element. Even if you split that into individual operations, comparing those strings directly still won't give you the numeric max/min you want.

To get the correct max and min based on the numeric values, you need to extract those numbers, convert them to integers, and use them as the basis for comparison.

Here's the corrected, concise code:

# Read and parse the log file properly
with open('ops.log', 'r') as f:
    # Read full content, strip extra whitespace, split into individual operations
    all_ops = f.read().strip().split()

# Use a key function to compare the numeric part of each operation
max_operation = max(all_ops, key=lambda op: int(op.split(',')[1]))
min_operation = min(all_ops, key=lambda op: int(op.split(',')[1]))

print("Correct max:", max_operation)
print("Correct min:", min_operation)

When you run this, you'll get the actual numeric extremes:

• Max: operation5,54546 (since 54546 is the largest number)
• Min: operation5,344 (since 344 is the smallest number)

The key parameter in max() and min() is the critical piece here:

• For each operation string (like "operation2,72321"), the lambda function splits the string at the comma, takes the second part (the number string), and converts it to an integer.
• max() and min() then use these integer values to determine which operation is the largest/smallest, instead of comparing the raw strings directly.

If you prefer a more explicit approach (great for debugging), you can pre-process operations into tuples of (numeric value, operation string):

with open('ops.log', 'r') as f:
    all_ops = f.read().strip().split()

# Convert each operation to a tuple of (number, operation string)
processed_ops = []
for op in all_ops:
    name, num_str = op.split(',')
    processed_ops.append( (int(num_str), op) )

# Max/min will use the first element of the tuple (the numeric value)
max_op = max(processed_ops)[1]
min_op = min(processed_ops)[1]

print(max_op, min_op)

This does the same thing, but makes the conversion step more visible.

内容的提问来源于stack exchange，提问作者Nishant Singh