Python从文件搜索指定内容时重复输出的代码优化问题

嘿，我懂你遇到的麻烦了——你的代码现在会跟着文件的行数重复输出结果，没找到目标内容时每一行都打印一次“Not in record!”，这肯定不是你想要的效果对吧？

问题出在哪？ #

你的代码把判断和输出逻辑放在了for循环内部，每遍历文件的一行就会执行一次判断并输出结果。也就是说，文件有多少行，就会输出多少次提示，完全忽略了我们只需要一次最终结果的需求。

修复方案 #

我们可以用一个标记变量来跟踪是否找到目标内容，循环结束后再统一输出结果，这样就只会得到一次提示了。这里有两种优化后的版本：

基础修复版

def name_search(): 
    target = open("customers.txt",'r') 
    name_search = input('Search >') 
    found = False  # 初始化标记，默认未找到
    for line in target: 
        if name_search in line: 
            print(f'{name_search} in record!') 
            found = True  # 找到后更新标记
            # 如果你只需要找到第一个匹配项就停止搜索，可以加上这行
            # break
    # 循环结束后根据标记输出最终结果
    if not found:
        print('Not in record!')
    target.close()  # 记得关闭文件

更安全的优化版（推荐）

用with语句自动管理文件，避免忘记关闭文件导致的资源泄漏：

def name_search(): 
    name_search = input('Search >') 
    found = False
    # with语句会在代码块结束后自动关闭文件
    with open("customers.txt",'r') as target:
        for line in target: 
            if name_search in line: 
                print(f'{name_search} in record!') 
                found = True
                # 可选：找到匹配后立即停止遍历，提升效率
                # break
    if not found:
        print('Not in record!')

为什么这样改？ #

• 标记变量found会全程跟踪是否找到目标内容，循环里只负责更新标记和打印找到的提示（如果需要）。
• 循环结束后再根据found的值决定是否打印未找到的提示，确保最终只会输出一次结果，不管文件有多少行。

内容的提问来源于stack exchange，提问作者Benjie Malinao