BASE: SDK & Python Diskussion (Coder's Lair)

[Cybah]

Funktioniert noch nicht richtig. Man kann eins mit beliebigen GP-Typen starten, aber so wird nur eine einzige Einheit entfernt.

Mein Code:

void CvPlayer::killGoldenAgeUnits(CvUnit* pUnitAlive)
{
    CvUnit* pLoopUnit;
    CvUnit* pBestUnit;
    bool* pabUnitUsed;
    int iUnitsRequired;
    int iValue;
    int iBestValue;
    int iLoop;
    int iI;

    pabUnitUsed = new bool[GC.getNumUnitInfos()];

    for (iI = 0; iI < GC.getNumUnitInfos(); iI++)
    {
        pabUnitUsed[iI] = false;
    }

    iUnitsRequired = unitsRequiredForGoldenAge();

    if (pUnitAlive != NULL)
    {
        pabUnitUsed[pUnitAlive->getUnitType()] = true;
        iUnitsRequired--;
    }

    for (iI = 0; iI < iUnitsRequired; iI++)
    {
        iBestValue = 0;
        pBestUnit = NULL;

        for(pLoopUnit = firstUnit(&iLoop); pLoopUnit != NULL; pLoopUnit = nextUnit(&iLoop))
        {
            if (pLoopUnit->isGoldenAge())
            {
                if (!(pabUnitUsed[pLoopUnit->getUnitType()]))
                {
                    iValue = 10000;

                    iValue /= (plotDistance(pLoopUnit->getX_INLINE(), pLoopUnit->getY_INLINE(), pUnitAlive->getX_INLINE(), pUnitAlive->getY_INLINE()) + 1);

                    if (iValue > iBestValue)
                    {
                        iBestValue = iValue;
                        pBestUnit = pLoopUnit;
                    }
                }
            }
        }

        FAssert(pBestUnit != NULL);
        if (pBestUnit != NULL)
        {

// Cybah: BASE - Nobelpreis Projekt Start
// Original Code
//             pabUnitUsed[pBestUnit->getUnitType()] = true;
//
            if (!(GET_TEAM(getTeam()).getProjectCount((ProjectTypes)GC.getInfoTypeForString("PROJECT_NOBEL")) > 0))
            {
                pabUnitUsed[pBestUnit->getUnitType()] = true;
            }
// Cybah: BASE - Nobelpreis Projekt End

            pBestUnit->kill(true);

            //play animations
            if (pBestUnit->plot()->isActiveVisible(false))
            {
                //kill removes bestUnit from any groups
                pBestUnit->getGroup()->pushMission(MISSION_GOLDEN_AGE, 0);
            }
        }
    }

    SAFE_DELETE_ARRAY(pabUnitUsed);
} 

[rucivfan]

Ich verstehe das Problem, es wird merhfach die gleiche Einheit so ausgewählt. Ausgewählte Einheiten haben die Mission MISSION_GOLDEN_AGE mit pBestUnit->getGroup()->pushMission(MISSION_GOLDEN_AGE, 0); gesetzt bekommen.

Man müsste die Mission oben noch abfragen, dass nur Einheiten ohne die Mission in Frage kommen. Muss aber noch nachschauen, wie man das macht.

[rucivfan]

Das Problem muss ein anderes sein.

pBestUnit->kill(true); tötet die Einheit, aber warum wird sie beim zweiten durchgang noch genommen, vertsehe ich nicht. edit: Man muss oben wohl noch überprüfen , ob die Einheit noch existiert sprich nicht kill ausgeführt wurde!

if (pUnitAlive != NULL)
    {
        pabUnitUsed[pUnitAlive->getUnitType()] = true;
        iUnitsRequired--;
    }

Das Rote muss oben auch noch blockiert werden!

[rucivfan]

Funktioniert noch nicht richtig. Man kann eins mit beliebigen GP-Typen starten, aber so wird nur eine einzige Einheit entfernt.

Wenn du den Roten Hinweis oben machst, sollte es gehen. Das Problem ist wohl, du hast nur mehrere Spezialisten einer Art gehabt. Das rote musst du dafür blocken(edit), weil sonst dieser Typ gesperrt wird und dann nicht mehr entfernt wird. Da verhindert sonst das killen der anderen.

[Cybah]

Hat nichts geändert.

[rucivfan]

Code bitte anhängen.

[Cybah]

int CvPlayer::unitsGoldenAgeReady() const
{
    PROFILE_FUNC();

    CvUnit* pLoopUnit;
    bool* pabUnitUsed;
    int iCount;
    int iLoop;
    int iI;

    pabUnitUsed = new bool[GC.getNumUnitInfos()];

    for (iI = 0; iI < GC.getNumUnitInfos(); iI++)
    {
        pabUnitUsed[iI] = false;
    }

    iCount = 0;

    for(pLoopUnit = firstUnit(&iLoop); pLoopUnit != NULL; pLoopUnit = nextUnit(&iLoop))
    {
        if (!(pabUnitUsed[pLoopUnit->getUnitType()]))
        {
            if (pLoopUnit->isGoldenAge())
            {
// Cybah: BASE - Nobelpreis Projekt Start
// Original Code
//                pabUnitUsed[pLoopUnit->getUnitType()] = true;
//
                if (!(GET_TEAM(getTeam()).getProjectCount((ProjectTypes)GC.getInfoTypeForString("PROJECT_NOBEL")) > 0))
                {
                    pabUnitUsed[pLoopUnit->getUnitType()] = true;
                }
// Cybah: BASE - Nobelpreis Projekt ENd
                iCount++;
            }
        }
    }

    SAFE_DELETE_ARRAY(pabUnitUsed);

    return iCount;
}


void CvPlayer::killGoldenAgeUnits(CvUnit* pUnitAlive)
{
    CvUnit* pLoopUnit;
    CvUnit* pBestUnit;
    bool* pabUnitUsed;
    int iUnitsRequired;
    int iValue;
    int iBestValue;
    int iLoop;
    int iI;

    pabUnitUsed = new bool[GC.getNumUnitInfos()];

    for (iI = 0; iI < GC.getNumUnitInfos(); iI++)
    {
        pabUnitUsed[iI] = false;
    }

    iUnitsRequired = unitsRequiredForGoldenAge();

    if (pUnitAlive != NULL)
    {
// Cybah: BASE - Nobelpreis Projekt Start
// Original Code
//         pabUnitUsed[pUnitAlive->getUnitType()] = true;
//
        if (!(GET_TEAM(getTeam()).getProjectCount((ProjectTypes)GC.getInfoTypeForString("PROJECT_NOBEL")) > 0))
        {
            pabUnitUsed[pUnitAlive->getUnitType()] = true;
        }
// Cybah: BASE - Nobelpreis Projekt End
        iUnitsRequired--;
    }

    for (iI = 0; iI < iUnitsRequired; iI++)
    {
        iBestValue = 0;
        pBestUnit = NULL;

        for(pLoopUnit = firstUnit(&iLoop); pLoopUnit != NULL; pLoopUnit = nextUnit(&iLoop))
        {
            if (pLoopUnit->isGoldenAge())
            {
                if (!(pabUnitUsed[pLoopUnit->getUnitType()]))
                {
                    iValue = 10000;

                    iValue /= (plotDistance(pLoopUnit->getX_INLINE(), pLoopUnit->getY_INLINE(), pUnitAlive->getX_INLINE(), pUnitAlive->getY_INLINE()) + 1);

                    if (iValue > iBestValue)
                    {
                        iBestValue = iValue;
                        pBestUnit = pLoopUnit;
                    }
                }
            }
        }

        FAssert(pBestUnit != NULL);
        if (pBestUnit != NULL)
        {

// Cybah: BASE - Nobelpreis Projekt Start
// Original Code
//             pabUnitUsed[pBestUnit->getUnitType()] = true;
//
            if (!(GET_TEAM(getTeam()).getProjectCount((ProjectTypes)GC.getInfoTypeForString("PROJECT_NOBEL")) > 0))
            {
                pabUnitUsed[pBestUnit->getUnitType()] = true;
            }
// Cybah: BASE - Nobelpreis Projekt End

            pBestUnit->kill(true);

            //play animations
            if (pBestUnit->plot()->isActiveVisible(false))
            {
                //kill removes bestUnit from any groups
                pBestUnit->getGroup()->pushMission(MISSION_GOLDEN_AGE, 0);
            }
        }
    }

    SAFE_DELETE_ARRAY(pabUnitUsed);
} 

[rucivfan]

Unterhalb von for(pLoopUnit = firstUnit(&iLoop); pLoopUnit != NULL; pLoopUnit = nextUnit(&iLoop)) musst du pUnitAlive und pLoopUnit vergleichen auf ungleichheit.

if (pUnitAlive == NULL || pUnitAlive->getID() != pLoopUnit->getID())

edit: Ab 3 Spezialisten sollten 2 sterben ohne die Änderung. Hast du das getestet?

[Cybah]

Kann ich mal schnell machen.

Edit: Jo, statt 3 sterben 2.

[rucivfan]

Dann musst du nur das letzt genannt noch machen.

[Cybah]

Du meinst so?

    for(pLoopUnit = firstUnit(&iLoop); pLoopUnit != NULL; pLoopUnit = nextUnit(&iLoop))
    {
// Cybah: BASE - Nobelpreis Projekt Start
        if (pUnitAlive == NULL || pUnitAlive->getID() != pLoopUnit->getID())
        {
// Cybah: BASE - Nobelpreis Projekt End

            if (!(pabUnitUsed[pLoopUnit->getUnitType()]))
            {
                if (pLoopUnit->isGoldenAge())
                {
// Cybah: BASE - Nobelpreis Projekt Start
// Original Code
//                    pabUnitUsed[pLoopUnit->getUnitType()] = true;
//
                    if (!(GET_TEAM(getTeam()).getProjectCount((ProjectTypes)GC.getInfoTypeForString("PROJECT_NOBEL")) > 0))
                    {
                        pabUnitUsed[pLoopUnit->getUnitType()] = true;
                    }
// Cybah: BASE - Nobelpreis Projekt End
                    iCount++;
                }
            }
// Cybah: BASE - Nobelpreis Projekt Start
        }
// Cybah: BASE - Nobelpreis Projekt End
    } 

[rucivfan]

Oh, ich meinte nur in killGoldenAgeUnits-methode. Die andere hat kein pUnitAlive!

edit: Schiebst du das dann in das Code-Thema?

[Cybah]

1>CvPlayer.cpp(10298) : error C2065: 'pUnitAlive' : undeclared identifier
1>CvPlayer.cpp(10298) : error C2227: left of '->getID' must point to class/struct/union
1> type is ''unknown-type''
1>CvPlayer.cpp(10298) : error C3861: 'pUnitAlive': identifier not found, even with argument-dependent lookup

Wo er Recht hat...

[Cybah]

Oh, ich meinte nur in killGoldenAgeUnits-methode. Die andere hat kein pUnitAlive!

edit: Schiebst du das dann in das Code-Thema?

Ach so.

Ja.

[rucivfan]

Klappt es jetzt einwandfrei?