hmm .. interesting .. I havent tested it but if you wrote the majority of it yourself and understand how it works then good ..
you do have a lot of stuff in your script that you dont need.. variables that you don't use, if statements that do nothing but make the file larger, a lot of empty white space.. and other things that have me scratching my head trying to figure out what in the world your trying to do..
but at least your trying and learning 
some errors you made:
Simba Code:
Mouse(x,y,5,5,True); // true = left click, false = right click
if InFight then // doing this means you look for randoms only while in a fight ....
begin
repeat
FindNormalRandoms;
until ((not LoggedIn) or (not InFight))
end;
FindObjEx(x,y,....) // when you use this it automatically puts the mouse coordinates into x and y .. you dont need to do GetMousePos
Useful links:
http://docs.villavu.com/simba/referencescript.html
http://docs.villavu.com/srl-5/srlref.html