英文:
How to convert a single object or a set into a set?
问题
以下是翻译好的代码部分，不包括问题的回答：
有以下方法：```pythondef send_message(content: str,slack_conn_ids: Union[SlackConnection, Set[SlackConnection]],send_only_in_production: bool = True,):...if isinstance(slack_conn_ids, set):set_slack_conn_ids = slack_conn_idselif isinstance(slack_conn_ids, SlackConnection):set_slack_conn_ids = {slack_conn_ids}else:raise ValueError("`slack_conn_ids`应该是`SlackConnection`或`Set[SlackConnection]`类型")...
这段代码是否可以更简化，更符合惯用法？<details><summary>英文:</summary>There's a method as follows:
def send_message(
content: str,
slack_conn_ids: Union[SlackConnection, Set[SlackConnection]],
send_only_in_production: bool = True,
):
...
if isinstance(slack_conn_ids, set):
set_slack_conn_ids = slack_conn_ids
elif isinstance(slack_conn_ids, SlackConnection):
set_slack_conn_ids = {slack_conn_ids}
else:
raise ValueError("slack_conn_ids should be of type SlackConnection or Set[SlackConnection]"
)
...
Can this code be simplified, implemented more idiomatically?</details># 答案1**得分**: 2可以使用模式匹配。```py匹配 slack_conn_ids:情况 set():set_slack_conn_ids = slack_conn_ids情况 SlackConnection():set_slack_conn_ids = { slack_conn_ids }情况 _:引发 ValueError("‘slack_conn_ids’ 应该是 ‘SlackConnection’ 或 ‘Set[SlackConnection]’ 类型")
英文:
You can use pattern matching.
match slack_conn_ids:case set():set_slack_conn_ids = slack_conn_idscase SlackConnection():set_slack_conn_ids = { slack_conn_ids }case _:raise ValueError("`slack_conn_ids` should be of type `SlackConnection` or `Set[SlackConnection]`")
答案2
得分: 2
不。让调用方负责提供一个集合，即使该集合只包含一个连接。
def send_message(content: str,slack_conn_ids: Set[SlackConnection],send_only_in_production: bool = True,):...s = SlackConnection(...)send_message("hi", {s})
英文:
Don't. Let the caller be responsible for providing a set, even if that set only contains one connection.
def send_message(content: str,slack_conn_ids: Set[SlackConnection],send_only_in_production: bool = True,):...s = SlackConnection(...)send_message("hi", {s})
答案3
得分: -2
你的代码已经非常简化，很难再有什么改进的余地。
以下是我设计的代码：
if isinstance(slack_conn_ids, SlackConnection) or isinstance(slack_conn_ids, set):try:slack_conn_ids = {slack_conn_ids}except TypeError:passelse:raise ValueError("‘slack_conn_ids’ 应该是 ‘SlackConnection’ 或 ‘Set[SlackConnection]’ 类型")
英文:
Your code is already very simplified, it is impossible to come out with something worth the difference.
Here is how I would design that code:
if isinstance(slack_conn_ids, SlackConnection) or isinstance(slack_conn_ids, set):try:slack_conn_ids = {slack_conn_ids}except TypeError:passelse:raise ValueError("`slack_conn_ids` should be of type `SlackConnection` or `Set[SlackConnection]`")
Explanation:
- Check in one-line if input is approved.
- Change variable slack_conn_ids to set if object. If already set, everything okay. No need to waste memory creating new variables and assigning more than needed.
通过集体智慧和协作来改善编程学习和解决问题的方式。致力于成为全球开发者共同参与的知识库，让每个人都能够通过互相帮助和分享经验来进步。
评论