Здравствуйте, The SochiX, Вы писали:
TS>Здравствуйте, уважаемые коллеги! Недавно искал работу, предложили выполнить задание для собеседования в AnyWayAnyDay. Кто не знает, это такая компания которая занимается поиском билетом, отелей и.т.д. Мое решение им не понравилось, почему не ответили. Собственно, прошу проревьюить мой код и сказать что с ним не так. Премного благодарен
Код не запускал, но в целом всё ок )
Пара моментов:
1) Непонятно, зачем в LocalHttpListener финализатор. У него с HttpListener (реализующий IDisposable) композиция, соответсвенно сам LocalHttpListener тоже должен его реализовывать, а SimpleServer — явно диспозить.
2) Следствие первого — непонятно, как SimpleServer.Stop() останавливает сервер. На что влияет установка флага isAlreadyStarted ?
3) SimpleServerException — чем не устроил стандартный Message? И нигде нету сохранения innerException