2

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

Пример использования:

getUsersInfo((users) => {
  console.log(users); // [ { name: 'Alex', age: 70 }, { name: 'Elon' } ]
});

Для получения данных вам предоставлены 2 асинхронные функции

getUsersIds - возвращает массив с идентификаторами пользователей. getUserInfo - возвращает данные пользователя по заданному идентификатору:

getUsersIds((ids) => {
  console.log(ids); // ['id2', 'id6']
});

getUserInfo('someUserId', (userInfo) => {
  console.log(userInfo); // { name: 'Alex', age: 70 }
});

Функция должна вызвать callback, переданный первым аргументом и передать туда массив данных о пользователях.

Порядок пользователей в результирующем массиве должен соответствовать порядку идентификаторов в массиве из getUsersIds

Решение должно быть следующего формата:

const { getUserInfo, getUsersIds } = db;

function getUsersInfo(onLoad) {
  // код решения
}
4
  • Жаль промисифицировать нормально не выйдет =) Коллбеки неправильные)
    – vp_arth
    19 ноя 2020 в 15:12
  • @vp_arth, почему не выйдет? просто ветки error не будет - а так все то же самое
    – Grundy
    19 ноя 2020 в 15:13
  • Нормально не выйдет — это значит нужно писать костыль вместо require('promisify')
    – vp_arth
    19 ноя 2020 в 15:14
  • 1
    @vp_arth, не костыль, а специализированное решение :-)
    – Grundy
    19 ноя 2020 в 15:14

2 ответа 2

1

Callback-hell во всей красе. :)

// const { getUserInfo, getUsersIds } = db;
// Демо функции, чтобы работало

const data = {
  id1: {name: 'Alice',   age: 62},
  id2: {name: 'Bob',     age: 28},
  id3: {name: 'Charlie', age: 10},
};
function getUsersIds(cb) {
  setTimeout(() => cb(Object.keys(data)), 100);
};
function getUserInfo(id, cb) {
  setTimeout(() => cb(data[id]), 10*data[id].age);
}


function getUsersInfo(onLoad) {
  // Начнём с получения идентификаторов
  getUsersIds(ids => {
    // Пустой массив, если пользователей нет
    if (ids.length === 0) return onLoad([]);
    
    // Подготовим массив необходимой длины
    const users = [];
    users.length = ids.length;
    // Счётчик коллбеков, нужен чтобы среагировать на последний
    let c = ids.length;

    // Перебираем идентификаторы, запускаем «асинхронные» функции
    // for (const [i, id] of Object.entries(ids)) {
    for(let i = 0; i < ids.length; ++i) {
      const id = ids[i];
      
      getUserInfo(id, user => {
        // Кладём полученного пользователя в нужное место в массиве
        users[i] = user;
        // Когда счётчик дошёл до нуля, вызываем onLoad
        if (!--c) onLoad(users);
      });
    }
  });
}

getUsersInfo(users => console.log(JSON.stringify(users)));


А теперь попробуем это облагородить)

// const { getUserInfo, getUsersIds } = db;
// Демо функции, чтобы работало
const getUsersIds = (cb) => cb([1, 2, 3]);
const getUserInfo = (id, cb) => cb({1: {name: 'Alice'}, 2: {name: 'Bob'}, 3: {name: 'Charlie'}}[id]);

// Функция, улучшающая интерфейс предоставленных функций
const promisify = fn => (...args) => 
    new Promise((resolve) => fn(...args, res => resolve(res)));

let getUsersIds$ = promisify(getUsersIds);
const getUserInfo$ = promisify(getUserInfo);

// Promise-реализация, чтобы не менять интерфейс функции, необходимой по ТЗ
async function getUsersInfo$() {
  const ids = await getUsersIds$();
  return await Promise.all(ids.map(id => getUserInfo$(id)));
}

function getUsersInfo(onLoad) {
  getUsersInfo$().then(onLoad)
}

// Вызов функций
getUsersInfo(users => console.log(JSON.stringify(users)));

setTimeout(() => {
  getUsersIds$ = () => Promise.resolve([]);
  getUsersInfo(users => console.log(JSON.stringify(users))); // []
}, 1000);

12
  • добавь пояснения в код, а то черт ногу сломит :)
    – Grundy
    19 ноя 2020 в 15:15
  • У меня самодокументированный код!! добавил)
    – vp_arth
    19 ноя 2020 в 15:18
  • плохая документация - стоит переписать :-D
    – Grundy
    19 ноя 2020 в 15:19
  • Надеюсь — это шутка про код) Потому что комментарии переписывать я не хотю)
    – vp_arth
    19 ноя 2020 в 15:28
  • Это было про "самодокументируемость" :-)
    – Grundy
    19 ноя 2020 в 15:39
0
const { getUserInfo, getUsersIds } = db;

function getUsersInfo(onLoad) {
  getUsersIds((ids) => {
    const users = [];
    let count = 0;
    ids.forEach((id, index) => {
      getUserInfo(id, (userInfo) => {
        users[index] = userInfo;
        count++;
        if (count === ids.length) {
          onLoad(users);
        }
      });
    });
  });
}
1
  • 1
    добро пожаловать на Stack Overflow на русском! пожалуйста, постарайтесь оставлять чуть более развёрнутые ответы. дополнить ответ можно, нажав править 11 апр в 17:13

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.