Python的v Unpythonic

Question

在代碼審查的挑戰，我最近有下面的代碼名爲unpythonic：

def word_count(string): 
wc_dict = {} 
word = "" 
index = 0 
while index < len(string): 

    if string[index].isalnum(): 
     word = word + string[index] 
     index += 1 
    else: 
     if word: 
      if word.lower() in wc_dict: 
       wc_dict[word.lower()] += 1 
      else: 
       wc_dict[word.lower()] = 1 
     word = "" 
     index += 1 

if word: 
    if word.lower() in wc_dict: 
     wc_dict[word.lower()] += 1 
    else: 
     wc_dict[word.lower()] = 1 

return wc_dict  

我還提交不同的代碼，我用Ruby寫了一個挑戰，有這樣的解決方案稱爲「Python化」 。

代碼是什麼意思是Pythonic/Unpythonic？

Answer 1

我會說「pythonic」意思是「符合python的通常習語，以便編寫易於閱讀的代碼」。你的例子不是pythonic，因爲它可能（可能）更容易寫出來。

例如使用正則表達式+ collections.Counter原來這成爲非常明顯的2 - 內膽：

words = re.findall(r'\w+', string) 
wc_dict = collections.Counter(words) 

如果你只期望能夠在空白分裂，這使得它更容易：時間

wc_dict = collections.Counter(string.split()) 

95％如果你使用字符串中的單個字符，還有更好的方法。

Answer 2

我非常同意@ mgilson對上面（un-）Pythonic的定義，但會在這個案例上再擴展一點。

我想說這個代碼可能被描述爲「非pythonic」的原因之一是因爲它似乎是一個相當直接的C代碼翻譯來做同樣的事情。我還看到許多Python代碼似乎基於熟悉Java或C＃以及它們各自的數據結構，標準庫和風格約定。

除了dict之外，OP的代碼似乎沒有使用任何Python的高級數據結構或迭代功能。

即使在使用dict時，它也是一種不太流利的方式。此代碼：

if word.lower() in wc_dict: 
    wc_dict[word.lower()] += 1 
else: 
    wc_dict[word.lower()] = 1 

...可以用更簡潔的版本替換平凡（這也可以縮短到一個班輪與defaultdict）：

這

wc_dict.setdefault(word.lower(), 0) 
wc_dict[word.lower()] += 1 

無意爲順便提一下，批評OP是一位深思熟慮的程序員：這是一個看似合理和正確的算法來解決這個問題。它只是沒有利用有經驗的用戶熟悉的許多Python功能。