Repository navigation
Conversation
EvgrafovMichail
left a comment
EvgrafovMichail
left a comment
There was a problem hiding this comment.
Хорошее решение. В следующий раз не допускайте обидных нарушений критериев.
Также рекомендация относительно модуля cart.py: функции получились достаточно длинными. Во время чтения этих функций можно выделить несколько логических действий. Старайтесь такие действия оформлять в отдельный функции.
| if cart[i][LINE_PRODUCT_ID_INDEX] == product_id: | ||
| cart[i] = new_cart_line | ||
| return new_cart_line | ||
| else: |
There was a problem hiding this comment.
Нарушение критерия T1. Штраф 0.25 балла
|
|
||
| if cart_line is not None: | ||
| new_cart_line = (product_id, quantity + cart_line[LINE_QUANTITY_INDEX]) | ||
| for i in range(len(cart)): |
There was a problem hiding this comment.
Нарушение критерия T5. Штраф 0.25 балла
| return 0 | ||
|
|
||
| name, price, quan = fields | ||
| n_id = generate_product_id(storage) |
There was a problem hiding this comment.
Лучше генерировать идентификатор после выполнения проверки. Так есть риск выполнения лишней работы в тех случаях, когда проверка уникальности имени не будет пройдена и сгенерированный идентификатор так и не будет использован.
No description provided.