АГ>>template <typename T>
АГ>>void some_function(const T& original)
АГ>>{
АГ>>  try
АГ>>  {
АГ>>    T copy = original;
АГ>>    // other work
АГ>>  }
АГ>>  catch (...)
АГ>>  {
АГ>>    // cleanup
АГ>>    throw;
АГ>>  }
АГ>>}
АГ>>

AS>try{
AS>  //do some work
AS>}
AS>catch(...){
AS>  //do some cleanup
AS>    throw;
AS>}
AS>

B>...обычно-то — упали с AV, взяли полученный dmp-файл, открыли в студии и смотрим как дошли до такой жизни....

IK>try{
IK>  //do some work
IK>}
IK>catch(Exception e){
IK>  //do some cleanup
IK>    throw;
IK>}
IK>

 AS> try{
 AS>   //do some work
 AS> }
 AS> catch(...){
 AS>   //do some cleanup
 AS>  throw;
 AS> }
 AS>

 AS> void some_class::foo()
 AS> {
 AS>   try{
 AS>     //do some work
 AS>   }
 AS>   catch(...){
 AS>     //log the message - there was error during some_class::foo
 AS>     throw;
 AS>   }
 AS> }
 AS>

 AS> void some_class::foo()
 AS> {
 AS>   struct local{
 AS>     bool _ok;
 AS>     local(){_ok = false;}
 AS>     do_work(){    
 AS>       //do some work
 AS>       _ok = true;
 AS>     }
 AS>     ~local(){
 AS>       if (!_ok){
 AS>         //log the message - there was error during some_class::foo
 AS>       }
 AS>     }
 AS>   } l;l.do_work();
 AS> }
 AS>

IK> catch (Exception e)
IK> {
IK> }
IK> catch (int i)
IK> {
IK> }
IK>

АГ>template <typename T>
АГ>void some_function(const T& original)
АГ>{
АГ>  try
АГ>  {
АГ>    T copy = original;
АГ>    // other work
АГ>  }
АГ>  catch (...)
АГ>  {
АГ>    // cleanup
АГ>    throw;
АГ>  }
АГ>}
АГ>

AS>void some_class::foo()
AS>{
AS>  struct local{
AS>    bool _ok;
AS>    local(){_ok = false;}
AS>    do_work(){          
AS>      //do some work
AS>      _ok = true;
AS>    }
AS>    ~local(){
AS>      if (!_ok){
AS>        //log the message - there was error during some_class::foo
AS>      }
AS>    }
AS>  } l;l.do_work();
AS>}
AS>

GUI>- юный возраст
GUI>- использование компонентов .NET (такое тестовое задание данному рецензенту попалось впервые);
GUI>

GUI> — предыдущий опыт работы был связан скорее с администрированием, нежели с кодированием;
GUI> — "знаком с языком VC" — этого может оказаться недостаточно;
GUI> — задание выполнено как "отписка", без цели показать "все, на что способен", а с целью сделать "чтобы отстали"
GUI> — совершенно неудобный UI (пользователь должен знать имена сервисов и имена машин, чтобы запустить или остановить сервис);
GUI> — многократное создание одних и тех же объектов (в VB при каждом нажатии на кнопку создается COM-объект MyServControl; в VC тоже самое происходит с объектом ServiceController);
GUI> — catch(...) — за это у нас "расстреливают";
GUI> — глобальные директивы using — это, по крайней мере, плохой стиль программирования. Кроме того, раз уж написал
GUI>

GUI>using namespace System::ServiceProcess;
GUI>

то зачем тогда дальше писать
GUI>

GUI>System::ServiceProcess::ServiceController *SC;
GUI>

?
GUI>

GUI>- человек явно стремится показать свои знания (проект выглядит крупнее и "продвинутее" среднестатистического тестового задания);
GUI> — не часто встречается в заданиях запуск и останов зависимых сервисов;
GUI> — использование обертки для SC_HANDLE;
GUI> — использование WNetAddConnection (очень редко встречается в заданиях);
GUI> — использование коллекций VB.
GUI>

GUI> — в коде не выбрасывается ни одно исключение, но встречается огромное количество конструкций

GUI>

GUI>    try {}
GUI>    catch(...) {}
GUI>


GUI> — обработка ошибок сделана на возвращаемых значениях (в качестве типа ошибки используется свой собственный enum);
GUI> — использование таких конструкций как
GUI> using CConnectionImpl::OpenConnection; (см. класс CManagerImpl) конечно говорит о том, что человек знает о их существовании, но на практике означает просчеты в проектировании (в проекте такого масштаба они явно не уместны!);
GUI> — встречаются "сишные" функции; используется CString вместо std::basic_string<> (стоит ли прикручивать целый MFC ради этого?);
GUI> — доступ к внутреннему представлению вектора (см., например, ServiceCtrl::QueryServiceConfig());
GUI> — std::vector<LPCTSTR> a_Array(a_Size); — выглядит пугающе (это отмечает в комментариях и сам автор);
GUI> — функция CManagerImpl::WaitForServiceState() содержит конструкцию

GUI>

GUI>for(;;)
GUI>    {
GUI>       ...
GUI>       Sleep(a_Wait);
GUI>    }
GUI>


GUI>и вызывается в основном потоке (фактически, прямо в обработчике сообщений от UI); как следствие, приложение "зависает", пока не запустится (не остановится) соответствующий сервис (может, это как-то реализовано в VB?).
GUI>

AS>public bool IsOk = false;
AS>