Альтернативное собеседование на позицию разработчика ПО

На фоне гремящих здесь на Хабре дискуссий про алгоритмические задачки и собеседования в Яндекс, вспомнил, как один мой приятель, работающий в одно довольно большой (не-российской) компании, как-то раз предлагал мне задачку с собеседований, которые проводят они. Сейчас задачка изменилась (они нашли более крутой пример кода), поэтому с его позволения, публикую старый вариант здесь.

Кандидату, приходящему на должность разработчика, не давали алгоритмических задачек, не гоняли по дебрям языка, а просто давали на ревью код. Код из реального проекта, который один раз пытался смёржить один зеленый джуниор. К счастью, вовремя заметили.

Предлагалось сделать следующее:

1. Изучить код и в двух словах предположить, что он вообще делает. (в проекте все довольно хорошо документировано, но вот у сторонних библиотек с документацией и комментариями иногда беда, и разработчикам приходится понимать, что там вообще происходит)

2. Провести code-review, указать на подозрительные и плохие места и предложить, как можно их улучшить или переделать. Можно задавать любые вопросы и гуглить все что угодно.

Код на C++, хотя большинство ошибок смогут найти даже разработчики на других языках. Код был примерно такой:

class SomeServiceClient { public:   SomeServiceClient();   virtual ~SomeServiceClient();   bool CallAsync(const std::string &uri, const std::string &param,                  const misc::BusServiceClient::ResponseCB &callback);   bool CallSync(const std::string &uri, const std::string &param,                 const misc::BusServiceClient::ResponseCB &callback);  private:   misc::BusServiceClient ss_client_;    static const int kSleepMs = 100;   static const int kSleepCountMax = 50; };  class SpecificUrlFetcher : public UrlFetcher { public:   SpecificUrlFetcher();   virtual ~SpecificUrlFetcher();    SomeData FetchData(const URL &url, const UrlFetcher::ResponseCB &callback);  private:   bool SsResponse_returnValue{false};   char SsResponse_url[1024];    void SsResponseCallback(const std::string &response);    SomeServiceClient *ss_client_; };  // ...  static const char ss_getlocalfile_uri[] = "bus://url_replace_service";  namespace net {  pthread_mutex_t g_url_change_callback_lock = PTHREAD_MUTEX_INITIALIZER;  SomeBusServiceClient::SomeBusServiceClient()     : ss_client_(misc::BusServiceClient::PrivateBus) {}  SomeBusServiceClient::~SomeBusServiceClient() {}  bool SomeBusServiceClient::CallAsync(     const std::string &uri, const std::string &param,     const misc::BusServiceClient::ResponseCB &callback) {   bool bRet;   bRet = ss_client_.callASync(uri, param, callback);   return bRet; }  bool SomeBusServiceClient::CallSync(     const std::string &uri, const std::string &param,     const misc::BusServiceClient::ResponseCB &callback) {   boold bRet bRet = false;   int counter;    pthread_mutex_lock(&g_url_change_callback_lock);   ss_client_.callASync(uri, param, callback);    counter = 0;   for (;;) {     int r = pthread_mutex_trylock(&g_url_change_callback_lock);     if (r == 0) {       bRet = true;       pthread_mutex_unlock(&g_url_change_callback_lock);     } else if (r == EBUSY) {       usleep(kSleepMs);       counter++;       if (counter >= kSleepCountMax) {         pthread_mutex_unlock(&g_url_change_callback_lock);         break;       } else         continue;     }     break;   }   return bRet; }  /**************************************************************************/  SpecificUrlFetcher::SpecificUrlFetcher() {} SpecificUrlFetcher::~SpecificUrlFetcher() {}  void SpecificUrlFetcher::SsResponseCallback(const std::string &response) {   std::unique_ptr<lib::Value> value(lib::JSONReader::Read(response));   if (!value.get() || !value->is_dict()) {     pthread_mutex_unlock(&g_url_change_callback_lock);     return;   }    lib::DictionaryValue *response_data =       static_cast<lib::DictionaryValue *>(value.get());   bool returnValue;   if (!response_data->GetBoolean("returnValue", &returnValue) || !returnValue) {     pthread_mutex_unlock(&g_url_change_callback_lock);     return;   }    std::string url;   if (!response_data->GetString("url", &url)) {     pthread_mutex_unlock(&g_url_change_callback_lock);     return;   }   SsResponse_returnValue = true;    size_t array_sz = arraysize(SsResponse_url);   strncpy(SsResponse_url, url.c_str(), array_sz);   SsResponse_url[array_sz - 1] = 0;    pthread_mutex_unlock(&g_url_change_callback_lock); }  SomeData SpecificUrlFetcher::FetchData(const URL &url,                                        const UrlFetcher::ResponseCB &callback) {   lib::DictionaryValue dictionary;   std::string ss_request_payload;    misc::BusServiceClient::ResponseCB response_cb =       lib::Bind(&SpecificUrlFetcher::SsResponseCallback, this);    SomeBusServiceClient *ss_client_ = new SomeBusServiceClient();    dictionary.SetString("url", url.to_string());   lib::JSONWriter::Write(dictionary, &ss_request_payload);    SsResponse_returnValue = false;   SsResponse_url[0] = 0x00;    ss_client_->CallSync(ss_getlocalfile_uri, ss_request_payload, response_cb);   URL new_url;   if (SsResponse_returnValue) {     new_url = URL::from_string(SsResponse_url);   }    delete ss_client_;   return UrlFetcher::FetchData(new_url, callback); }  } // namespace net

Ответы будут под спойлером, нажимайте на него осознанно, пути назад уже нет.

Ответы и замечания от кандидата позволяли составить представление о его уровне владения современными стандартами C++ и хорошими практиками, понимании асинхронности и многопоточности, дотошности на ревью и умении «отлаживать» код в голове. Ну и задать темы для дальнейшего разговора по душам.