使用超过4个条件的列表理解

Question

我有一份清单清单。每个子列表包含两个整数元素。我想将这些元素中的每一个乘以1.2，如果它们是正数且大于某个数字，则将它们更改为该特定数字。同样的事情，如果他们更小。我使用列表理解实现的乘法位。有了这些额外的条件，我有点迷失 - 对我来说，它似乎过于复杂，甚至是非pythonic。我想知道我是否能得到一些意见。

我使用列表理解实现的乘法位。有了这些额外的条件，我有点迷失 - 对我来说，它似乎过于复杂，甚至是非pythonic。我对如何将所有条件都放在列表理解中有一个粗略的想法，但是当我开始实现它们时，我感到困惑。我想知道我是否能得到一些意见。

MAX_VOL = 32767  # audio data is the list of lists
MIN_VOL = -32767
audio_list = [[int(x*1.2), int(y*1.2)] for x, y in audio_data]

# My idea was something like:
audio_list = [[int(x*1.2), int(y*1.2) if x > MAX_VOL x == MAX VOL if x < MIN_VOL....] for x, y in audio_data]

首先，我很确定存在语法问题。但是，对于每个数字，我必须检查它是正还是负，然后，如果它大于或小于最大/最小音量。那么，这是否可以使用列表理解？它是高效和pythonic？

非常感谢！

Answer 1

使用min和max：

MAX_VOL = 32767  # audio data is the list of lists
MIN_VOL = -32767

audio_list = [[int(max(MIN_VOL, min(MAX_VOL, x*1.2))), int(max(MIN_VOL, min(MAX_VOL, y*1.2)))] for x, y in audio_data]

由于这些是长表达式，因此也使用函数：

def clip(s):
    return int(max(MIN_VOL, min(MAX_VOL, s)))
audio_list = [[clip(x*1.2)), clip(y*1.2)] for x, y in audio_data]

Answer 2

定义一个不允许值超过最大或最小音量的边界函数。然后你仍然可以使用列表理解。

MAX_VOL = 32767  # audio data is the list of lists
MIN_VOL = -32767

# Ensure volume does not exceed limits.
def bounded(val):
    if val > MAX_VOL:
        return MAX_VOL
    elif val < MIN_VOL:
        return MIN_VOL
    return val

audio_list = [[bounded(int(x*1.2)), bounded(int(y*1.2))] for x, y in audio_data]

Answer 3

如果列表理解太复杂，则不要使用它。您应该尝试让您的解决方案易于让其他人理解并让您了解您是否在一年内回归它。

for x,y in audio_data:
    if x > MAX_VOL:
        foo = int(x*1.2)
        bar = int(y*1.2)
    if x == MAX_VOL:
        ...