ilnar-geekbrains / Test_11_18

2 stars 94 forks source link

Выполненные задание Соломонов Д.В. #1

Closed Solomon89 closed 2 years ago

ilnar-geekbrains commented 2 years ago

Добрый день! Благодарю за присланное решение!

Приведу комментарии по решению: Есть ряд замечаний по неймингам (много опечаток), например, ethelon, rezult. В этом на занятиях точно нужно быть гораздо аккуратнее. Тесты вынесли в отдельные классы - это хорошо. Но в примере мы не пользовались ничем кроме if и самописных методов, например, для сравнения массивов. Было бы здорово реализовать это также (на базовом курсе мы пока дальше этого не идем). При отправке pr принято свои предложения пушить из отдельной ветки, а не из основной main/master. Но это уже детали. Не принимайте этот комментарий на свой счет, просто вы потратили время на выполнение тестового, я со своей стороны считаю важным дать обратную связь, не более того. По дальнейшим шагам с вами свяжется hr, спасибо. pr закрою в ближайшее время. Еще раз спасибо.