Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Информация выводится датаклассами #21

Merged
merged 16 commits into from
Nov 6, 2023

Conversation

Stanislav-Gutnikov
Copy link

@Stanislav-Gutnikov Stanislav-Gutnikov commented Oct 18, 2023

Парсинг таблицы -- !словарь по каждому игроку! -- словари в объекты класса. Сейчас это работает так. По сути шаг где я создаю словари можно убрать, но я это сделал чтобы код был более читаемым, чтобы было наглядно и понятно какие данные идут в объекты класса.
Еще есть заминка. я использую try except без Exception. Соответственно flake8 ругается. В тех случаях где я это делаю это реально удобно. Я добавил пустой Exception для обхода flake8

@Stanislav-Gutnikov
Copy link
Author

Новый парсер к новой таблице

@Raidzin Raidzin merged commit 219b1eb into dev Nov 6, 2023
@Raidzin Raidzin deleted the feature/xlsx_parser branch November 6, 2023 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants