Решение на От ливадите до Лас Вегас (и назад) от Александър Сариков
Към профила на Александър Сариков
Резултати
- 10 точки от тестове
- 0 бонус точки
- 10 точки общо
- 15 успешни тест(а)
- 0 неуспешни тест(а)
Код
Лог от изпълнението
............... ---------------------------------------------------------------------- Ran 15 tests in 0.162s OK
История (5 версии и 9 коментара)
Александър обнови решението на 05.11.2022 08:19 (преди над 2 години)
Използването на две долни черти тук според мен е излишно. Една стига. Не мисля, че би искал да си толкова стриктен в това дали някой пипа в този лист. НО, това не е никакъв проблем. така си решил - така си направил. Тук мисля, че мнението ми е по-скоро субективно.
Реално погледнато този минус преди r
е излишен. Все пак работиш със случайно число - слагайки минус просто променяш случайното число, но то пак си е случайно :)
self.__cards.extend(cards)
прави същото.
Да. Дори бих казал, че слуачйният индекс е излишен. Ако просто върнеш self.__players
ще опростиш класа си, а реално този случай е частен случай на твоя, така че не внасяш никаква промяна. От функционална гледна точка, смятам, че дори е по-логично всеки път да връщаш един и същи ред.
С този if
бих определил посоката, т.е. стъпката 1/-1 след което просто да използвам посоката в цикъла на ред 112.
if self.dealing_direction == 'rtl':
step = -1
else:
step = 1
# Помисли дори как можеш да опростиш горния израз :)
...
for p in self__players[::step]:
...
Избягвай еднобукени имена за променливи.
Тази проверка е излишна. Щом си казал, че методът ти е private, ти си в пълен контрол на инпута. В случая подаваш self.number_of_players
като инпут. Не е нужно да проверяваш дали е по-малко, или не.
Слагай по един празен ред след class ....
реда
Добро решение!
Генерално бих те посъветвал да украсиш с docstring-ове. Казахме, че не държим на това, но това е единственото, което ми липсва. Харесва ми, че си слагал коментари. Всички са напълно намясто и си направил правилните предположения.
Използването на две долни черти тук според мен е излишно. Една стига. Не мисля, че би искал да си толкова стриктен в това дали някой пипа в този лист. НО, това не е никакъв проблем. така си решил - така си направил. Тук мисля, че мнението ми е по-скоро субективно.
Слагай по един празен ред след
class ....
реда