Kuznetsov Artur task8

[i-am-dak0ta]

Изменения в тесты не вносил, но все проходило

Буду ждать комментарии о работе в ПР или же в ТГ

[mak-42]

Изменения в тесты не вносил, но все проходило

Не представляю, как это возможно. Сборка явно указывает на ошибки компиляции тестов из-за проблем с объявлением исключения "unreported exception croc.education.ws2023spb.knightsmove.IllegalPositionException; must be caught or declared to be thrown": https://github.com/I-am-DaKoTa/ws2023spb-knightsmove/actions/runs/6749448271/job/18349808044

[mak-42]

Код оставляет приятное впечатление. Из того, что особо понравилось - использование регулярного выражения для проверки позиции.

Указал на явные проблемы в местах их возникновения. Но стоит на них посмотреть и в других местах, т.к. некоторые повторяются.

Из того, что можно ещё доработать, но это выходит на рамки курса:

1. ConcreteKnightsMoveChecker не содержит состояния, потому при реализации KnightsMoveCheckerFactory можно воспользоваться шаблоном проектирования Singleton.

Коллеги подсказывают, что вы сдали задание на проверку позже контрольного срока. Пока единственное, что могу обещать, что проверю исправления.

[mak-42]

Спасибо за хорошую работу.