Решение на Шахматни фенове от Александър Сариков
Към профила на Александър Сариков
Резултати
- 10 точки от тестове
- 1 бонус точка
- 11 точки общо
- 17 успешни тест(а)
- 0 неуспешни тест(а)
Код
Лог от изпълнението
................. ---------------------------------------------------------------------- Ran 17 tests in 0.165s OK
История (4 версии и 9 коментара)
Александър обнови решението на 23.11.2022 13:54 (преди почти 2 години)
Александър обнови решението на 23.11.2022 14:09 (преди почти 2 години)
Въобще да не те притеснява това name. Това показва откъде идва дефиницята на ексепшъна. Няма нужда да го закачаш.
Реално погледнато тук нищо не правиш, така че и без този метод, постигаш същото.
В крайна сметка един pass
в тялото на класа е достатъчен.
Бих използвал малки букви. Да, абревиатура е, но главните подсказват константа.
Двойните долни черти според мен за излишни. Една стига. https://fmi.py-bg.net/tips/2
Бих използвал None
за тези дефиниции. Както сам си написал по-долу, вече сме сигурни, че има точно по един цар от двата цвята, така че дори не би трябвало да се притесняваме за тези два реда. Можеш да ги пропуснеш и да скочиш на циклите, които ще ги инициализират с правилни стойности.
Ако направиш малко аритметика около индексите, можеш да спестиш този дълъг ред и пренасянето с \
, което въобще не толерирам.
Тук RegEx-а е излишен. self.FEN.count('k')
дава същото.
Въпреки това, приветствам факта, че тренираш RegEx след вчерашната лекция.
Това е прекалено обфускирано :(
По-добре го раздели на няколко реда.
Правилно си го интерпретирал - трябва да е списък. Инструкцията в примера е грешна и я оправих. Получаваш една бонус точка за тази ремарка.