Показать сообщение отдельно

  #6  
Старый 24.09.2007, 23:27
sni4ok
Участник форума
Регистрация: 04.11.2006
Сообщений: 150
С нами: 10271420

Репутация: 175
По умолчанию

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

собсно это то, что попалось при беглом просмотре, удачи в развитии)
 
Ответить с цитированием