Mod für PB-Spiele: PB Mod_v1

[Ziankali]

Die neuen Funktionen in FDataStreamBase.h werden also gar nicht wirklich benutzt?

[Ramkhamhaeng]

Genau, die sind dann nicht mehr notwendig. Beim Testen ich festgestellt, dass meine Änderungen die Save-Größe nur unwesentlich verkleinern. Da ich nicht alles löschen wollte schwirren da noch die Reste rum

[Ziankali]

Ich fasse dann mal die Änderungen zusammen:

• CvDefines.h: MAX_CIV_PLAYERS erhöht
• CvMessageControl: Neue Funktion "sendTurnCompleteAll"
• CyGame.cpp: Neue Funktionen "setPausePlayer" und "setCivPassword"
• CyGameInterface.cpp: Python-defs für "setPausePlayer" und "setCivPassword"
• CyGlobalContext.h: Die Klasse "CyGlobalContext" wird ergänzt um

  	mutable CvString tmp;
  	const char * getAltrootDir( ) const { 
  		tmp.Convert( gDLL->GetPitbossSmtpLogin() );
  		return tmp.GetCString();
  	}

• CyGlobalContextInterface4.cpp: .def für "getAltrootDir"
• CyMessageControl: Neue Funktion "sendTurnCompleteAll"
• CyMessageControlInterface.cpp: .def für "sendTurnCompleteAll"
• FDataStreamBase.h: "Hilfsfunktion zum Auslesen von Arrays"

Ich sehe da nur neue Funktionen (in der DLL). Den Python-Teil schaue ich mir als nächstes an.

[Ramkhamhaeng]

Gute Zusammenfassung
Im Pythonteil bezieht sich der meiste Code (bei Spielern) auf die Änderung der Punkteliste. Außerdem wurden die Dateien für den PB-Server entkernt und der eigentliche Inhalt ist unter PBStats/PBs/Python/v1 zu finden. Damit wird verhindert, dass Änderungen am Code des Servers vom Programm als Änderung der Mod aufgefasst werden.

[mauz]

ich muss nochmal ein großes Lob und Danke aussprechen an Ram und Zulan für eure Bemühungen - es funktioniert tadellos. Civ4 wird leben, ihr habt großen Anteil daran.

[Ramkhamhaeng]

Die XML-Dateien müssen noch einen versteckten Fehler enthalten. Ich kann dazu nur sagen, dass mir das Spiel auf meinem Linux-Rechner sofort crasht, wenn ich einen Eintrag im Hauptmenü anwähle. Wenn ich den XML-Ordner umbenenne läuft es aber! Bin vor ein paar Monaten dem Problem schon mal begegnet und mit der Debug-DLL wird unter Windows auch eine Assert-Meldung ausgelöst. Allerdings kann man mit der Meldung wenig anfangen…

[Flunky]

Debugst du mit Visual Studio? Da wird die Zeile/Datei genannt.

[Ramkhamhaeng]

Ja, das Problem an der Stelle ist, dass die Assertmeldung erst durch den Return-Value einer Funktion ausgelöst wird und die Funktion außerhalb der DLL liegt. D.h. ich weiß nur, dass beim Parsen der (welcher? weiß ich gerade nicht.) XML-Datei ein Fehler auftrat. Screenshot hab ich davon aber gerade nicht zur Hand.

[Ramkhamhaeng]

Evtl. löst sich das, was ich vorhin geschrieben habe auch in Rauch auf und es liegt doch an fehlenden dds-Files, die in den XML-Dateien verlinkt sind. Muss ich morgen mal weiter gucken

[mauz]

ich find das Layout des neuen Civstats übrigens gelinde gesagt ziemlich suboptimal,
- total unübersichtlich (horizontale Dreiteilung)
- den Satz "Display and Manage Pitboss Games" braucht kein Mensch
- die Nav ganz rechts
- die Formatierung des mittleren Frames

-> Nav nach oben, dann brauchts auch keine horizontale Dreiteilung;
-> der Metatext (Name: Pitboss 73, Runde: 0, Datum: 4000 BC, Timer: 171:42:08 from 200 hours left, Spiel pausiert: Nein) brauchen nicht in einzelne Zeilen umgebrochen werden, dafür reichen 2 Zeilen und 3 Spalten
-> die eigentliche Statistik braucht in der Breite mehr Platz, momentan gibt's da Zeilenumbrüche
-> der Farbsalat braucht vom Layout her Trennelemente

[alpha civ]

Benutze doch die zweite Version.

[Zulan]

ich find das Layout des neuen Civstats übrigens gelinde gesagt ziemlich suboptimal,
- total unübersichtlich (horizontale Dreiteilung)
- den Satz "Display and Manage Pitboss Games" braucht kein Mensch
- die Nav ganz rechts
- die Formatierung des mittleren Frames

-> Nav nach oben, dann brauchts auch keine horizontale Dreiteilung;
-> der Metatext (Name: Pitboss 73, Runde: 0, Datum: 4000 BC, Timer: 171:42:08 from 200 hours left, Spiel pausiert: Nein) brauchen nicht in einzelne Zeilen umgebrochen werden, dafür reichen 2 Zeilen und 3 Spalten
-> die eigentliche Statistik braucht in der Breite mehr Platz, momentan gibt's da Zeilenumbrüche
-> der Farbsalat braucht vom Layout her Trennelemente

Die Version 2 (http://civ.zulan.net/pbspy/) kann dank templates leichter angepasst werden:

https://github.com/Zulan/PBStats/blo...me_detail.html

Patches fuer die Datei sind wilkommen. Gerne mit getrennten CSS.

[Ramkhamhaeng]

@Mauz: Ja, du bist nicht der erste, der damit nicht zufrieden ist Aber wie Zulan schon schrieb wird die php-Version ersetzt werden und daher ändere ich da jetzt nicht mehr viel.
Damit es nicht wieder so ein Farbterror wird, muss diese Auswahl dann aber jmd. anderes treffen

Evtl. löst sich das, was ich vorhin geschrieben habe auch in Rauch auf und es liegt doch an fehlenden dds-Files, die in den XML-Dateien verlinkt sind. Muss ich morgen mal weiter gucken

Das war letzten Endes was ganz anderes. Heute ging wieder mal gar nichts und ein Entfernen des XML-Ordners brachte nix. Ein typisches Nullpointer-Problem Es muss ein Bug in der Wine-Version (1.6.1) gewesen sein, die ich verwendet habe. Mit der aktuellen Version 1.7.18 crasht das Spiel nicht mehr.

[Ziankali]

Die Version 2 (http://civ.zulan.net/pbspy/) kann dank templates leichter angepasst werden:

https://github.com/Zulan/PBStats/blo...me_detail.html

Patches fuer die Datei sind wilkommen. Gerne mit getrennten CSS.

Das werde ich mir am Wochenende mal ansehen

[Ramkhamhaeng]

Das werde ich mir am Wochenende mal ansehen

Hallo Ziankali,

an der Codepassage habe ich gestern Nacht noch was geändert. D.h. derzeit ist die Version in meinem Repo aktueller.