Pourquoi ne pycharm proposer de changer de méthode statique

La nouvelle pycharm libération (3.1.3 community edition) propose de transformer les méthodes qui ne fonctionnent pas avec l'objet courant de l'état statique.

Pourquoi ne pycharm proposer de changer de méthode statique

Qu'est-ce que la raison pratique pour qui? Une sorte de micro-performance (ou la mémoire)-optimisation?

  • Avez-vous cliquez sur "plus..."? Faites-vous référence self n'importe où à l'intérieur de la méthode? (Si la véritable question est "pourquoi PyCharm, les concepteurs de la conception de cette façon... vous allez leur demander, pas TELLEMENT...)
  • il est return 1 sur une seule ligne, la mise en œuvre de la méthode. "Plus" ne contiennent pas quelque chose d'utile
InformationsquelleAutor zerkms | 2014-05-09
  1. 134

    PyCharm "pense" que vous pourriez avoir voulais d'avoir une méthode statique, mais vous avez oublié de le déclarer à être statique.

    PyCharm propose cela parce que la méthode n'est pas utilisation self dans son corps et donc n'a pas fait changer la classe de l'instance. D'où la méthode pourrait être statique, c'est à dire remboursables sans avoir créé une instance de la classe avant.

    • De nombreuses personnes ont répondu à cette saveur de réponse. Je voudrais ajouter que, si vous savez qu'il est certainement ne va pas être une méthode statique, puis d'inclure une "jeter NotImplementedError" pendant que vous êtes là pour être certain que vous ne l'utilisez pas sans l'avoir terminé.
    • Il pourrait y avoir des cas où PyCharm de l'avertissement n'est pas justifiée en ce que nous ne voulons d'une méthode statique ni de modifier l'état. D'autre part, si la méthode n'est pas encore mis en œuvre, il semble toujours une bonne idée d'élever un NotImplementedError.
    • J'ai le cas que mon défaut de mise en œuvre renvoie une constante, mais mon sous-classes sont autorisés à retourner une valeur en fonction de self. Dans ce cas, l'avertissement est négligeable, et que je marque avec # noinspection PyMethodMayBeStatic. C'est dommage que IntelliJ IDEA n'offre pas l'ajout de cette désactivation des commentaires dans le contexte des menus pour cet avertissement.
    • Je suggère de changer la gravité de cette PyCharm inspection de l' "Avertissement" pour "la mise en évidence, uniquement à fixer" dans PyCharm préférences. (Il s'agit de produire de nombreux faux positifs pour moi.)
    InformationsquelleAutor jolvi
  2. 41

    D'accord avec @jolvi, @ArundasR, et d'autres, l'avertissement se produit sur une fonction membre qui n'utilise pas self.

    Si vous êtes sûr de PyCharm est mal, que la fonction ne doit pas être un @staticmethod, et, si vous la valeur zéro mises en garde, vous pouvez faire ce un aller de deux façons différentes:

    Solution n ° 1

    def bar(self):
    self.is_not_used()
    doing_something_without_self()

def is_not_used(self):
    pass

    Solution n ° 2 [Merci @DavidPärsson]

    # noinspection PyMethodMayBeStatic
def bar(self):
    doing_something_without_self()

    L'application que j'ai eu pour cette (raison pour laquelle je ne pouvais pas utiliser @staticmethod) a été dans un tableau de fonctions de gestionnaire pour répondre à un protocole sous-type de terrain. Tous les gestionnaires devaient être de la même forme de cours (statiques ou non). Mais certains n'arrivent pas à faire n'importe quoi avec l'instance. Si j'ai fait ces statique que j'obtiendrais "TypeError: 'staticmethod' objet n'est pas appelable".

    À l'appui de l'OP est la consternation, ce qui suggère que vous ajoutez staticmethod chaque fois que vous pouvez, va à l'encontre de la principe qu'il est plus facile de rendre le code moins restrictif, plus tard, que pour le rendre plus-création d'une méthode statique rend moins restrictives, désormais, que vous pouvez appeler la classe.f() à la place de l'instance.f().

    Devine pourquoi cet avertissement existe:

    • Il annonce staticmethod. Il rend les développeurs au courant de quelque chose qu'ils peuvent bien avoir prévu.
    • Comme @JohnWorrall points, il obtient votre attention lorsque auto a été par inadvertance de la fonction.
    • C'est un repère de repenser le modèle d'objet; peut-être la fonction n'appartient pas dans cette classe à tous.
    • "faire une méthode statique rend moins restrictives, désormais," - - - ce n'est pas du tout. Par exemple: les méthodes polymorphes
    • Je pense que le dernier point mérite d'être répété: "pourquoi pensez-vous en faire une méthode, quand il s'agit clairement d'une fonction?" Garder les choses qui a vraiment besoin d'une instance distincte de la structure qui traite de certains aspects. Vous pouvez ensuite facilement le subdiviser en module séparé.
    • Les règles sont de belles choses jusqu'à ce que la journée vous pensez à un raisonnable exception. J'ai décrit l'un, une liste de rappels.
    • si elles ne sont pas à l'aide de variables d'instance, alors pourquoi ne pas utiliser le module pour le groupement? Assurez-vous parfois une méthode statique s'inscrit dans l'histoire, c'est pourquoi il existe, mais je pensais que ce point particulier de vous, c'est important.
    • Je trouve cette fonctionnalité de PyCharm utile et je suis fortement d'accord avec le dernier point. À l'aide de non-statiques les méthodes de la classe qui n'ont pas de référence de soi est odeur de code.
    • L'ajout de # noinspection PyMethodMayBeStatic au-dessus de la méthode ou de la classe supprime l'avertissement, et il est à mon avis de mieux que d'appeler une méthode vide.
    • Convenu @DavidPärsson, ajouté à la réponse que la solution n ° 2. C'est la façon dont je le fais aujourd'hui. En gardant #1 parce que c'est un peu moins arcanes. Les lecteurs peuvent choisir.
    • Juste l'ajout de mes pensées, je suis à l'aide de python 3 et l'auto est déjà supprimé, alors pourquoi pycharm est encore de donner à ces mises en garde? Ma conjecture est que de la Charpie est toujours viser la version 2.x.
    • n'est pas retiré dans Python3 à tous.
    • Retiré n'était pas le bon mot à utiliser il y a, il devrait être, "pas nécessaire d'écrire maintenant".
    • Je ne comprends pas votre point de vue. Je ne pense pas de toute façon que self en Python 2 ne serait pas nécessaire en Python 3. Pouvez-vous donner un exemple ou un lien?

    InformationsquelleAutor Bob Stein
  3. 10

    Je pense que la raison de cet avertissement est config en Pycharm.
    Vous pouvez décocher la sélection Méthode peut être statique dans l'Éditeur->Inspection

    • Ma question était de savoir pourquoi une telle inspection de même qu'il existe. Je comprends que je ne peut l'éteindre. Désolé, pas de réponse.
    InformationsquelleAutor mincom
  4. 7

    Je peux imaginer les avantages d'avoir une méthode d'une classe définie comme statique:

    • vous pouvez appeler la méthode de la juste en utilisant un nom de classe, pas besoin d'instancier.

    restant avantages sont probablement marginal s'il est présent à tous:

    • pourraient courir un peu plus vite
    • économiser un peu de mémoire
    • Yep. Mais la chose est - je ne suis pas à l'utiliser comme une méthode statique. Sinon, il serait déjà en statique. Donc, PyCharm des conseils pour le faire sans une bonne raison (?). "en restant avantages sont probablement marginal s'il est présent à tous les" --- oui, exactement. Mais si c'est le cas - c'est un drôle de conseils de PyCharm
    • c'est comment ça se passe avec certains de charme instances 🙂
    • Les méthodes statiques sont un ennemi à la construction d'un bon logiciel. Ils void de nombreux principes de la bit courir plus vite, n'est pas le point (parce qu'ils s'exécuter dans la mémoire ram de manière rapide dans les deux cas) et comme vous le savez, les ordinateurs ont maintenant bunch de mémoire, ce qui n'est pas plus un problème. Notez également que votre première pensée: C'est une formalité de comportement n'est pas un objet orienté.
    InformationsquelleAutor Jan Vlcinsky
  5. 6

    Je suis d'accord avec les réponses données ici (méthode n'utilise pas self et, par conséquent, peuvent être décorés avec @staticmethod).

    Je tiens à ajouter peut-être que vous souhaitez déplacer la méthode à une fonction de niveau supérieur au lieu d'une méthode statique dans une classe. Pour les détails, voir à cette question, et l'on a accepté la réponse: python - dois-je utiliser des méthodes ou des fonctions de niveau supérieur

    Le déplacement de la méthode à un niveau supérieur de la fonction de corriger les PyCharm avertissement, trop.

    • Vraiment utile de répondre - peut-être PyCharm devrait renommer l'avertissement de la méthode peut être statique ou de la fonction de haut niveau'. Lors d'un refactoring une méthode, pycharm permettra de créer une fonction de haut niveau et non pas une méthode statique de toute façon, si self n'est pas un paramètre.
    InformationsquelleAutor tlo
  6. 3

    Ce message d'erreur juste m'a aidé un tas, que je n'avais pas réalisé que j'avais écrit accidentellement ma fonction à l'aide de mes tests exemple le joueur

    my_player.attributes[item]

    au lieu de la bonne façon

    self.attributes[item]
    InformationsquelleAutor John Worrall
  7. 3

    Puisque vous n'avez pas référence à self dans le bar corps de la méthode, PyCharm est pour vous demander si vous pourrait ont voulu faire de bar statique. Dans d'autres langages de programmation comme Java, il existe des raisons évidentes de déclaration d'une méthode statique. En Python, le seul réel avantage d'une méthode statique (AFIK) est d'être capable de l'appeler sans une instance de la classe. Cependant, si c'est votre seule raison, vous êtes probablement mieux d'aller avec une fonction de haut niveau - comme le remarque ici.

    En bref, je ne suis pas à cent pour cent sûr de savoir pourquoi il est là. Je devine qu'ils seront probablement retirer dans une prochaine version.

    InformationsquelleAutor Don
  8. 0

    Il pourrait être un peu brouillon, mais parfois vous n'avez pas besoin d'accéder à self, mais vous souhaitez conserver la méthode de la classe et pas rendre statique. Ou vous voulez juste pour éviter d'ajouter un tas de inesthétiques, les décorateurs. Voici quelques solutions potentielles de cette situation.

    Si votre méthode a des effets secondaires et vous ne se soucient pas de quoi il en retourne:

    def bar(self):
    doing_something_without_self()
    return self

    Si vous avez besoin de la valeur de retour:

    def bar(self):
    result = doing_something_without_self()
    if self:
        return result

    Maintenant, votre méthode est d'utiliser self, et l'avertissement s'en va!

    InformationsquelleAutor Junuxx