Repository navigation
2. Кривчанская Елена Ивановна. 614 - #139
Flounderena wants to merge 25 commits into
Conversation
first task 25.09.2026
first task 25.09.2026
update
Second task utils
second task crud
second task storage
second task cart
| if cart[ind][LINE_PRODUCT_ID_INDEX] == product_id: | ||
| new_c_prod: CartLine = ( | ||
| product_id, | ||
| cart[ind][LINE_QUANTITY_INDEX] - quantity, |
There was a problem hiding this comment.
Очень правильно, что вы определили эту функцию и вынесли сюда логику обновления количества. Единственное, следовало использовать переданное значение quantity напрямую, без каких-либо модификаций. Тогда эту функцию можно было бы использовать и в функции добавления новых товаров в корзину, и в функции удаления позиций из корзины. Без этого в коде получается дублирование.
| product_id, | ||
| (prod[NAME_INDEX], prod[PRICE_INDEX], prod[QUANTITY_INDEX] - quantity), | ||
| ) | ||
| for ind in range(len(cart)): |
There was a problem hiding this comment.
Фактически, логика этого обновления вынесена в функцию change_cart_quantity, а тут присутствует дублирование логики. Это нарушение критерия S1, минус балл.
| """ | ||
| # TODO: реализуйте функцию | ||
| return 0 | ||
| if len(storage) == 0: |
There was a problem hiding this comment.
Нарушение критерия T4. Минус 0.25 балла
| """ | ||
| # TODO: реализуйте функцию | ||
| return (0, "", Decimal(0), 0) | ||
| for ind in range(len(storage)): |
There was a problem hiding this comment.
Нарушение критерия T5. В данном коде следовало бы использовать enumerate, чтобы увеличить читаемость кода. Минус 0.25 балла
Task 2, repull