python - 如何将 lambda 函数分解成它自己的函数？ (Lambda 目前是 125+ 个字符)

Question

我有一个 lambda 函数，我想添加它并让它变得更健壮。但是，我也想遵循 PEP 8 并将我的行保持在 79 个字符以内、正确的缩进等。但是我在思考如何拆分行时遇到了麻烦。

当前行是:

styled_df = df.style.apply(lambda x: ["background: rgba(255,0,0,.3)" if('BBC' in x['newsSource'] or 'Wall' in x['newsSource']) and idx==0 else "" for idx, v in enumerate(x)], axis = 1)

到目前为止，我可以做到这一点:

styled_df = df.style.apply(
    lambda x: ["background: rgba(255,0,0,.3)" 
        if('BBC' in x['newsSource'] or 'Wall' in x['newsSource']) and
        idx == 0 else ""
        for idx, v in enumerate(x)], axis=1)

但第三行 (if('BBC' in ...) 与 PEP 8 (E128) 冲突。而且，无可否认，这不是我分解它时最清晰的代码.

此外，我正计划为此添加更多条件，并且想知道这样做的最佳方式是什么。只是继续添加，并尽我所能打破线路？或者是否有针对此类问题的“最佳实践”？

编辑:如我所想，但忘了提及，我可能可以将此 lambda 更改为一个函数，但我正在苦苦思索如何做。我是 lambda 的新手，完全是从另一个问题中得到的......所以是的，我同意 Lambda 是根本问题。

Answer 1

一般来说，我认为依靠自动化工具来重新格式化您的代码比自己尝试弄清楚如何像计算机一样应用所有这些规则要好。

我所知道的三个主要选择是:

• black :在语法级别重新格式化；除了最大行长度外，完全没有配置。
• yapf :在语法级别重新格式化；高度可配置。
• autopep8 (以及像 pep8ify 这样的前辈):仅在表层重新格式化；不会更改任何已经符合 PEP 8 的内容。如果您想要尽可能少的更改(例如，因为您正在重新格式化长期存在的代码并且不希望源代码管理中有大量更改列表)，这很有用。

这是black对你的代码做:

styled_df = df.style.apply(
    lambda x: [
        "background: rgba(255,0,0,.3)"
        if ("BBC" in x["newsSource"] or "Wall" in x["newsSource"]) and idx == 0
        else ""
        for idx, v in enumerate(x)
    ],
    axis=1,
)

这会占用大量的垂直空白。但是你不能配置 black以不同的方式对待它。那么，您可以做什么？

每当black坚持将我的代码重新格式化为一些对我来说不好看的东西，这意味着我必须将我的代码重新组织成更容易格式化的东西。

这里最明显的事情就是转动那个巨人lambda进入def ，或者甚至两个:

def highlight(x):
    return "BBC" in x["newsSource"] or "Wall" in x["newsSource"]

def style(x):
    return [
        "background: rgba(255,0,0,.3)" if highlight(x) and idx==0 else ""
        for idx, v in enumerate(x)
    ]

styled_df = df.style.apply(style, axis=1)

完成之后……您甚至没有使用 v ;您所做的只是为第一个 ( and idx == 0 ) 设置样式，并且前提是新闻来源包括 BBC 或 Wall。

所以，对于 BBC 和 Wall 的东西，你要返回一个 background加上 len(x)-1空字符串；对于其他事情，您只是返回 len(x)空字符串。

假设这是你想要的逻辑，让我们明确一点:

def style(x):
    if "BBC" in x["newsSource"] or "Wall" in x["newsSource"]:
        first = "background: rgba(255,0,0,.3)"
        return [first] + [""]*(len(x)-1)
    return [""]*len(x)

styled_df = df.style.apply(style, axis=1)

您可能更喜欢 ["" for _ in range(x)]至 [""]*len(x) ;我不确定这里哪个更具可读性。

---

I likely could change this lambda in to a function, but am struggling with how. I'm new to lambdas and got this one from another problem altogether...So yes, agreed the Lambda is the root issue.

A lambda 是一个函数，就像def是。唯一的区别是:

• def是一个语句，所以你不能把它放在表达式的中间。
• lambda是一个表达式，因此您不能在其中包含任何语句。
• def给一个函数一个名字。

除此之外，它们编译的功能完全相同。例如:

func = lambda x: expr(x)

def func(x): return expr(x)

… 定义了两个具有相同字节码的函数，几乎所有其他的都相同，除了 func.__name__。是'func'对于 def但像 '<lambda>'对于 lambda .

更重要的是，如果你想在函数中进行循环或测试，用 lambda您必须将其扭曲为理解或 if 表达式；与 def ，如果合适，您可以这样做，如果不合适，您可以使用复合语句。

但是，另一方面，如果函数没有好的名称，并且除了用作回调函数之外真的不值得考虑，lambda更好。例如，如果您定义一个函数只是 return x - 3这只会被使用一次，def会很傻。