PEP 8 und Liste Verständnis

Question

Ich habe ein bisschen ein Problem mit PEP 8 getroffen.

Ich benutze Liste Verständnis, die länger als 79 Zeichen lang ist. Also schreit mein Text-Editor mich an, etwas dagegen zu tun, und es ist ein Blick, den man beim Codieren sehen kann.

return [(i['user_id'], i['id']) for i in j['collection'] if i and i['user_id']] 

So versuche ich die Linie zu brechen, aber jetzt klagt sie brechen meine Linie für visuelle Zwecke ist.

return [(i['user_id'], i['id']) for i in j['collection'] 
    if i and i['user_id']] 

Was für ein Mann ist in einem solchen Gurke zu tun?

Edit: Basierend auf den Antworten, entschied ich mich für die Lesbarkeit über Liste Verständnis in diesem Fall, und jetzt macht es mehr Sinn für den Leser:

tracks = set() 

for track in json['collection']: 
    if track and track['user_id']: 
     tracks.add((track['user_id'], track['id'])) 

Answer 1

Bei der Verwendung von Pep8 gilt der gesunde Menschenverstand. Wenn Sie eine Logikzeile nicht innerhalb von 80 Zeichen einfügen können, ist dies fast immer ein Zeichen dafür, dass diese Zeile neu geschrieben werden muss. Und es ist in diesem Fall wahr, da ich nicht einmal anfangen kann zu versuchen, zu verstehen, was diese Linie tun soll.

Aber wenn Sie es in ein viel besser lesbares Format verschieben, dann wird nicht nur die Anzahl Ihrer Zeilen steigen, sondern auch Ihre Lesbarkeit, so dass Sie Platz für richtige Variablennamen wie user statt i haben. Dies wird Ihre eigene Wartung in der Zukunft viel einfacher machen, ganz zu schweigen davon, ob jemand anderes es jemals anschauen muss.

So dieses Beispiel in tatsächlichen Code zu setzen, das ist, was ich mit ihm tun würde (mit einer netten Funktion wickelt die Probe viel leichter lesbar zu machen!):

j = {'collection': [{'id': 1, 'user_id': 1}]} 

def get_qualified_users(users): 
    qualified_users = [] 
    for user in users: 
     if user and user['user_id']: 
      qualified_users.append((user['user_id'], user['id'])) 
    return qualified_users 

print(get_qualified_users(j['collection'])) 

Sie können ganz einfach kopieren/fügen Sie es in Ihren Interpetor ein, um zu sehen, dass es funktioniert. Und mehr als das - es ist extrem einfach zu pflegen und zu folgen, mit klarem api eingekapselt in einer versiegelten Funktion.

Answer 2

, wenn Sie sich strikt an PEP8 halten wollen, dann Der hängenden Eindruck ist dein Freund.

return [ 
    (i['user_id'], i['id']) 
    for i in j['collection'] 
    if i and i['user_id']] 

Answer 3

Während ich ein Fan der Entspanntheit der strikten 80-Char-Grenze bin, und (wie erwähnt) so ist Raymond Hettinger in diesem zwingenden Vortrag von PyCon 2015, in diesem speziellen Fall könnte das sauberste Ding zu sein aufgeteilt in zwei Zeilen:

valid_things = (i for i in j['collection'] if i and i['user_id']) 
return [(i['user_id'], i['id']) for i in valid_things] 

diese Weise wird die „Filterung“ und „Auswahl“ Logik klar getrennt sind und in einer lesbaren Weise dargestellt, als sie zusammen Verklemmen.

Eine weitere Option für die funktionell geneigten:

return map(itemgetter('user_id', 'id'), valid_things) 

Answer 4

Sie müssen nur richtig die zweite Zeile einrücken:

return [(i['user_id'], i['id']) for i in j['collection'] 
     if i and i['user_id']] 

Confirmed mit PEP8 Online, aber lassen Sie mich wissen, ob es funktioniert für dich auch.

Persönlich mag ich nicht Ausdruck und Quelle zusammen, aber die Bedingung getrennt. Ich würde den Ausdruck eher deutlich sehen, nicht den Zustand. Markieren Sie, was Sie bekommen. So würde ich eine von ihnen tun:

return [(i['user_id'], i['id']) 
     for i in j['collection'] if i and i['user_id']] 

return [(i['user_id'], i['id']) 
     for i in j['collection'] 
     if i and i['user_id']] 

Klarstellung: Dies ist, wie ich indent würde/brechen. Ich habe die Variablennamen nicht berücksichtigt, da ich nur eine direkte Antwort geben und erklären wollte, was der PEP8-Fehler war und wie er behoben werden konnte, weil niemand sonst etwas getan hatte.