Repository navigation
1. Веснина Мария Александровна. 613 - #26
mvesnina91-debug wants to merge 6 commits into
Conversation
| print("Timestaps overflows") | ||
| return None | ||
| return ( | ||
| (time_ms << (NODE_ID_BITS + SEQUENCE_ID_BITS)) |
There was a problem hiding this comment.
Вычисление сдвига дублировано в функции decode_timestamp_ms. Это может быть опасно, так как если логика поменяется, то надо будет исправлять в 2х местах, а это можно забыть, поэтому лучше было вынести вычисление в отдельную константу.
Пункт S1, минус балл.
|
|
||
| new_id = generate_product_id(storage) | ||
| normal_price = normalize_price(fields[1]) | ||
| quantity = fields[2] |
There was a problem hiding this comment.
Магические константы. Для индексов есть константы.
Пункт S3
| PRICE_PRECISION: Final[int] = 0 | ||
| PRICE_STEP: Final = Decimal(0) | ||
| PRICE_PRECISION: Final[int] = 2 | ||
| PRICE_STEP: Final = Decimal(1).scaleb(PRICE_PRECISION) |
There was a problem hiding this comment.
Decimal(1).scaleb(-PRICE_PRECISION)| # measured per call from the data (see the function). | ||
| # TODO: задайте заголовки столбцов таблицы | ||
| TABLE_HEADERS: Final[tuple[str, ...]] = () | ||
| TABLE_HEADERS: Final[tuple[str, ...]] = ("ID", "name", "price", "quanyity") |
There was a problem hiding this comment.
Должно быть quantity. У меня даже тесты упали, но снижать не буду. Будьте внимательнее.
| normal_price = normalize_price(fields[1]) | ||
| quantity = fields[2] | ||
|
|
||
| update_product = (product_id, normal_name, normal_price, quantity) |
There was a problem hiding this comment.
А обновление storage где?
| return None | ||
|
|
||
| if target_index == -1: | ||
| print(f"no product with id {product_id}") |
There was a problem hiding this comment.
Тут, наверное планировался return None
| args = words[1:] | ||
| len_args = len(args) | ||
|
|
||
| if cmd == "help" and len_args == 0: |
There was a problem hiding this comment.
match case был бы проще и нагляднее
В результате постоянная проверка len_args, копипастой
Пункт S1
No description provided.