АГ>>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>