替代连续尝试/排除块

Question

我对 Python 还很陌生，所以我想知道是否有比运行大量连续的 try/except 块更简洁的替代方法，如下所示？

try:
    project_type = body.find_element_by_xpath('./div[contains(@class, "discoverableCard-type")]').text
except Exception:
    project_type = 'Error'
try:
    title = body.find_element_by_xpath('./div[contains(@class, "discoverableCard-title")]').text
except Exception:
    title = 'Error'
try:
    description = body.find_element_by_xpath('./div[contains(@class, "discoverableCard-description")]').text
except Exception:
    description = 'Error'
try:
    category = body.find_element_by_xpath('./div[contains(@class, "discoverableCard-category")]').text
except Exception:
    category = 'Error'
...

正如this thread 或this thread, 中所建议的那样，我想我可以创建变量名和查询列表，然后使用for 循环为每个容器项构造一个字典，但真的没有其他选择哪些可能更具可读性？

Answer 1

你的代码有很多混乱的原因是因为你有重复的代码。您连续四次表达相同的想法（查找一个值，如果失败则设置默认值），这自然意味着您也必须编写四次相同的支持代码。

循环是修复重复代码的好方法 - 因此使用名称列表查找并创建值字典是您的完美解决方案。这使您可以编写逻辑一次，然后多次使用它。

（另外：您的代码重复导致了一个错误！前两个 try-except 块将 'Error' 值分配给 description 而不是适当的变量。重复代码可能会咬人！）

Answer 2

您可以将调用抽象为find_element_by_xpath；这避免了代码重复，并使您的代码更具可读性：

def _find_element_by_xpath(body, xpath)
    try:
        return body.find_element_by_xpath(xpath).text
    except Exception:   # <-- Be specific about the Exception to catch
        return 'Error'

def get_a_specific_xpath(element):
    return f'./div[contains(@class, "discoverableCard-{element}")]'

那么你的代码就变成了：

project_type = _find_element_by_xpath(body, get_a_specific_xpath('project_type'))
title = _find_element_by_xpath(body, get_a_specific_xpath('title'))
description = _find_element_by_xpath(body, get_a_specific_xpath('description'))
category = _find_element_by_xpath(body, get_a_specific_xpath('category'))
...