[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...
Oops, missing the skill values.  Will see about fixing, but feel free to beat me to it as I may have to break for dinner shortly.