Sivovaalex patch 2#22
Conversation
|
изменила код, учитывая замечания:
и другие замечания тоже |
romvano
left a comment
romvano
left a comment
There was a problem hiding this comment.
немного поднял оценку. Но кода очень много - можно сокращать и сокращать. Больше переиспользовать общие паттерны, выделяя их в отдельные функции или декораторы
|
|
||
| @first_name.setter | ||
| def first_name(self, first_name): | ||
| if hasattr(self, 'first_name') == False: |
There was a problem hiding this comment.
мб в кавычках имелось в виду '_first_name'?
| def __init__(self, first_name, last_name, birth_date, phone, document_type, document_id): | ||
| self.logger_e = logging.getLogger("errors") | ||
| self.logger_e.setLevel(logging.ERROR) | ||
| self.formatter = logging.Formatter("%(filename)s[LINE:%(lineno)d]# %(levelname)-8s [%(asctime)s] %(message)s") |
There was a problem hiding this comment.
работу с логгерами договорились в отдельном месте делать
| and (len(birthday_list[2]) == 1 or len(birthday_list[2]) == 2) and ( | ||
| int(birthday_list[2]) >= 1 and int(birthday_list[2]) <= 31): | ||
| birthday_year = birthday_list[0] | ||
| birthday_month = birthday_list[1] |
There was a problem hiding this comment.
лучше всю эту функцию реализовать через datetime
| {'first_name': self.first_name, 'last_name': self.last_name, 'birth_date': self.birth_date, | ||
| 'phone': self.phone, 'document_type': self.document_type, 'document_id': self.document_id}) | ||
| except OSError: | ||
| self.log_error("Error: Данные не сохранены") |
There was a problem hiding this comment.
except (OSError, UnicodeError, ....)
|
|
||
| def limit(self, n): | ||
| raise NotImplementedError() | ||
| self.lim = n |
There was a problem hiding this comment.
из-за общего ресурса не получится использовать один объект коллекции параллельно
2 participants
No description provided.