Dangerous default value [] as argument
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia con la PR #456 e cerca nella codebase Python gli argomenti delle funzioni il cui valore predefinito è []. Conferma quali occorrenze rientrano nell’ambito usando l’esempio Test in questa issue per comprendere il bug dello stato condiviso. Il lavoro è completato quando i valori predefiniti delle liste mutabili interessati usano un’inizializzazione sicura e la suite di test esistente ha esito positivo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
This issue came up in PR #456.
I wasn't really aware of this issue ... thought it would be just coding style. We should open an issue and fix it in a new PR, maybe like this:
def __init__(self, some_list=None):
self.some_list = some_list or []
Codacy is giving me "Dangerous default value [] as argument" but this happens a lot elsewhere. I agree this should be changed, but maybe not in this PR...
I think this should really be changed, because this can lead to very hard to find bugs:
class Test: def __init__(self, some_list=[]): self.some_list = some_list t1 = Test() t2 = Test() t2.some_list.append(1) print(t1.some_list) # prints [1]
Originally posted by @cehbrecht in https://github.com/geopython/pywps/pull/456#issuecomment-471551513
- Lingua principale
- Python
- Stelle
- 186
- Fork
- 116
- Merge medio
- 11g 20h
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di geopython/pywps
-
ReadTheDocs OAuth App InstallationForse di nuovo libera @tomkralidis l’ha presa 71 giorni fa e non c’è nessuna pull request aperta. Apertadocumentation
-
Modernizing the packaging of the libraryForse di nuovo libera @Zeitsperre l’ha presa 284 giorni fa e non c’è nessuna pull request aperta. Apertadocumentation enhancement packaging
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
-
ComplexOutput data as reference always generates a file named "input.csv", should be "output".Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Tutte le issue di geopython/pywps
Issue simili
-
Claiming namespace `apoint`Apertanamespace operations
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
EclipseFdn/open-vsx.org#13573 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
collective/icalendar#1854 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
rancher/rancher-ai-agent#412 ·
I maintainer di solito rispondono entro 6 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
TUDelftGeodesy/DePSI#134 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
HenriquesLab/rxiv-maker#335 ·