Der Pythonthread

[delnan]

Der Code macht so eh keinen Sinn, außer man will Team 1 oder 0 auswählen und selbst dann ist es recht umständlich Wenn das den Hegomon rausfinden soll, indem er für alle Teams durchprobiert, ob das Vasall zu diesem Team gehört, dann muss es heißen:

next(i for i in range(gc.getMAX_CIV_TEAMS()) if pPreviousTeam.isVassal(i))

Das kann man auf Kompabilität trimmen indem man daraus das macht:

(i for i in range(gc.getMAX_CIV_TEAMS()) if pPreviousTeam.isVassal(i)).next()

[alpha civ]

Wieso will man einen Iterator, wenn man sowieso nur an dem ersten Element interessiert ist?
Oder warum das "next"?

[delnan]

Weil man so zum ersten i in range(...) springt, für das pPreviousTeam.isVassal(i) gilt.

[alpha civ]

Weil man so zum ersten i in range(...) springt, für das pPreviousTeam.isVassal(i) gilt.

Dann muss man doch aber vorher sicherstellen, dass ein solches i tatsächlich existiert, sonst bekommt man eine "StopIteration-Exception".

Und statt "range" sollte man doch lieber "xrange" verwenden.

[delnan]

Dann muss man doch aber vorher sicherstellen, dass ein solches i tatsächlich existiert, sonst bekommt man eine "StopIteration-Exception".

Wird ja vorher mit any() gemacht. Könnte man wohl auch anders regeln (chain(..., [None]); if pHegomon is not None: ... z.B.), aber jetzt isses nunmal so

Und statt "range" sollte man doch lieber "xrange" verwenden.

Stimmt, vergess ich immer wieder. Das ist der Nachteil, wenn man normalerweise 3.x benutzt

[alpha civ]

Wird ja vorher mit any() gemacht. Könnte man wohl auch anders regeln (chain(..., [None]); if pHegomon is not None: ... z.B.), aber jetzt isses nunmal so

Oder man macht einfach sowas:

try:
    pHegemon = gc.getTeam(i for i in range(gc.getMAX_CIV_TEAMS()) if pPreviousTeam.isVassal(i)).next())
except StopIteration:
    pHegemon = None

Dann spart man sich das any (was eine überflüssige Schleife ist). Man braucht dann nur zu testen, ob pHegemon None ist, also

if not pHegemon is None:
    irgendwas

[Kathy]

Ich habe einmal den Vorschlag von alpha civ eingebaut. Der Code sieht nun so aus:

    def onCityAcquired(self, argsList):
        'City Acquired'
        iPreviousOwner,iNewOwner,pCity,bConquest,bTrade = argsList
        CvUtil.pyPrint('City Acquired Event: %s' %(pCity.getName()))

### Vasallenbonus entziehen beginn ###
        pPreviousOwner = gc.getPlayer(iPreviousOwner)
        if pPreviousOwner.getNumCities() == 0:
            pPreviousTeam = gc.getTeam(pPreviousOwner.getTeam())
            try:
                pHegemon = gc.getTeam(i for i in range(gc.getMAX_CIV_TEAMS()) if pPreviousTeam.isVassal(i)).next()
            except StopIteration:
                pHegemon = None
            if not pHegemon is None:
                pHegemon = gc.getTeam(i for i in xrange(gc.getMAX_CIV_TEAMS()) if pPreviousTeam.isVassal(i)).next() 

Leider sorgt diese Zeile für ein Problem:

                pHegemon = gc.getTeam(i for i in range(gc.getMAX_CIV_TEAMS()) if pPreviousTeam.isVassal(i)).next() 

Ich verstehe leider weder die Zeile noch die Fehlermeldung zu hundert Prozent.

[delnan]

Alpha civ hat sich beim platzieren des .next() um eine Klammer vertan. Muss direkt an den Generator statt an das Ergebnis des .getTeam(), also:

pHegemon = gc.getTeam((i for i in range(gc.getMAX_CIV_TEAMS()) if pPreviousTeam.isVassal(i)).next())

[Kathy]

Danke, delnan. Der Code funktioniert nun für fast alle Situationen - außer an dieser Stelle:

                if pPreviousOwner.hasTrait(gc.getInfoTypeForString("TRAIT_STADTSTAAT_ARTISTIC")):
                    (loopCity, iter) = pHegemon.firstCity(false)
                    while(loopCity):
                        loopCity.setNumRealBuilding(gc.getCivilizationInfo(pHegemon.getCivilizationType()).getCivilizationBuildings(gc.getInfoTypeForString("BUILDINGCLASS_SUPPORT_STADTSTAAT_ARTISTIC")),0)
                                        (loopCity, iter) = pHegemon.nextCity(iter, false) 

Das Problem ist, dass pHegemon ein Team ist, ich aber dieses Gebäude nur aus den Städte eines Spielers entfernen kann. Ich muss also irgendwie alle Spieler von pHegemon herausfinden und in der Schleife durchgehen. Wie bekomme ich möglichst elegant alle betroffenen Spieler heraus?

                    (loopCity, iter) = pHegemon.firstCity(false) 

[delnan]

Das ist blöde. Muss man wohl das draus machen:

                hegomonPlayers = (gc.getPlayer(i) for i in range(gc.getMAX_CIV_PLAYERS()) if gc.getPlayer(i).getTeam() == pHegomon.getID())
                if pPreviousOwner.hasTrait(gc.getInfoTypeForString("TRAIT_STADTSTAAT_ARTISTIC")):
                    for pHegomonPlayer in hegomonPlayers:
                        (loopCity, iter) = pHegemon.firstCity(false)
                        while(loopCity):
                            loopCity.setNumRealBuilding(gc.getCivilizationInfo(pHegomonPlayer.getCivilizationType()).getCivilizationBuildings(gc.getInfoTypeForString("BUILDINGCLASS_SUPPORT_STADTSTAAT_ARTISTIC")),0)
                                            (loopCity, iter) = pHegemon.nextCity(iter, false) 

[Kathy]

Danke vielmals, delnan! Es funktioniert!

[Cybah]

triggert on improvementbuild auch bei automatischen upgrades wie hütte -> weiler? ja

ach und lässt sich die stadt abfragen, die diese hütte bearbeitet?

[The_J]

oh, übersehen, mycity = pPlot.getWorkingCity ()


----------
alpha, wenn ich mich richtig erinnere hast du schonmal mit matplotlib gearbeitet, ja ? Falls dem richtig ist, bist du fit genug drin um eine Frage zu beantworten ?

[alpha civ]

oh, übersehen, mycity = pPlot.getWorkingCity ()


----------
alpha, wenn ich mich richtig erinnere hast du schonmal mit matplotlib gearbeitet, ja ? Falls dem richtig ist, bist du fit genug drin um eine Frage zu beantworten ?

Nein, intensiv habe ich mich damit noch nicht ausgesetzt. Was willst du denn machen?

[Cybah]

Naja, wenn eine Gemeinde entsteht, soll die Stadt, die diese bearbeitet, 1 Bevölkerung dazu bekommen. Bisher hab ich das mit der nächstgelegenen Stadt gelöst, aber das muss ja nicht immer stimmen je nach Grenzverlauf.

Edith fragt sich gerade, ob die Frage überhaupt an mich ging. Vermutlich ists mit The_Js Hilfe schon getan.