mkholodnyak / shri2015-task2

Тестовое задание в ШРИ'15. Вторая часть.

Демо

http://kholodnyak.ru/shri/africa-population

Описание ошибок

Скрипт имеет несколько проблем:

• Проблема: В функции callback отсутствует переменная request. Когда функция вызывается, она берёт это переменную из внешнего LexicalEnvironment. На момент выполнения передаваемого в getData() коллбэка внешний цикл уже завершился => последнее значение переменной response == '/populations'. В результате все переданные в getData() коллбэки будут использовать одно и то же значение response ( == '/populations'), поэтому результат выполнения отличается от ожидаемого.

  Решение: Исправить код можно, "поймав" текущее значение response с помощью дополнительной внешней функции (в ней response является аргументом). В своей реализации я назвал эту функцию callbackGenerator.

  var callback = (function (request) {
      // Далее идёт изначальный код из условия.
      return function (error, result) {
          responses[request] = result;
          ...
      };
  })(request);

  Остальные ошибки и неточности не влияют на выполнение исходного кода, но их всё равно желательно исправить.

• Если использовать переменную без объявления через var, она попадет в глобальную область видимости (добавляется свойством в объект window/global/etc.) или выдаст ReferenceError в strict mode. Необходимо всегда объявлять переменные с помощью var/let (ES6)

• Нет обработки переданной в коллбэк ошибки.

  function (error, result) { ... }

• В функции getData() при вызове setTimeout() вторым параметром передается NaN. Исправить можно, добавив скобки функции Math.random().

• Неинтуитивные имена переменных, магические цифры, большая вложенность, именование констант lowercase, отсутствие декомпозиции.

Условие задания

Существует API, которое умеет отвечать по трём URL: /countries, /cities и /populations. Клиентское приложение подсчитывает численность населения в Африке. Запросы друг от друга не зависят. Чтобы браузер пользователя не простаивал, клиентскому приложению важно уметь делать все три запроса одновременно. Реализацией API является функция getData(url, callback), которая принимает строку с URL запроса и функцию обратного вызова. В случае ошибки в callback первым аргументом будет передана строка ошибки, в случае успеха вторым аргументом будет передан ответ API.

Вам досталась реализация клиентского приложения, которое должно решать описанную выше задачу. Но в коде приложения есть ошибки, из-за которых фактический результат работы отличается от ожидаемого.

Смотри. код здесь

Как должно быть: приложение выводит в консоль суммарную популяцию в Африке. Как на самом деле: приложение не выводит в консоль ничего.