Dlaczego pycharm proponuje zmianę metody na statyczną

Nowe wydanie pycharm (wydanie społeczności 3.1.3) proponuje konwersję metod, które nie działają z bieżącym stanem obiektu, na statyczne.

Jaki jest tego praktyczny powód? Jakiś rodzaj mikro-optymalizacji (-lub-pamięci) -optymalizacji?

Pycharm „myśli”, że mogłeś chciał mieć metody statyczne, ale zapomniałeś o stwierdzenie, że jest statyczny (za pomocą @staticmethoddekoratora).

PyCharm proponuje to, ponieważ metoda nie używa self jej w treści i dlatego w rzeczywistości nie zmienia instancji klasy . W związku z tym metoda może być statyczna, tj. Wywoływalna bez przekazywania instancji klasy lub nawet bez jej utworzenia.

Uzgodnione z @jolvi, @ArundasR i innymi, ostrzeżenie pojawia się w przypadku funkcji członkowskiej, która nie używa self.

Jeśli jesteś pewien, że PyCharm się myli, że funkcja nie powinna być a @staticmethod, a jeśli cenisz zero ostrzeżeń, możesz to zrobić na dwa różne sposoby:

Obejście nr 1

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

def is_not_used(self):
    pass

Obejście nr 2 ^{[Thanks @ DavidPärsson ]}

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

Aplikacja, którą miałem w tym celu (powód, dla którego nie mogłem użyć @staticmethod), polegała na tworzeniu tabeli funkcji obsługi odpowiadających na pole podtypu protokołu. Wszyscy przewodnicy musieli oczywiście mieć tę samą formę (statyczną lub niestatyczną). Ale niektórzy nie zrobili nic z instancją. Gdybym uczynił te statyczne, dostałbym "TypeError: 'staticmethod' object is not callable".

Na poparcie konsternacji PO, sugerowanie dodawania metody statycznej, kiedy tylko jest to możliwe, jest sprzeczne z tą zasadą , że łatwiej jest uczynić kod mniej restrykcyjnym później, niż uczynić go bardziej - uczynienie metody statyczną czyni ją mniej restrykcyjną teraz, ponieważ można wywołaj class.f () zamiast instance.f ().

Zgadnij, dlaczego istnieje to ostrzeżenie:

• To reklamuje staticmethod . Uświadamia programistom coś, co mogli zamienić.
• Jak wskazuje @ JohnWorrall's, przyciąga twoją uwagę, gdy jaźń została nieumyślnie pominięta w funkcji.
• Jest to wskazówka do ponownego przemyślenia modelu obiektowego; może funkcja w ogóle nie należy do tej klasy.

Myślę, że powodem tego ostrzeżenia jest config w Pycharm. Możesz odznaczyć wybór Metoda może być statyczna w Edytorze-> Inspekcja

Zgadzam się z odpowiedziami podanymi tutaj (metoda nie używa selfi dlatego można ją ozdobić @staticmethod).

Chciałbym dodać, że być może chcesz przenieść metodę do funkcji najwyższego poziomu zamiast metody statycznej wewnątrz klasy. Po szczegóły zobacz to pytanie i zaakceptowaną odpowiedź: python - czy powinienem używać metod statycznych czy funkcji najwyższego poziomu

Przeniesienie metody do funkcji najwyższego poziomu naprawi również ostrzeżenie PyCharm.

Mogę sobie wyobrazić następujące zalety posiadania metody klasowej zdefiniowanej jako statyczna:

• możesz wywołać metodę używając nazwy klasy, bez potrzeby jej tworzenia.

pozostałe zalety są prawdopodobnie marginalne, jeśli w ogóle występują:

• może działać trochę szybciej
• zaoszczędzić trochę pamięci

Ponieważ nie odwołałeś się selfw bartreści metody, PyCharm pyta, czy mógłbyś chcieć zrobić barstatyczne. W innych językach programowania, takich jak Java, istnieją oczywiste powody deklarowania metody statycznej. W Pythonie jedyną realną korzyścią dla metody statycznej (AFIK) jest możliwość wywołania jej bez instancji tej klasy. Jeśli jednak to twój jedyny powód, prawdopodobnie lepiej będzie, jeśli wybierzesz funkcję najwyższego poziomu - jak zauważ tutaj .

Krótko mówiąc, nie jestem w stu procentach pewien, dlaczego tam jest. Zgaduję, że prawdopodobnie usuną go w nadchodzącej wersji.

Ten komunikat o błędzie tylko mi pomógł, ponieważ nie zdawałem sobie sprawy, że przypadkowo napisałem swoją funkcję za pomocą mojego testowego odtwarzacza

my_player.attributes[item] 

zamiast we właściwy sposób

self.attributes[item]

Może to być trochę bałaganiarskie, ale czasami po prostu nie potrzebujesz dostępu self, ale wolisz zachować metodę w klasie i nie ustawiać jej statycznie. Lub po prostu chcesz uniknąć dodawania wielu nieestetycznych dekoratorów. Oto kilka potencjalnych obejść tej sytuacji.

Jeśli Twoja metoda ma tylko skutki uboczne i nie obchodzi Cię, co zwróci:

def bar(self):
    doing_something_without_self()
    return self

Jeśli potrzebujesz wartości zwracanej:

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

Teraz Twoja metoda używa self, a ostrzeżenie zniknie!

Powód, dla którego Pycharm robi to jako ostrzeżenie, ponieważ Python przekaże self jako pierwszy argument podczas wywoływania metody statycznej none (nie dodaje @staticmethod). Pycharm o tym wie.

Przykład:

class T:
    def test():
        print "i am a normal method!"

t = T()
t.test()
output:
Traceback (most recent call last):
  File "F:/Workspace/test_script/test.py", line 28, in <module>
    T().test()
TypeError: test() takes no arguments (1 given)

Jestem z Javy, w Javie "self" nazywa się "this", nie musisz pisać self (lub this) jako argumentu w metodzie klasowej. Możesz po prostu zadzwonić do siebie w ramach metody. Ale Python „musi” przekazać self jako argument metody.

Rozumiejąc to, nie potrzebujesz żadnego Obejścia, ponieważ odpowiedź @BobStein.