The reason I mentioned $i is that you reuse the variable (not just the name). I typically view that as bad, especially when it's trivial to declare it for the loop itself.

    ||
    vv
for my $i (1..5) {
   ...
}
[download]