"Switch" loops not working as it should

[tonitoni1998]

hey guys. i want the monster to loop 5 nodes all and all over again.

with the use of a switch loop i want the monster to wait 1 sec when it reaches the first path node only at the first loop. so as i understood it i would need a case that says "when i = 1 (because of the first node) set x= 1 (the waiting time). and a default that sets all other cases to x=0.001"

so this would look like this for me:

Code: (Select All)

void reloop_1(string &in asParent, string &in asChild, int alState)
{
    for(int i = 1; i <=5; i++)
    {
    int x;
    switch( i )
    {
    case 1:
        x= 1;
    break;
    default:
        x=0.001;
    }
    AddEnemyPatrolNode("grunt_1", "PathNodeLoop_"+i, x, "");
    }
}

but it waits 1 ec at every node in every loop

[NaxEla]

Don't think this will make a difference, but in this situation, it would probably be better to use an if-else statement.

PHP Code: (Select All)

void reloop_1(string &in asParent, string &in asChild, int alState)
{
    for(int i = 1; i <=5; i++)
    {
    float x;
    if(i == 1) {
        x = 1.0f;
    }
    else {
        x = 0.001f;
    }
    AddEnemyPatrolNode("grunt_1", "PathNodeLoop_"+i, x, "");
    }
} 


Also, I made x a float. That was your problem there. You were trying to set x to 0.001 when it was an int.

[tonitoni1998]

Yeah i see. I also got an idea how i could do it, that it only waits the first loop, but i dont know it is worth it

[NaxEla]

You could also just add the first path node, then after one second (using a timer) add the other path nodes.

[tonitoni1998]

Yeah when you think about it, there are enought ways to do it.

[palistov]

That's not an issue with script; your code is perfectly fine

A lot of people have this misunderstanding about AddEnemyPathNode. A wait time of 0 does not mean the monster waits for 0 seconds. The monster will pause at any node he is assigned, regardless of the duration being 0 or otherwise. If you want him to loop in a circle, you need to clear his nodes then assign him a new one before he reaches his destination. Use collision callbacks for this. If he reaches his destination node, he will pause before executing his next movement, regardless of the wait time. It sucks, but that's how the game is coded

Edit to expand on your circle loop:
A good way to implement a grunt moving in a circle is to make a circle, and assign him every 2nd or third node along the circle. And before he reaches this node, clear his assigned nodes and add the next one in the loop. This way constantly moves the same direction along the circle; if you just repeatedly add the same node he could potentially turn around and go backwards, depending on how your nodes are set up.

[TheGreatCthulhu]

@OP: The problem in you original code was that the x variable was of type int, so when you assigned "x=0.001", the value got implicitly converted to the nearest integer, which is 0. And, as palistov said, since with AddEnemyPathNode() zero is not zero... you got the idea (I suppose you knew this, since you tried to assign 0.001).