Conversation
|
/request-review @chernishev |
Godrik0
left a comment
Godrik0
left a comment
There was a problem hiding this comment.
ISBN10: 5/10
Бинарный поиск: 5/5
Оформление: 2/10 (snake_case, новые файлы с тестами, нет лицензии, нет .gitignore, проблемы с форматированием, нет внятного описания в PR, неинформативные и плохо сформулированные тексты коммитов)
| num = input() | ||
|
|
||
| while num != "-1": | ||
| res = modulo11Checksum(num) | ||
|
|
||
| if res: | ||
| print("correct") | ||
|
|
||
| else: | ||
| print("incorrect") | ||
|
|
||
| num = input() |
There was a problem hiding this comment.
Когда pytest импортирует этот файл для тестов, он зависнет, ожидая ввода данных.
Код, который относится к запуску нужно оборачивать в if __name__ == "__main__".
| if len(digits) != 10: | ||
| print("Цифр должно быть 10", end=' - ') | ||
| return | ||
|
|
There was a problem hiding this comment.
У Вас пропущена обработка символа X, который может служить контрольной цифрой.
|
|
||
|
|
||
| if len(digits) != 10: | ||
| print("Цифр должно быть 10", end=' - ') |
There was a problem hiding this comment.
Такое не должно выводиться принтом, тут следовало кинуть исключение.
|
|
||
| if len(digits) != 10: | ||
| print("Цифр должно быть 10", end=' - ') | ||
| return |
There was a problem hiding this comment.
Это вернет None, хотя функция предполагает возврат True/False.
There was a problem hiding this comment.
А что не так с тестами из папки test? Для чего новые файлы с тестами?
|
|
||
| def test1(): | ||
| num = "99921-58-10-7" | ||
| assert modulo11Checksum(num) == True |
There was a problem hiding this comment.
Правильно писать assert modulo11Checksum(num) или assert not modulo11Checksum(num).
2 participants
исправлены ошибки
сделал тесты
ruff ci исправлен