🇧S
flake8 поставь
Size: a a a
2020 May 25
i
ну как то так :) это мое первое тестовое :)
Ну а чего ожидал, что ослепишь своим кодом с первого раза? ))) В принципе получилось))))
А
ilya
Ну а чего ожидал, что ослепишь своим кодом с первого раза? ))) В принципе получилось))))
я ничего не ожидал, мне просто была интересна обратная связь, а ее не дали...вот и попросил помощи у вас
MB
pep8 стоит
Потыкай wemake-python-styleguide, мб часть плагинов и правил приглянется:)
i
я ничего не ожидал, мне просто была интересна обратная связь, а ее не дали...вот и попросил помощи у вас
Основная проблема - логическая организация кода. Остальное на фоне уже не заметно
м
не увидел, где предусмотрены возможные сбои при записи на диск.
а чё там, при попытке записи эксепшн летит?
я не в курсе просто
я не в курсе просто
R
а чё там, при попытке записи эксепшн летит?
я не в курсе просто
я не в курсе просто
ну да по идее
А
Потыкай wemake-python-styleguide, мб часть плагинов и правил приглянется:)
ок, попробую глянуть
MO
ну в run.py идет сходу какая-то аццкая копипаста два одинаковых по сути куска
R
приветы. Подавал отклик на вакансию, прислали тестовое. Сделал, отправил. Сегодня прислали ответ, что не готовы пригласить на интервью, потому что не устроило качество выполнения тестового задания.
Ссылка на условия тестового: https://docs.google.com/document/d/1TbwwaWk6Oai-qpx7Jd59NIjxzKnva_42aYtpbydu8gQ/edit
Ссылка на мое решение: https://github.com/yelrik/medrating_test
Может кто-нибудь глянет, настолько там все плохо?
Ссылка на условия тестового: https://docs.google.com/document/d/1TbwwaWk6Oai-qpx7Jd59NIjxzKnva_42aYtpbydu8gQ/edit
Ссылка на мое решение: https://github.com/yelrik/medrating_test
Может кто-нибудь глянет, настолько там все плохо?
Вопрос а чего ты ожидал когда твоя реализация без прикрас это адовый говнокод ?
Ты вообще о линтерах слышал ?
Когда я увидел functions.py у меня глаз начал дергаться. 😖
Это тебе ещё мягко сказали что качество не устроило.
Ты вообще о линтерах слышал ?
Когда я увидел functions.py у меня глаз начал дергаться. 😖
Это тебе ещё мягко сказали что качество не устроило.
м
ну да по идее
на самом деле удобно, что в документации пишут об этом, жаль я не смог найти(
А
ну в run.py идет сходу какая-то аццкая копипаста два одинаковых по сути куска
ну почему адская? :) забыл исключение поправить. Если там одинаковый код, то почему не скопировать :)
А
Вопрос а чего ты ожидал когда твоя реализация без прикрас это адовый говнокод ?
Ты вообще о линтерах слышал ?
Когда я увидел functions.py у меня глаз начал дергаться. 😖
Это тебе ещё мягко сказали что качество не устроило.
Ты вообще о линтерах слышал ?
Когда я увидел functions.py у меня глаз начал дергаться. 😖
Это тебе ещё мягко сказали что качество не устроило.
я установил линтер с pep8
м
есть разные линтеры для разных задач
А
Вопрос а чего ты ожидал когда твоя реализация без прикрас это адовый говнокод ?
Ты вообще о линтерах слышал ?
Когда я увидел functions.py у меня глаз начал дергаться. 😖
Это тебе ещё мягко сказали что качество не устроило.
Ты вообще о линтерах слышал ?
Когда я увидел functions.py у меня глаз начал дергаться. 😖
Это тебе ещё мягко сказали что качество не устроило.
что именно не так там?
MB
ну почему адская? :) забыл исключение поправить. Если там одинаковый код, то почему не скопировать :)
i
ну почему адская? :) забыл исключение поправить. Если там одинаковый код, то почему не скопировать :)
😁👍 главное в нашем деле это не унывать
MO
ну почему адская? :) забыл исключение поправить. Если там одинаковый код, то почему не скопировать :)
прочитай какую-нибудь книжку по ооп в питоне, например
А
ilya
😁👍 главное в нашем деле это не унывать
ну а чего мне унывать, ошибки постараюсь учесть на будущее...не плакать же...Откуда я узнаю, что не правильно пишу...я нормально отношусь к критике, поэтому и попросить ревью