ループ処理のネストでは、配列の位置を示す変数を別にしないといけないというのが肝になります。
初心者の内は、これを忘れてバグが取れずに苦しんだりします。
慣れてきた人でも、コードのコピペなどで、こういったバグに出くわしたりします。