Repository navigation
2. Шемякин Николай Александрович. 613 - #33
shiba-rose wants to merge 12 commits into
Conversation
shiba-rose
changed the title
shiba-rose
changed the title
| if timestamp_ms > TIMESTAMP_MS_MAX: | ||
| return print("overflows") | ||
|
|
||
| timestamp_ms_snowflake_id = timestamp_ms << (SEQUENCE_ID_BITS + NODE_ID_BITS) |
There was a problem hiding this comment.
Вычисление сдвига дублировано в функции decode_timestamp_ms. Это может быть опасно, так как если логика поменяется, то надо будет исправлять в 2х местах, а это можно забыть, поэтому лучше было вынести вычисление в отдельную константу.
Пункт S1, минус балл.
| """ | ||
| # TODO: реализуйте функцию | ||
| return (0, "", Decimal(0), 0) | ||
| for product in storage: |
There was a problem hiding this comment.
Цикл:
for index, product in enumerate(storage):
if product[PRODUCT_ID_INDEX] == product_id:встречается несколько раз, лучше было бы использовать везде read_product. Код был бы понятнее.
Пункт S1
| """ | ||
| # TODO: реализуйте функцию | ||
| return (0, 0) | ||
| product = read_product(storage, product_id) |
There was a problem hiding this comment.
Лучше не экономить строки и оставлять пробелы (пустые строки) между логическими блоками. Так читабельность кода улучшается.
| # TODO: реализуйте функцию | ||
| return (0, 0) | ||
| line = find_cart_line(cart, product_id) | ||
| if line == None: |
| """ | ||
| # TODO: реализуйте функцию | ||
| return 0 | ||
| if len(storage) == 0: |
| for index, product in enumerate(storage): | ||
| if product[PRODUCT_ID_INDEX] == product_id: | ||
| for other_product in storage: | ||
| if ( |
There was a problem hiding this comment.
Слишком большая вложеность
Пункт M1
| return "\n".join(table_rows) | ||
|
|
||
|
|
||
| storage: list[tuple[int, str, Decimal, int]] = [] |
| return "" | ||
| column_width = [len(header) for header in TABLE_HEADERS] | ||
| for product in storage: | ||
| column_width[0] = max(column_width[0], len(str(product[0]))) |
There was a problem hiding this comment.
А почему просто не сделать цикл? Ниже тоже самое.
Пункт S1
| for product in storage: | ||
| id, name, price, quantity = map(str, product) | ||
| row_mas = [ | ||
| f" {id.ljust(column_width[0])} ", |
There was a problem hiding this comment.
Тут 0, 1, 2, 3 фактически являются магическими константами, так как если у вас поменяются местами name и price, то и индексы надо будет менять местами.
Пункт S3
No description provided.