如何改进Python循环以实现多FASTA序列存入字典？

你的代码失效核心原因是变量名冲突：一开始用name存储文件路径，后续又用它存储序列名称，同时缺少对序列名称变量的初始校验，导致多序列场景下逻辑混乱。另外，处理序列名称时的字符截取方式不够健壮，容易保留多余的换行符或空格。

改进后的代码 #

from collections import defaultdict

# 重命名文件路径变量，避免和序列名冲突
file_path = '/Users/Tom/OneDrive/Desktop/projekt/sequences.fasta'
seq_dict = defaultdict(str)
current_name = None  # 初始化当前序列名，避免未定义错误

# 使用with语句自动管理文件资源，更安全
with open(file_path, 'r') as f:
    for line in f:
        line = line.strip()  # 先去掉每行首尾的空白（换行、空格等）
        if not line:  # 跳过空行
            continue
        if line.startswith('>'):
            # 提取序列名称：去掉开头的>，再去一次空白避免多余空格
            current_name = line[1:].strip()
        else:
            # 确保当前有合法的序列名才拼接序列
            if current_name is not None:
                seq_dict[current_name] += line

# 测试输出
for name, sequence in seq_dict.items():
    print(f"{name}: {sequence}")

关键改进点 #

• 变量名分离：将文件路径变量改为file_path，和存储序列名的current_name彻底区分，避免覆盖。
• 初始化校验：提前初始化current_name为None，处理序列行前先判断是否已获取合法的序列名称，避免文件开头无>行时的报错。
• 健壮的文本处理：对每行先执行strip()，跳过空行，同时提取序列名称时再做一次strip()，确保名称干净无多余空白。
• 资源安全：使用with语句打开文件，无需手动调用close()，避免文件泄漏。

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