Mengapa pycharm mengusulkan untuk mengubah metode menjadi statis

Rilis pycharm baru (edisi komunitas 3.1.3) mengusulkan untuk mengubah metode yang tidak berfungsi dengan keadaan objek saat ini menjadi statis.

Apa alasan praktis untuk itu? Semacam optimasi kinerja mikro (-atau-memori)?

PyCharm "berpikir" bahwa Anda mungkin ingin memiliki metode statis, tetapi Anda lupa menyatakannya sebagai statis (menggunakan @staticmethoddekorator).

PyCharm mengusulkan ini karena metode ini tidak digunakan self dalam tubuhnya dan karenanya tidak benar-benar mengubah instance kelas . Karenanya metode ini bisa statis, yaitu callable tanpa melewati instance kelas atau bahkan tanpa membuat instance kelas.

Setuju dengan @jolvi, @ArundasR, dan lainnya, peringatan tersebut terjadi pada fungsi anggota yang tidak digunakan self.

Jika Anda yakin PyCharm salah, bahwa fungsinya seharusnya tidak menjadi @staticmethod, dan jika Anda menghargai peringatan nol, Anda bisa membuat yang ini menghilangkan dua cara berbeda:

Solusi # 1

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

def is_not_used(self):
    pass

Penanganan masalah # 2 ^{[Terima kasih @ DavidPärsson ]}

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

Aplikasi yang saya miliki untuk ini (alasan saya tidak bisa menggunakan @staticmethod) adalah membuat tabel fungsi handler untuk menanggapi bidang subtipe protokol. Semua penangan harus berbentuk kursus yang sama (statis atau tidak statis). Tetapi beberapa tidak melakukan apa-apa dengan instance. Jika saya membuat yang statis saya akan mendapatkan "TypeError: objek 'staticmethod' tidak bisa dipanggil".

Untuk mendukung kekuatiran OP, menyarankan Anda menambahkan metode statis kapan pun Anda bisa, bertentangan dengan prinsip bahwa lebih mudah membuat kode menjadi tidak terlalu ketat nanti, daripada membuatnya lebih banyak - membuat metode statis membuatnya kurang ketat sekarang, karena Anda dapat panggil class.f () alih-alih instance.f ().

Tebak mengapa peringatan ini ada:

• Ini mengiklankan metode statis . Itu membuat pengembang menyadari sesuatu yang mungkin mereka maksudkan.
• Seperti yang ditunjukkan oleh @ JohnWorrall, itu menarik perhatian Anda ketika diri Anda secara tidak sengaja dihilangkan dari fungsi tersebut.
• Ini adalah isyarat untuk memikirkan kembali model objek; mungkin fungsinya tidak termasuk dalam kelas ini sama sekali.

Saya pikir alasan untuk peringatan ini adalah konfigurasi di Pycharm. Anda dapat menghapus centang pada pilihan Metode mungkin statis di Editor-> Inspeksi

Saya setuju dengan jawaban yang diberikan di sini (metode tidak digunakan selfdan karenanya dapat dihiasi dengan @staticmethod).

Saya ingin menambahkan bahwa Anda mungkin ingin memindahkan metode ke fungsi tingkat atas daripada metode statis di dalam kelas. Untuk detail, lihat pertanyaan ini dan jawaban yang diterima: python - haruskah saya menggunakan metode statis atau fungsi tingkat atas

Memindahkan metode ke fungsi tingkat atas akan memperbaiki peringatan PyCharm juga.

Saya bisa membayangkan keuntungan berikut memiliki metode kelas yang didefinisikan sebagai metode statis:

• Anda dapat memanggil metode hanya menggunakan nama kelas, tidak perlu instantiate.

keuntungan yang tersisa mungkin marjinal jika ada sama sekali:

• mungkin berjalan sedikit lebih cepat
• menghemat sedikit memori

Karena Anda tidak merujuk selfpada barbadan metode, PyCharm bertanya apakah Anda mungkin ingin membuat barstatis. Dalam bahasa pemrograman lain, seperti Java, ada alasan yang jelas untuk mendeklarasikan metode statis. Dalam Python, satu-satunya manfaat nyata untuk metode statis (AFIK) adalah dapat memanggilnya tanpa turunan dari kelas. Namun, jika itu satu-satunya alasan Anda, Anda mungkin lebih baik menggunakan fungsi tingkat atas - seperti dicatat di sini .

Singkatnya, saya tidak seratus persen yakin mengapa itu ada di sana. Saya kira mereka mungkin akan menghapusnya dalam rilis mendatang.

Pesan kesalahan ini hanya membantu saya banyak, karena saya tidak menyadari bahwa saya tidak sengaja menulis fungsi saya menggunakan contoh pemain pengujian saya

my_player.attributes[item] 

bukannya dengan cara yang benar

self.attributes[item]

Mungkin agak berantakan, tetapi kadang-kadang Anda tidak perlu mengaksesnya self, tetapi Anda lebih suka menyimpan metode di kelas dan tidak membuatnya statis. Atau Anda hanya ingin menghindari menambahkan dekorator yang tidak sedap dipandang. Berikut adalah beberapa solusi potensial untuk situasi itu.

Jika metode Anda hanya memiliki efek samping dan Anda tidak peduli dengan pengembaliannya:

def bar(self):
    doing_something_without_self()
    return self

Jika Anda membutuhkan nilai pengembalian:

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

Sekarang metode Anda gunakan self, dan peringatan itu hilang!

Alasan mengapa Pycharm menjadikannya sebagai peringatan karena Python akan menganggap diri sebagai argumen pertama ketika memanggil metode tidak ada statis (tidak menambahkan @staticmethod). Pycharm tahu itu.

Contoh:

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)

Saya dari Jawa, di Jawa "diri" disebut "ini", Anda tidak perlu menulis sendiri (atau ini) sebagai argumen dalam metode kelas. Anda bisa memanggil diri sendiri sesuai kebutuhan di dalam metode ini. Tapi Python "harus" lulus sebagai argumen metode.

Dengan memahami ini, Anda tidak memerlukan Penanganan Masalah karena jawaban @BobStein.