Ашихмина Людмила Александровна Б42#9
Ашихмина Людмила Александровна Б42#9Lintari wants to merge 14 commits intopython-course-matmex:mainfrom
Conversation
|
в конце несколько коммитов fixed some little problems Относились чисто к косметическим изменениям и там я просто строчку забыла убрать |
Godrik0
left a comment
Godrik0
left a comment
There was a problem hiding this comment.
ISBN10: 5/10
Бинарный поиск: 4/5
Оформление: 5/10 (snake_case, нет лицензии, нет .gitignore, нет описания к PR)
| @@ -1,6 +1,8 @@ | |||
| def binSearch(xs: list[int], x: int): | |||
| if type(x) != int: | |||
| return "wrong type" | |||
There was a problem hiding this comment.
Нужно бросать исключение. Такой return может быть неожиданным для пользователя, потому что он ожидает, что возвращаемым значением будет число, а возвращается строка.
| elif char != '-' and char != ' ': | ||
| return False | ||
|
|
||
|
|
||
| if len(digits) != 10: | ||
| return False |
There was a problem hiding this comment.
Рекомендация: при неверной длине строки (или других ошибках) лучше выбрасывать исключение, чтобы различать некорректный формат входных данных и просто невалидную контрольную сумму.
Тогда в вызывающем коде нужно будет добавить блок try-except.
|
|
||
| def test_good(): | ||
| assert modulo11Checksum("2-266-11156-3") | ||
| assert modulo11Checksum("1-563-68745-X") |
There was a problem hiding this comment.
В задании требовалось также реализовать консольную утилиту.
There was a problem hiding this comment.
Не хватает тестов на пустой массив, на массив из 1 элемента.
2 participants
No description provided.