2010-11-02 10 views
5

मुझे शब्दों की एक सूची लेनी है और इसमें सभी शब्द गिनना है जो 2 या अधिक वर्ण लंबे हैं और जहां पहला और अंतिम वर्ण बराबर है।लूप के लिए सूची समझ और लेन() बनाम सरल

result = 0 
for word in words: 
    if len(word) >= 2 and word[0] == word[-1]: 
     result += 1 
return result 

बनाम

return len([word for word in words if len(word) >= 2 and word[0] == word[-1]]) 

कौन सा पसंदीदा समाधान होगा:

मैं दो संभव समाधान के साथ आया था? या यहां तक ​​कि बेहतर भी हैं?

+0

मैं # 2 लेगा: लेकिन कई लाइनों पर अच्छी तरह से स्वरूपित किया गया। सिर्फ इसलिए कि आप इसे * लिख सकते हैं, इसका मतलब यह नहीं है कि आपको (जेनरेटर या फॉर-कॉम्प आदि के साथ भी) –

+0

कृपया, कृपया, कृपया मापें। इसे मापने के लिए कृपया 'टाइमिट' का उपयोग करें। कृपया परिणामों को मापें और पोस्ट करें। –

+1

@ एसएलओटी: उन्होंने प्रदर्शन के बारे में कुछ भी नहीं कहा। कुछ करने के लिए * पसंदीदा * तरीका कुछ करने के लिए * सबसे तेज़ * तरीका नहीं दर्शाता है। –

उत्तर

14

आपके दूसरे उदाहरण में generator expression आपकी सूची बड़ी है तो सूची-कंप से बेहतर होगा।

sum(1 for word in words if len(word) >= 2 and word[0] == word[-1]) 
+1

टाइप एरर: प्रकार 'जेनरेटर' की ऑब्जेक्ट में कोई लेन नहीं है() – dan04

+0

धन्यवाद! मैं लगभग सुझाव देने जा रहा था। – pyfunc

+0

@ dan04: धन्यवाद; अभी तय @pyfunc: धन्यवाद! – bernie

1

दोनों बहुत अच्छे हैं।

सूची समझ एक और सूची है जो आप लेन करने से गुजर रहे हैं देता है:

छोटे मतभेद हैं। पहला समाधान किसी अन्य सूची के निर्माण से बचाता है।

3

पहला पायथन में निश्चित रूप से पसंदीदा समाधान होगा।

अजगर के अपने ज़ेन न भूलें:

The Zen of Python, by Tim Peters

Beautiful is better than ugly.

Explicit is better than implicit.

Simple is better than complex.

Complex is better than complicated.

Flat is better than nested.

Sparse is better than dense.

Readability counts.

Special cases aren't special enough to break the rules.

Although practicality beats purity.

Errors should never pass silently.

Unless explicitly silenced.

In the face of ambiguity, refuse the temptation to guess.

There should be one-- and preferably only one --obvious way to do it.

Although that way may not be obvious at first unless you're Dutch.

Now is better than never.

Although never is often better than right now.

If the implementation is hard to explain, it's a bad idea.

If the implementation is easy to explain, it may be a good idea.

Namespaces are one honking great idea -- let's do more of those!

अन्य की तुलना में है कि अपने समाधान अच्छे हैं।

+4

ज़ेन-ऑफ-पायथन-तोता का सामान्य सूत्र: दावा करें, और इसके बारे में एक पंक्ति पर औचित्य के रूप में इंगित करें, जैसे कि यह कुछ धार्मिक पाठ है। (मुझे दूसरा संस्करण बहुत स्पष्ट लगता है; पहली बार सी से पोर्ट की तरह दिखता है) –

+0

मैं पूरी तरह से असहमत हूं। सूची की समझ * बहुत * पाइथोनिक और पठनीय हैं। –

2

मुझे व्यक्तिगत रूप से स्पष्ट लूप अधिक पठनीय लगता है, लेकिन यह स्वाद का मामला है (कुछ सामान्य रूप से छोटे कोड को पसंद करते हैं, खासकर जब उन्हें इसे लिखना होता है)।

या तो संस्करण आगे छोटा किया जा सकता है/बेहतर:

result = 0 
for word in words: 
    result += int(len(word) >= 2 and word[0] == word[-1]) 
return result 

पूर्णांक() रूपांतरण सख्ती से, अनावश्यक बात कर रहा है यह सच है के बाद से 1 का एक प्रकार है, लेकिन यह पठनीयता के लिए बेहतर हो सकता है।

len(1 for word in words if len(word) >= 2 and word[0] == word[-1]) 
+0

'ए + = int (बूल) 'उस बिंदु पर अनौपचारिक है जहां मैं इसे बुरा कहूंगा। ऐसा मत करो; मूल स्थिति बहुत स्पष्ट है। –

1

:

return sum(len(word) >= 2 and word[0] == word[-1] for word in words) 

आप लेन() का उपयोग करना चाहते हैं, मैं तथ्य यह है कि मूल्यों को वास्तव में बात नहीं करने के लिए पाठक इंगित चाहते हैं: एक ही दृष्टिकोण समझ लिए आवेदन कर सकते कुछ अन्य वेरिएंट जिन्हें आप विचार करना चाहते हैं:

सबसे पहले, आप फ़ंक्शन में फ़िल्टर स्थिति को तोड़ सकते हैं। यह स्थिति किसी भी तरह से ठीक है, लेकिन अगर यह किसी भी अधिक जटिल हो जाता मैं निश्चित रूप से ऐसा करने चाहते हैं:

def check(word): 
    return len(word) >= 2 and word[0] == word[-1] 
sum(1 for word in words if check(word)) 

इसके बाद, यदि (मूल सूची समझ के रूप में) सूची तैयार स्वीकार्य है, तो आप क्या कर सकते हैं यह :

len(filter(check, words)) 

वहाँ itertools.ifilter है, लेकिन आप का उपयोग करें कि आप sum अभिव्यक्ति फिर से उपयोग करने की आवश्यकता है, इसलिए इसे किसी भी स्पष्ट खत्म नहीं करता है।

sum चाल इतनी बार आती है कि मुझे आश्चर्य है कि एक इटेटरेटर में आइटमों की संख्या गिनने के लिए मानक लाइब्रेरी कॉल नहीं है (यदि वहां है, तो मुझे यह नहीं मिला है)। वैकल्पिक रूप से, अगर len किसी इटरेटर में प्रविष्टियों की संख्या का उपभोग और गणना करेगा तो इसका अर्थ होगा यदि उसके पास __len__ नहीं है, लेकिन ऐसा नहीं है।

संबंधित मुद्दे