🚀 go-pugleaf

{AGUT2}MeltDown <meltdownUSE@YOURunrealtower.imagination.org> wrote:

>Spike enlightened us with:
>> CTC_TMale1BGot DM-Ablee.CTC_TMale1Bot23 (Function
>> CatchTheChicken.CTCBot.findChicken:0043) Runaway loop detected (over
>> 10000000 iterations)
>
>This is the only looping code:
>
>for(item = someone.Inventory; item != None; item=item.Inventory)
>   if(item.IsA('CTC_Chicken')) return item;

Hmm... I don't know UnrealScript, but AFAIK it's like C - and if this
was C code, the error might be in the "item=item.Inventory" part. The
loop *will* hang if this isn't guaranteed to set "item" to either a
value it never had before or to "None".

I guess you're trying to check if the actor "someone" possesses the
chicken?

If so, I'd find something like "item=someone.Inventory.next",
"item=item.next" or "item++" more intuitive - depends on how
UnrealScript handles iterators :)

HTH...

>Apparently, one of the bots has an inventory item pointing to itself.

I can't comment on this as I don't know UScript and the rest of the
code, but a dirty hack might be to check if the item is, in fact, an
actor (ie a bot). Pseudocode:

for(item = someone.Inventory; item != None; item=item.Inventory) {
   if(item.IsA('CTC_Chicken')) return item;
   if(item.IsA('Actor')) return NULL;  
     // place after the chicken test if chicken is an Actor itself
}

>New features for 0.1:
>
>- A big arrow on the HUD pointing to the chicken. It'll be green when
>  the chicken is walking around, and red when it's caught.
>- Some sort of graphical HUD element on which you can see how long it'll
>  take before you score again, when you hold the chicken.

That sounds great :)

Cheers,
Robert aka The_Monk