पायथन: बहुत से तर्कों के बारे में पिलिंट चेतावनियों से परहेज करना

Question

मैं छोटे पायथन फ़ंक्शन को छोटे में पुन: सक्रिय करना चाहता हूं। उदाहरण के लिए, इस कोड स्निपेट पर विचार करें:

x = x1 + x2 + x3 + x4 + x5 + x6 + x7 + x8 + x9 

बेशक, यह एक मामूली उदाहरण है। अभ्यास में, कोड अधिक जटिल है।

def mysum(x1, x2, x3, x4, x5, x6, x7, x8, x9): 
    x = x1 + x2 + x3 + x4 + x5 + x6 + x7 + x8 + x9 
    return x 

समस्या यह है कि pylint भी कई तर्क बारे में एक चेतावनी ट्रिगर करेगा है: मेरा मुद्दा यह है कि यह कई स्थानीय दायरे चर निकाले कार्य करने के लिए पारित किया जा करने के लिए होता है कि, जो दिखाई दे सकता है शामिल हैं। मैं की तरह कुछ कर रही द्वारा चेतावनी से बच सकते हैं:

def mysum(d): 
    x1 = d['x1'] 
    x2 = d['x2'] 
    ... 
    x9 = d['x9'] 
    x = x1 + x2 + x3 + x4 + x5 + x6 + x7 + x8 + x9 
    return x 

def mybigfunction(): 
    ... 
    d = {} 
    d['x1'] = x1 
    ... 
    d['x9'] = x9 
    x = mysum(d) 

लेकिन इस दृष्टिकोण मेरे लिए बदसूरत घरों में शौचालय, यह कोड का एक बहुत है कि अनावश्यक है लेखन की आवश्यकता है।

क्या ऐसा करने का कोई बेहतर तरीका है?

Answer 1

फ़ंक्शन को सरल या विभाजित करें ताकि उसे नौ तर्कों की आवश्यकता न हो (या पिलिंट को अनदेखा न करें, लेकिन जो लोग आप प्रस्तावित कर रहे हैं जैसे कि लिंट टूल के उद्देश्य को हराते हैं)।

संपादित करें: http://lists.logilab.org/pipermail/python-projects/2006-April/000664.html

बाद में, आप विकलांग चेतावनी के सभी के लिए grep कर सकते हैं: अगर यह एक अस्थायी उपाय है, के रूप में यहाँ वर्णित एक टिप्पणी का उपयोग कर प्रश्न में विशेष कार्य के लिए चेतावनी को अक्षम करें।

Answer 2

आप Python's variable arguments सुविधा का उपयोग कर की कोशिश कर सकते:

def myfunction(*args): 
    for x in args: 
     # Do stuff with specific argument here 

Answer 3

शायद आप सदस्य चर में बहस के कुछ बदल सकता है। यदि आपको उस राज्य की आवश्यकता है तो एक वर्ग मुझे एक अच्छा विचार की तरह लगता है।

Answer 4

पायथन के कुछ अच्छे कार्यात्मक प्रोग्रामिंग टूल हैं जो आपकी आवश्यकताओं को अच्छी तरह फिट करने की संभावना रखते हैं। lambda functions और map देखें। साथ ही, जब आप ऐसा लगता है कि आप सूचियों के साथ बेहतर सेवा करेंगे तो आप डिक्ट्स का उपयोग कर रहे हैं। आपके द्वारा प्रदान किए गए सरल उदाहरण के लिए, इस मुहावरे को आजमाएं। ध्यान दें कि नक्शा बेहतर और तेज होगा, लेकिन अपनी आवश्यकताओं फिट नहीं हो सकता है:

def mysum(d): 
    s = 0 
    for x in d: 
     s += x 
    return s 

def mybigfunction(): 
    d = (x1, x2, x3, x4, x5, x6, x7, x8, x9) 
    return mysum(d) 

आप स्थानीय चर का एक बहुत होने का उल्लेख किया है, लेकिन स्पष्ट रूप से अगर आप सूचियों (या tuples) के साथ काम कर रहे हैं, आप सूचियों का उपयोग करना चाहिए और लंबे समय तक उन सभी स्थानीय चरों को कारक बनाएं।

Answer 5

पहले, Perlis's epigrams में से एक:

"आप 10 मानकों के साथ एक प्रक्रिया है, तो आप शायद कुछ याद किया।"

10 में से कुछ तर्क संभावित रूप से संबंधित हैं। उन्हें एक वस्तु में समूहित करें, और इसके बजाय इसे पास करें।

ऊपर एक उदाहरण बनाना, प्रश्न में पर्याप्त जानकारी सीधे जवाब देने के लिए नहीं है, क्योंकि वहाँ:

class PersonInfo(object): 
    def __init__(self, name, age, iq): 
    self.name = name 
    self.age = age 
    self.iq = iq 

फिर अपने 10 तर्क समारोह:

def f(x1, x2, name, x3, iq, x4, age, x5, x6, x7): 
    ... 

हो जाता है:

def f(personinfo, x1, x2, x3, x4, x5, x6, x7): 
    ... 

और कॉलर में परिवर्तन:

personinfo = PersonInfo(name, age, iq) 
result = f(personinfo, x1, x2, x3, x4, x5, x6, x7) 

Answer 6

क्या आप तर्कों को पारित करने के लिए एक बेहतर तरीका चाहते हैं या pylint को कठिन समय देने से रोकने का एक तरीका चाहते हैं?

#pylint: disable-msg=R0913 

या:

#pylint: disable-msg=too-many-arguments 

उन्हें वापस चालू करने के लिए याद बाद, मैं की तर्ज पर अपने कोड में pylint -controlling टिप्पणियां डाल कर याद करने के लिए है कि आप सता रोक सकता है लगता है जितनी जल्दी संभव हो।

मेरी राय में, वहाँ कुछ भी नहीं स्वाभाविक तर्क और समाधान कुछ कंटेनर बहस में उन सब को ऊपर लपेटकर वास्तव में किसी भी समस्याओं, आप :-) सता से pylint रोक के अलावा अन्य का समाधान नहीं है की वकालत की एक बहुत कुछ गुजर साथ कुछ गड़बड़ है।

यदि आपको बीस तर्क पारित करने की आवश्यकता है, तो उन्हें पास करें। ऐसा हो सकता है कि यह आवश्यक है क्योंकि आपका कार्य बहुत अधिक कर रहा है और फिर से फैक्टरिंग वहां सहायता कर सकती है, और यह कुछ ऐसा है जिसे आपको देखना चाहिए। लेकिन यह कोई निर्णय नहीं है हम वास्तव में तब तक बना सकते हैं जब तक कि हम देखते हैं कि 'असली' कोड क्या है।

Answer 7

आप आसानी से पिलिंट में अधिकतम स्वीकृत संख्याओं को बदल सकते हैं।

अधिकतम-आर्ग = 5

लिए: बस अपने pylintrc फ़ाइल (यह उत्पन्न करता है, तो आप पहले से ही एक नहीं है) और परिवर्तन को खोलने

अधिकतम-आर्ग = 6 # या कि किसी भी मूल्य सूट आप

pylint के manual

निर्दिष्ट सभी उपयुक्त अपने स्थापना के लिए विकल्प और कोडिंग मानकों हो सकता से टेडि ous, तो डिफ़ॉल्ट मान निर्दिष्ट करने के लिए एक आरसी फ़ाइल का उपयोग करना संभव है। पिलिंट/etc/pylintrc और ~/.pylintrc के लिए दिखता है। Thegenerate-rcfile विकल्प कॉन्फ़िगरेशन फ़ाइल मानक आउटपुट और बाहर निकलने पर वर्तमान कॉन्फ़िगरेशन के अनुसार एक टिप्पणी उत्पन्न करेगा। आप विकल्पों को कॉन्फ़िगरेशन में उपयोग करने से पहले या डिफ़ॉल्ट मानों के साथ शुरू करने से पहले कॉन्फ़िगरेशन को ट्यून कर सकते हैं।

Answer 8

paxdiablo के प्रत्युत्तर के बारे टिप्पणी - के रूप में मैं वहाँ सीधे टिप्पणी करने के लिए पर्याप्त प्रतिष्ठा नहीं है: -/

मैं संख्या की चर्चा करते हुए पसंद नहीं है, sybolic नाम और अधिक अर्थपूर्ण है और जोड़ने के लिए होने से बचने एक टिप्पणी जो समय के साथ अप्रचलित हो सकती है।

तो मैं नहीं बल्कि करना चाहते हैं:

#pylint: disable-msg=too-many-arguments 

और मैं भी वहाँ झूलते इसे छोड़ नहीं करने की सिफारिश करेंगे: जब तक फ़ाइल समाप्त हो जाती है या वह अक्षम है, जो भी पहले हो यह सक्रिय रहेंगे।

तो बेहतर कर रहा:

#pylint: disable-msg=too-many-arguments 
code_which_would_trigger_the_msg 
#pylint: enable-msg=too-many-arguments  

मैं भी सक्षम/एक ही चेतावनी/प्रति पंक्ति त्रुटि को अक्षम करने की सिफारिश करेंगे।