Der SDK-Fragen-Thread

[rucivfan]

71 sind aber nichts im Vergleich zu den Einheiten.

Die "DefaultUnit" werden, wie Ramkhamhaeng geschrieben hat, gelöst. Kannst dir das zum Beispiel mal ansehen.

Ach die redundante Variable für den String, wie Ramkhamhaeng schrieb, werden nach den nachträglichen lesen geleert. Die Redundanz existiert also nur beim Spielstart.

Um es etwas zu präzisieren: Ich habe in der Datei Civ4BuildInfos.xml einen neuen Eintrag für "UnitType" gemacht, aber diese Inhalte werden, obwohl im SDK definiert, nicht eingelesen. Der Eintrag wird schlicht nicht erkannt, ich nehme an, weil UnitInfos.xml erst nach BuildInfos.xml geladen wird, was ja ganz logisch ist, da die Build für die UnitInfo.xml schon bekannt sein müssen.

Das ist absolut der Fall. Suche mal nach CvXMLLoadUtility::LoadPreMenuGlobals() in der Datei CvXMLLoadUtilitySet.cpp. Dort die Reihenfolge nicht ändern!

Manche sachen werden auch erst in der CvXMLLoadUtility::LoadPostMenuGlobals() geladen. Das heißt, sie werden erst beim Starten einer Partie geladen.

[Kathy]

Ich habe irgendwie den Wurm drin. So sieht es im SDK aus

bool CvPlot::canBuild(BuildTypes eBuild, PlayerTypes ePlayer, bool bTestVisible) const
{
	ImprovementTypes eImprovement;
	ImprovementTypes eFinalImprovementType;
	//Kathy Plant Feature Beginn
	FeatureTypes eFeaturePlantType;
	//Kathy Plant Feature End
	RouteTypes eRoute;
	bool bValid;
(...)
	//Kathy Build Tower beginn

	if (GC.getBuildInfo(eBuild).getTowerUnit() != -1)//Überspringe alle Builds, bei denen es keinen Eintrag in TowerUnitTypes gibt - also Straße, Farmen etc, Removes
	{
		CvPlayerAI pPlayer;
		pPlayer = GET_PLAYER(getOwnerINLINE());Ermittelt den Besitzer eines Plots
		CivilizationTypes ePlayerCiv = pPlayer.getCivilizationType();Gibt die Zivilisation des Besitzers zurücl
		UnitTypes eTowerUnitType = (UnitTypes)(GC.getCivilizationInfo(ePlayerCiv).getCivilizationUnits(GC.getBuildInfo(eBuild).getTowerUnit()));Soll ermitteln: Den Unittype, der bei dieser Zivilisation für die Unitclas rauskommt, die in der entsprechenden Build-Info im Tag "ToweUnit" gespeichert ist.

		if (isWater())
		{
			if ( GC.getUnitInfo(eTowerUnitType).getDomainType() == DOMAIN_LAND)Darum geht es hier eigentlich: Ein Bauauftrag soll nur mäglich sein, wenn die Domain der gebauten Einheit der Art des Plots entspricht
			{
				return false;
			}
			if ( GC.getUnitInfo(eTowerUnitType).getDomainType() == DOMAIN_IMMOBILE)
			{
				return false;
			}
		}
		else
		{
			if ( GC.getUnitInfo(eTowerUnitType).getDomainType() == DOMAIN_SEA)
			{
				return false;
			}
		}

		if (!bTestVisible)
		{
			if (GET_PLAYER(ePlayer).getTeam() != getTeam())
			{
				return false;
			}
		}

		bValid = true;
	}
//Kathy Build Tower end

Und immer, wenn jetzt ein Baufahrzeug kommt, stürzt das Spiel ab. Außer, es steht in der Stadt, wo es sowieso nichts bauen kann. Ich sehe aber meinen Fehler nicht. Habe ich irgendwo einen offensichtlichen Denkfehler?

Edit: @ruvicfan: Habe deinen Beitrag noch nicht gesehen gehabt, als ich das hier gepostet habe.

[rucivfan]

Wahrscheinlich wird der Code canBuild einfach in einer Stadt nicht ausgeführt.

Siehe oben mein Beitrag.

[Kathy]

Ich kann dir nicht ganz folgen. Der Code, den ich jetzt gepostet habe, setzt ja was ganz anderes vorraus als dein Post oben. Ich habe dort versucht, entsprechend der Ladereihenfolge vorzugehen, das heißt, in der BuildInfo.xml ist jetzt eine Unitclass gespeichert. Diese soll dann erst, wenn es ums bauen geht, in eine Einheit übersetzt werden, damit deren Domain geprüft werden kann. Der Vorteil in diesem Fall ist auch, dass Zivilisationen, die wirklich eine UU in diesem Bereich haben, diese auch bauen können. Darum scheint mit dieses Vorgehen ohnehin besser, als den Weg über eine bestimmte Einheit zu gehen. Aber irgendwo ist in meinem neuen Code der Wurm drin. Und ich wüsste gerne, wo.

[rucivfan]

Deine Lösung ist der beste Ansatz.

[Kathy]

War das jetzt Ironie?

Sie funktioniert leider nicht. Da ist irgendwo ein Denkfehler drin.

[rucivfan]

Das war kein Ironie. Ohne Code kann ich dir nicht helfen.

[Kathy]

Dear Code ist doch da oben Oder brauchst du noch anderen Code, der davor ausgeführt wird?

Zur Präzision: getTowerUnit() soll in dre Theorie eine Unitclass zurückgeben.
CvInfos.h:

		//Kathy Build tower Beginn
	int getTowerUnit() const;				// Exposed to Python
	//Kathy Build tower Beginn

CvInfos.cpp:

	//Kathy Buld tower Beginn
	pXML->GetChildXmlValByName(szTextVal, "TowerUnitType");
	m_iUnitclass = pXML->FindInInfoClass(szTextVal);
	//Kathy Buld tower ENd

//Kathy Build Tower Beginn
int CvBuildInfo::getTowerUnit() const	
{
	return m_iUnitclass;
}
//Kathy Build Tower End

So, zu guter Letzt noch der überarbeitete Code.

bool CvPlot::canBuild(BuildTypes eBuild, PlayerTypes ePlayer, bool bTestVisible) const
{
	ImprovementTypes eImprovement;
	ImprovementTypes eFinalImprovementType;
	//Kathy Plant Feature Beginn
	FeatureTypes eFeaturePlantType;
	//Kathy Plant Feature End
	RouteTypes eRoute;
	bool bValid;

(...)//unveränderter Originalcode, kann ich nachreichen, wenn es von Belang sein sollte.
	//Kathy Build Tower beginn

	if (GC.getBuildInfo(eBuild).getTowerUnit() != NO_UNITCLASS)//Wähle nur Builds, die auch einen Eintrag unter TowerUnit haben. TowerUnit ist eine Unitclass!
	{
		if (getOwnerINLINE() != -1)//Auf Felden ohne Besitzer kann keine ToweUnit gebaut werden
		{
			CvPlayerAI pPlayer;
			pPlayer = GET_PLAYER(getOwnerINLINE());//Ermittle den Besitzer des Plats
			CivilizationTypes ePlayerCiv = pPlayer.getCivilizationType();//Ermittle die Zivilisation des Besitzers
			UnitTypes eTowerUnitType = (UnitTypes)(GC.getCivilizationInfo(ePlayerCiv).getCivilizationUnits(GC.getBuildInfo(eBuild).getTowerUnit()));//Suche die zur Unitclass (aus der BuildInfo) gehörende Einheit der Zivilisation ePlayerCiv

			if (isWater())
			{
				if ( GC.getUnitInfo(eTowerUnitType).getDomainType() == DOMAIN_LAND)
				{
					return false;
				}
				if ( GC.getUnitInfo(eTowerUnitType).getDomainType() == DOMAIN_IMMOBILE)
				{
					return false;
				}
			}
			else
			{
				if ( GC.getUnitInfo(eTowerUnitType).getDomainType() == DOMAIN_SEA)
				{
					return false;
				}
			}

			if (!bTestVisible)
			{
				if (GET_PLAYER(ePlayer).getTeam() != getTeam())
				{
					return false;
				}
			}

			bValid = true;
		}
	}
//Kathy Build Tower end

[Kathy]

Fehlt eventuell im Textblock

getCivilizationUnits(GC.getBuildInfo(eBuild).getTowerUnit())

etwas vor

GC.getBuildInfo(eBuild).getTowerUnit()

?
Wenn das eine Einheit wäre, müsste da ja noch

(UnitTypes)

stehen, wie sieht das bei einer Unitclass aus?

[Ramkhamhaeng]

So, bin wieder da. Die Klasse statt der Unit zu nehmen war eine gute Idee!
Zum aktuellen Fehler:
Wie ich schon schrieb () musst du den Typ noch nutzen, um den InfoTyp zu holen. Du kannst nicht einfach den Integerwert casten

Also statt

UnitTypes eTowerUnitType = (UnitTypes)(GC.getCivilizationInfo(ePlayerCiv).getCivilizationUnits(GC.getBuildInfo(eBuild).getTowerUnit()));

so was:

UnitTypes eTowerUnitType = (UnitTypes) GC.getCivilizationInfo(ePlayerCiv).getCivilizationUnits(GC.getBuildInfo(eBuild).getTowerUnit());
 if (eTowerUnitType != NO_UNIT)
	CvUnitInfo& kUnit = GC.getUnitInfo(eTowerUnitType);
//hier dann weiterarbeiten
}

[Kathy]

Also so?

	if (GC.getBuildInfo(eBuild).getTowerUnit() != NO_UNITCLASS)
	{
		if (getOwnerINLINE() != -1)
		{
			CvPlayerAI pPlayer;
			pPlayer = GET_PLAYER(getOwnerINLINE());
			CivilizationTypes ePlayerCiv = pPlayer.getCivilizationType();
			UnitTypes eTowerUnitType = (UnitTypes)(GC.getCivilizationInfo(ePlayerCiv).getCivilizationUnits(GC.getBuildInfo(eBuild).getTowerUnit()));
			if (eTowerUnitType != NO_UNIT)
			{
				CvUnitInfo& kUnit = GC.getUnitInfo(eTowerUnitType);

				if (isWater())
				{
					if ( kUnit.getDomainType() == DOMAIN_LAND)
					{
						return false;
					}
					if ( kUnit.getDomainType() == DOMAIN_IMMOBILE)
					{
						return false;
					}
				}
				else
				{
					if ( kUnit.getDomainType() == DOMAIN_SEA)
					{
						return false;
					}
				}
					
				if (!bTestVisible)
				{
					if (GET_PLAYER(ePlayer).getTeam() != getTeam())
					{
						return false;
					}
				}

				bValid = true;

			}

		}
	}
//Kathy Build Tower end

[Ramkhamhaeng]

Ja, so müsste es meiner Meinung nach gehen.

[Kathy]

Leider ist er beim Test gerade wieder abgestürzt.

[Ramkhamhaeng]

Kannst du den Debugger benutzen? Ein Breakpoint könnte helfen.

[Kathy]

Kann ich, aber dann wohl eher morgen. Bis mein Computer das Spiel zum Debuggen geladen hat braucht er immer ewig lange.