[FreshKilla]

If its using to check if they have weapons why not just make the code simpler and just use:

pawn Code:

if(GetPlayerWeapon(i) == weaponid)
  {
    if(GetPlayerAmmo(i) > 0)
    {
      // Code here...
    }
  }

Then you would follow the pattern with:

pawn Code:

if(GetPlayerWeapon(i) == weaponid)
  {
    if(GetPlayerAmmo(i) > 0)
    {
      // Code here...
    }
  }
  else if(GetPlayerWeapon(i) == weaponid)
  {
    if(GetPlayerAmmo(i) > 0)
    {
      // Code here...
    }
  }

But if its just to check weapons, I recommend using OnPlayerUpdate, its commonly known to be used for Anti Cheats.