This makes no sense no need to save return value if you are saving something else to it at next line
Also you should leave attacking for harassExecuteOld