From d615eda6c73629dea1a34d513b04f5cc191738f4 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Wed, 23 Sep 2026 17:41:30 +0300 Subject: [PATCH 01/23] hhh --- src/part1/snowflake.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index d3abf050..a5eba8ee 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -9,7 +9,7 @@ caller passes the sequence counter on every call and is responsible for advancing it within a millisecond and resetting it when the clock ticks over. -Each packed field can be read back on its own with :func:`decode_timestamp_ms`, +Each packed field can be read back on its own with :func:`decode_timestamp_ms`,hhjhj :func:`decode_node_id` and :func:`decode_sequence_id`. """ From e24ec21fd3f4fc69ad2b34ba820ac9170bd39f6c Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Wed, 23 Sep 2026 17:45:05 +0300 Subject: [PATCH 02/23] =?UTF-8?q?=D0=93=D0=BE=D1=82=D0=BE=D0=B2=D1=8B?= =?UTF-8?q?=D0=B9=20=D0=BA=D0=BE=D0=B4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/constants.py | 46 +++------------- src/part1/snowflake.py | 122 ++++++++--------------------------------- 2 files changed, 31 insertions(+), 137 deletions(-) diff --git a/src/part1/constants.py b/src/part1/constants.py index 9abfbba2..05135d68 100644 --- a/src/part1/constants.py +++ b/src/part1/constants.py @@ -1,38 +1,8 @@ -"""Bit layout and default epoch for the Twitter Snowflake ID generator. - -A Snowflake identifier is a 63-bit positive integer packed as follows, from -the most significant bit downwards: - - 1 bit unused sign bit, always zero - 41 bits milliseconds elapsed since a custom epoch - 10 bits node identifier - 12 bits per-millisecond sequence counter - -The shift and mask constants should be derived from the bit widths so that -changing a width keeps everything else consistent. -""" - -from typing import Final - - -# TODO: замените заглушки (0) на корректные значения, см. TASK.md. -# Все константы, зависящие от ёмкостей *_BITS, должны вычисляться из них. - -# Twitter's original Snowflake epoch: 2010-11-04 01:42:54.657 UTC. -EPOCH_MS_DEFAULT: Final[int] = 0 - -# Node identifier used when the caller does not supply one. -NODE_ID_DEFAULT: Final[int] = 0 - -# Width of each field, in bits. -TIMESTAMP_BITS: Final[int] = 0 -NODE_ID_BITS: Final[int] = 0 -SEQUENCE_ID_BITS: Final[int] = 0 - -# Largest value each field can hold. -TIMESTAMP_MS_MAX: Final[int] = 0 -NODE_ID_MAX: Final[int] = 0 -SEQUENCE_ID_MAX: Final[int] = 0 - -# Здесь можно добавить собственные вспомогательные константы -# (например, сдвиги полей при сборке идентификатора). +EPOCH_MS_DEFAULT = 1288834974657 +NODE_ID_DEFAULT = 0 +TIMESTAMP_BITS, NODE_ID_BITS, SEQUENCE_ID_BITS = 41, 10, 12 +TIMESTAMP_MS_MAX = 2**TIMESTAMP_BITS - 1 +NODE_ID_MAX = 2**NODE_ID_BITS - 1 +SEQUENCE_ID_MAX = 2**SEQUENCE_ID_BITS - 1 +NODE_ID_SHIFT = SEQUENCE_ID_BITS +TIMESTAMP_SHIFT = SEQUENCE_ID_BITS + NODE_ID_BITS diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index a5eba8ee..ec0dc666 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -1,114 +1,38 @@ -"""Twitter Snowflake identifier generator. - -Snowflake produces roughly time-ordered 64-bit integer identifiers without -coordination between nodes. Each identifier packs the milliseconds elapsed -since a custom epoch, a node identifier and a per-millisecond sequence counter -into a single 63-bit positive integer (see ``constants`` for the layout). - -The public entry point is :func:`generate_snowflake_id`. It is stateless: the -caller passes the sequence counter on every call and is responsible for -advancing it within a millisecond and resetting it when the clock ticks over. - -Each packed field can be read back on its own with :func:`decode_timestamp_ms`,hhjhj -:func:`decode_node_id` and :func:`decode_sequence_id`. -""" - -import time # noqa: F401 - -from .constants import ( # noqa: F401 - EPOCH_MS_DEFAULT, - NODE_ID_DEFAULT, - NODE_ID_MAX, - SEQUENCE_ID_MAX, - TIMESTAMP_MS_MAX, -) +import time +from constants import * def read_current_millis(epoch_ms: int) -> int: - """Read the number of milliseconds elapsed since the custom epoch. - - Args: - epoch_ms: The custom epoch expressed as Unix milliseconds. - - Returns: - The count of whole milliseconds between ``epoch_ms`` and now. May be - negative if ``epoch_ms`` lies in the future. - """ - # TODO: реализуйте функцию - return 0 - + return time.time_ns() // 1_000_000 - epoch_ms def decode_timestamp_ms(snowflake_id: int, epoch_ms: int = EPOCH_MS_DEFAULT) -> int: - """Read the timestamp field out of a Snowflake identifier. - - Args: - snowflake_id: An identifier produced against the same epoch. - epoch_ms: The epoch the identifier was generated against. Defaults to - the original Twitter epoch (2010-11-04 01:42:54.657 UTC). - - Returns: - The absolute Unix time in milliseconds at which the identifier was - generated. - """ - # TODO: реализуйте функцию - return 0 - + stored_ms = snowflake_id >> TIMESTAMP_SHIFT + return stored_ms + epoch_ms def decode_node_id(snowflake_id: int) -> int: - """Read the node identifier field out of a Snowflake identifier. - - Args: - snowflake_id: An identifier produced by :func:`generate_snowflake_id`. - - Returns: - The node identifier packed into ``snowflake_id``, in the range - ``[0, NODE_ID_MAX]``. - """ - # TODO: реализуйте функцию - return 0 - + return (snowflake_id >> NODE_ID_SHIFT) & NODE_ID_MAX def decode_sequence_id(snowflake_id: int) -> int: - """Read the sequence counter field out of a Snowflake identifier. - - Args: - snowflake_id: An identifier produced by :func:`generate_snowflake_id`. - - Returns: - The per-millisecond sequence counter packed into ``snowflake_id``, in - the range ``[0, SEQUENCE_ID_MAX]``. - """ - # TODO: реализуйте функцию - return 0 - + return snowflake_id & SEQUENCE_ID_MAX def generate_snowflake_id( - sequence_id: int, - node_id: int = NODE_ID_DEFAULT, - epoch_ms: int = EPOCH_MS_DEFAULT, + sequence_id: int, node_id: int = NODE_ID_DEFAULT, epoch_ms: int = EPOCH_MS_DEFAULT ) -> int | None: - """Build and return a Snowflake identifier for the current millisecond. - The function is stateless: the caller passes the per-millisecond sequence - counter explicitly. It is the caller's responsibility to increment - ``sequence_id`` for identifiers minted within the same millisecond and to - reset it once the clock advances. + if not (0 <= node_id <= NODE_ID_MAX): + print(f"node_id must be in [0, {NODE_ID_MAX}], got {node_id}") + return None + + if not (0 <= sequence_id <= SEQUENCE_ID_MAX): + print(f"sequence_id must be in [0, {SEQUENCE_ID_MAX}], got {sequence_id}") + return None - Args: - sequence_id: The per-millisecond sequence counter, in the range - ``[0, SEQUENCE_ID_MAX]``. - node_id: The identifier of this node, in the range ``[0, NODE_ID_MAX]``. - Optional; defaults to ``NODE_ID_DEFAULT``. - epoch_ms: The start of the epoch as Unix milliseconds. Optional; - defaults to the original Twitter epoch (2010-11-04 01:42:54.657 - UTC). + elapsed_ms = read_current_millis(epoch_ms) + if elapsed_ms > TIMESTAMP_MS_MAX: + print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") + return None - Returns: - The Snowflake identifier as a positive 63-bit integer, or ``None`` if - ``node_id`` is outside ``[0, NODE_ID_MAX]``, ``sequence_id`` is outside - ``[0, SEQUENCE_ID_MAX]``, or the elapsed time no longer fits in the - timestamp field (roughly 69 years after ``epoch_ms``). In each of those - cases an explanatory message is printed to stdout first. - """ - # TODO: реализуйте функцию - return 0 + snowflake_id = ( + (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | (sequence_id) + ) + return snowflake_id \ No newline at end of file From bc9c9364072788f6304b968089ab4aeb42674bd2 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Wed, 23 Sep 2026 17:52:47 +0300 Subject: [PATCH 03/23] =?UTF-8?q?=D0=93=D0=BE=D1=82=D0=BE=D0=B2=D1=8B?= =?UTF-8?q?=D0=B9=20=D0=BA=D0=BE=D0=B4=202?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index ec0dc666..af2eff98 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -1,6 +1,14 @@ import time -from constants import * +from constants import ( + EPOCH_MS_DEFAULT, + NODE_ID_DEFAULT, + NODE_ID_MAX, + NODE_ID_SHIFT, + SEQUENCE_ID_MAX, + TIMESTAMP_MS_MAX, + TIMESTAMP_SHIFT, +) def read_current_millis(epoch_ms: int) -> int: return time.time_ns() // 1_000_000 - epoch_ms @@ -16,7 +24,9 @@ def decode_sequence_id(snowflake_id: int) -> int: return snowflake_id & SEQUENCE_ID_MAX def generate_snowflake_id( - sequence_id: int, node_id: int = NODE_ID_DEFAULT, epoch_ms: int = EPOCH_MS_DEFAULT + sequence_id: int, + node_id: int = NODE_ID_DEFAULT, + epoch_ms: int = EPOCH_MS_DEFAULT, ) -> int | None: if not (0 <= node_id <= NODE_ID_MAX): @@ -32,7 +42,8 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - snowflake_id = ( - (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | (sequence_id) - ) - return snowflake_id \ No newline at end of file + return ( + (elapsed_ms << TIMESTAMP_SHIFT) + | (node_id << NODE_ID_SHIFT) + | sequence_id + ) \ No newline at end of file From c08ad7071075016d5e583c007c45d5c8d4909a87 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 22:34:28 +0300 Subject: [PATCH 04/23] =?UTF-8?q?=D0=92=D0=B5=D1=80=D1=81=D0=B8=D1=8F=203?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index af2eff98..049c065d 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -11,7 +11,7 @@ ) def read_current_millis(epoch_ms: int) -> int: - return time.time_ns() // 1_000_000 - epoch_ms + return time.time_ns() // 1000000 - epoch_ms def decode_timestamp_ms(snowflake_id: int, epoch_ms: int = EPOCH_MS_DEFAULT) -> int: stored_ms = snowflake_id >> TIMESTAMP_SHIFT @@ -43,7 +43,5 @@ def generate_snowflake_id( return None return ( - (elapsed_ms << TIMESTAMP_SHIFT) - | (node_id << NODE_ID_SHIFT) - | sequence_id + (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id ) \ No newline at end of file From a09a31cf3424e9341ae4b0ad8333b7899ac711bc Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 22:47:27 +0300 Subject: [PATCH 05/23] =?UTF-8?q?=D0=92=D0=B5=D1=80=D1=81=D0=B8=D1=8F=204?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index 049c065d..4589acb3 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -10,23 +10,30 @@ TIMESTAMP_SHIFT, ) + def read_current_millis(epoch_ms: int) -> int: return time.time_ns() // 1000000 - epoch_ms + def decode_timestamp_ms(snowflake_id: int, epoch_ms: int = EPOCH_MS_DEFAULT) -> int: stored_ms = snowflake_id >> TIMESTAMP_SHIFT return stored_ms + epoch_ms + def decode_node_id(snowflake_id: int) -> int: return (snowflake_id >> NODE_ID_SHIFT) & NODE_ID_MAX + def decode_sequence_id(snowflake_id: int) -> int: return snowflake_id & SEQUENCE_ID_MAX + def generate_snowflake_id( sequence_id: int, node_id: int = NODE_ID_DEFAULT, epoch_ms: int = EPOCH_MS_DEFAULT, + + ) -> int | None: if not (0 <= node_id <= NODE_ID_MAX): From 7069e220f3466c1109a90a8391f941297acc4c3d Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 22:51:10 +0300 Subject: [PATCH 06/23] =?UTF-8?q?=D0=92=D0=B5=D1=80=D1=81=D0=B8=D1=8F=204?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index 4589acb3..d7096813 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -35,7 +35,6 @@ def generate_snowflake_id( ) -> int | None: - if not (0 <= node_id <= NODE_ID_MAX): print(f"node_id must be in [0, {NODE_ID_MAX}], got {node_id}") return None @@ -49,6 +48,4 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - return ( - (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id - ) \ No newline at end of file + return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file From 40434a025fdf2564482f27d01c8506f1cc4581b3 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 22:54:51 +0300 Subject: [PATCH 07/23] Update snowflake.py MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Версия 5 --- src/part1/snowflake.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index d7096813..2b2e5927 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -32,8 +32,6 @@ def generate_snowflake_id( sequence_id: int, node_id: int = NODE_ID_DEFAULT, epoch_ms: int = EPOCH_MS_DEFAULT, - - ) -> int | None: if not (0 <= node_id <= NODE_ID_MAX): print(f"node_id must be in [0, {NODE_ID_MAX}], got {node_id}") From e4cc97b07465d4b3ff1c7062b17dcd2da8b5296b Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 22:59:02 +0300 Subject: [PATCH 08/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=205?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index 2b2e5927..1d56c62b 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -46,4 +46,5 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file + return ( + elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file From 8ff5dbf8c9235fe0b9058bb1135342229adf2db9 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 23:02:01 +0300 Subject: [PATCH 09/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=206?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index 1d56c62b..be626222 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -46,5 +46,4 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - return ( - elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file + return ( elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file From b913ec417b20e37f0e6557aed181e2a353a746a6 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 23:03:14 +0300 Subject: [PATCH 10/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=207?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index be626222..2b2e5927 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -46,4 +46,4 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - return ( elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file + return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file From c921b9f98c98bc47d83b9977c7551b2c404ebea8 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 23:10:43 +0300 Subject: [PATCH 11/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=208?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index 2b2e5927..9e92b7ad 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -46,4 +46,8 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file + return ( + (elapsed_ms << TIMESTAMP_SHIFT) + | (node_id << NODE_ID_SHIFT) + | sequence_id + ) \ No newline at end of file From 62f28bb8993b075a352f1f544936ada756b76e3f Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 23:12:19 +0300 Subject: [PATCH 12/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=209?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/snowflake.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index 9e92b7ad..2b2e5927 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -46,8 +46,4 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - return ( - (elapsed_ms << TIMESTAMP_SHIFT) - | (node_id << NODE_ID_SHIFT) - | sequence_id - ) \ No newline at end of file + return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file From 9bf797b29b9087e62af5d33489b386cabec0122f Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Thu, 24 Sep 2026 23:59:22 +0300 Subject: [PATCH 13/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=2010?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/constants.py | 20 ++++++++++++-------- src/part1/snowflake.py | 2 +- 2 files changed, 13 insertions(+), 9 deletions(-) diff --git a/src/part1/constants.py b/src/part1/constants.py index 05135d68..478de728 100644 --- a/src/part1/constants.py +++ b/src/part1/constants.py @@ -1,8 +1,12 @@ -EPOCH_MS_DEFAULT = 1288834974657 -NODE_ID_DEFAULT = 0 -TIMESTAMP_BITS, NODE_ID_BITS, SEQUENCE_ID_BITS = 41, 10, 12 -TIMESTAMP_MS_MAX = 2**TIMESTAMP_BITS - 1 -NODE_ID_MAX = 2**NODE_ID_BITS - 1 -SEQUENCE_ID_MAX = 2**SEQUENCE_ID_BITS - 1 -NODE_ID_SHIFT = SEQUENCE_ID_BITS -TIMESTAMP_SHIFT = SEQUENCE_ID_BITS + NODE_ID_BITS +from typing import Final + +EPOCH_MS_DEFAULT: Final[int] = 1288834974657 +NODE_ID_DEFAULT: Final[int] = 0 +TIMESTAMP_BITS: Final[int] = 41 +NODE_ID_BITS: Final[int] = 10 +SEQUENCE_ID_BITS: Final[int] = 12 +TIMESTAMP_MS_MAX: Final[int] = 2**TIMESTAMP_BITS - 1 +NODE_ID_MAX: Final[int] = 2**NODE_ID_BITS - 1 +SEQUENCE_ID_MAX: Final[int] = 2**SEQUENCE_ID_BITS - 1 +NODE_ID_SHIFT: Final[int] = SEQUENCE_ID_BITS +TIMESTAMP_SHIFT: Final[int] = SEQUENCE_ID_BITS + NODE_ID_BITS diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index 2b2e5927..fc8a3fa9 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -46,4 +46,4 @@ def generate_snowflake_id( print(f"timestamp overflows: {elapsed_ms} > {TIMESTAMP_MS_MAX}") return None - return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id \ No newline at end of file + return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id From 97247760f0b296349774e56b6190bc72edcc9157 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 25 Sep 2026 00:03:31 +0300 Subject: [PATCH 14/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=2011?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part1/constants.py | 1 + 1 file changed, 1 insertion(+) diff --git a/src/part1/constants.py b/src/part1/constants.py index 478de728..0339d24f 100644 --- a/src/part1/constants.py +++ b/src/part1/constants.py @@ -1,5 +1,6 @@ from typing import Final + EPOCH_MS_DEFAULT: Final[int] = 1288834974657 NODE_ID_DEFAULT: Final[int] = 0 TIMESTAMP_BITS: Final[int] = 41 From 66079fe9790091d966b857772852e0aee75d70af Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 25 Sep 2026 00:28:49 +0300 Subject: [PATCH 15/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=2012?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- pyproject.toml | 7 +++++++ src/part1/__init__.py | 1 - src/part1/snowflake.py | 2 +- 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index 06fbe913..83913475 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -33,3 +33,10 @@ avoid-escape = true [tool.ruff.lint.isort] lines-after-imports = 2 split-on-trailing-comma = true + +[tool.mypy] +strict = true +mypy_path = "src" +namespace_packages = true +explicit_package_bases = true + diff --git a/src/part1/__init__.py b/src/part1/__init__.py index 242b07ce..e69de29b 100644 --- a/src/part1/__init__.py +++ b/src/part1/__init__.py @@ -1 +0,0 @@ -"""Homework 3: a Twitter Snowflake identifier generator and its bit layout.""" diff --git a/src/part1/snowflake.py b/src/part1/snowflake.py index fc8a3fa9..a3f3fe8f 100644 --- a/src/part1/snowflake.py +++ b/src/part1/snowflake.py @@ -1,6 +1,6 @@ import time -from constants import ( +from part1.constants import ( EPOCH_MS_DEFAULT, NODE_ID_DEFAULT, NODE_ID_MAX, From 67e0e5d2dd33e9212704f1d1b143f5576e39e78c Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 25 Sep 2026 18:58:46 +0300 Subject: [PATCH 16/23] FINAL --- src/part1/constants.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/part1/constants.py b/src/part1/constants.py index 0339d24f..09033e11 100644 --- a/src/part1/constants.py +++ b/src/part1/constants.py @@ -2,7 +2,7 @@ EPOCH_MS_DEFAULT: Final[int] = 1288834974657 -NODE_ID_DEFAULT: Final[int] = 0 +NODE_ID_DEFAULT: Final[int] = 1 TIMESTAMP_BITS: Final[int] = 41 NODE_ID_BITS: Final[int] = 10 SEQUENCE_ID_BITS: Final[int] = 12 From 514d4b060f3fba603dc27f57de60b2a45a656703 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 2 Oct 2026 08:59:33 +0300 Subject: [PATCH 17/23] 2 task --- src/part2/cart.py | 133 +++++++++++++++++++++++++++---------------- src/part2/crud.py | 107 +++++++++++----------------------- src/part2/storage.py | 8 +-- src/part2/utils.py | 19 ++----- 4 files changed, 123 insertions(+), 144 deletions(-) diff --git a/src/part2/cart.py b/src/part2/cart.py index 063f9aea..b98bfd49 100644 --- a/src/part2/cart.py +++ b/src/part2/cart.py @@ -11,10 +11,11 @@ from typing import Final -from .crud import read_product, update_product # noqa: F401 -from .storage import ( # noqa: F401 +from crud import read_product +from storage import ( NAME_INDEX, PRICE_INDEX, + PRODUCT_ID_INDEX, QUANTITY_INDEX, Product, ) @@ -22,9 +23,16 @@ type CartLine = tuple[int, int] -# TODO: задайте позиции полей внутри кортежа CartLine LINE_PRODUCT_ID_INDEX: Final = 0 -LINE_QUANTITY_INDEX: Final = 0 +LINE_QUANTITY_INDEX: Final = 1 + + +def find_cart_line(cart: list[CartLine], product_id: int) -> CartLine | None: + """Return the cart line for ``product_id``, or ``None`` if absent.""" + for line in cart: + if line[LINE_PRODUCT_ID_INDEX] == product_id: + return line + return None def add_to_cart( @@ -33,28 +41,40 @@ def add_to_cart( product_id: int, quantity: int, ) -> CartLine | None: - """Move ``quantity`` units of ``product_id`` from the store into ``cart``. - - The product is looked up in ``storage``. If it is missing, a message is - printed (by :func:`~src.part2.crud.read_product`) and nothing - changes. If the store holds fewer units than requested, an explanatory - message is printed and nothing changes. Otherwise the store record is - decremented by ``quantity`` and the cart line for ``product_id`` gains - ``quantity`` units -- a new line is created when the cart had none. - - Args: - storage: The product store to draw stock from; modified in place on - success. - cart: The cart to add the units to; modified in place on success. - product_id: The identifier of the product to add. - quantity: Number of units to move into the cart. - - Returns: - The cart line for ``product_id`` after the addition, or ``None`` - when the product is unknown or the store cannot cover the request. - """ - # TODO: реализуйте функцию - return (0, 0) + product = read_product(storage, product_id) + if product is None: + return None + + available = product[QUANTITY_INDEX] + if available < quantity: + print( + f"not enough stock for product {product_id}: " + f"{available} available, {quantity} requested" + ) + return None + + for index, item in enumerate(storage): + if item[PRODUCT_ID_INDEX] == product_id: + storage[index] = ( + item[PRODUCT_ID_INDEX], + item[NAME_INDEX], + item[PRICE_INDEX], + available - quantity, + ) + break + + for index, line in enumerate(cart): + if line[LINE_PRODUCT_ID_INDEX] == product_id: + updated = ( + product_id, + line[LINE_QUANTITY_INDEX] + quantity, + ) + cart[index] = updated + return updated + + new_line = (product_id, quantity) + cart.append(new_line) + return new_line def remove_from_cart( @@ -63,27 +83,40 @@ def remove_from_cart( product_id: int, quantity: int, ) -> CartLine | None: - """Move ``quantity`` units of ``product_id`` from ``cart`` back to the store. - - The cart line for ``product_id`` is looked up. If there is none, a - message is printed and nothing changes. If the line holds fewer units - than requested, an explanatory message is printed and nothing changes. - Otherwise the cart line loses ``quantity`` units -- the line is dropped - when it reaches zero -- and the store record gains ``quantity`` units - back. - - Args: - storage: The product store to return stock to; modified in place on - success. - cart: The cart to take the units from; modified in place on - success. - product_id: The identifier of the product to remove. - quantity: Number of units to move back into the store. - - Returns: - The cart line for ``product_id`` after the removal (a quantity of - zero means the line was dropped), or ``None`` when the cart has no - line for the product or holds too few units. - """ - # TODO: реализуйте функцию - return (0, 0) + line = find_cart_line(cart, product_id) + if line is None: + print(f"product {product_id} is not in the cart") + return None + + cart_qty = line[LINE_QUANTITY_INDEX] + if cart_qty < quantity: + print( + f"cart holds only {cart_qty} unit(s) of product {product_id}, " + f"cannot remove {quantity}" + ) + return None + + product = read_product(storage, product_id) + if product is None: + return None + + for index, item in enumerate(storage): + if item[PRODUCT_ID_INDEX] == product_id: + storage[index] = ( + item[PRODUCT_ID_INDEX], + item[NAME_INDEX], + item[PRICE_INDEX], + item[QUANTITY_INDEX] + quantity, + ) + break + + for index, cart_line in enumerate(cart): + if cart_line[LINE_PRODUCT_ID_INDEX] == product_id: + remaining = cart_qty - quantity + if remaining == 0: + del cart[index] + return (product_id, 0) + updated = (product_id, remaining) + cart[index] = updated + return updated + return None diff --git a/src/part2/crud.py b/src/part2/crud.py index d026cbcb..9dc92d52 100644 --- a/src/part2/crud.py +++ b/src/part2/crud.py @@ -14,66 +14,40 @@ from decimal import Decimal -from .storage import ( # noqa: F401 +from storage import ( NAME_INDEX, PRODUCT_ID_INDEX, PRODUCT_ID_MIN, Product, ) -from .utils import normalize_price # noqa: F401 +from utils import normalize_price def generate_product_id(storage: list[Product]) -> int: - """Choose the identifier for the next product added to ``storage``. - - Args: - storage: The product store to inspect. - - Returns: - One past the greatest identifier currently held in ``storage``, or - :data:`~src.part2.storage.PRODUCT_ID_MIN` when ``storage`` is - empty. - """ - # TODO: реализуйте функцию - return 0 + if not storage: + return PRODUCT_ID_MIN + return max(p[PRODUCT_ID_INDEX] for p in storage) + 1 def create_product( storage: list[Product], fields: tuple[str, Decimal, int] ) -> int | None: - """Append a new product to ``storage`` and return its new identifier. - - Args: - storage: The product store to append to; modified in place on - success. - fields: A ``(name, price, quantity)`` tuple describing the product. - ``price`` is a :class:`~decimal.Decimal` amount and is rounded - to the stored money precision before it is saved. - - Returns: - The identifier generated for the new product, or ``None`` when a - product with the same name already exists. In the ``None`` case - ``storage`` is left unchanged and a message naming the clashing - name is printed. - """ - # TODO: реализуйте функцию - return 0 + name, price, quantity = fields + if any(p[NAME_INDEX] == name for p in storage): + print(f"product name '{name}' is already taken") + return None + product_id = generate_product_id(storage) + storage.append((product_id, name, normalize_price(price), quantity)) + return product_id def read_product(storage: list[Product], product_id: int) -> Product | None: - """Return the product stored under ``product_id``. + for product in storage: + if product[0] == product_id: + return product - Args: - storage: The product store to search. - product_id: The identifier to look up. - - Returns: - The matching ``(product_id, name, price, quantity)`` record, or - ``None`` when no product carries that identifier (a message is - printed in that case). - """ - # TODO: реализуйте функцию - return (0, "", Decimal(0), 0) + print(f"no product with id {product_id}") + return None def update_product( @@ -81,40 +55,23 @@ def update_product( product_id: int, fields: tuple[str, Decimal, int], ) -> Product | None: - """Overwrite the fields of the product stored under ``product_id``. - - The identifier itself is preserved; only ``name``, ``price`` and - ``quantity`` are replaced. + name, price, quantity = fields - Args: - storage: The product store to modify; the matching record is - replaced in place on success. - product_id: The identifier of the product to change. - fields: A ``(name, price, quantity)`` tuple with the new values. - ``price`` is a :class:`~decimal.Decimal` amount and is rounded - to the stored money precision before it is saved. + for index, product in enumerate(storage): + if product[0] == product_id: + updated = (product_id, name, normalize_price(price), quantity) + storage[index] = updated + return updated - Returns: - The updated ``(product_id, name, price, quantity)`` record, or - ``None`` when no product carries that identifier (``storage`` is - left unchanged and a message is printed). - """ - # TODO: реализуйте функцию - return (0, "", Decimal(0), 0) + print(f"no product with id {product_id}") + return None def delete_product(storage: list[Product], product_id: int) -> int | None: - """Remove the product stored under ``product_id`` from ``storage``. - - Args: - storage: The product store to remove from; modified in place on - success. - product_id: The identifier of the product to remove. - - Returns: - ``product_id`` when a product was removed, or ``None`` when no - product carried that identifier (``storage`` is left unchanged and - a message is printed). - """ - # TODO: реализуйте функцию - return 0 + for index, product in enumerate(storage): + if product[0] == product_id: + del storage[index] + return product_id + + print(f"no product with id {product_id}") + return None diff --git a/src/part2/storage.py b/src/part2/storage.py index b63d5fc7..35821c5f 100644 --- a/src/part2/storage.py +++ b/src/part2/storage.py @@ -26,9 +26,9 @@ # TODO: задайте позиции полей внутри кортежа Product PRODUCT_ID_INDEX: Final = 0 -NAME_INDEX: Final = 0 -PRICE_INDEX: Final = 0 -QUANTITY_INDEX: Final = 0 +NAME_INDEX: Final = 1 +PRICE_INDEX: Final = 2 +QUANTITY_INDEX: Final = 3 # TODO: задайте идентификатор первого товара в пустом хранилище -PRODUCT_ID_MIN: Final[int] = 0 +PRODUCT_ID_MIN: Final[int] = 1 diff --git a/src/part2/utils.py b/src/part2/utils.py index ec0348d3..955a239f 100644 --- a/src/part2/utils.py +++ b/src/part2/utils.py @@ -6,24 +6,13 @@ currency values free of binary floating-point error. """ -from decimal import ROUND_HALF_UP, Decimal # noqa: F401 +from decimal import ROUND_HALF_UP, Decimal from typing import Final -# TODO: задайте число знаков после запятой и шаг квантования -PRICE_PRECISION: Final[int] = 0 -PRICE_STEP: Final = Decimal(0) +PRICE_PRECISION: Final[int] = 2 +PRICE_STEP: Final = Decimal(1).scaleb(-PRICE_PRECISION) def normalize_price(price: Decimal) -> Decimal: - """Round a price to the precision every stored record uses. - - Args: - price: The raw price amount. - - Returns: - ``price`` quantised to :data:`PRICE_PRECISION` fractional digits, - with halves rounded up. - """ - # TODO: реализуйте функцию - return Decimal(0) + return price.quantize(PRICE_STEP, rounding=ROUND_HALF_UP) From d60611c7bf1348cd16d46125756ae227020a28b9 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 2 Oct 2026 09:24:23 +0300 Subject: [PATCH 18/23] =?UTF-8?q?=D0=9F=D0=BE=D0=BF=D1=8B=D1=82=D0=BA?= =?UTF-8?q?=D0=B0=202?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- pyproject.toml | 2 +- src/part2/cart.py | 8 ++++---- src/part2/storage.py | 8 ++++---- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index 83913475..a7400a1c 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -39,4 +39,4 @@ strict = true mypy_path = "src" namespace_packages = true explicit_package_bases = true - +mypy_path = "src/part2" diff --git a/src/part2/cart.py b/src/part2/cart.py index b98bfd49..6704d5d6 100644 --- a/src/part2/cart.py +++ b/src/part2/cart.py @@ -11,8 +11,8 @@ from typing import Final -from crud import read_product -from storage import ( +from src.part2.crud import read_product +from src.part2.storage import ( NAME_INDEX, PRICE_INDEX, PRODUCT_ID_INDEX, @@ -23,8 +23,8 @@ type CartLine = tuple[int, int] -LINE_PRODUCT_ID_INDEX: Final = 0 -LINE_QUANTITY_INDEX: Final = 1 +LINE_PRODUCT_ID_INDEX: Final[int] = 0 +LINE_QUANTITY_INDEX: Final[int] = 1 def find_cart_line(cart: list[CartLine], product_id: int) -> CartLine | None: diff --git a/src/part2/storage.py b/src/part2/storage.py index 35821c5f..73cc4464 100644 --- a/src/part2/storage.py +++ b/src/part2/storage.py @@ -25,10 +25,10 @@ type Product = tuple[int, str, Decimal, int] # TODO: задайте позиции полей внутри кортежа Product -PRODUCT_ID_INDEX: Final = 0 -NAME_INDEX: Final = 1 -PRICE_INDEX: Final = 2 -QUANTITY_INDEX: Final = 3 +PRODUCT_ID_INDEX: Final[int] = 0 +NAME_INDEX: Final[int] = 1 +PRICE_INDEX: Final[int] = 2 +QUANTITY_INDEX: Final[int] = 3 # TODO: задайте идентификатор первого товара в пустом хранилище PRODUCT_ID_MIN: Final[int] = 1 From 58f91e08d6fd22e507402f04a18996e08907d0b0 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 2 Oct 2026 09:32:59 +0300 Subject: [PATCH 19/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=203?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- pyproject.toml | 1 - 1 file changed, 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index a7400a1c..f115ba0f 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -36,7 +36,6 @@ split-on-trailing-comma = true [tool.mypy] strict = true -mypy_path = "src" namespace_packages = true explicit_package_bases = true mypy_path = "src/part2" From 807a12403b5d3e78d9c1aa0978008d7fc7b28c01 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 2 Oct 2026 09:44:44 +0300 Subject: [PATCH 20/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=203?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- pyproject.toml | 2 +- src/part2/cart.py | 10 +++++----- src/part2/crud.py | 4 ++-- src/part2/storage.py | 10 +++++----- 4 files changed, 13 insertions(+), 13 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index f115ba0f..b43c018e 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -38,4 +38,4 @@ split-on-trailing-comma = true strict = true namespace_packages = true explicit_package_bases = true -mypy_path = "src/part2" +mypy_path = "src" \ No newline at end of file diff --git a/src/part2/cart.py b/src/part2/cart.py index 6704d5d6..87d853f6 100644 --- a/src/part2/cart.py +++ b/src/part2/cart.py @@ -9,10 +9,10 @@ an explanatory message to stdout and returns ``None``. """ -from typing import Final +from typing import Final, Literal -from src.part2.crud import read_product -from src.part2.storage import ( +from part2.crud import read_product +from part2.storage import ( NAME_INDEX, PRICE_INDEX, PRODUCT_ID_INDEX, @@ -23,8 +23,8 @@ type CartLine = tuple[int, int] -LINE_PRODUCT_ID_INDEX: Final[int] = 0 -LINE_QUANTITY_INDEX: Final[int] = 1 +LINE_PRODUCT_ID_INDEX: Final[Literal[0]] = 0 +LINE_QUANTITY_INDEX: Final[Literal[1]] = 1 def find_cart_line(cart: list[CartLine], product_id: int) -> CartLine | None: diff --git a/src/part2/crud.py b/src/part2/crud.py index 9dc92d52..feec1212 100644 --- a/src/part2/crud.py +++ b/src/part2/crud.py @@ -14,13 +14,13 @@ from decimal import Decimal -from storage import ( +from part2.storage import ( NAME_INDEX, PRODUCT_ID_INDEX, PRODUCT_ID_MIN, Product, ) -from utils import normalize_price +from part2.utils import normalize_price def generate_product_id(storage: list[Product]) -> int: diff --git a/src/part2/storage.py b/src/part2/storage.py index 73cc4464..893b1a61 100644 --- a/src/part2/storage.py +++ b/src/part2/storage.py @@ -19,16 +19,16 @@ """ from decimal import Decimal -from typing import Final +from typing import Final, Literal type Product = tuple[int, str, Decimal, int] # TODO: задайте позиции полей внутри кортежа Product -PRODUCT_ID_INDEX: Final[int] = 0 -NAME_INDEX: Final[int] = 1 -PRICE_INDEX: Final[int] = 2 -QUANTITY_INDEX: Final[int] = 3 +PRODUCT_ID_INDEX: Final[Literal[0]] = 0 +NAME_INDEX: Final[Literal[1]] = 1 +PRICE_INDEX: Final[Literal[2]] = 2 +QUANTITY_INDEX: Final[Literal[3]] = 3 # TODO: задайте идентификатор первого товара в пустом хранилище PRODUCT_ID_MIN: Final[int] = 1 From abb738e4791a215df748cce673feba293a90ad54 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 2 Oct 2026 09:49:10 +0300 Subject: [PATCH 21/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=204?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- pyproject.toml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index b43c018e..9baf12f7 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -38,4 +38,7 @@ split-on-trailing-comma = true strict = true namespace_packages = true explicit_package_bases = true -mypy_path = "src" \ No newline at end of file +mypy_path = "src" + +[tool.ruff.lint] +ignore = ["PYI064"] \ No newline at end of file From 893d974289d5e14de6359f3807d88fc4cd3d1402 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 2 Oct 2026 09:51:32 +0300 Subject: [PATCH 22/23] =?UTF-8?q?=D0=92=D0=B0=D1=80=D0=B8=D0=B0=D0=BD?= =?UTF-8?q?=D1=82=205?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- pyproject.toml | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index 9baf12f7..33695916 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -23,6 +23,7 @@ skip-magic-trailing-comma = false [tool.ruff.lint] extend-select = ["I", "Q"] +per-file-ignores = { "src/part2/storage.py" = ["PYI064"], "src/part2/cart.py" = ["PYI064"] } [tool.ruff.lint.flake8-quotes] inline-quotes = "double" @@ -39,6 +40,3 @@ strict = true namespace_packages = true explicit_package_bases = true mypy_path = "src" - -[tool.ruff.lint] -ignore = ["PYI064"] \ No newline at end of file From 1dfb32c0bd7c19e04cd209248a07615206b4d205 Mon Sep 17 00:00:00 2001 From: SimencaSs <74181505+SimencaSs@users.noreply.github.com> Date: Fri, 9 Oct 2026 22:05:32 +0300 Subject: [PATCH 23/23] =?UTF-8?q?=D0=9D=D0=BE=D1=80=D0=BC=D0=B0=D0=BB?= =?UTF-8?q?=D0=B8=D0=B7=D0=B0=D1=86=D0=B8=D1=8F=20=D0=B8=D0=BC=D1=91=D0=BD?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/part3/TASK.md | 486 +++++++++++++++++++++++++++++++++++++++++ src/part3/__init__.py | 1 + src/part3/cart.py | 111 ++++++++++ src/part3/crud.py | 80 +++++++ src/part3/run_admin.py | 112 ++++++++++ src/part3/storage.py | 12 + src/part3/utils.py | 39 ++++ 7 files changed, 841 insertions(+) create mode 100644 src/part3/TASK.md create mode 100644 src/part3/__init__.py create mode 100644 src/part3/cart.py create mode 100644 src/part3/crud.py create mode 100644 src/part3/run_admin.py create mode 100644 src/part3/storage.py create mode 100644 src/part3/utils.py diff --git a/src/part3/TASK.md b/src/part3/TASK.md new file mode 100644 index 00000000..c8fedd8c --- /dev/null +++ b/src/part3/TASK.md @@ -0,0 +1,486 @@ +# Задача: нормализация имён, печать хранилища таблицей и админ-консоль + +## Легенда + +CRUD-функции из [части 2](../part2/TASK.md) уже умеют создавать, читать, +изменять и удалять товары, но пользоваться ими можно только из кода: чтобы +добавить в каталог одну позицию или посмотреть, что лежит на складе, приходится +писать скрипт. Для администратора магазина это неудобно, а при первых же +реальных данных вылезают новые проблемы: + +- один и тот же товар заводят несколько раз под разными написаниями — + `"Cordless Drill"`, `"cordless drill"`, `" cordless drill "` — и в + каталоге появляются дубликаты, которые невозможно найти по имени; +- содержимое хранилища выводится как «сырой» список кортежей, в котором трудно + что-либо разобрать, особенно когда товаров становится больше нескольких; +- любое действие над каталогом требует запуска отдельного кода, хотя + администратору нужен простой интерактивный инструмент. + +Чтобы решить эти задачи, мы научим хранилище приводить имена товаров к единому +виду и следить за их уникальностью, отрисовывать себя выровненной текстовой +таблицей и построим над ним консоль администратора — маленькую программу, в +которой команды вводятся строками, а результат сразу печатается на экране. + +## Контекст + +Часть 3 продолжает [часть 2](../part2/TASK.md) — тот же интернет-магазин, то же +хранилище товаров (список кортежей `Product` в оперативной памяти) и те же +CRUD-операции над ним. Корзина (`cart.py`) в этой части не используется. То, что +в этой части не меняется, берите из своего решения части 2. Новых возможностей +три: + +- имя товара приводится к канонической форме перед сохранением и сравнением; +- хранилище можно отрисовать выровненной текстовой таблицей; +- появляется интерактивная консоль администратора над этим хранилищем. + +Модули: + +- `storage.py` — используйте своё решение части 2 (форма записи `Product`, + константы `*_INDEX`, `PRODUCT_ID_MIN`); +- `utils.py` — к существующему округлению цен (`PRICE_PRECISION`, `PRICE_STEP`, + `normalize_price` — используйте своё решение части 2) добавляются + нормализация имени и отрисовка таблицы; +- `crud.py` — `generate_product_id`, `read_product`, `delete_product` — + используйте свои решения части 2; `create_product` и `update_product` + дорабатываются (см. ниже); +- `run_admin.py` — **новый модуль**, консоль администратора. + +Всё, что не изменилось по сравнению с частью 2, здесь не описывается — +реализуйте это так, как вы делали в своём решении части 2 (см. +[описание части 2](../part2/TASK.md)). + +> **Важно.** Тексты, которые функции печатают в stdout или возвращают как +> строку (сообщения об ошибках, таблица хранилища, ответы консоли), в тестах +> проверяются **дословно**: лишний пробел, другой регистр, пропущенная точка или +> перевод строки считаются ошибкой. Берите тексты из описания и примеров как +> есть. + +--- + +## Модуль `utils.py` + +### Константа `TABLE_HEADERS` + +Кортеж заголовков столбцов таблицы, слева направо: +`("ID", "name", "price", "quantity")`. Аннотирован `typing.Final`. Ширина +столбцов здесь не фиксируется — она измеряется по данным в момент вызова +`get_storage_str_representation`. + +### `normalize_product_name(name: str) -> str` + +Приводит сырое имя товара к единственному написанию, которое хранит склад. + +- **Вход:** `name` — произвольная строка. +- **Выход:** та же строка без начальных и конечных пробельных символов; + каждая внутренняя последовательность пробельных символов (пробелы, + табуляции и т. п.) заменяется на один пробел; всё остальное переведено в + нижний регистр. Если в строке нет ничего, кроме пробельных символов, — + возвращается пустая строка. + +Примеры: `" Cordless Drill " → "cordless drill"`, +`"abc def\tghi\tjkl" → "abc def ghi jkl"`, `" \t\n " → ""`, +`"claw hammer" → "claw hammer"` (уже канонично — не меняется). + +### `get_storage_str_representation(storage: list[Product]) -> str` + +Отрисовывает всё хранилище одной мультистрокой — выровненной таблицей, +пригодной для печати в терминал. + +«Выровненная» значит, что при печати моноширинным шрифтом все строки таблицы +имеют одинаковую длину, а разделители `|` стоят строго друг под другом: +значения в каждом столбце начинаются с одной и той же позиции, а ширина столбца +подбирается под самое длинное из его значений. Сравните: так выглядит вывод +хранилища, если просто напечатать значения через пробел: + +``` +ID name price quantity +10 compressor 1299.99 3 +2 nail 2.50 500 +``` + +А так он выглядит после выравнивания: + +``` +| ID | name | price | quantity | +|----|------------|---------|----------| +| 10 | compressor | 1299.99 | 3 | +| 2 | nail | 2.50 | 500 | +``` + +Точные правила выравнивания приведены в разделе «Формат» ниже. + +- **Вход:** `storage` — хранилище товаров. +- **Выход:** мультистрока (строки соединены `"\n"`, завершающего + перевода строки нет): + + 1. строка заголовков — значения из `TABLE_HEADERS`; + 2. разделительная строка из дефисов; + 3. по одной строке на каждый товар из `storage`, **в том порядке, в котором + хранилище их держит**. + + Когда `storage` пусто — возвращаются только строки 1 и 2. + +**Формат.** + +- Значения ячеек товара берутся как `str(...)` от полей записи в порядке + `TABLE_HEADERS`: `str(id)`, имя как есть, `str(price)` (у `Decimal`, уже + нормализованного, это, например, `"9.90"`, `"12.00"`, `"1299.99"`), + `str(quantity)`. +- Ширина каждого столбца равна длине самого длинного значения в этом столбце, + причём метка заголовка из `TABLE_HEADERS` тоже участвует в подсчёте. Поэтому + столбец никогда не уже своего заголовка, в том числе на пустом хранилище. +- Ячейка выравнивается по левому краю и дополняется пробелами до ширины + столбца; вокруг каждой ячейки — по одному пробелу отступа. Строка данных и + строка заголовков начинаются и заканчиваются символом `|`, соседние ячейки + тоже разделяются `|`. +- В разделительной строке каждый прогон дефисов на два символа длиннее ширины + своего столбца (закрывает отступы по краям ячейки); прогоны разделены `|`, + строка начинается и заканчивается `|`. + +Пустое хранилище отрисовывается ровно так: + +``` +| ID | name | price | quantity | +|----|------|-------|----------| +``` + +Хранилище `[(1, "saw", Decimal("9.90"), 5), (2, "axe", Decimal("12.00"), 8)]`: + +``` +| ID | name | price | quantity | +|----|------|-------|----------| +| 1 | saw | 9.90 | 5 | +| 2 | axe | 12.00 | 8 | +``` + +Хранилище `[(10, "compressor", Decimal("1299.99"), 3), (2, "nail", Decimal("2.50"), 500)]` +(столбцы расширяются под самое длинное значение): + +``` +| ID | name | price | quantity | +|----|------------|---------|----------| +| 10 | compressor | 1299.99 | 3 | +| 2 | nail | 2.50 | 500 | +``` + +--- + +## Модуль `crud.py` + +Остальные функции модуля (`generate_product_id`, `read_product`, +`delete_product`) реализуйте как в своём решении части 2. Изменяются только две +функции, описанные ниже; их сигнатуры прежние. + +### `create_product(storage: list[Product], fields: tuple[str, Decimal, int]) -> int | None` + +Порядок действий (при первой же неудаче хранилище не меняется, печатается +сообщение, возвращается `None`): + +1. Имя из `fields` приводится через `normalize_product_name`. Дальше везде — + и при сравнении, и при сохранении — используется нормализованное имя. +2. Если нормализованное имя пустое — печатается сообщение + `product name must not be blank`, возвращается `None`. +3. Если в хранилище уже есть запись с таким же **нормализованным** именем — + печатается сообщение, содержащее это имя + (`product name '<имя>' is already taken`), возвращается `None`. Сравнение + идёт по канонической форме, поэтому `" Cordless Drill "` (иной регистр, + лишние пробелы по краям и между словами) конфликтует с уже сохранённым + `"cordless drill"`. +4. Иначе — как в части 2: кортеж вида `(идентификатор, нормализованное имя, + нормализованная цена, количество)` добавляется в хранилище, возвращается + идентификатор. Значение идентификатора должно быть сгенерировано + с помощью функции `generate_product_id`, цена нормализована с помощью + функции `normalize_price`. + +**Примеры.** Каждый пример — отдельный фрагмент скрипта, который начинается с +пустого хранилища. В комментариях указано, что функция возвращает или печатает. + +Успешное создание: имя нормализуется, цена округляется, идентификаторы +выдаются по порядку: + +```python +storage: list[Product] = [] + +first_id = create_product(storage, (" Cordless Drill ", Decimal("89.999"), 12)) +second_id = create_product(storage, ("Claw Hammer", Decimal("9.9"), 40)) + +print(first_id) # 1 +print(second_id) # 2 +print(storage) +# [(1, 'cordless drill', Decimal('90.00'), 12), +# (2, 'claw hammer', Decimal('9.90'), 40)] +``` + +Имя занято (сравнение идёт по нормализованной форме), хранилище не меняется: + +```python +storage: list[Product] = [] + +create_product(storage, ("cordless drill", Decimal("50"), 3)) +result = create_product(storage, (" CORDLESS DRILL ", Decimal("75"), 1)) +# печатает: product name 'cordless drill' is already taken + +print(result) # None +print(storage) # [(1, 'cordless drill', Decimal('50.00'), 3)] +``` + +Имя из одних пробелов, хранилище не меняется: + +```python +storage: list[Product] = [] + +result = create_product(storage, (" ", Decimal("10"), 5)) +# печатает: product name must not be blank + +print(result) # None +print(storage) # [] +``` + +### `update_product(storage: list[Product], product_id: int, fields: tuple[str, Decimal, int]) -> Product | None` + +Порядок действий: + +1. Имя из `fields` приводится через `normalize_product_name`. +2. Если нормализованное имя пустое — печатается сообщение + `product name must not be blank`, возвращается `None`, хранилище не + меняется. Эта проверка выполняется **до** поиска записи по идентификатору. +3. Если записи с `product_id` нет — печатается сообщение с идентификатором + (`no product with id `), возвращается `None`, хранилище не меняется. +4. Если нормализованное имя уже занято **другим** товаром (запись с таким + же нормализованным именем и другим `product_id`) — печатается сообщение, + содержащее это имя (`product name '<имя>' is already taken`), возвращается + `None`, хранилище не меняется. Переименование товара в его же текущее имя + (и обновление только цены/количества) конфликтом не считается. +5. Иначе запись заменяется на `(product_id, нормализованное имя, + normalize_price(цена), количество)`; идентификатор сохраняется прежним. + Возвращается изменённая запись. + +**Примеры.** Каждый пример — отдельный фрагмент скрипта, который начинается с +хранилища из двух товаров: + +```python +storage: list[Product] = [ + (1, "cordless drill", Decimal("90.00"), 12), + (2, "claw hammer", Decimal("9.90"), 40), +] +``` + +Успешное изменение: имя нормализуется, цена округляется, идентификатор +сохраняется: + +```python +result = update_product(storage, 2, (" Rubber Mallet ", Decimal("7.505"), 25)) + +print(result) # (2, 'rubber mallet', Decimal('7.51'), 25) +print(storage) +# [(1, 'cordless drill', Decimal('90.00'), 12), +# (2, 'rubber mallet', Decimal('7.51'), 25)] +``` + +Обновление товара под его же именем (даже в другом написании) конфликтом не +считается — меняются только цена и количество: + +```python +result = update_product(storage, 1, ("CORDLESS Drill", Decimal("85"), 10)) + +print(result) # (1, 'cordless drill', Decimal('85.00'), 10) +``` + +Имя уже занято другим товаром, хранилище не меняется: + +```python +result = update_product(storage, 1, (" Claw Hammer ", Decimal("85"), 10)) +# печатает: product name 'claw hammer' is already taken + +print(result) # None +print(storage) +# [(1, 'cordless drill', Decimal('90.00'), 12), +# (2, 'claw hammer', Decimal('9.90'), 40)] +``` + +Товара с таким идентификатором нет, а пустое имя отвергается раньше, чем +выполняется поиск по идентификатору: + +```python +result = update_product(storage, 99, ("saw", Decimal("20"), 5)) +# печатает: no product with id 99 +print(result) # None + +result = update_product(storage, 99, (" ", Decimal("20"), 5)) +# печатает: product name must not be blank (а не no product with id 99) +print(result) # None +``` + +--- + +## Модуль `run_admin.py` (новый) + +Интерактивная консоль администратора над одним хранилищем товаров, которое +живёт в памяти всё время работы процесса. Цикл «прочитать строку — выполнить — +напечатать результат» продолжается, пока не введена команда `exit`. + +Пример сессии (после `admin> ` идёт ввод пользователя, остальное печатает +программа): + +``` +admin> create Cordless Drill 89.99 12 +1 +admin> create claw hammer 9.9 40 +2 +admin> show +| ID | name | price | quantity | +|----|----------------|-------|----------| +| 1 | cordless drill | 89.99 | 12 | +| 2 | claw hammer | 9.90 | 40 | +admin> read 5 +no product with id 5 +admin> exit +``` + +### Константы + +- `PROMPT: Final[str] = "admin> "` — приглашение, печатается перед чтением + каждой строки ввода. +- `HELP_TEXT: Final[str]` — многострочный текст-справка: перечень всех команд с + кратким описанием каждой и пояснением, что для `create` и `update` цена и + количество — это два последних слова строки, а всё между ними — имя товара + (поэтому имя может содержать пробелы и не требует кавычек). + + Пример текста справки: + + ``` + Available commands: + help show this message + exit leave the console + show print the whole store as a table + create add a product, print its new id + read print the product with that id + update overwrite that product's fields + delete remove the product with that id + + For create and update the price and quantity are the last two words of the + line; everything before them is the product name, so it may contain spaces + (for example "Gibson SG Junior") and needs no quoting. + ``` + +### Разбор команд + +Команда определяется по строке, разбитой по пробельным символам. +Распознаются следующие формы (число слов должно совпадать точно): + +| Форма ввода | Действие | +| --- | --- | +| `help` | напечатать `HELP_TEXT` | +| `exit` | завершить цикл | +| `show` | напечатать всё хранилище выравненной таблицей | +| `create <имя...> <цена> <количество>` | добавить товар; `<имя...>` — одно или больше слов | +| `read ` | напечатать один товар | +| `update <имя...> <цена> <количество>` | перезаписать поля товара; `<имя...>` — одно или больше слов | +| `delete ` | удалить товар | + +`<имя...>` может состоять из произвольного количества слов (но не меньше +одного), разделённых пробелами; слова склеиваются через один пробел в имя +товара. Все остальные параметры команд (``, `<цена>`, `<количество>`) — ровно +по одному слову. Поэтому `create` и `update` без слов имени (`create 89.90 12`) +командой не считаются. + +Разобранные аргументы приводятся к типам: `` и `<количество>` — через +`int(...)` (десятичное целое), `<цена>` — через `Decimal(...)`. Получившийся +кортеж `(имя, цена, количество)` передаётся в соответствующую функцию `crud.py`. + +### `show_help() -> None` + +Печатает `HELP_TEXT` в stdout как есть. + +### `print_result(result: object) -> None` + +- **Вход:** `result` — значение, вернувшееся из CRUD-операции. +- **Поведение:** если `result` не `None` — печатает `result` в stdout; если + `None` — не печатает ничего (операция уже сообщила о своей неудаче сама). + +### `run_command(storage: list[Product], line: str) -> bool` + +Разбирает одну строку консоли и выполняет названную ею команду. + +- **Вход:** `storage` — общее хранилище сессии (команды `create`, `update`, + `delete` меняют его на месте); `line` — одна строка ввода, уже очищенная от + окружающих пробелов. +- **Выход:** `True` — цикл нужно продолжать; `False` — введена команда `exit`. +- **Поведение по формам:** + - `help` — печатается справка; цикл продолжается; + - `exit` — цикл нужно остановить; + - `show` — печатается таблица со всем хранилищем; цикл продолжается; + - `create`, `read`, `update`, `delete` — выполняется соответствующая + CRUD-операция с разобранными аргументами, её результат выводится на экран + (если операция ничего не вернула, не выводится ничего); цикл продолжается; + - строка, которая не подошла ни под одну форму — неизвестная команда + (`frobnicate ...`) либо известная команда с неверным числом аргументов + (`read 1 2 3`) — печатается сообщение `'<строка>' is not a command`; + хранилище не меняется; вернуть `True`. + +### `main() -> None` + +Запускает цикл до команды `exit`. В шаблоне задания эта функция **уже +реализована** — писать её не нужно; ниже описано её поведение, чтобы вы понимали, +как консоль использует `run_command`. + +- Заводит одно пустое хранилище `storage` и держит его в памяти всю сессию. +- На каждой итерации: читает строку через `input(PROMPT)`, очищает её от + окружающих пробелов, передаёт в `run_command`. Значение `run_command` + становится флагом продолжения цикла. +- `ValueError` и `decimal.InvalidOperation` из `run_command` перехватываются + здесь: печатается сообщение вида + `'<строка>' names a command but its arguments are invalid`, хранилище не + меняется, цикл продолжается. +- Цикл останавливает только `exit`. + +### Как запустить консоль + +Откройте терминал **в корне репозитория** (там, где лежит `pyproject.toml`) и +выполните: + +``` +uv run python -m src.part3.run_admin +``` + +Если `uv` у вас не установлен, см. [инструкцию](../../manuals/local_checks.md); +вместо него можно использовать обычный Python 3.12 или новее: + +``` +python -m src.part3.run_admin +``` + +После запуска появится приглашение `admin> `. Введите `help`, чтобы увидеть +список команд (после того как вы реализуете `show_help()`), и `exit`, чтобы +выйти. Хранилище живёт только пока работает программа: при следующем запуске оно +снова пустое. + +Обратите внимание: + +- запускать нужно именно как модуль (`-m src.part3.run_admin`, имя модуля с + точками, без `.py`). Запуск файла напрямую (`python src/part3/run_admin.py`) + завершится ошибкой `ImportError: attempted relative import with no known + parent package`, потому что модули пакета импортируют друг друга; +- запуск из другой папки (например, из `src/part3`) тоже не сработает — модуль + `src` в этом случае не будет найден; +- если вы запускаете скрипт из PyCharm или VS Code, настройте запуск «как + модуль» (параметр `-m` / `module`) с тем же именем `src.part3.run_admin` и + рабочей папкой в корне репозитория. + +--- + +## Инварианты + +- Всякое сохранённое имя каноническое: `create_product` и `update_product` + пропускают имя через `normalize_product_name` и при сравнении, и при записи, + поэтому в хранилище не встречаются имена с окружающими пробелами, с + несколькими пробельными символами подряд внутри или с буквами верхнего + регистра. +- Имена уникальны с точностью до нормализации: два имени, совпадающие после + `normalize_product_name`, не могут оба быть в хранилище. +- `get_storage_str_representation` не теряет и не переставляет товары: тело + таблицы — это записи `storage` в исходном порядке, по строке на запись; ни + одно значение не обрезается (каждый столбец не уже самого длинного своего + значения). +- Для непустого `storage` в выводе `get_storage_str_representation` все строки + имеют одинаковую длину, а «столбики» `|` стоят друг под другом. \ No newline at end of file diff --git a/src/part3/__init__.py b/src/part3/__init__.py new file mode 100644 index 00000000..e953a49e --- /dev/null +++ b/src/part3/__init__.py @@ -0,0 +1 @@ +"""Homework 4: a CRUD interface over an in-memory list-of-tuples product store.""" diff --git a/src/part3/cart.py b/src/part3/cart.py new file mode 100644 index 00000000..0bb651e5 --- /dev/null +++ b/src/part3/cart.py @@ -0,0 +1,111 @@ +from typing import Final + +from .crud import read_product +from .storage import ( + NAME_INDEX, + PRICE_INDEX, + PRODUCT_ID_INDEX, + QUANTITY_INDEX, + Product, +) + + +type CartLine = tuple[int, int] + +LINE_PRODUCT_ID_INDEX: Final = 0 +LINE_QUANTITY_INDEX: Final = 1 + + +def find_cart_line(cart: list[CartLine], product_id: int) -> CartLine | None: + """Return the cart line for ``product_id``, or ``None`` if absent.""" + for line in cart: + if line[LINE_PRODUCT_ID_INDEX] == product_id: + return line + return None + + +def add_to_cart( + storage: list[Product], + cart: list[CartLine], + product_id: int, + quantity: int, +) -> CartLine | None: + product = read_product(storage, product_id) + if product is None: + return None + + available = product[QUANTITY_INDEX] + if available < quantity: + print( + f"not enough stock for product {product_id}: " + f"{available} available, {quantity} requested" + ) + return None + + for index, item in enumerate(storage): + if item[PRODUCT_ID_INDEX] == product_id: + storage[index] = ( + item[PRODUCT_ID_INDEX], + item[NAME_INDEX], + item[PRICE_INDEX], + available - quantity, + ) + break + + for index, line in enumerate(cart): + if line[LINE_PRODUCT_ID_INDEX] == product_id: + updated = ( + product_id, + line[LINE_QUANTITY_INDEX] + quantity, + ) + cart[index] = updated + return updated + + new_line = (product_id, quantity) + cart.append(new_line) + return new_line + + +def remove_from_cart( + storage: list[Product], + cart: list[CartLine], + product_id: int, + quantity: int, +) -> CartLine | None: + line = find_cart_line(cart, product_id) + if line is None: + print(f"product {product_id} is not in the cart") + return None + + cart_qty = line[LINE_QUANTITY_INDEX] + if cart_qty < quantity: + print( + f"cart holds only {cart_qty} unit(s) of product {product_id}, " + f"cannot remove {quantity}" + ) + return None + + product = read_product(storage, product_id) + if product is None: + return None + + for index, item in enumerate(storage): + if item[PRODUCT_ID_INDEX] == product_id: + storage[index] = ( + item[PRODUCT_ID_INDEX], + item[NAME_INDEX], + item[PRICE_INDEX], + item[QUANTITY_INDEX] + quantity, + ) + break + + for index, cart_line in enumerate(cart): + if cart_line[LINE_PRODUCT_ID_INDEX] == product_id: + remaining = cart_qty - quantity + if remaining == 0: + del cart[index] + return (product_id, 0) + updated = (product_id, remaining) + cart[index] = updated + return updated + return None diff --git a/src/part3/crud.py b/src/part3/crud.py new file mode 100644 index 00000000..11de16ed --- /dev/null +++ b/src/part3/crud.py @@ -0,0 +1,80 @@ +from decimal import Decimal + +from .storage import ( + NAME_INDEX, + PRODUCT_ID_INDEX, + PRODUCT_ID_MIN, + Product, +) +from .utils import normalize_price, normalize_product_name + + +def generate_product_id(storage: list[Product]) -> int: + if not storage: + return PRODUCT_ID_MIN + return max(p[PRODUCT_ID_INDEX] for p in storage) + 1 + + +def create_product( + storage: list[Product], fields: tuple[str, Decimal, int] +) -> int | None: + name, price, quantity = fields + name = normalize_product_name(name) + if not name: + print("product name must not be blank") + return None + if any(p[NAME_INDEX] == name for p in storage): + print(f"product name '{name}' is already taken") + return None + product_id = generate_product_id(storage) + storage.append((product_id, name, normalize_price(price), quantity)) + return product_id + + +def read_product(storage: list[Product], product_id: int) -> Product | None: + for product in storage: + if product[PRODUCT_ID_INDEX] == product_id: + return product + + print(f"no product with id {product_id}") + return None + + +def update_product( + storage: list[Product], + product_id: int, + fields: tuple[str, Decimal, int], +) -> Product | None: + name, price, quantity = fields + name = normalize_product_name(name) + if not name: + print("product name must not be blank") + return None + + index = None + for i, product in enumerate(storage): + if product[PRODUCT_ID_INDEX] == product_id: + index = i + break + + if index is None: + print(f"no product with id {product_id}") + return None + + for product in storage: + if product[NAME_INDEX] == name and product[PRODUCT_ID_INDEX] != product_id: + print(f"product name '{name}' is already taken") + return None + updated = (product_id, name, normalize_price(price), quantity) + storage[index] = updated + return updated + + +def delete_product(storage: list[Product], product_id: int) -> int | None: + for index, product in enumerate(storage): + if product[PRODUCT_ID_INDEX] == product_id: + del storage[index] + return product_id + + print(f"no product with id {product_id}") + return None diff --git a/src/part3/run_admin.py b/src/part3/run_admin.py new file mode 100644 index 00000000..5608f2e3 --- /dev/null +++ b/src/part3/run_admin.py @@ -0,0 +1,112 @@ +from decimal import Decimal, InvalidOperation +from typing import Final + +from .crud import ( + create_product, + delete_product, + read_product, + update_product, +) +from .storage import Product +from .utils import get_storage_str_representation + + +PROMPT: Final[str] = "admin> " +HELP_TEXT: Final[str] = """Available commands: + help show this message + exit leave the console + show print the whole store as a table + create add a product, print its new id + read print the product with that id + update overwrite that product's fields + delete remove the product with that id + +For create and update the price and quantity are the last two words of the +line; everything before them is the product name, so it may contain spaces +(for example "Gibson SG Junior") and needs no quoting.""" + + +def show_help() -> None: + print(HELP_TEXT) + + +def print_result(result: object) -> None: + if result is not None: + print(result) + + +def run_command(storage: list[Product], line: str) -> bool: + words = line.split() + if not words: + return True + + command = words[0] + + if command == "help" and len(words) == 1: + show_help() + return True + + if command == "exit" and len(words) == 1: + return False + + if command == "show" and len(words) == 1: + print_result(get_storage_str_representation(storage)) + return True + + if command == "create": + if len(words) < 4: + print(f"{line!r} is not a command") + return True + name = " ".join(words[1:-2]) + price = Decimal(words[-2]) + quantity = int(words[-1]) + print_result(create_product(storage, (name, price, quantity))) + return True + + if command == "read": + if len(words) != 2: + print(f"{line!r} is not a command") + return True + product_id = int(words[1]) + print_result(read_product(storage, product_id)) + return True + + if command == "update": + if len(words) < 5: + print(f"{line!r} is not a command") + return True + product_id = int(words[1]) + name = " ".join(words[2:-2]) + price = Decimal(words[-2]) + quantity = int(words[-1]) + print_result(update_product(storage, product_id, (name, price, quantity))) + return True + + if command == "delete": + if len(words) != 2: + print(f"{line!r} is not a command") + return True + product_id = int(words[1]) + print_result(delete_product(storage, product_id)) + return True + + print(f"{line!r} is not a command") + return True + + +def main() -> None: + storage: list[Product] = [] + running = True + + while running: + line = input(PROMPT).strip() + + try: + running = run_command(storage, line) + + except (ValueError, InvalidOperation): + print(f"{line!r} names a command but its arguments are invalid") + + +if __name__ == "__main__": + main() diff --git a/src/part3/storage.py b/src/part3/storage.py new file mode 100644 index 00000000..81651a75 --- /dev/null +++ b/src/part3/storage.py @@ -0,0 +1,12 @@ +from decimal import Decimal +from typing import Final + + +type Product = tuple[int, str, Decimal, int] + +PRODUCT_ID_INDEX: Final = 0 +NAME_INDEX: Final = 1 +PRICE_INDEX: Final = 2 +QUANTITY_INDEX: Final = 3 + +PRODUCT_ID_MIN: Final[int] = 1 diff --git a/src/part3/utils.py b/src/part3/utils.py new file mode 100644 index 00000000..9b2b1a72 --- /dev/null +++ b/src/part3/utils.py @@ -0,0 +1,39 @@ +from decimal import ROUND_HALF_UP, Decimal +from typing import Final + +from .storage import Product + + +PRICE_PRECISION: Final[int] = 2 +PRICE_STEP: Final[Decimal] = Decimal(1).scaleb(-PRICE_PRECISION) +TABLE_HEADERS: Final[tuple[str, str, str, str]] = ("ID", "name", "price", "quantity") + + +def normalize_price(price: Decimal) -> Decimal: + return price.quantize(PRICE_STEP, rounding=ROUND_HALF_UP) + + +def normalize_product_name(name: str) -> str: + return " ".join(name.split()).lower() + + +def get_storage_str_representation(storage: list[Product]) -> str: + rows: list[tuple[str, str, str, str]] = [ + (str(product[0]), product[1], str(product[2]), str(product[3])) + for product in storage + ] + widths: list[int] = [ + max(len(header), *(len(row[i]) for row in rows)) if rows else len(header) + for i, header in enumerate(TABLE_HEADERS) + ] + header_line = ( + "|" + + "|".join(f" {cell:<{width}} " for cell, width in zip(TABLE_HEADERS, widths)) + + "|" + ) + dop_line = "|" + "|".join("-" * (width + 2) for width in widths) + "|" + data_lines = [ + "|" + "|".join(f" {cell:<{width}} " for cell, width in zip(row, widths)) + "|" + for row in rows + ] + return "\n".join([header_line, dop_line, *data_lines])