Форматирование длинных строк python

Question

Что было бы красивым способом отступать/форматировать строку в функции ниже? Или я не пытаюсь написать это как один лайнер?

def rects_bound(rects): 
    """ Returns a rectangle that bounds all given rectangles 
    """ 
    return reduce(lambda (ax,ay,aw,ah), (bx,by,bw,bh): (min(ax,bx), 
                 min(ay,by), 
                 max(ax+aw, bx+bw), 
                 max(ay+ah, by+bh)), rects) 

или, может быть,

def rects_bound(rects): 
    """ Returns a rectangle that bounds all given rectangles 
    """ 
    return reduce(lambda (ax,ay,aw,ah), 
         (bx,by,bw,bh): (min(ax,bx), min(ay,by), 
             max(ax+aw, bx+bw), max(ay+ah, by+bh)), 
        rects) 

Я обычно просто своего рода «получить творческий» в таких ситуациях, и я знаю, что, вероятно, нет «правильный» путь, я просто интересно ваше мнение и привычки.

Answer 1

Если вы беспокоитесь о длинных очередях, не используйте lambda. Вместо этого используйте обычную именованную функцию.

def rects_bound(rects): 
    """ Returns a rectangle that bounds all given rectangles 
    """ 
    def bounding_rect_reducer((ax, ay, aw, ah), (bx, by, bw, bh)): 
     return (min(ax,bx), 
       min(ay,by), 
       max(ax+aw, bx+bw), 
       max(ay+ah, by+bh)) 

    return reduce(bounding_rect_reducer, rects) 

Answer 2

Я думаю, что это зависит от программиста и от ситуации, но я обычно не люблю выделять переменную, чтобы иметь более короткие строки.

Глядя на ваши 2 примера, я хотел бы пойти на второй, или это один:

def rects_bound(rects): 
    """ Returns a rectangle that bounds all given rectangles 
    """ 
    return reduce(
     lambda (ax,ay,aw,ah), (bx,by,bw,bh): 
     (min(ax,bx), min(ay,by), max(ax+aw, bx+bw), max(ay+ah, by+bh) 
    ), 
     rects 
    ) 

Answer 3

Я хотел бы предложить следующее:

def rects_bound(rects): 
    """ Returns a rectangle that bounds all given rectangles 
    """ 
    return reduce(lambda (X,Y,W,H), (x,y,w,h): (min(X,x), min(Y,y), 
               max(X+W, x+w), 
               max(Y+H, y+h)), rects) 

Снижение каждый аргумент к одному символу действительно экономит место, что помогает этому выглядеть чище. Другим вариантом было бы определить лямбда-функцию на отдельной строке (возможно, даже с def), чтобы аргументы были не так уж далеки.

Answer 4

Прежде всего, избегайте длинных линий, если это возможно. Этот конкретный пример может быть написан гораздо более читаемо, как

def rects_bound(rects): 
    x0 = min(x for x, y, w, h in rects) 
    y0 = min(y for x, y, w, h in rects) 
    x1 = max(x + w for x, y, w, h in rects) 
    y1 = max(y + h for x, y, w, h in rects) 
    return x0, y0, x1, y1 

Если вы предпочитаете, чтобы избежать переменных, вы можете также использовать

def rects_bound(rects): 
    return (min(x for x, y, w, h in rects), 
      min(y for x, y, w, h in rects), 
      max(x + w for x, y, w, h in rects), 
      max(y + h for x, y, w, h in rects)) 

, который я до сих пор находят гораздо более удобным для чтения, чем ваш исходный код.

(Обратите внимание, что я предполагаю, что rects допускает несколько итераций.)

Answer 5

Ваша функция лямбда неправильна. Чтобы исправить это, он сделает его еще длиннее и потребует избыточных вычислений. Используйте вместо Защиты:

def rects_bound(rects): 
    """ Returns a rectangle that bounds all given rectangles 
    """ 
    def bound_2_rects((ax, ay, aw, ah), (bx, by, bw, bh)): 
     x = min(ax, bx) 
     y = min(ay, by) 
     return x, y, max(ax+aw, bx+bw) - x, max(ay+ah, by+bh) - y 

    return reduce(bound_2_rects, rects)