Refactor cannoncalls to missles and replace usages of for-each iterator