The CheckMaxLives Function playing twice?

EggXzaB · Aug 6, 2003

Can anyone tell me why the STANDARD function checkmaxlives() in ut2k3 plays twice? This has to check if you have any lives left, and if everbody died or not ...
Put a Log in it, and play with it, you'll see it shows twice ...
And it SHOULD be in the function, cause it's called only once

Code:

function bool CheckMaxLives(PlayerReplicationInfo Scorer)
{
    local Controller C;
    local PlayerReplicationInfo Living;
    local bool bNoneLeft;

    if ( MaxLives > 0 )
    {
		if ( (Scorer != None) && !Scorer.bOutOfLives )
			Living = Scorer;
        bNoneLeft = true;
        for ( C=Level.ControllerList; C!=None; C=C.NextController )
            if ( (C.PlayerReplicationInfo != None) && C.bIsPlayer
                && !C.PlayerReplicationInfo.bOutOfLives )
            {
				if ( Living == None )
					Living = C.PlayerReplicationInfo;
				else if ( (C.PlayerReplicationInfo != Living) &&
 (C.PlayerReplicationInfo.Team != Living.Team) )
			   	{
    	        	bNoneLeft = false;
	            	break;
				}
            }
        if ( bNoneLeft )
        {
			if ( Living != None )
				EndGame(Living,"LastMan");
			else
				EndGame(Scorer,"LastMan");
			return true;
		}
    }
    return false;
}

I'm using the

Code:

if ( Living != None )
EndGame(Living,"LastMan");
else
EndGame(Scorer,"LastMan");

part currently as :

Code:

if ( Living != None )
{
Living.Team.Score += 1;
Endround();
}
else
{
log("Else");
Endround();
}

But here it gives my team 2 points when everyone died

My thoughts are with the "For" loop..

Help plz?
thx

BlackHornet · Aug 6, 2003

maybe it is replicated and called on the server and on the client......or check the the gametype class where the CheckMaxLives function is called....would be the better way first.

Maybe it is called on every kill and at roundend

Wormbo · Aug 6, 2003

As a GameInfo function it's hardly replicated anywhere.

Like the name suggests it's a function that checks something for a certain player. There is no way to figure out, how often it's executed. Try finding a different function for your code that is really executed only once for a certain event.

EggXzaB · Aug 7, 2003

Yeah, but I can't find a function that checks if someone is really dead, only the died() in the xPawn, but thats not useable in what I do ..
I'm trying to make a round system like in TO/CS, so when your dead, you spectate, and when everybody in a team is dead, a new round starts and the winning team gets a point...

BlackHornet · Aug 7, 2003

There is a bDead boolean in the PlayerController, check for this

Mychaeel · Aug 7, 2003

...or just check whether the Controller has a Pawn. If not, they're dead.

EggXzaB · Aug 7, 2003

if (c.Pawn = None)

?

BlackHornet · Aug 7, 2003

Code:

if(C.Pawn == None)
  Log("PlayerController without Pawn");

this or

Code:

if(C.bDead)
  Log("Player is dead");

Both should work

EggXzaB · Aug 8, 2003

I'll try, thx

TaoPaiPai · Aug 8, 2003

EggXzaB said:
Yeah, but I can't find a function that checks if someone is really dead, only the died() in the xPawn, but thats not useable in what I do ..
I'm trying to make a round system like in TO/CS, so when your dead, you spectate, and when everybody in a team is dead, a new round starts and the winning team gets a point...

I don't know what you did wrong but I've done something siimilar and it's working okay.
I use checkmaxLives and it doesn't seem to be called twice for me.

EggXzaB · Aug 8, 2003

hmm, put a log in one of the lines, the showlog, you'll see..
Or I did something stupid

TaoPaiPai · Aug 9, 2003

maybe you put a bracket at the wrong place.If you are testing the gametype with two players ineach team then it's very likely that it's a bracket in a wrong spot.
post the whole function so tha we can better see.
Also make sure the endround() function doesn't call checkmaxlives again.
FYI chekmaxlives is supposed to be called every seconds or so during match.

Doc_EDo · Aug 9, 2003

There already is a round system in ....INVASION!!

For the team round based you need to write your own function and just delete checkmaxlives. Flankly it's confusing and sucks.

Search

Search

The CheckMaxLives Function playing twice?

EggXzaB

BinaryZero Error maker .. o_O

BlackHornet

Global Warzone Project Leader

Wormbo

Administrator

EggXzaB

BinaryZero Error maker .. o_O

BlackHornet

Global Warzone Project Leader

Mychaeel

New Member

EggXzaB

BinaryZero Error maker .. o_O

BlackHornet

Global Warzone Project Leader

EggXzaB

BinaryZero Error maker .. o_O

TaoPaiPai

Commisaire Van Loc

EggXzaB

BinaryZero Error maker .. o_O

TaoPaiPai

Commisaire Van Loc

Doc_EDo

LEFT