पायथन सूची-समझ को कैसे इंडेंट करें?

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)] 

आम तौर पर, लंबी लाइनों चर में पूर्व कंप्यूटिंग subexpressions से बचा जा सकता है, जो एक मामूली प्रदर्शन लागत जोड़ सकते हैं:

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

मेरे लिए यह बहुत अधिक है। हो सकता है कि यह सिर्फ एक भयानक उदाहरण है, क्योंकि "टाइप" और "हटाया गया" स्पष्ट रूप से डीबी क्वेरी का हिस्सा होगा।

मुझे लगता है कि अगर एक सूची समझ कई लाइनों में फैली हुई है तो शायद यह एक सूची समझ नहीं होनी चाहिए। ऐसा कहकर, मैं आम तौर पर इस चीज़ को "अगर" अन्य लोगों की तरह यहां विभाजित करता हूं और यहां जवाब दूंगा।

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) 

वास्तव में समान व्यवहार, और अधिक पठनीय:

इतने लंबे स्निपेट के लिए, आप साधारण ब्लॉक का उपयोग करना चाहिए। Guido आपको गर्व होगा :-)

Answer 7

यदि आप एक समझ पर सेट हैं orestis's answer अच्छा है।

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