python 文件对象是单次迭代器，首次遍历后即被耗尽；若在外部循环外打开命令文件，内部循环仅对第一个主机生效，后续主机因文件指针已到末尾而无命令可执行。
python 文件对象是单次迭代器，首次遍历后即被耗尽；若在外部循环外打开命令文件，内部循环仅对第一个主机生效，后续主机因文件指针已到末尾而无命令可执行。
在使用 pexpect 或 pxssh 批量操作网络设备时，一个常见却易被忽视的陷阱是：文件对象（file object）并非可重复使用的序列，而是一次性迭代器（iterator）。这正是你遇到“嵌套 for 循环只执行一次”问题的根本原因。
在你的第一段代码中：
h_ls = open(hostls, "r")
c_ls = open(commands, "r") # ← 命令文件仅打开一次
for host in h_ls:
# ... 登录逻辑 ...
for cmd in c_ls: # ← 第一个 host 迭代完后，c_ls 已耗尽
s.sendline(cmd.strip() + "\n")
# ...
c_ls 是一个文件对象，其迭代行为等价于逐行调用 __next__()。当第一个 host 的循环执行完毕时，c_ls 的内部指针已到达文件末尾（EOF）。后续所有 for cmd in c_ls: 都不会进入循环体——不是报错，而是静默跳过，导致其余主机看似“未执行命令”。
✅ 正确做法有两类，推荐后者：
方案一：每次循环重新打开文件（简单直接）
如你第二段代码所示，在 for host in h_ls: 内部打开命令文件：
for host in h_ls:
host = host.strip()
try:
# ... pxssh 登录 ...
with open(commands, "r") as c_file: # ✅ 使用 with 确保自动关闭
for cmd in c_file:
cmd = cmd.strip()
if not cmd: # 跳过空行
continue
s.sendline(cmd)
s.prompt()
print(s.before.decode() if hasattr(s.before, 'decode') else s.before)
s.logout()
except pxssh.ExceptionPxssh as e:
print(f"***pxssh failed on {host}: {e}***")
方案二：一次性读入内存，复用列表（更高效、更 Pythonic）
文件内容通常不大（配置命令一般几十行），将其加载为列表是最优实践：
# 预加载所有命令（推荐）
with open(commands, "r", encoding="utf-8") as f:
commands_list = [line.strip() for line in f if line.strip()]
# 外层循环中直接复用
for host in h_ls:
host = host.strip()
try:
s = pxssh.pxssh(timeout=5, logfile=sys.stdout, options={
"StrictHostKeyChecking": "no",
"UserKnownHostsFile": "/dev/null",
"AddKeysToAgent": "yes"
}, encoding='utf-8')
s.login(host, user, password, auto_prompt_reset=False, original_prompt=r'.*#')
print(f">>>>> Working on {host} @ {now} <p>? <strong>关键补充说明</strong>： </p>
- f.readlines() 返回带换行符的列表（如 ['show version\n', 'conf t\n']），而 [line.strip() for line in f] 更安全，自动清理空白与换行；
- 使用 with open(...) as f: 不仅语义清晰，更能确保异常时文件句柄被释放，避免资源泄漏；
- TCL/Expect 行为不同，是因为其 foreach 对文件内容默认做“重读”或内部缓存，而 Python 显式区分了 流（stream） 和 数据容器（list），这是语言设计哲学差异，非 bug。
? 总结：永远不要假设文件对象可多次迭代；若需复用，请显式转为 list、tuple 或其他可迭代容器。 这一原则同样适用于 csv.reader、jsonlines.Reader 等一次性迭代器场景。
Python免费学习笔记（深入）：立即使用
在学习笔记中，你将探索 Python 的核心概念和高级技巧！
