
24.09.2007, 23:27
|
|
Участник форума
Регистрация: 04.11.2006
Сообщений: 150
С нами:
10271420
Репутация:
175
|
|
замечания по поводу кода:
1. довольна слабая обработка ошибок, в частности везде предпологается, что запрашиваемая память всегда выделится, обработки исключений нет,
нету даже обработки ошибок вызова connect.
2. обработка команд реализована большим свичём- это очень не удобно,и в случае расширения функциональности будет очень сильно мешать, стоит заменить на фактори. Уже сейчас размер этого свича на 250 строчек.. т.е не читаемо.
3. весь код вообще не подразумевает о том, что где-то могут возникнуть исключения, опять таки это чревато последствиями, т.е даже если пишешь (пока) без исключений, то освобождать ресурсы(крит.секция, память, сокеты и пр.) лучше всёже в деструкторе их владельцев, а не вызовом функций напрямую.
собсно это то, что попалось при беглом просмотре, удачи в развитии)
|
|
|