Skip to content

решение задачи - #1

Open
rybakovegor59 wants to merge 15 commits into
Fork3141592:mainfrom
rybakovegor59:main
Open

решение задачи#1
rybakovegor59 wants to merge 15 commits into
Fork3141592:mainfrom
rybakovegor59:main

Conversation

@rybakovegor59

Copy link
Copy Markdown

задача1.1

Comment on lines +11 to +12
if kind == "copy":
second = original.copy()

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Решение хорошее, но не следует использовать вещи, которых нет в теории или которые явно не прописаны в задании. Попробуй реализовать данную часть без метода copy()

Comment on lines +9 to +13
case (x, 0):
return "on-x"
case (0, y):
return "on-y"
case (x, y):

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вместо переменных в этом случае лучше использовать wildcard, так как на не нужны далее сохранённые значения

Comment on lines +36 to +37
else:
return None

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ветка else здесь излишня, так как раньше не вводилось никакое ветвление

Comment on lines +14 to +15
def leaderboard(rows: list[tuple[str, int]]) -> list[tuple[str, int]]:
return sorted(rows,key = lambda x: (-x[1],x[0]))

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Решение хорошее, но хочу также увидеть вариант, который показывает именно свойство стабильности сортировки

Comment on lines +21 to +31
def caesar(s: str, k: int) -> str:
alfa = "abcdefghijklmnopqrstuvwxyz"
s_new = ""

for i in s:
if i in alfa:
i_new = (alfa.index(i) + k) % 26
s_new += alfa[i_new]
else:
s_new += i
return s_new

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Решение считаю неверным, т.к. функция занимает слишком много памяти из-за постоянного создания новых строк.

Comment on lines +36 to +37
def is_sorted_words(words: list[str]) -> bool:
return sorted(words)== words

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Решение рабочее, но при огромных списках не оптимальное, ведь работает за время: O(n log n) и память O(n). есть решение оптимальнее.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants