Вот это точно совершенно не нужно. Понятно, почему?

Вообще, это плохой тон -- менять перемную итерации в цикле