Python: evitare gli avvertimenti di pylint su troppi argomenti

Question

Voglio refactoring una grande funzione Python in quelle più piccole. Ad esempio, si consideri questo frammento di codice seguente:

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

Naturalmente, questo è un esempio banale. In pratica, il codice è più complesso. Il mio punto è che contiene molte variabili local-scope che avrebbero dovuto essere passati alla funzione estratto, che potrebbe apparire come:

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

Il problema è che pylint innescherebbe un avvertimento circa troppi argomenti. ho potuto evitare l'avvertimento facendo qualcosa di simile:

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) 

ma questo approccio Loos brutto per me, richiede la scrittura di un sacco di codice che è anche ridondante.

Esiste un modo migliore per farlo?

Answer 1

Semplifica o interrompe la funzione in modo che non richieda nove argomenti (o ignori il puntatore, ma gli schivati ​​come quelli che stai proponendo vanificano lo scopo di uno strumento sfilaccia).

EDIT: se si tratta di una misura temporanea, disattivare l'avviso per la particolare funzione in questione con un commento come descritto qui: http://lists.logilab.org/pipermail/python-projects/2006-April/000664.html

Successivamente, è possibile grep per tutte le avvertenze disabili.

Answer 2

Si potrebbe provare a utilizzare Python's variable arguments funzione:

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

Answer 3

Forse si potrebbe trasformare alcuni degli argomenti in variabili membro. Se hai bisogno di quello stato, una classe mi sembra una buona idea.

Answer 4

Python dispone di alcuni strumenti di programmazione funzionale che sono adatti alle vostre esigenze. Controlla lambda functions e map. Inoltre, stai usando dicts quando sembra che tu possa essere servito molto meglio con le liste. Per il semplice esempio che hai fornito, prova questo idioma. Si noti che la mappa sarebbe meglio e più velocemente, ma non può soddisfare le vostre esigenze:

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) 

Lei ha parlato di avere un sacco di variabili locali, ma francamente se hai a che fare con le liste (o tuple), è necessario utilizzare gli elenchi e eliminare tutte quelle variabili locali nel lungo periodo.

Answer 5

In primo luogo, una delle Perlis's epigrams:

"Se si dispone di una procedura con 10 parametri, probabilmente perso un po '."

Alcuni dei 10 argomenti sono presumibilmente correlati. Raggruppali in un oggetto e passa quello.

Fare un esempio su, perché non c'è abbastanza informazioni nella domanda a cui rispondere direttamente:

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

Allora la vostra funzione 10 argomento:

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

diventa:

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

e il chiamante cambia in:

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

Answer 6

Vuoi un modo migliore per passare gli argomenti o solo un modo per impedire a pylint di darti dei problemi? In quest'ultimo caso, mi sembra di ricordare che si potrebbe fermare il fastidioso mettendo pylint commenti -controlling nel codice lungo le linee di:

#pylint: disable-msg=R0913 

o:

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

ricordando di riattivarli il prima possibile.

A mio parere, non c'è niente di intrinsecamente sbagliato con il superamento di un sacco di argomenti e soluzioni che sostengono tutti avvolgendolo in qualche argomento contenitore in realtà non risolvere eventuali problemi, altro che fermarsi pylint da voi :-) fastidioso.

Se è necessario passare venti argomenti, quindi passarli. Può essere che questo sia richiesto perché la tua funzione sta facendo troppo e un ri-factoring potrebbe aiutarti, ed è qualcosa che dovresti guardare. Ma non è una decisione che possiamo fare a a meno che non vediamo quale sia il codice "reale".

Answer 7

È possibile modificare facilmente il numero massimo consentito di argomenti in pylint. Basta aprire il file pylintrc (generarlo se non ne hai già uno) e il cambiamento:

max-args = 5

a:

max-args = 6 # o qualsiasi valore che si adatta si

Da pylint di manual

Specifica tutte le opzioni adatte per la configurazione e standard di codifica può essere tedi ous, quindi è possibile utilizzare un file rc per specificare i valori di default . Pylint cerca/etc/pylintrc e ~/.pylintrc. L'opzione --generate-rcfile genererà un file di configurazione commentato in base alla configurazione corrente dello sullo standard output e exit. È possibile aggiungere altre opzioni prima di utilizzarle nella configurazione o iniziare con i valori predefiniti e regolare manualmente la configurazione .

Answer 8

Commento su risposta di paxdiablo - come io non ho abbastanza fama di commentare lì direttamente: -/

non mi piace riferimento al numero, il nome sybolic è molto più espressiva e evitare di dover aggiungere un commento che potrebbe diventare obsoleto nel tempo.

Quindi io preferirei fare:

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

e vorrei anche consigliare di non lasciarlo lì penzoloni: resterà attiva fino a quando il file termina o si è disattivato, si verifica per primo.

Quindi meglio fare:

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

Auspico inoltre abilitare/disabilitare un singolo avviso/errore per riga.