Civilisationen wiederbeleben/befreien

[Monaldinio]

Na dann kopiere ich mal den Code den du gepostet hast und setz ihn mal ein...^^

[Monaldinio]

Also der Kopf meiner Funktion sieht jetzt so aus...

[Kathy]

Nein! Du darfst nichts an dieser Args-List ändern, vor allem nichts hinzufügen oder entfernen. Der fehlende iOwner muss anders herausgefunden werden. Bzw. er ist schon bekannt, aber er heißt nund halt iNewOwner. Du musst den iOwner durch iNewOwner ersetzen.

[Monaldinio]

Muss ich den iOwner durch iNewOwner In der argsListe oder im Code ersetzen...

[deepwater]

Du musst einfach jeden iOwner durch iNewOwner ersetzen.
Wießt du, die argslist... das ist einfach eine Sammlung von Integern und boolean (wenn man so will ein array), die dem Python geschrippsel durch das SDK beim Funktionsaufruf übergeben wird. Sie ist ein so genanntes "non-mutable Tuple" also ein nicht veränderbarer Schlauch, auf den man entweder über xxx = argslist[2] (also wie bei einem Array) zugreifen kann, oder man überträgt den gesamten Schlauch auf einzelne Variablen mit ixx, ivv, bss = argslist Was dasselbe ist wie

ixx = argslist[0]
ivv = argslist[1]
bss = argslist[2] 

, wobei es natürlich auf die Reihenfolge ankommt.

Edit: Du streichst jetzt den iOwner wieder aus der argslist und ersetzt im Code jeden iOwner durch iNewOwner.
Anm.: Werte werden von rechts nach links zugewiesen.

[Monaldinio]

och meno...ich habe auch ein pOwner...muss der durch pNewOwner erstetzt werden?!?


Im Code selbst habe ich gar kein iOwner mehr...nur noch an dieser Stelle hier

pNewOwner = gc.getPlayer(iOwner) 

...

[deepwater]

Nein!

Edit: Und du kannst diese Zeilen mit derhier ersetzen:

        iOriginalOwner = pCity.getOriginalOwner () 
        pOriginalOwner = gc.getPlayer(iOriginalOwner) 
        ##ersetzen durch:
        pOriginalOwner = gc.getPlayer(iPreviousOwner) 

[Kathy]

Vorsicht, deepwater! Es gibt in diesem Code 3 verschiedene Owner, die man nicht durcheinanderbringen sollte.
- iNewOwner ist der, der die Stadt eingenommen hat, und wird durch die argsList übergeben
- iPreviousOwner ist der, dem die Stadt vor der Eroberung gehört hat, und er kommt ebenfalls aus der argsList.

Aber: iOriginalOwner ist derjenige, der die Stadt als allererstes besessen hat, als ihr Gründer! Der wird durch diesen Code ermittelt (und der ist letztlich der, der befreit werden soll):

        iOriginalOwner = pCity.getOriginalOwner () 

Bei deinem Code aber wird pOriginalOwner der Spieler von iPreviousOwner. Mach das nicht, sonst funktioniert mein ganzer Code nicht mehr, und damit die ganze Befreien-Funktion!

[Monaldinio]

(iPlayer,5) 

so?!?!

Fehler

...in line 571...

global name iOwner is not defined!

Es nervt mitlerweile nur noch...

[Kathy]

Nein, siehe meinen letzten Beitrag (Crossposting).

Und hier (3.Zeile) wird immer noch iOwner aufgerufen.

        pNewOwner = gc.getPlayer(iOwner) 

iOwner ist aber nicht definiert, es muss heißen:

        pNewOwner = gc.getPlayer(iNewOwner) 

[Monaldinio]

Mhmmm...es gab nun keine Fehlermeldung bei der Eroberung.
Den Rest prüfe ich mal morgen.

Danke für eure hilfe! Ich stell mich manchaml echt doof an...oh gott...

[Kathy]

Das mag sein, aber der Code prüft mit deepwaters Änderung etwas ganz anderes: Ob der Vorbesitzer, also dein Feind, lebt, und du wirst gefragt werden, ob du ihn wiederbeleben willst, wenn er einmal bei der Eroberung stirbt. Das ist aber ja nicht der Sinn der Sache, den Effekt könnte man mit einer einfachen Kapitulation auch erreichen.
Mit meinem Code kannst du dem Gründer der Stadt - der diese einmal verloren hat und später vernichtet wurde - diese Stadt zurückgeben. Beispiel: Du (Santiago) bist mit Yang (iPreviosOwner) im Krieg und nimmst ihm die Stadt weg. Zum Beispiel "NewJerusalem". Da dir Miriam Godwinson eine gute Vasallin zu sein scheint - leider ist sie tot, Yang hat sie vernichtet - kannst du mit meinem Code ihr die Stadt zurückgeben, uns sie wird dir nun dienen. Mit deepwaters Code kannst du nach der Eroberung nun Yang die Stadt zurückgeben und er wird dein Vasall - wenn er bei der Eroberung umgekommen wäre. Es ist aber unwahrscheinlich, dass du ihn dann noch haben willst, denn sonst hättest du vorher schon seine Kapitulation verlangen können.

[deepwater]

Achso, jetzt verstehe ich , habe das übersehen, sorry.

[Monaldinio]

Das mag sein, aber der Code prüft mit deepwaters Änderung etwas ganz anderes: Ob der Vorbesitzer, also dein Feind, lebt, und du wirst gefragt werden, ob du ihn wiederbeleben willst, wenn er einmal bei der Eroberung stirbt. Das ist aber ja nicht der Sinn der Sache, den Effekt könnte man mit einer einfachen Kapitulation auch erreichen.
Mit meinem Code kannst du dem Gründer der Stadt - der diese einmal verloren hat und später vernichtet wurde - diese Stadt zurückgeben. Beispiel: Du (Santiago) bist mit Yang (iPreviosOwner) im Krieg und nimmst ihm die Stadt weg. Zum Beispiel "NewJerusalem". Da dir Miriam Godwinson eine gute Vasallin zu sein scheint - leider ist sie tot, Yang hat sie vernichtet - kannst du mit meinem Code ihr die Stadt zurückgeben, uns sie wird dir nun dienen. Mit deepwaters Code kannst du nach der Eroberung nun Yang die Stadt zurückgeben und er wird dein Vasall - wenn er bei der Eroberung umgekommen wäre. Es ist aber unwahrscheinlich, dass du ihn dann noch haben willst, denn sonst hättest du vorher schon seine Kapitulation verlangen können.

...also muss das hier

iOriginalOwner = pCity.getOriginalOwner ()  
pOriginalOwner = gc.getPlayer(iOriginalOwner) 

wieder eingefügt werden...

Edit: Habe

pOriginalOwner = gc.getPlayer(iPreviousOwner) 

durch

habe....iOriginalOwner = pCity.getOriginalOwner ()  
        pOriginalOwner = gc.getPlayer(iOriginalOwner) 

ersetzt.
Fehlermeldung beim laden...Traceback...

[Monaldinio]

Ist Zustand...mit Fehlermeldung!