Repository navigation
2. Устинов Семён Михайлович. 614 #78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
SimencaSs
wants to merge
24
commits into
python-department:main
Choose a base branch
from
SimencaSs:completed-task-2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
d615eda
hhh
SimencaSs e24ec21
Готовый код
SimencaSs bc9c936
Готовый код 2
SimencaSs c08ad70
Версия 3
SimencaSs a09a31c
Версия 4
SimencaSs 7069e22
Версия 4
SimencaSs 40434a0
Update snowflake.py
SimencaSs e4cc97b
Вариант 5
SimencaSs 8ff5dbf
Вариант 6
SimencaSs b913ec4
Вариант 7
SimencaSs c921b9f
Вариант 8
SimencaSs 62f28bb
Вариант 9
SimencaSs 9bf797b
Вариант 10
SimencaSs 9724776
Вариант 11
SimencaSs 66079fe
Вариант 12
SimencaSs 67e0e5d
FINAL
SimencaSs 25f91a9
Merge branch 'python-department:main' into main
SimencaSs 514d4b0
2 task
SimencaSs d60611c
Попытка 2
SimencaSs 58f91e0
Вариант 3
SimencaSs 807a124
Вариант 3
SimencaSs abb738e
Вариант 4
SimencaSs 893d974
Вариант 5
SimencaSs 1dfb32c
Нормализация имён
SimencaSs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +0,0 @@ | ||
| """Homework 3: a Twitter Snowflake identifier generator and its bit layout.""" | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,38 +1,13 @@ | ||
| """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: Final[int] = 1288834974657 | ||
| NODE_ID_DEFAULT: Final[int] = 1 | ||
| 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,114 +1,49 @@ | ||
| """Twitter Snowflake identifier generator. | ||
| import time | ||
|
|
||
| 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`, | ||
| :func:`decode_node_id` and :func:`decode_sequence_id`. | ||
| """ | ||
|
|
||
| import time # noqa: F401 | ||
|
|
||
| from .constants import ( # noqa: F401 | ||
| from part1.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: | ||
| """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() // 1000000 - 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, | ||
| ) -> int | None: | ||
| """Build and return a Snowflake identifier for the current millisecond. | ||
| if not (0 <= node_id <= NODE_ID_MAX): | ||
| print(f"node_id must be in [0, {NODE_ID_MAX}], got {node_id}") | ||
| return None | ||
|
|
||
| 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 <= 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 | ||
| return (elapsed_ms << TIMESTAMP_SHIFT) | (node_id << NODE_ID_SHIFT) | sequence_id |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -9,22 +9,30 @@ | |
| an explanatory message to stdout and returns ``None``. | ||
| """ | ||
|
|
||
| from typing import Final | ||
| from typing import Final, Literal | ||
|
|
||
| from .crud import read_product, update_product # noqa: F401 | ||
| from .storage import ( # noqa: F401 | ||
| from part2.crud import read_product | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Вы поменяли относительные импорты на абсолютные. Из-за этого тесты сломались. |
||
| from part2.storage import ( | ||
| NAME_INDEX, | ||
| PRICE_INDEX, | ||
| PRODUCT_ID_INDEX, | ||
| QUANTITY_INDEX, | ||
| Product, | ||
| ) | ||
|
|
||
|
|
||
| type CartLine = tuple[int, int] | ||
|
|
||
| # TODO: задайте позиции полей внутри кортежа CartLine | ||
| LINE_PRODUCT_ID_INDEX: Final = 0 | ||
| LINE_QUANTITY_INDEX: Final = 0 | ||
| 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: | ||
| """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 | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Так делать нельзя. Нельзя добавлять игнорирование ошибок линтера