在with语句内部返回函数是否为不良实践？哪种写法更规范？

先给结论：你推测的没错——Python中不管return语句写在with块内部还是外部，上下文管理器的__exit__方法都会被调用，资源会被正确释放。这两段代码在当前场景下功能完全等价，但从可读性角度，第二种写法更优。

为什么两种写法行为一致？ #

在你的代码里，out = list(res)已经把数据库查询结果一次性加载到内存中，out是独立于数据库连接conn的普通列表。不管你是在with块内return，还是块外return：

• 当函数执行到return时，Python会先触发with块的退出流程，调用conn的__exit__方法（通常是关闭数据库连接），然后再把out返回给调用方。
• 因为out已经和数据库连接无关，所以两种写法返回的结果完全一样，资源也都能正常释放。

什么时候在with块内return会有问题？ #

只有当你返回的对象依赖上下文管理器的资源存活时，才会出问题。比如如果你的代码是直接返回res（数据库游标，惰性迭代器）而不是转成list：

@classmethod
def find(cls, *args, **kwargs):
    with cls.connect() as conn:
        coll = conn[cls.DB][cls.COLLECTION]
        res = coll.find(*args, **kwargs)
        return res  # 错误：返回游标，with块退出后conn关闭，游标失效

这种情况下，调用方拿到游标后迭代时，数据库连接已经关闭，会抛出异常。但你的代码里已经把结果转成了list，所以不存在这个问题。

哪种是最佳实践？ #

推荐第二种写法：把return放在with块外部。原因是：

• 更符合直观逻辑：资源（数据库连接）释放完成后，再返回处理好的结果，代码意图更清晰。
• 避免误解：不熟悉Python上下文机制的开发者看到第一种写法，可能会担心“return的时候连接还没关闭会不会泄漏资源”，或者误以为返回值和连接有关，而第二种写法完全消除了这种疑虑。

当然，如果你的业务逻辑需要在with块内提前返回（比如某些分支条件下直接返回），在确保返回值不依赖上下文资源的前提下，写在块内也没问题——Python会正确处理__exit__的调用，不会造成资源泄漏。

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