Сложновато читать неформатированный код. Но самое первое, что бросается в глаза - это запрос в цикле. Считается очень грубой ошибкой.