From 306f76dba5ca68d650c4fd032c556fa44de67b69 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=BC=D0=B0=D0=BA=D1=81=D0=B8=D0=BC?= Date: Fri, 9 Oct 2026 19:18:18 +0300 Subject: [PATCH] task3 done --- src/part2/cart.py | 67 +++++++++++++++++++++++++++++++++++++----- src/part2/crud.py | 45 ++++++++++++++++++++-------- src/part2/storage.py | 10 +++---- src/part2/utils.py | 10 +++---- src/part3/crud.py | 60 +++++++++++++++++++++++++++++-------- src/part3/run_admin.py | 58 +++++++++++++++++++++++++++++------- src/part3/storage.py | 11 +++---- src/part3/utils.py | 40 +++++++++++++++---------- 8 files changed, 224 insertions(+), 77 deletions(-) diff --git a/src/part2/cart.py b/src/part2/cart.py index 063f9aea..cf3ad751 100644 --- a/src/part2/cart.py +++ b/src/part2/cart.py @@ -11,8 +11,8 @@ from typing import Final -from .crud import read_product, update_product # noqa: F401 -from .storage import ( # noqa: F401 +from .crud import read_product, update_product +from .storage import ( NAME_INDEX, PRICE_INDEX, QUANTITY_INDEX, @@ -22,9 +22,15 @@ 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``, if present.""" + return next( + (line for line in cart if line[LINE_PRODUCT_ID_INDEX] == product_id), None + ) def add_to_cart( @@ -53,8 +59,30 @@ def add_to_cart( 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}: {available} available, " + f"{quantity} requested" + ) + return None + update_product( + storage, + product_id, + (product[NAME_INDEX], product[PRICE_INDEX], available - quantity), + ) + line = find_cart_line(cart, product_id) + if line is None: + line = (product_id, quantity) + cart.append(line) + return line + index = cart.index(line) + updated = (product_id, line[LINE_QUANTITY_INDEX] + quantity) + cart[index] = updated + return updated def remove_from_cart( @@ -85,5 +113,28 @@ def remove_from_cart( 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 + held = line[LINE_QUANTITY_INDEX] + if held < quantity: + print( + f"cart holds only {held} unit(s) of product {product_id}, cannot remove {quantity}" + ) + return None + product = read_product(storage, product_id) + if product is None: + return None + update_product( + storage, + product_id, + (product[NAME_INDEX], product[PRICE_INDEX], product[QUANTITY_INDEX] + quantity), + ) + remaining = held - quantity + index = cart.index(line) + if remaining == 0: + del cart[index] + else: + cart[index] = (product_id, remaining) + return (product_id, remaining) diff --git a/src/part2/crud.py b/src/part2/crud.py index d026cbcb..4e219a8e 100644 --- a/src/part2/crud.py +++ b/src/part2/crud.py @@ -14,13 +14,13 @@ 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: @@ -34,8 +34,11 @@ def generate_product_id(storage: list[Product]) -> int: :data:`~src.part2.storage.PRODUCT_ID_MIN` when ``storage`` is empty. """ - # TODO: реализуйте функцию - return 0 + return ( + max(product[PRODUCT_ID_INDEX] for product in storage) + 1 + if storage + else PRODUCT_ID_MIN + ) def create_product( @@ -56,8 +59,13 @@ def create_product( ``storage`` is left unchanged and a message naming the clashing name is printed. """ - # TODO: реализуйте функцию - return 0 + name, price, quantity = fields + if any(product[NAME_INDEX] == name for product in storage): + print(f"product name {name!r} 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: @@ -72,8 +80,11 @@ def read_product(storage: list[Product], product_id: int) -> Product | None: ``None`` when no product carries that identifier (a message is printed in that case). """ - # TODO: реализуйте функцию - return (0, "", Decimal(0), 0) + 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( @@ -99,8 +110,14 @@ def update_product( ``None`` when no product carries that identifier (``storage`` is left unchanged and a message is printed). """ - # TODO: реализуйте функцию - return (0, "", Decimal(0), 0) + name, price, quantity = fields + for index, product in enumerate(storage): + if product[PRODUCT_ID_INDEX] == product_id: + updated = (product_id, name, normalize_price(price), quantity) + storage[index] = updated + return updated + print(f"no product with id {product_id}") + return None def delete_product(storage: list[Product], product_id: int) -> int | None: @@ -116,5 +133,9 @@ def delete_product(storage: list[Product], product_id: int) -> int | None: product carried that identifier (``storage`` is left unchanged and a message is printed). """ - # TODO: реализуйте функцию - return 0 + 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/part2/storage.py b/src/part2/storage.py index b63d5fc7..e4bceff1 100644 --- a/src/part2/storage.py +++ b/src/part2/storage.py @@ -24,11 +24,9 @@ type Product = tuple[int, str, Decimal, int] -# 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 11bd7a76..6fbf19f7 100644 --- a/src/part2/utils.py +++ b/src/part2/utils.py @@ -6,13 +6,12 @@ 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: @@ -25,5 +24,4 @@ def normalize_price(price: Decimal) -> Decimal: ``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) diff --git a/src/part3/crud.py b/src/part3/crud.py index 68811d7e..e67db3f5 100644 --- a/src/part3/crud.py +++ b/src/part3/crud.py @@ -20,13 +20,13 @@ 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, normalize_product_name # noqa: F401 +from .utils import normalize_price, normalize_product_name def generate_product_id(storage: list[Product]) -> int: @@ -40,8 +40,11 @@ def generate_product_id(storage: list[Product]) -> int: :data:`~src.part3.storage.PRODUCT_ID_MIN` when ``storage`` is empty. """ - # TODO: реализуйте функцию (используйте своё решение части 2) - return 0 + return ( + max(product[PRODUCT_ID_INDEX] for product in storage) + 1 + if storage + else PRODUCT_ID_MIN + ) def create_product( @@ -64,8 +67,17 @@ def create_product( normalised name already exists. In the ``None`` case ``storage`` is left unchanged and an explanatory message is printed. """ - # TODO: реализуйте функцию - return 0 + name, price, quantity = fields + name = normalize_product_name(name) + if not name: + print("product name must not be blank") + return None + if any(product[NAME_INDEX] == name for product in storage): + print(f"product name {name!r} 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: @@ -80,8 +92,11 @@ def read_product(storage: list[Product], product_id: int) -> Product | None: ``None`` when no product carries that identifier (a message is printed in that case). """ - # TODO: реализуйте функцию (используйте своё решение части 2) - return (0, "", Decimal(0), 0) + 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( @@ -110,8 +125,25 @@ def update_product( unchanged and an explanatory message is printed. Renaming a product to its own current name is allowed. """ - # TODO: реализуйте функцию - return (0, "", Decimal(0), 0) + name, price, quantity = fields + name = normalize_product_name(name) + if not name: + print("product name must not be blank") + return None + for index, product in enumerate(storage): + if product[PRODUCT_ID_INDEX] != product_id: + continue + if any( + other[PRODUCT_ID_INDEX] != product_id and other[NAME_INDEX] == name + for other in storage + ): + print(f"product name {name!r} is already taken") + return None + updated = (product_id, name, normalize_price(price), quantity) + storage[index] = updated + return updated + print(f"no product with id {product_id}") + return None def delete_product(storage: list[Product], product_id: int) -> int | None: @@ -127,5 +159,9 @@ def delete_product(storage: list[Product], product_id: int) -> int | None: product carried that identifier (``storage`` is left unchanged and a message is printed). """ - # TODO: реализуйте функцию (используйте своё решение части 2) - return 0 + 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 index c5feeb0b..84e68bbb 100644 --- a/src/part3/run_admin.py +++ b/src/part3/run_admin.py @@ -29,22 +29,32 @@ it. """ -from decimal import Decimal, InvalidOperation # noqa: F401 +from decimal import Decimal, InvalidOperation from typing import Final -from .crud import ( # noqa: F401 +from .crud import ( create_product, delete_product, read_product, update_product, ) from .storage import Product -from .utils import get_storage_str_representation # noqa: F401 +from .utils import get_storage_str_representation -# TODO: задайте приглашение и текст справки -PROMPT: Final[str] = "" -HELP_TEXT: Final[str] = "" +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: @@ -54,7 +64,7 @@ def show_help() -> None: is: a heading, one line per command with a short description, and a note on how multi-word names are parsed. """ - # TODO: реализуйте функцию + print(HELP_TEXT) def print_result(result: object) -> None: @@ -65,7 +75,8 @@ def print_result(result: object) -> None: operation already reported its own failure, so nothing is printed in that case. """ - # TODO: реализуйте функцию + if result is not None: + print(result) def run_command(storage: list[Product], line: str) -> bool: @@ -88,8 +99,35 @@ def run_command(storage: list[Product], line: str) -> bool: decimal.InvalidOperation: If the price argument of ``create`` or ``update`` does not parse as a decimal number. """ - # TODO: реализуйте функцию - return False + words = line.split() + if words == ["help"]: + show_help() + elif words == ["exit"]: + return False + elif words == ["show"]: + print(get_storage_str_representation(storage)) + elif len(words) >= 4 and words[0] == "create": + name, price, quantity = ( + " ".join(words[1:-2]), + Decimal(words[-2]), + int(words[-1]), + ) + print_result(create_product(storage, (name, price, quantity))) + elif len(words) == 2 and words[0] == "read": + print_result(read_product(storage, int(words[1]))) + elif len(words) >= 5 and words[0] == "update": + product_id = int(words[1]) + name, price, quantity = ( + " ".join(words[2:-2]), + Decimal(words[-2]), + int(words[-1]), + ) + print_result(update_product(storage, product_id, (name, price, quantity))) + elif len(words) == 2 and words[0] == "delete": + print_result(delete_product(storage, int(words[1]))) + else: + print(f"{line!r} is not a command") + return True def main() -> None: diff --git a/src/part3/storage.py b/src/part3/storage.py index c5e34be5..c516645b 100644 --- a/src/part3/storage.py +++ b/src/part3/storage.py @@ -24,12 +24,9 @@ type Product = tuple[int, str, Decimal, int] -# TODO: задайте позиции полей внутри кортежа Product (используйте своё решение -# части 2) 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/part3/utils.py b/src/part3/utils.py index 318ded0b..09f61211 100644 --- a/src/part3/utils.py +++ b/src/part3/utils.py @@ -18,7 +18,7 @@ the longest value it holds in that particular call. """ -from decimal import ROUND_HALF_UP, Decimal # noqa: F401 +from decimal import ROUND_HALF_UP, Decimal from typing import Final from .storage import Product @@ -26,16 +26,13 @@ # Number of fractional digits every stored price is rounded to. # Quantisation step derived from PRICE_PRECISION, e.g. Decimal("0.01"). -# TODO: задайте число знаков после запятой и шаг квантования (используйте своё -# решение части 2) -PRICE_PRECISION: Final[int] = 0 -PRICE_STEP: Final = Decimal(0) +PRICE_PRECISION: Final[int] = 2 +PRICE_STEP: Final = Decimal(1).scaleb(-PRICE_PRECISION) # Column headers of the table produced by get_storage_str_representation, # left to right. The width of each column is not fixed here -- it is # measured per call from the data (see the function). -# TODO: задайте заголовки столбцов таблицы -TABLE_HEADERS: Final[tuple[str, ...]] = () +TABLE_HEADERS: Final[tuple[str, ...]] = ("ID", "name", "price", "quantity") def normalize_price(price: Decimal) -> Decimal: @@ -48,8 +45,7 @@ def normalize_price(price: Decimal) -> Decimal: ``price`` quantised to :data:`PRICE_PRECISION` fractional digits, with halves rounded up. """ - # TODO: реализуйте функцию (используйте своё решение части 2) - return Decimal(0) + return price.quantize(PRICE_STEP, rounding=ROUND_HALF_UP) def normalize_product_name(name: str) -> str: @@ -66,11 +62,7 @@ def normalize_product_name(name: str) -> str: The result is an empty string when ``name`` holds nothing but whitespace. """ - # TODO: реализуйте функцию - return "" - - -# TODO: при необходимости добавьте свои вспомогательные функции + return " ".join(name.split()).lower() def get_storage_str_representation(storage: list[Product]) -> str: @@ -91,5 +83,21 @@ def get_storage_str_representation(storage: list[Product]) -> str: is empty only the header and separator rows are returned, sized to the header labels. """ - # TODO: реализуйте функцию - return "" + rows = [TABLE_HEADERS, *(tuple(map(str, product)) for product in storage)] + widths = tuple( + max(len(row[index]) for row in rows) for index in range(len(TABLE_HEADERS)) + ) + + def format_row(row: tuple[str, ...]) -> str: + return ( + "|" + + "|".join( + f" {value:<{width}} " for value, width in zip(row, widths, strict=True) + ) + + "|" + ) + + separator = "|" + "|".join("-" * (width + 2) for width in widths) + "|" + return "\n".join( + (format_row(TABLE_HEADERS), separator, *(format_row(row) for row in rows[1:])) + )