ValueError错误含义解析及Python读取文件取最大整数代码问题

首先说下这个错误的意思：ValueError: invalid literal for int() with base 10: '-' 表示你尝试把单独的减号'-'转换成整数，但这并不是一个有效的十进制整数——有效的整数格式应该是-123、456这类，单独的减号不符合整数的定义，所以触发了转换失败的报错。

你的代码问题分析 #

1. 换行符处理逻辑有漏洞：你用line[0:-1]来移除换行符，但这种方式很不可靠：
   • 如果某一行内容就是'-\n'，截取后会得到纯'-'，转int时直接报错；
   • 如果文件最后一行没有换行符，line[0:-1]会切掉最后一个字符，比如行内容是'100'会变成'10'，要是行只有单个字符还会变成空字符串，同样会触发转int错误。
2. 没处理无效行/空行：如果文件里存在空行、或者只有符号/字母的无效内容，你的代码会直接强行转int，必然报错。
3. 文件资源管理不严谨：手动调用infile.close()有风险，要是代码中途报错，文件可能无法正常关闭，最好用自动管理资源的写法。

修复后的代码 #

def max_num_in_file(filename):
    """ returns the largest integer found in file, as an integer. """
    num_list = []
    # 用with语句自动管理文件，无需手动close
    with open(filename, "r") as infile:
        for line in infile:
            # 移除前后所有空白（包括换行、空格、制表符）
            stripped_line = line.strip()
            # 跳过空行
            if not stripped_line:
                continue
            try:
                # 尝试转换为整数，捕获转换失败的情况
                num = int(stripped_line)
                num_list.append(num)
            except ValueError:
                # 遇到无效行可以选择跳过，也可以打印提示方便排查
                print(f"跳过无效行内容: {stripped_line}")
                continue
    # 先检查列表是否为空，避免无有效整数时调用max报错
    if num_list:
        return max(num_list)
    else:
        raise ValueError("文件中未找到有效的整数")

关键修改说明 #

• 改用with语句打开文件，不管代码是否报错，都能自动释放文件资源；
• 用line.strip()替代line[0:-1]，更安全地处理换行符和多余空白；
• 增加try-except捕获转换错误，遇到无效行时跳过，避免程序直接崩溃；
• 加入空列表检查，防止文件全是无效内容时调用max()触发新的报错。

内容的提问来源于stack exchange，提问作者James Lee