BACK-END COVID-19 PYTHON#18
Conversation
romvano
left a comment
romvano
left a comment
There was a problem hiding this comment.
Код хороший, но часто перегружен такими нюансами, как, например, преобразование True/False/None к текстовым полям и обратно. С точки зрения оптимизации, это экономия на спичках. С точки зрения поддерживаемости кода - становится сильно сложнее
|
|
||
|
|
||
| def check_first_name(first_name): | ||
| if isinstance(first_name, str): |
There was a problem hiding this comment.
лучше наоборот: if not isinstance: raise, а дальше просто логику, чтобы не плодить вложенность
| if first_name.isalpha(): | ||
| first_name = first_name.capitalize() | ||
|
|
||
| page = requests.get(f"http://imenator.ru/search/?text={first_name}") |
There was a problem hiding this comment.
такие константы лучше в отдельное место сложить
|
|
||
| def check_phone(phone): | ||
| if not isinstance(phone, str): | ||
| if 10_000_000_000 <= phone <= 99_999_999_999: |
There was a problem hiding this comment.
а если phone не число, не залогируется ошибка
|
|
||
|
|
||
| class Patient: | ||
| _DOCUMENT_TYPES = {None: "Паспорт РФ", |
There was a problem hiding this comment.
лучше сразу текстовые переменные возвращать после проверки. Так сложно уследить, что откуда берется
| @document_type.getter | ||
| def document_type(self): | ||
| """Для оптимизации кода тип документа | ||
| храним как None, True или False. |
There was a problem hiding this comment.
излишняя оптимизация. Код понимать сложнее, а по памяти слабая оптимизация. Если говорить об огромной БД, то такие оптимизации лучше проводить ближе к хранению данных и дальше от бизнес-логики
|
|
||
| if check.is_typo_in_name(self._last_name, new_last_name): | ||
| INFO_LOG.info(f"Изменена фамилия на '{new_last_name}' у пациента {self}.") | ||
| self._last_name = new_last_name |
There was a problem hiding this comment.
по ТЗ фамилию нельзя менять
|
|
||
| if check.is_typo_in_name(self._first_name, new_first_name): | ||
| INFO_LOG.info(f"Изменено имя на '{new_first_name}' у пациента {self}.") | ||
| self._first_name = new_first_name |
| for i, line in enumerate(f): | ||
| if i != 0: | ||
| if i <= last: | ||
| params = line.decode().split(',') |
There was a problem hiding this comment.
можно же не в байтовом режиме файл открыть
2 participants
No description provided.