Очень прошу ревью кода. И есть пара вопросов по выбору файла с появлением/исчезанием фрейма

[Andrey6570]

Я только начал разбираться с Python, всего две недели. До этого только Basic в школе и HTML в универе.
Попробовал написать программу, которая проверяет знание иностранного языка, в частности греческих слов.

ТЗ: Есть файл Excel, который содержит три листа с парами слов на греческом и русском - общая лексика, числа и дни недели. Пользователь выбирает направление перевода, тип тестирования и какие слова проверять. У него есть 10 вопросов, счетчик внизу считает количество вопросов и правильные и неправильные ответы. Переход к следующему вопросу по кнопке Далее.
Как написал в теме, вроде работает. Но хочется получить рекомендации и советы по коду, что можно улучшить. И есть несколько вопросов:

1. После 10 вопроса должен пропадать фрейм с заданием, и на его месте должен появляться фрейм frame_about. Не работает.

2. Хочу сделать, чтобы из меню Файл можно открывалось окно с выбором файла Excel - не знаю как, может кто-то подскажет?

Код не могу разместить из-за ограничений количества символов в сообщении, поэтому размещаю его в виде txt

Делал всё в PyCharm

[Ципихович Эндрю]

Сообщение от Andrey6570

Код не могу разместить из-за ограничений количества символов в сообщении

вау

Сообщение от Andrey6570

2. Хочу сделать, чтобы из меню Файл можно открывалось окно с выбором файла Excel - не знаю как, может кто-то подскажет?

не Эксель-рядом, разберётесь

# типы файла: (('HTML-файлы', '*.html'),) или ('Изображение', '*.jpg *.gif *.png'),) или (('Любой', '*'),) или (('Hyper Text Markup Language file', '*.html'),) или (('Текст в формате RTF', '*.rtf'),) или (('Текстовый файл', '*.txt'),)
# если писать несколько типов файла, в конце запятая не нужна: (('Текстовый файл', '*.txt'), ('Изображение', '*.jpg *.gif *.png'), ('Любой', '*'))
# multiple=True|False множественный выбор-по умолчанию=False
str_file_name = fd.askopenfilename(title='Выберете файл с расширением txt', initialdir='d:/', filetypes=(('Текстовый файл', '*.txt'),))

[Ципихович Эндрю]

попробовал ради интереса
ну да
Вы ввели слишком длинный текст (14903 символов). Пожалуйста, сократите его до 8000 символов.
не знал
на мой беглый взгляд global-зло

c=0
d=0
e=0

хозяин барин, но в одну строку

c=d=e=0

далее

if i < 10:
        start()
else:
        stop()

хозяин барин, но укоротим

if i < 10:
    start()
else:stop()

а так - не вникал

[Andrey6570]

Сообщение от Ципихович Эндрю

попробовал ради интереса
ну да
Вы ввели слишком длинный текст (14903 символов). Пожалуйста, сократите его до 8000 символов.
не знал
на мой беглый взгляд global-зло

c=0
d=0
e=0

хозяин барин, но в одну строку

c=d=e=0

далее

if i < 10:
        start()
else:
        stop()

хозяин барин, но укоротим

if i < 10:
    start()
else:stop()

а так - не вникал

Мелочь, но замечания важные. Спасибо большое.

[VeryStupidPerson]

на мой беглый взгляд global-зло

Вы ещё скажите GOTO - зло. (Хотя, кстати, в питоне его нет). Иногда бывает проще использовать глобальную переменную, чем устраивать передачи в функцию и тд. Кстати, Ципихович Эндрю, интерпретатор питон, если встречает в функциях неизвестную переменную, ищет её в основном коде и, если находит, объявляет глобальной.
Это какой-то максималистский подход, брать и объявлять то, чем вы не пользуетесь, злом. Всё, что есть в языке, порой необходимо.

[VeryStupidPerson]

далее
Код:
if i < 10:
start()
else:
stop()
хозяин барин, но укоротим
if i < 10:
start()
else:stop()

На мой взгляд, более длинный вариант читабельней и приятней для глаз. Вряд ли Вы используете возможность записи в одну строку (из разряда, a = input("a = ");print('a');b=input("b = ");print('b')), а ведь интерпретатор позволяет запись в строку, если разделять команды точкой с запятой.

[Andrey6570]

Сообщение от VeryStupidPerson

На мой взгляд, более длинный вариант читабельней и приятней для глаз. Вряд ли Вы используете возможность записи в одну строку (из разряда, a = input("a = ");print('a');b=input("b = ");print('b')), а ведь интерпретатор позволяет запись в строку, если разделять команды точкой с запятой.

Спасибо. Может будут еще какие-то замечания по форматингу?

[BDA]

Сообщение от Andrey6570

Может будут еще какие-то замечания по форматингу?

Ага. Проблемки с пробелами (то есть после запятой, то нет; двойные пробелы; вокруг "равно" то есть, то нет), лишние пустые строки (то есть пустая строка после заголовка функции, то нет; то отделяются пустыми строками if-elif, то нет; то есть пустая строка между функциями, то нет; то есть строка после global, то нет). Из-за этого код выглядит неряшливо.

[Ципихович Эндрю]

Сообщение от VeryStupidPerson

Вряд ли Вы используете возможность записи в одну строку (из разряда, a = input("a = ");print('a');b=input("b = ");print('b')), а ведь интерпретатор позволяет запись в строку, если разделять команды точкой с запятой.

промах-так как я бы с радостью так сделал, но пользуюсь в пайчарме реформатом кода-и он всё равно раскидает построчно

[Ципихович Эндрю]

Сообщение от BDA

Проблемки с пробелами (то есть после запятой, то нет; двойные пробелы; вокруг "равно" то есть, то нет), лишние пустые строки (то есть пустая строка после заголовка функции, то нет; то отделяются пустыми строками if-elif, то нет; то есть пустая строка между функциями, то нет; то есть строка после global, то нет). Из-за этого код выглядит неряшливо.

тоже из этого много решить реформатом кода