Ревьюером было предложено использованием именованных кортежей для читабельности кода. Почитал статью посмотрел видео в ютубе, но пока трудно с пониманием реализацией. Я так понимаю в моем случае его надо прописывать в методе transformations, и первое что я должен создать класс и его атрибуты. Например Competitors = namedtuple('Competitors', 'name pi fi'), а дальше я должен через созданную переменную Competitors создать экземпляры класса переданные в метод transformations. А как это сделать туплю и не могу понять что делать.
В метод tranformations передается кортеж с такими данными (<имя>, <числовое значение>, <числовое значение>) и возвращает тот же кортеж только в другом порядке.
def partition(competitors, left, right):
if right <= left:
return
pivot = (left + right) // 2
part = competitors[pivot]
begin = left
end = right
while begin <= end:
while part > competitors[begin]:
begin += 1
while part < competitors[end]:
end -= 1
if begin <= end:
competitors[begin], competitors[end] = competitors[
end], competitors[begin]
begin += 1
end -= 1
partition(competitors, left, end)
partition(competitors, begin, right)
def transformations(competitors):
competitors[1] = - int(competitors[1])
competitors[2] = int(competitors[2])
return [competitors[1], competitors[2], competitors[0]]
def quick_sort(competitors):
partition(competitors, 0, len(competitors) - 1)
return [line[2] for line in competitors]
if __name__ == '__main__':
number = int(input())
competitors = [transformations(input().split()) for _ in range(number)]
result = quick_sort(competitors)
print(*result, sep="\n")
transformations
все этиcompetitors[1]
и[competitors[1], competitors[2], competitors[0]]
- непонятно что там внутри и что в итоге. Поэтому вместо получения и возврата массива с индексами (которые ничего не означают для читателя) вы можете заменить их на словарь или классы (кортежи), у которых есть имена и код станет более читабельный. И не стоит забывать про типизированный способ создания кортежей черезfrom typing import NamedTuple
В метод tranformations передается кортеж с такими данными (<имя>, <числовое значение>, <числовое значение>) и возвращает тот же кортеж только в другом порядке.
- судя по тому, что вы в этой функции перезаписываете элементы competitors, то вы явно туда не кортеж передаете, а скорее всего список. На выходе почему-то первый элемент (имя?) оказывается последним. Если менять на namedtuple, то нужно быть готовым к тому, что это неизменяемая структура данных. Ну или вам на самом деле нужен класс (или датакласс), а не namedtuple.