1

Есть массив объектов пользоватилей

[{
  username: 'Alice',
  status: 'online',
  lastActivity: 10
}, {
  username: 'Lucy',
  status: 'offline',
  lastActivity: 22
}, {
  username: 'Bob',
  status: 'online',
  lastActivity: 104
}]

из него нужно получить новый объект в котором перечислено, кто из пользователей онлайн или оффлайн, или если lastActivity > 10, то статус away.

{
  online: ['Alice'],
  offline: ['Lucy'],
  away: ['Bob']
}

мой код работает не правильно, помогите разобраться

function whoIsOnline(friends) {
  if (!friends) {
    return {};
  }

  const newOb = {};

  for (const us in friends) {
    if (us.status === 'online' && us.lastActivity <= 10) {
      newOb.online = us.username;
    } else if (us.status === 'offline') {
      newOb.offline = us.username;
    } else if (us.lastActivity > 10) {
      newOb.away = us.username;
    }
  }

  return newOb;
}

// Получается вот:

expect(received).toMatchObject(expected)
- Expected  - 11
+ Received  +  1

- Object {
-   "away": Array [
-     "Bob",
-   ],
-   "offline": Array [
-     "Lucy",
-   ],
-   "online": Array [
-     "Alice",
-   ],
- }
+ Object {}

1
  • У вас на первый взгляд такие недочёты: 1. Для перебора массива лучше использовать for...of, потому что for...in перебирает ключи объектов, а для массива это просто цифровые индексы; 2. При переборе вы просто присваиваете имена ключам нового объекта, а нужно создавать там массивы и добавлять имена к ним. 9 мая 2021 в 12:02

1 ответ 1

1

Попробуйте так:

const users = [{
  username: 'Alice',
  status: 'online',
  lastActivity: 10
}, {
  username: 'Lucy',
  status: 'offline',
  lastActivity: 22
}, {
  username: 'Bob1',
  status: 'online',
  lastActivity: 104
}, {
  username: 'Bob2',
  status: 'online',
  lastActivity: 105
}, {
  username: 'Bob3',
  status: 'online',
  lastActivity: 106
}];

const stats = users.reduce(
  (acc, { username, status, lastActivity }) => {
    if (status === 'online' && lastActivity > 10) status = 'away';
    acc[status] ??= [];
    acc[status].push(username);
    return acc;
  },
  {},
);

console.log(stats);

2
  • спасибо за ответ, но по Вашему коду если несколько пользователей away, в массив попадает только одно имя, я так поняла, вот ошибка:expect(received).toMatchObject(expected) - Expected - 2 + Received + 0 Object { "away": Array [ - "Lucy", - "Bob", "David", ], }
    – Таня
    10 мая 2021 в 9:36
  • @ТаняЕвдунова К сожалению, я не знаком с вашей библиотекой тестирования и не понимаю её сообщения об ошибках. Но я изменил ответ, чтобы проиллюстрировать, что в массив c away попадают все отвечающие условиям пользователи. 10 мая 2021 в 9:47

Ваш ответ

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

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