Python: példányok nyilvántartása osztályon belül

Nem néztem meg a programodat, de miért kellene egy osztálynak ismernie a létező példányait?
És mit értesz pontosan az alatt, hogy az osztály? Az összes példány osztozni fog egy közös listán?

Persze lehet, hogy nem értem az alap problémát. Esetleg leírnád, hogy mit is szeretnél megoldani?

Klasszik OOP szerint:

- csinálnék egy listát amiben ellenség típusú objektumok lehetnek
- ez a lista legyen egy singleton

--
Gábriel Ákos

Hozz létre egy __del__(self) metódust, ami kitörli a nyilvántartásból az objektumát, mielőtt a szemétgyűjtő törölné magát az objektumot.

Szerk.: Ez mégsem oldja meg automatikusan a problémát, mert a __del__() meghívásának időpontja nem látható előre.
--
eutlantis

Valami ilyesmit tennék:

#!/usr/bin/env python3
#-*- coding:utf-8 -*-

class Soldier():
 
    instances = {} # name:self
 
    def __init__(self, name):

        self.name= name
        if name in self.instances:
                raise NameError("Duplikált név: '"+name+"'")
        else:
            self.instances[name]=self

    @classmethod
    def isalive(cls,name):
        return name in cls.instances

    def fight(self):
        if not self.isalive(self.name):
            return None
        else:
            # ....
            return True

    def shoot(self):
        if not self.isalive(self.name):
            return None
        else:
            # ....
            return True

    @classmethod
    def trupp(cls):
        return list(cls.instances.keys())

    @classmethod
    def kill(cls,name):
        ident= cls.instances.pop(name,None)

if __name__ == "__main__":

    enemy=[ Soldier(name) for name in ['Alpha', 'Beta', 'Gamma'] ]
    print( Soldier.trupp()) 

    try:
        alpha2=Soldier('Alpha')
    except NameError as event:
        print(event)

    Soldier.kill('Gamma')
    print( Soldier.trupp())

    jedi=Soldier("Jedi")
    print( Soldier.trupp())
    print( jedi.fight() )
    jedi.kill("Jedi")  # öngyilkosság?
    print( jedi.fight() )

És egy katona bármit is tenne (fight,shoot), előtte megnézné, hogy egyáltalán még életben van-e. Még meghalni is csak akkor szabad, ha még él:-)
--
eutlantis

Eventek kellenek neked szerintem.

https://stackoverflow.com/questions/1092531/event-system-in-python/

a linkelt SO kérdésben olyat csinálnak, hogymikor a GC szabadítaná fel az objektumot, akkor kerüljön ki a listából.

Ha jól értem, neked jól megfogható halál eseményed van, amikor ki tudod szedni kézzel a listából - és minden referenciát megszüntetsz rá, a GC meg majd összekaparja valamikor. Ehhez nem kell weakreference, mehet az az út, amit elkezdtél.

Lehet, hogy öreg vagyok már, de nem statikus metódust nem egy példányon keresztül hívni elég betegnek tűnik. @classmethod helyett miért nem @staticmethod van? A this(itt cls) paraméter meg nem kell, hisz ezek nem egy példányhoz kötött metódusok.

Én így csinálnám:

https://pastebin.com/RU5GG0Pk

Hát én már nagyon rég átnyergeltem Go-ra, úgyhogy előre is bocsi.
De anno se szerettem ezeket a típusokat, osztályokat, öröklődést - sokszor megégettem magam...

Ha jól látom, itt egy darab listába (dict, ha gyorsabbat akarsz) kell karbantartani az élő példányokat.

Tehát kell egy globális vagy csilivili osztály PÉLDÁNY aminek a pop/push metódusát meg kell hívni __init__ és kill esetén.
Én ezt nem keverném bele az osztályba, simán kapja meg az __init__-ben hogy melyik dict-ben kell ezt nyilvántartani, és kész.

Alapvetően "jóvanazúgy," ha az neked megfelel, csak majd visszaüthet ha megnő. De akkor úgyis igazodsz.
A static metódusokat szokták kerülni, ha lehetséges, mivel az öröklődéssel nem működik szépen együtt.
Itt ez megoldható lenne egy külső osztállyal, akinek pont a lista nyilvántartása lenne a feladata és lenne két nagyon egyszerű osztály. Az egyszerűség jó.

Ami nekem személy szerint sokkal jobban szúrja a szememet, az a printelős tesztelés. Nem számottevően nehezebb unittest-esen megfogalmazni, viszont sokkal hasznosabb.

_______Peter
I am a stone. I do not move.