Conversation
| self.__tax_rate = {'чипсы': 20, 'кола': 20, 'печенье': 20, 'молоко': 10, 'кефир': 10} | ||
|
|
||
| @property | ||
| def get_name_items(self): |
There was a problem hiding this comment.
Можно лучше: название пропертей стоит давать по названию атрибута
| return self.__number_items | ||
|
|
||
| def add_item_to_cheque(self, name): | ||
| if len(name) != 0 or len(name) < 40: |
There was a problem hiding this comment.
Нужно исправить: условие составлено не верно
There was a problem hiding this comment.
Коммент необходимо описать более развернуто, указать, что именно мы получим при такой реализации условия, а что мы должны быти получить по заданию, чтобы дать студенту направление мысли для исправления
| def add_item_to_cheque(self, name): | ||
| if len(name) != 0 or len(name) < 40: | ||
| raise ValueError('Нельзя добавить товар, если в его названии нет символов или их больше 40') | ||
| else: |
There was a problem hiding this comment.
Нужно исправить: отсутствует проверка наличия товара в справочнике
| if len(self.__name_items) > 10: | ||
| return sum(total) - (sum(total) * 10) / 100 | ||
| else: | ||
| return sum(total) * 0.9 |
There was a problem hiding this comment.
Нужно исправить: в данном случае должна возвращаться полная стоимость товаров
| if len(twenty_percent_tax) > 10: | ||
| return (sum(total) - (sum(total) * 10) / 100) | ||
| else: | ||
| return sum(total) |
There was a problem hiding this comment.
Нужно исправить: не верная формула расчета НДС. Рассчитана полная стоимость всех позиций
| if len(ten_percent_tax) > 10: | ||
| return (sum(total) - (sum(total) * 10) / 100) | ||
| else: | ||
| return sum(total) |
There was a problem hiding this comment.
Нужно исправить: не верная формула расчета НДС. Рассчитана полная стоимость всех позиций
| ten_percent_tax = [] | ||
| for i in range(0, len(self.__name_items)): | ||
| if self.__name_items[i] in self.__tax_rate: | ||
| if self.__tax_rate.get(self.__name_items[i]) == 20: |
There was a problem hiding this comment.
Нужно исправить: неверное условие фильтрации
| return sum(total) | ||
|
|
||
| def total_tax(self): | ||
| return OnlineSalesRegisterCollector.ten_percent_tax_calculation() + OnlineSalesRegisterCollector.twenty_percent_tax_calculation() |
There was a problem hiding this comment.
Нужно исправить: для вызова данных методов необходимо создать экземпляр класса, а раз вызов происходит внутри класса, то вызов должен происходить через self
There was a problem hiding this comment.
Коммент дает студенту вариативность написания, что является неверным. Создавать экземпляр класса внутри методов этого же класса нельзя, корректным будет только обращение через self
| if len(str(telephone_number)) > 10: | ||
| raise ValueError('Необходимо ввести 10 цифр после "+7"') | ||
| try: | ||
| float(telephone_number) |
There was a problem hiding this comment.
Нужно исправить: необходимо проверять является ли введенное значение целочисленным. Лучше всего для этого подойдет функция isinstance
| if twenty_percent_tax[i] in self.__item_price: | ||
| total.append(self.__item_price.get(twenty_percent_tax[i])) | ||
| if len(twenty_percent_tax) > 10: | ||
| return (sum(total) - (sum(total) * 10) / 100) |
There was a problem hiding this comment.
Отсутствует коммент: Нужно исправить: применена формула для расчета 10% ндс а не 20 по методу
2 participants
No description provided.