What good would removing the inner loop do, when it was the inner loop that was the both the heart of the question and the source of the misrefactoring?

Turning it into a subroutine makes the logic easier to see?