Python-Instanz hat keine call - Methode

Habe ich gelernt, wie man Python für die bessere Teil von heute, nachdem er bis auf einen hässlichen bash-Skript.

Ich versuche, die 2 Klassen zu definieren, die ein paar von arrays von Objekten, in denen zu speichern, einige einzigartige strings und zahlen (1-10). Die Objekte bestehen aus den folgenden:

object[i].user
         .n     # n = i
         .name
         .coords
         .hero

(param1, param2, param3) wird anders sein für jedes Objekt.n und Objekt.Nutzer, so dass ich versuche, eine Zuweisung, die nicht Aussehen wie Müll, nachdem er 90 einzigartige Zeichenfolgen. Verschachtelung ein Beispiel habe ich gefunden, hat nicht funktioniert, hier also der Kompromiss:

class CityBean:
    def __init__(self,name,coords,hero):
        self.name = name
        self.coords = coords
        self.hero = hero

class Castles:
    def __init__(self,user,n):
        self.user = user
        self.n = n
        if self.user == 'user1':
            temp = {
                1:  CityBean( "name1"  , "coord1" , "hero1"),
                ... blah blah blah
                10: CityBean( "name10" , "coord10" , "hero10" )}[self.n]()
        if self.user == 'user2':
            temp = {
                1:  CityBean( "name11" , "coord11" , "hero11" ),
                ... blah blah blah
                10: CityBean( "name20" , "coord20" , "hero20" ) }[self.n]()
        if self.user == 'user3':
            temp = {
                1:  CityBean( "name21" , "coord21" , "hero21" ),
                ... blah blah blah
                10: CityBean( "name30" , "coord30" , "hero30" ) }[self.n]()
        self.name = temp.name
        self.coords = temp.coords
        self.hero = temp.coords
        __del__(temp)

Ich nenne es mit so etwas wie dies:

cities = list( Castles("user2",i) for i in range(1,11) )

Es gibt mir diese Fehlermeldung:

AttributeError: CityBean instance has no __call__ method

Und es macht diese Zeile:

                10: CityBean( "name20" , "coord20" , "hero20" ) }[self.n]() # pseudo
                10: CityBean( "" , "" , "" ) }[self.n]() # what is actually looks like

Was ist Los mit meinem cruddy Klassen? Bin ich etwas zurückgeblieben, nicht wahr?

Was wollen Sie erreichen mit [self.n]()?
Ich habe nur versucht, wählen Sie eine der Optionen, nicht ahnend, dass die Klammern bezeichnet das Ergebnis als Funktion... oops!

InformationsquelleAutor tyblu | 2010-02-22

Es ist wirklich schwer zu erraten, was Sie wollen, was Sie zur Verfügung gestellt, denn, anstatt zu sagen, was Sie wollen, sofern Sie Ihre newbie-code, so hat man eine Festplatte denke, die Zeit.

Ich würde denken, etwas wie dies tun würde:

data = {
        (1, 'user1'): ("name1", "coord1", "hero1"),
        (2, 'user1'): ("name2", "coord2", "hero2"),
        #...
        (1, 'user2'): ("name11", "coord11", "hero11"),
        (2, 'user2'): ("name12", "coord12", "hero12"),
        # ...
    }


class CityBean:
    def __init__(self,name,coords,hero):
        self.name = name
        self.coords = coords
        self.hero = hero

class Castles:
    def __init__(self,user,n):
        self.user = user
        self.n = n
        name, coords, hero = data.get((n, user))
        self.citybean = CityBean(name, coords, hero)

+1 ich verneige mich vor Ihrer Fähigkeit, Sinn zu machen von dem Ziel, den code in dieser Frage.
Hey, das sieht echt gut aus! Dank nosklo.
Beachten Sie die Verwendung des Tupel (1,'user1') als key des dict. Ein Tupel ist unveränderlich und kann daher als Schlüssel verwendet. Eine Liste [1, 'user1'] können nicht als Schlüssel verwendet.
Ein Tupel, hey... Scheint ein Mehrdeutiger name. 🙂 Danke für den Tipp.

InformationsquelleAutor nosklo

4

Warum habt Ihr dieses schreiben?
```
temp = {
            1:  CityBean( "name21" , "coord21" , "hero21" ),
            ... blah blah blah
            10: CityBean( "name30" , "coord30" , "hero30" ) }[self.n]()
```
Was denkst du temp= {...}[something]() tun?
1. Schafft es ein Wörterbuch. {...}
2. Es wählt ein Element aus dem dictionary. {...}[something]. Dies wird eine CityBean Objekt.
3. Wertet er, dass ein Element als eine Funktion {...}[something]() Ist CityBean(...)()
Warum fordern Sie die CityBean Objekt, als wenn es wurde eine Funktion?

Weiter, warum erstellen Sie komplette Wörterbücher nur um einen einzigen Punkt davon? Was ist falsch mit if Aussagen?

InformationsquelleAutor S.Lott
2

Ihrem tun
```
{
1:  CityBean( "name1"  , "coord1" , "hero1"),
... blah blah blah
10: CityBean( "name10" , "coord10" , "hero10" )}[self.n]()
```
welcher im wesentlichen aus einem dict get-Wert basiert auf Schlüssel, Werte auf Ihre dict sind CityBean Instanzen, so kurz Sie sind, dies zu tun
```
CityBean( "name1"  , "coord1" , "hero1")()
```
gültig ist, aber dann rufen wir die spezielle Methode __call__ von der Instanz, also entweder entfernen () oder fügen Sie ein __call__ Methode, je nach Bedarf

InformationsquelleAutor Anurag Uniyal
1

Ich nicht wirklich verstehen, was Sie zu tun versuchen genau das, aber:
```
 10: CityBean( "name10" , "coord10" , "hero10" )}[self.n]()
```
wenn etwas rufen (die () am Ende machen es zu einem Funktionsaufruf).

Was würden Sie wollen, ist, denke ich
temp = { ...
10: CityBean( "name10" , "coord10" , "hero10" )}[self.n]

In diesem Fall würden Sie bekommen das dict-Eintrag mit-Taste n und-Aufkleber, die temp.
- Ja, das wäre meine quick-fix -- danke!
InformationsquelleAutor extraneon
1

Beachten Sie, dass in python, wenn Sie schreiben
```
CityBean( "name1"  , "coord1" , "hero1")
```
Initialisieren Sie das Objekt CityBean. Es gibt keine Notwendigkeit für eine zusätzliche ().

InformationsquelleAutor Thomas Ahle
1

Können Sie dieses problem, wenn Sie etwas falsch machen
```
os.environ("METAG_DATA")
```
Dem richtigen Weg ist
```
os.environ["METAG_DATA"]
```
InformationsquelleAutor Kemin Zhou
0

Setzen Klammern nach dem name einer Klasse in einem normalen Ausdruck instanziiert es sofort. Wenn Sie möchten, zu verzögern, Instanziierung, dann müssen Sie mit lambda oder functools.partial():
```
10: lambda: CityBean("", "", "")}...
```
InformationsquelleAutor Ignacio Vazquez-Abrams

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.