[etc]

Стандартные грабли начинающего программера, который полагает, что если он счетчик i написал внутри какой-то функции, объявленной в цикле, то этот самый счетчик i внутри этой функции будет заморожен и иметь значение, равное тому, что было в момент создания функции. Это на самом деле не так, значение i будет равно тому, которое будет после выполнения цикла (т.е. равен n при схеме i<n). Выходом обычно является присвоение объекту в цикле переменной, содержащее текущее значение i, а внутри функции использовать эту переменную.
Код приводить не буду, на эти граблы наступает куча человек и лучше вникнуть и написать код самостоятельно.