2013-04-23 22 views
1

這是非常微不足道的,但它是竊聽我,我想我會把它扔出去。pep8和使用構造函數作爲驗證程序

python netaddr庫很棒。我使用它來處理一些事情,包括IP地址驗證。要做到這一點,我使用類似

try: 
    ddd = IPRange(split[0], split[1]) 
except: 
    return False 

而且工作很好,但PEP8討厭我創建'DDD',但從來沒有使用它。是否有另一種更好,更清潔的方式來做到這一點?謝謝

+5

不要'IPRange(分裂[0],分裂[1])結合'到一個變量? – Volatility 2013-04-23 11:30:59

+0

我還會在'try:'行的上面加註釋,表示你想檢查一個IP地址的有效性。這應該足夠清楚! – 2013-04-23 11:37:45

+0

嘗試/除了塊之後是否使用'ddd'? – gatto 2013-04-23 11:48:16

回答

2

如果你不需要「ddd」變量,那麼不要聲明/分配它。如果它是一個可以從其他模塊使用的模塊變量,則不能真正避免它。我建議你添加一個註釋來解釋爲什麼這個變量在模塊中單獨存在。

而且它被認爲是不好的做法,有一個裸露except:,你應該使用至少except Exception:

它在PEP8太:

當捕捉異常,提到具體的例外儘可能而是採用了裸除了:子句。

+0

凡異常'也可以是http://pythonhosted.org/netaddr/api.html#custom-exceptions – invert 2013-04-23 13:23:05

相關問題