Now that getFacing returns null if an NPC is attacking client.getMyPlayer(), you have to use this fact in conjunction with the fact that getFacingId = -1 if the NPC is not attacking any player to find a monster that is attacking you. Here's my version:

	NPC getAttackingNPC() {
		List<NPC> npcs = client.getLocalNPCs();
		int n = 0;
		for (n = 0; n < npcs.size(); n++) {
			if (npcs.get(n).getFacing() == null
					&& npcs.get(n).getFacingId() != -1) {
				return npcs.get(n);
			}
		}
		return null;
	}

a neat little trick is to use an object for loop (not sure on the correct name)

for (NPC n : npcs) {
  if (n.getFacing() == null && blahblah) {
  }
}

So basically it iterates through whatever list you do.  Easy way of not having to find the size of the list and count up

 

*I believe it's called an advancing for loop

Edited September 28, 201312 yr by dreamliner