Conversation
Godrik0
left a comment
Godrik0
left a comment
There was a problem hiding this comment.
ISBN10: 4/10
Бинарный поиск: 3/5
Оформление: 7/10 (нужно было переписать всё в snake_case)
| calculated_checksum = total % 11 | ||
| return calculated_checksum == checkDigit |
There was a problem hiding this comment.
Ошибка в логике.
Нужно было либо оставить checksum = total + checkDigit и сравнивать остаток от деления на 11 с 0.
Либо высчитывать вот так (11 - (sum % 11)) % 11 и тогда уже сравнивать с контрольной цифрой.
There was a problem hiding this comment.
Тесты упадут, потому что импорт в файле тестов test/test_bin_searh.py не был обновлен, там так и осталось from src.bin_search import binSearch.
There was a problem hiding this comment.
В задании требовалось также реализовать консольную утилиту.
| left, right = 0, len(xs) - 1 | ||
| while left < right: | ||
|
|
||
| while left <= right: |
There was a problem hiding this comment.
Нужно было добавить тесты, которые проверяют этот случай
| elif char.upper() == 'X': | ||
| remaining_chars = ISBNNumber[i:].replace('-', '').replace(' ', '') |
There was a problem hiding this comment.
Нужно было добавить тестов на проверку X
2 participants
добавила недостающие файлы в репозиторий, затем исправила баги в файлах checksum.py и binary_search.py
Евневич Екатерина. группа: Б42