HW2 with tests (all ok)#16
Conversation
romvano
left a comment
romvano
left a comment
There was a problem hiding this comment.
Логика реализовано достаточно грамотно, главное замечание - к двойным подчеркиваниям
|
|
||
| class LogDescriptor: | ||
| def __init__(self, file_path, log_name, log_level): | ||
| self.__log = logging.getLogger(log_name) |
There was a problem hiding this comment.
двойное нижнее подчеркивание нужно только при реализации сложной логики наследования. Для того, чтобы показать другим программистам, что не нужно трогать это поле извне, лучше использовать одинарное подчеркивание
| class Patient: | ||
| def __init__(self, *args, **kwargs): | ||
| pass | ||
| __document_types = {'паспорт': 10, |
There was a problem hiding this comment.
тоже стоит изменить двойные подчеркивания на одинарные
| from calendar import monthrange | ||
|
|
||
|
|
||
| class LogDescriptor: |
|
|
||
| def __init__(self, first_name, last_name, birth_date, phone, document_type, document_id): | ||
| self.__first_name = self._get_correct_name(first_name) | ||
| self.__last_name = self._get_correct_name(last_name) |
| f"Тип документа: {self.__document_type}\n" | ||
| f"Номер документа: {self.__document_id}\n") | ||
|
|
||
| def __getattr__(self, item): |
| raise ValueError('Wrong number of digits in document id') | ||
| return document_id | ||
|
|
||
| @classmethod |
There was a problem hiding this comment.
classmethod нужен, если есть сложная логика наследования или используются атрибуты класса. В этом случае стоит обойтись статическим методом
|
|
||
| class PatientCollectionIterator: | ||
| def __init__(self, path_to_file, limit=None): | ||
| self.__path_to_file = path_to_file |
There was a problem hiding this comment.
то же замечание к подчеркиваниям
| if self.__limit is None: | ||
| end = file_len | ||
| else: | ||
| if self.__limit > len(pd.read_csv(self.__path_to_file, header=None)): |
There was a problem hiding this comment.
дважды приходится считывать весь файл
| end = self.__limit | ||
| if self.__index < end: | ||
| self.__index += 1 | ||
| patient_info = pd.read_csv(self.__path_to_file, nrows=self.__index, dtype=str, |
There was a problem hiding this comment.
даже не дважды, а на каждой итерации в оперативу загоняется целиковый файл. А если он большой?
|
|
||
| def __init__(self, path_to_file): | ||
| if not os.path.isfile(path_to_file): | ||
| raise FileNotFoundError(f'Not exist: {path_to_file}') |
2 participants
Все тесты пройдены.