Python解析XML时多层嵌套if代码臃肿，如何优化提升可读性？

以下为三种可行方案，可按需选择： #

方案1：卫语句提前终止 #

通过把不满足条件的分支提前结束，从根源消除嵌套层级：

root = ET.fromstring(xml_output)
host = root.find('host')
# 不满足条件直接结束，无需进入后续逻辑
if not host:
    # 可根据需求替换为日志打印、异常抛出逻辑，主脚本层级无外层函数时可换为exit()
    return
ports = host.find('ports')
if not ports:
    return
for port in ports:
    service = port.find('service')
    # 无service节点直接跳过当前端口
    if not service:
        continue
    print(service.attrib.get('name'))

优势：逻辑直观无嵌套，后续新增同层级字段获取逻辑直接在对应判断后追加即可，无需额外嵌套。 #

方案2：XPath直接定位节点 #

利用Python内置ElementTree支持的XPath语法，直接定位目标节点，省去逐层判断逻辑：

root = ET.fromstring(xml_output)
# 直接匹配所有符合路径的service节点，无匹配结果时返回空列表，不会抛出异常
services = root.findall('./host/ports/port/service')
for service in services:
    print(service.attrib.get('name'))

优势：代码高度精简，没有多余的层级判断，后续需要新增其他字段提取时，新增对应XPath查询逻辑即可，和原有逻辑完全解耦。 #

方案3：封装公共节点查询函数 #

如果后续需要频繁查询不同路径的节点，可把判断逻辑封装成公共函数复用：

def safe_find(parent_node, tag_name):
    """安全查询子节点，父节点为空时直接返回None"""
    return parent_node.find(tag_name) if parent_node is not None else None

root = ET.fromstring(xml_output)
host = safe_find(root, 'host')
ports = safe_find(host, 'ports')
for port in ports if ports is not None else []:
    service = safe_find(port, 'service')
    if service:
        print(service.attrib.get('name'))

优势：消除重复的节点存在性判断代码，后续拓展时维护成本更低。 #

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