[CR] Monattack.cpp clean up

Moved the visual range checking and fake NPC making code to local helper functions, which should have a positive impact on readability of the code.

Only two commits to see if this kind of changes is welcome, I'm quite positive more stuff can be turned into descriptively named local functions...
Wrote a fix and then found it wasn't necessary.  >_<