Как отступы в Python?

Question

В некоторых ситуациях могут быть полезны списки, но они также могут быть довольно ужасны для чтения. Как слегка преувеличенный пример, как вы могли бы отступать следующим образом?

allUuids = [x.id for x in self.db.query(schema.allPostsUuid).execute(timeout = 20) if x.type == "post" and x.deleted is not False] 

Answer 1

Это зависит от того, как долго они находятся. Я склонен их структурировать так:

[x.id for x 
in self.db.query(schema.allPostsUuid).execute(timeout=20) 
if x.type == 'post' 
    and x.deleted is not False 
    and ... 
    and ...] 

Таким образом, каждое выражение имеет свою собственную линию.

Если строка становится слишком большой, я хотел, чтобы извлечь его в лямбда или выражения:

transform = lambda x: x.id 
results = self.db.query(schema.allPostsUuid).execute(timeout=20) 
condition = lambda x: x.deleted is not False and ... and ... 
[transform(x) for x in results if condition(x)] 

И тогда, если лямбда становится слишком долго он продвинут к функции.

Answer 2

Как насчет:

allUuids = [x.id for x in self.db.query(schema.allPostsUuid).execute(timeout = 20) 
        if (x.type == "post" and x.deleted is not False)] 

Как правило, длинные очереди можно избежать путем предварительного вычисления подвыражения в переменные, которые могли бы добавить незначительную стоимость исполнения:

query_ids = self.db.query(schema.allPostsUuid).execute(timeout = 20) 
allUuids = [x.id for x in query_ids 
        if (x.type == "post" and x.deleted is not False)] 

Кстати , не является «is not False» видом избытка? Вы беспокоитесь о различии между Нием и Ложь? Потому что в противном случае достаточно оставить условие только: i f (x.type == "post" and x.deleted)

Answer 3

Для меня это слишком много. Возможно, это просто ужасный пример, так как «тип» и «удаленный» явно будут частью запроса db.

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

Answer 4

Где я работаю, наши принципы кодирования бы нам сделать что-то вроде этого:

all_posts_uuid_query = self.db.query(schema.allPostsUuid) 
all_posts_uuid_list = all_posts_uuid_query.execute(timeout=20) 
all_uuid_list = [ 
    x.id 
    for x in all_posts_uuid_list 
    if (
     x.type == "post" 
     and 
     not x.deleted # <-- if you don't care about NULLs/None 
    ) 
] 

Answer 5

allUuids = [x.id 
      for x in self.db.query(schema.allPostsUuid).execute(timeout = 20) 
      if x.type == "post" and x.deleted is not False] 

Answer 6

Вы не должны использовать список понимание для этого.

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

Для такого длинного фрагмента кода, вы должны использовать обычные блоки:

allUuids = [] 
for x in self.db.query(schema.allPostsUuid).execute(timeout = 20) : 
    if x.type == "post" and x.deleted is not False : 
     allUuids.append(x.id) 

Точно такое же поведение, гораздо более удобным для чтения. Гвидо будет гордиться вами :-)

Answer 7

Если вы настроились на понимание, то orestis's answer это хорошо.

Для более сложных постижений, как я предложил бы использовать генератор с yield:

allUuids = list(self.get_all_uuids()) 


def get_all_uuids(self): 
    for x in self.db.query(schema.allPostsUuid).execute(timeout = 20): 
     if x.type == "post" and x.deleted is not False: 
      yield x.id