JS Количество вхождений в массиве и сортировка по наибольшему

Question

JS Количество вхождений в массиве и сортировка по наибольшему

1

Я пытаюсь написать функцию подсчета голосов, у меня есть массив объектов, и каждый объект имеет свойство currentVote которое имеет значение userId. Я хочу узнать, у какого пользователяID больше всего голосов. Используя lodash, это то, что я получил до сих пор:

function countVotes(players) {
  return new Promise(function(resolve, reject) {
    //votes should be empty, filled it with some sample values
    let votes = ['312139659792875521','360445341989863434', '312139659792875521','360445341989863435','1','9999999999999999999999'];
    for (let i = 0, j = players.length; i < j; i++) {
      votes.push(players[i].currentVote);
    }
    let tally = _.chain(votes)
      .countBy()
      .toPairs()
      .sortBy(1).reverse()
      .map(0)
      .value()[0];
    resolve(tally);
  });

Как я могу выбрать случайное значение, если у меня есть несколько userIds с таким же количеством голосов. На данный момент кажется, что самый маленький идентификатор всегда будет выбран?

Trax 04 нояб. 2017, в 18:07

Источник

1

Вместо того, чтобы выбрать первый, проверьте, сколько из них с таким же количеством голосов, и выберите случайный среди них.
Bergi 04 нояб. 2017, в 16:27
0

Почему ненужное использование обещания?
charlietfl 04 нояб. 2017, в 16:29
0

Использование .then () где-то еще.
Trax 04 нояб. 2017, в 16:32
0

Но почему ? Показанный код является синхронным
charlietfl 04 нояб. 2017, в 16:34
0

Получал ошибку, обещание исправило это,
Trax 04 нояб. 2017, в 16:37

Показать ещё 3 комментария

Теги:

javascript

arrays

2 ответа

1

Вы можете использовать _.shuffle и _.first. Может быть, что-то вроде:

_.chain(votes).countBy()
  .groupBy() // added this to get a list of all the ones that have matching vote counts
  .toPairs()
  .sortBy(0)
  .reverse()
  .first() // most votes
  .last() // get the user ids
  .shuffle() // randomize order
  .first() // get whatever is first
  .value()

ktilcu 04 нояб. 2017, в 14:07

Ещё вопросы

Вместо того, чтобы выбрать первый, проверьте, сколько из них с таким же количеством голосов, и выберите случайный среди них.
Почему ненужное использование обещания?
Но почему ? Показанный код является синхронным
Получал ошибку, обещание исправило это,

Edison Trutwein · Accepted Answer · 2017-11-04T15-29-00.000Z

Основываясь на вашем вопросе, я считаю, что ваш объект следующий, я использовал сортировку метода массива для сортировки массива в нисходящем порядке и взял первый элемент. Надеюсь это поможет.

let players = [{
                 "userId":1,
                 "currentVotes":2220
               },
               {
                 "userId":2,
                 "currentVotes":383830
               },
               {
                 "userId":3,
                 "currentVotes":6894740
               },
               {
                 "userId":4,
                 "currentVotes":6894740
               },
               {
                 "userId":5,
                 "currentVotes":1
               }
              ];


function getHighestVoteByRandowm(players){

    let arrSameVotes = [];

    let i = 0;
    do{
        arrSameVotes.push(players[i]);
        temp = players[i].currentVotes;
        i++;
    }while(players[i].currentVotes == temp);

    let rndTill = arrSameVotes.length - 1;
    let rndVal = Math.round(Math.random() * rndTill);

    return arrSameVotes[rndVal];

}

function sortVotes(a,b){
    return b.currentVotes - a.currentVotes;
}

let highestVotesPlayer = getHighestVoteByRandowm(players.sort(sortVotes));

console.log(highestVotesPlayer);

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