Решение на Астрологични забави от Калоян Витанов
Резултати
- 10 точки от тестове
- 1 отнета точка
- 9 точки общо
- 6 успешни тест(а)
- 0 неуспешни тест(а)
Код
Лог от изпълнението
...... ---------------------------------------------------------------------- Ran 6 tests in 0.006s OK
История (3 версии и 3 коментара)
Калоян обнови решението на 08.03.2015 22:07 (преди над 9 години)
Във втората версия има един space по-малко :P
Имам няколко забележки за стила ти.
Ако западните зодиите бяха 5000? Щеше ли да напишеш 5000 if-a? Не мислиш ли, че функцията за китайските зодии е прекалено сложна? Също така, прочети по - внимателно как се предава домашно. Наблегни на pep8.
Калоян обнови решението на 09.03.2015 22:37 (преди над 9 години)
Дойчине, благодаря за забележките ти! :) Изглежда бях разбрал погрешно някои от нещата в pep8, сега мисля съм оправил нещата. Ако зодиите бяха 5000 щях да помисля за друг начин. Впрочем даже и за 12-те помислих за друг начин, но понеже не съм програмирал нищо от доста време (аз съм ПМ 4-ти курс) и съм позабравил нещата, не се сетих и се задоволих с този. Напълно наложително ли е да направя нещата по друг начин и ако да може ли някакъв hint коя структура евентуално би свършила работа? Какво по- точно не харесваш във функцията за китайските зодии? Ако изключим предишната забележка като възможност то не съм сигурен, че те разбирам.