Решение на От ливадите до Лас Вегас (и назад) от Теодора Петкова
Към профила на Теодора Петкова
Резултати
- 10 точки от тестове
- 0 бонус точки
- 10 точки общо
- 15 успешни тест(а)
- 0 неуспешни тест(а)
Код
Лог от изпълнението
............... ---------------------------------------------------------------------- Ran 15 tests in 0.166s OK
История (4 версии и 12 коментара)
Теодора обнови решението на 13.11.2022 15:05 (преди над 2 години)
Спрямо PEP8, докстринговете трябва да се дефинират в """
, без интервали след отварящите кавички и преди затварящите, и завършващи с точка.
Има изисквания и за дължината на дефиницията. Не е нужно да го преправяш. Просто имай предвид за следващия път.
Бих използвал protected имена на атрибути като тези, които не използваш извън класа - self._suit
А ако е None
? Тестето остава празно, а не трябва да има игра без тесте. Дори да не си наследила този клас, той пак трябва да инициализира стандартно тесте.
Моля не използвай еднобуквени променливи.
Можеш да минеш и без дефиниране на двете временни променливи. Само споменавам. Не е проблем.
return self.cards.pop()
ще направи същото.
Ако няма да използваш променливата, именувай я _
, както е по конвенция.
Освен това, range(0, x)
е същото като range(x)
.
Не мисля, че си заслужава да инициализираш ново тесте при всяко раздаване. Можеш просто да изпразниш старото. Реално и така ще работи, но е прахосване на ресурси.
start_index
е по-добре. Иначе може да се интерпретира като start
, standard
, string
...
Тук отново са валидни коментарите ми за еднобуквено променливи и range()
. Останалото ми изглежда ок.
Теодора обнови решението на 14.11.2022 15:09 (преди над 2 години)
Ще е проблем ли, ако направя самото Game да създава пълно тесте, когато се инициализира или е задължително да е в случая с None на Deck.
Deck
трябва да работи самостоятелно, без да се имат предвид игрите по-долу. Тестовете ни ще разчитат, че при инициализиране на Deck
имаме карти, с които да изтестваме класа. Инициализиране на Deck
трябва да обект с готово за работа стандартното тесте от 52 карти.
А ако е
None
? Тестето остава празно, а не трябва да има игра без тесте. Дори да не си наследила този клас, той пак трябва да инициализира стандартно тесте.Ще е проблем ли, ако направя самото Game да създава пълно тесте, когато се инициализира или е задължително да е в случая с None на Deck.
Deck
трябва да работи самостоятелно, без да се имат предвид игрите по-долу. Тестовете ни ще разчитат, че при инициализиране наDeck
имаме карти, с които да изтестваме класа. Инициализиране наDeck
трябва да обект с готово за работа стандартното тесте от 52 карти.Можеш да минеш и без дефиниране на двете временни променливи. Само споменавам. Не е проблем.
Не мисля, че си заслужава да инициализираш ново тесте при всяко раздаване. Можеш просто да изпразниш старото. Реално и така ще работи, но е прахосване на ресурси.