UE1 - UT Accessed Non in Touch()

[Rajada]

Getting an accessed none in this function.

function Touch( actor Other )
{
	if (
        Other.IsA('Pawn')
        && !PlayerPawn(other).IsInState('CheatFlying') 
			&& other.Physics!=PHYS_Swimming &&
       ( (LimitedToName ==  '')
       ||(Other.IsA(LimitedToName)) )
       )
	{
		Pending = Pawn(Other);
		SetTimer(0.01, false);
		if ( bOnceOnly )
			Disable('Touch');
	}
}

Am I missing a stupid mistake? What could possibly be the none here?

 

[gopostal]

Add a wrapper to your code to check for bad conditions:

function Touch( actor Other )
{
	if (Other !=none)
        {

Touch can error because you may be touching the actor as it destroys or moves away from touch. If you code is running at the very moment then you could get a return of =none.

Wide swaths of access nones are cleared by adding the right

if(actor !=none)

checks.

 

[meowcat]

I think the problem is you check to see if it is pawn, and then assume it is a playerpawn in the next check (see highlighted check below) in which the cast could fail (resulting in trying to check IsInstate on a null reference)

if (
        Other.IsA('Pawn') && ![COLOR="Blue"]PlayerPawn[/COLOR](other).IsInState('CheatFlying') 
            && other.Physics!=PHYS_Swimming
            &&  ( (LimitedToName ==  '') || (Other.IsA(LimitedToName)) )
    )

Since the IsInState function should work for any actor you could probably just leave it at !Other.IsInState('CheatFlying')