Есть куча замечаний по поводу оформления. Не стоит писать функцию внутри другой. Ты проверяла работоспособность программы? По стилю оформления кажется, что нет. Смотри изменения