Repository navigation
Conversation
esnvox
changed the title
| """ | ||
| # TODO: реализуйте функцию | ||
| return 0 | ||
| timestamp_ms = (snowflake_id >> 22) & (2**TIMESTAMP_BITS - 1) |
There was a problem hiding this comment.
Магические константы усложняют понимание кода.
Пункт S3, минус балл.
| # TODO: реализуйте функцию | ||
| return 0 | ||
| if node_id < 0 or NODE_ID_MAX < node_id: | ||
| print(f"node_id must be in [{0}; {NODE_ID_MAX}]") |
There was a problem hiding this comment.
Забыли return None и ниже тоже
| if node_id < 0 or NODE_ID_MAX < node_id: | ||
| print(f"node_id must be in [{0}; {NODE_ID_MAX}]") | ||
|
|
||
| if 0 < sequence_id or sequence_id < SEQUENCE_ID_MAX: |
There was a problem hiding this comment.
Тут как будто знак больше должен быть:
0 < sequence_id or sequence_id > SEQUENCE_ID_MAX
| index = 0 | ||
| for line in cart: | ||
| if line[LINE_PRODUCT_ID_INDEX] == product_id: | ||
| updated_product = (product_id, line[LINE_PRODUCT_ID_INDEX] + quantity) |
There was a problem hiding this comment.
Вы используете LINE_PRODUCT_ID_INDEX, а это индекс id продукта, а не индекс количества
| for line in cart: | ||
| if line[LINE_PRODUCT_ID_INDEX] == product_id: | ||
| updated_product = (product_id, line[LINE_PRODUCT_ID_INDEX] + quantity) | ||
| cart[index] = updated_product |
There was a problem hiding this comment.
Кажется вы не меняете index, и он у вас всегда 0
| for index in range(len(cart)): | ||
| if cart[index][LINE_PRODUCT_ID_INDEX] == product_id: | ||
| if new_quantity == 0: | ||
| del cart[index] |
There was a problem hiding this comment.
Это уже вкусовщина, но мне больше нравится удалять через cart.pop(index)
| # TODO: реализуйте функцию | ||
| return (0, "", Decimal(0), 0) | ||
|
|
||
| for index, product in enumerate(storage): |
There was a problem hiding this comment.
Цикл:
for index, product in enumerate(storage):
if product[PRODUCT_ID_INDEX] == product_id:встречается 3 раза, лучше было бы поиск вынести в отдельную функцию. Код был бы понятнее.
Пункт S1
| for product in storage: | ||
| if product[PRODUCT_ID_INDEX] == product_id: | ||
| return product | ||
| print(f"no product with id {product_id}") |
There was a problem hiding this comment.
Сообщение пишется каждую итерацию
Домашнее задание на 26 сентября.
Домашнее задание на 2 октября.