La meilleure pratique pour le réglage de la valeur par défaut d'un paramètre qui est censé être une liste en Python?

J'ai une fonction Python qui prend une liste en tant que paramètre. Si j'ai mis la valeur par défaut du paramètre à une liste vide comme ceci:

def func(items=[]):
    print items

Pylint me dire "Dangereux valeur par défaut [] comme argument". Donc, je me demandais quelle est la meilleure pratique ici?

  • c'est quelque chose que chaque débutant en python talons de leurs pieds sur une ou deux fois, c'est assez cool que pylint a arrêté d'écrire un horrible bug !
  • Une simple astuce consiste à utiliser un tuple vide: def func(items=()). Un tuple est pas mutable donc pylint ferme, mais la question demande une liste, donc ce ne sera pas toujours pertinentes ou possibles.
InformationsquelleAutor Jack Z | 2012-03-02
  1. 88

    Utilisation None en tant que valeur par défaut:

    def func(items=None):
    if items is None:
        items = []
    print items

    Le problème avec une mutable argument par défaut c'est qu'il va être partagé entre toutes les invocations de la fonction -- voir "avertissement important" dans la section pertinente du tutoriel Python.

    • Ah wow, je suis surpris que les gens savent à ce sujet. =) J'ai détesté cette fonctionnalité de python tellement que j'ai écrit mon propre décorateur bibliothèque pour permettre func(items=new([])) de la syntaxe.
    • Merci! Il semble que Python pense qu'Aucun n'est d'un tout autre type de liste, alors je me demandais est-il généralement acceptable en Python pour définir la valeur par défaut d'un paramètre d'un type différent (comme Aucun) (je sais que Python est un type de langage, mais je viens de C++... lol)?
    • Il est tout à fait différent d'un langage fortement typé comme le C++. Il suffit de penser de variables que des noms qui font référence à des objets, et tout est passé par référence.
    • Je pense que tu veux dire par 'statique', pas fortement; python est fortement, mais typées dynamiquement.
    • Merci, wim! Ensuite, lorsque vous utilisez une variable, comment pouvez-vous décider si c'est une liste réelle ou gardant toujours sa valeur par défaut None? Ou avez-vous toujours de vérifier avant de l'utiliser?
    • Vous n'avez pas à vérifier à chaque fois, et seulement quand vous vous attendez à ce que ce ne sera pas ce que vous pensez qu'il devrait être. C'est comme vérifier si un pointeur/référence est nulle en C++, .. vous n'avez pas de vérifier tous les temps, mais dans de nombreux cas, il est logique de se prémunir contre elle.
    • J'ai l'habitude d'écrire des items = items or [] si je m'attends à items être un objet iterable type.

    InformationsquelleAutor Sven Marnach
  2. 8

    J'ai juste rencontré pour la première fois, et ma pensée est: "eh bien, je ne veux pas muter la liste de toute façon, donc ce que je veux vraiment, c'est par défaut d'une liste immuable afin de Python va me donner une erreur si j'ai accidentellement pour les transformer." Une liste immuable est juste un n-uplet. Donc:

     def func(articles=()): 
imprimer les éléments

    Sûr, si vous le passer à quelque chose qui n'a vraiment envie d'une liste (par exemple isinstance(articles, liste)), ensuite ce sera vous obtenez en difficulté. Mais c'est une odeur de code de toute façon.

    • Et si à l'intérieur de la fonction, vous devez faire une copie, l'utilisation my_copy = list(items). Il est venu avec un simple et très ingénieuse solution à un problème commun.
    InformationsquelleAutor sfink
  3. 3

    Pour mutable objet en tant que paramètre par défaut de la fonction et de la méthode de déclarations le problème, c'est que l'évaluation et de la création a lieu exactement au même moment. Le python-analyseur lit de la fonction-tête et l'évalue au même moment.

    Plus beginers asume qu'un nouvel objet est créé à chaque appel, mais ce n'est pas correct! UN objet (dans votre exemple d'une liste) est créé au moment de la DÉCLARATION et non à la demande lorsque vous appelez la méthode.

    Pour imutable objets qui n'est pas un problème, parce que même si tous les appels partagent le même objet, c'est imutable et donc ses propriétés restent les mêmes.

    Comme une convention vous utilisez le None objet pour les valeurs par défaut pour indiquer l'utilisation d'une initialisation par défaut, qui peut maintenant prendre place dans la fonction-corps, qui, naturellement, est évalué à du temps d'appel.

    InformationsquelleAutor Don Question
  4. 1

    En plus et aussi à mieux comprendre ce que python est, ici, ma petite sur le thème de l'extrait de code:

    from functools import wraps
def defaultFactories(func):
    'wraps function to use factories instead of values for defaults in call'
    defaults = func.func_defaults
    @wraps(func)
    def wrapped(*args,**kwargs):
        func.func_defaults = tuple(default() for default in defaults)
        return func(*args,**kwargs)
    return wrapped

def f1(n,b = []):
    b.append(n)
    if n == 1: return b
    else: return f1(n-1) + b

@defaultFactories
def f2(n,b = list):
    b.append(n)
    if n == 1: return b
    else: return f2(n-1) + b

>>> f1(6)
[6, 5, 4, 3, 2, 1, 6, 5, 4, 3, 2, 1, 6, 5, 4, 3, 2, 1, 6, 5, 4, 3, 2, 1, 6, 5, 4, 3, 2, 1, 6, 5, 4, 3, 2, 1]
>>> f2(6)
[1, 2, 3, 4, 5, 6]
    InformationsquelleAutor Odomontois