Димитър обнови решението на 14.11.2022 18:11 (преди около 2 години)
По-добре само една долна черта. Моля виж становището ми тук: https://fmi.py-bg.net/tips/2
Не е нужно да го преправяш сега. Просто имай предвид за рдуги проекти.
Смятам, че value
не е добро име за този аргумент. faces
седи по-добре.
Реално можеш да направиш една променлива, която определя лицата, за да спестиш повторение на двата блока от почти идентични редове.
Ако няма да използваш дадена променлива, именувай я _
, както е по конвенция.
Освен това range(0, x)
е същото като range(x)
.
Може и така. Смятам, че е по-чисто:
self.__cards = self.__cards[start:] + self.__cards[:start]
Този похват за справяне с различните тестета е доста хитър. Не видях някой друг да го използва.
Насилих се да измисля по-кратък начин за това, само и само да цитирам Стоичков - "first part good second part I'm not like":
self.__players[starting_index+1:][::-1]
Шегата настрана. Твоето решение е ок. Просто не се сдържах.
for player in first_part + second_part
и минаваш с един цикъл.
Заради чистото решение и идеята за _set_deck
получаваш бонус точка.
По-добре само една долна черта. Моля виж становището ми тук: https://fmi.py-bg.net/tips/2
Не е нужно да го преправяш сега. Просто имай предвид за рдуги проекти.
Смятам, че
value
не е добро име за този аргумент.faces
седи по-добре.Реално можеш да направиш една променлива, която определя лицата, за да спестиш повторение на двата блока от почти идентични редове.
Ако няма да използваш дадена променлива, именувай я
_
, както е по конвенция.Освен това
range(0, x)
е същото катоrange(x)
.Може и така. Смятам, че е по-чисто:
self.__cards = self.__cards[start:] + self.__cards[:start]
Този похват за справяне с различните тестета е доста хитър. Не видях някой друг да го използва.
Насилих се да измисля по-кратък начин за това, само и само да цитирам Стоичков - "first part good second part I'm not like":
self.__players[starting_index+1:][::-1]
Шегата настрана. Твоето решение е ок. Просто не се сдържах.
for player in first_part + second_part
и минаваш с един цикъл.