TFS 1.X+ Bad reaction/dodge from monsters

[Drucken]

Monsters in TFS 1.5 have a very different reaction(dodge, in TFS 0.4 the monster could dodge the players, which is quite necessary, in TFS 1.5 (it may happen the same in some other TFS) this doesn't happen... Besides other bugs which were seen by @Levi999x

Well, I have added the changes of @Gesior.pl in the post of:

Solved - Weird monster behaviour in TFS 1.1?

The monster AI in tfs 1.1 (idk about 1.0) is EXTREMELY dumb. This has discouraged me from using TFS 1.1 even though i've spent so much time with it. A couple of examples: If you chase any ranged mob, let's take lizard legionaire, they will turn around every time they make an attack and this...

otland.net

It has improved the reaction when the monster escapes, but the problem of dodge is still there, the monster gets stuck in the corners.

​

 

[johnsamir]

mine looks like this is this okey?

void Creature::goToFollowCreature()
{
    if (followCreature)
    {
        FindPathParams fpp;
        getPathSearchParams(followCreature, fpp);

        Monster* monster = getMonster();
        if (monster && !monster->getMaster() && (monster->isFleeing() || fpp.maxTargetDist > 1)) {
            Direction dir = DIRECTION_NONE;

            if (monster->isFleeing()) {
                const Position& targetPosition = followCreature->getPosition();
                const Position& thisPosition = getPosition();

                const int_fast32_t distanceToTarget = Position::getDistanceX(thisPosition, targetPosition) + Position::getDistanceY(thisPosition, targetPosition);
                if (distanceToTarget <= 2 && !hasFollowPath) {
                    listWalkDir.clear();
                    if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                        hasFollowPath = true;
                        startAutoWalk();
                    }
                }
              
                    if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                        hasFollowPath = true;
                        startAutoWalk();
                    }
                    else {
                        hasFollowPath = false;
                    }
                    return;
                }
            }

            if (dir != DIRECTION_NONE) {
                listWalkDir.clear();
                listWalkDir.push_back(dir);

                hasFollowPath = true;
                startAutoWalk();
            }
        }
        else {
            listWalkDir.clear();
            if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                hasFollowPath = true;
                startAutoWalk();
            }
            else {
                hasFollowPath = false;
            }
        //}
    }

    onFollowCreatureComplete(followCreature);
}

 

[johnsamir]

up

 

[johnsamir]

void Creature::goToFollowCreature()
{
    if (followCreature) {
        FindPathParams fpp;
        getPathSearchParams(followCreature, fpp);

        Monster* monster = getMonster();
        if (monster && !monster->getMaster() && (monster->isFleeing() || fpp.maxTargetDist > 1)) {
            Direction dir = DIRECTION_NONE;

            if (monster->isFleeing()) {
                monster->getDistanceStep(followCreature->getPosition(), dir, true);
            } else { // maxTargetDist > 1
                if (!monster->getDistanceStep(followCreature->getPosition(), dir)) {
                    // if we can't get anything then let the A* calculate
                    listWalkDir.clear();
                    if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                        hasFollowPath = true;
                        startAutoWalk();
                    } else {
                        hasFollowPath = false;
                    }
                    return;
                }
            }

            if (dir != DIRECTION_NONE) {
                listWalkDir.clear();
                listWalkDir.push_back(dir);

                hasFollowPath = true;
                startAutoWalk();
            }
        } else {
            listWalkDir.clear();
            if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                hasFollowPath = true;
                startAutoWalk();
            } else {
                hasFollowPath = false;
            }
        }
    }

    onFollowCreatureComplete(followCreature);
}

to this

void Creature::goToFollowCreature()
{
    if (followCreature)
    {
        FindPathParams fpp;
        getPathSearchParams(followCreature, fpp);

        if (getPathTo(followCreature->getPosition(), listWalkDir, fpp))
        {
            hasFollowPath = true;
            startAutoWalk(listWalkDir);
        }
        else
            hasFollowPath = false;
    }

    onFollowCreatureComplete(followCreature);
}

so i must change this

void Creature::goToFollowCreature()
{
    if (followCreature)
    {
        FindPathParams fpp;
        getPathSearchParams(followCreature, fpp);

        Monster* monster = getMonster();
        if (monster && !monster->getMaster() && (monster->isFleeing() || fpp.maxTargetDist > 1)) {
            Direction dir = DIRECTION_NONE;

            if (monster->isFleeing()) {
                const Position& targetPosition = followCreature->getPosition();
                const Position& thisPosition = getPosition();

                const int_fast32_t distanceToTarget = Position::getDistanceX(thisPosition, targetPosition) + Position::getDistanceY(thisPosition, targetPosition);
                if (distanceToTarget <= 2 && !hasFollowPath) {
                    listWalkDir.clear();
                    if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                        hasFollowPath = true;
                        startAutoWalk();
                    }
                }
                else {
                    monster->getDistanceStep(followCreature->getPosition(), dir, true);
                    // hasFollowPath = false;
                }
            }
            else { // maxTargetDist > 1
                if (!monster->getDistanceStep(followCreature->getPosition(), dir)) {
                    // if we can't get anything then let the A* calculate
                    listWalkDir.clear();
                    if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                        hasFollowPath = true;
                        startAutoWalk();
                    }
                    else {
                        hasFollowPath = false;
                    }
                    return;
                }
            }

            if (dir != DIRECTION_NONE) {
                listWalkDir.clear();
                listWalkDir.push_back(dir);

                hasFollowPath = true;
                startAutoWalk();
            }
        }
        else {
            listWalkDir.clear();
            if (getPathTo(followCreature->getPosition(), listWalkDir, fpp)) {
                hasFollowPath = true;
                startAutoWalk();
            }
            else {
                hasFollowPath = false;
            }
        }
    }

    onFollowCreatureComplete(followCreature);
}

to this?

void Creature::goToFollowCreature()
{
    if (followCreature)
    {
        FindPathParams fpp;
        getPathSearchParams(followCreature, fpp);

        if (getPathTo(followCreature->getPosition(), listWalkDir, fpp))
        {
            hasFollowPath = true;
            startAutoWalk(listWalkDir);
        }
        else
            hasFollowPath = false;
    }

    onFollowCreatureComplete(followCreature);
}