配列内で最も出現頻度の高い要素を取得します


85

JavaScript配列で最も出現頻度の高い要素(モード)を判別するためのエレガントな方法を探しています。

たとえば、

['pear', 'apple', 'orange', 'apple']

'apple'要素は、最も頻繁一つです。


このStackoverflowの質問からいくつかのアイデアを適応させることができます。stackoverflow.com/questions/840781/...
Nosredna

解決策をあまり詳しく読んでいませんが、出現回数が最も多い要素ではなく、出現回数が最も多い要素を特定するという要件に基づいて、次のニュアンス(最適化?)を組み込んでいますか?そして、そのニュアンスは、配列がループオーバーするときに、最高と2番目に高いオカレンスの差がループオーバーする残りの要素の数よりも少ないときにカウントを停止でき、ループを停止でき、現在の最高値が最高になります
Dexygen 2010

1
また、これらのソリューションは結びつきを説明していないようです。
Dexygen 2010

1
ケースでは、(少し小さい)別の解決策を探しているstackoverflow.com/questions/40410470/...
daanvanham

回答:


94

これはただのモードです。これは、最適化されていない迅速なソリューションです。O(n)である必要があります。

function mode(array)
{
    if(array.length == 0)
        return null;
    var modeMap = {};
    var maxEl = array[0], maxCount = 1;
    for(var i = 0; i < array.length; i++)
    {
        var el = array[i];
        if(modeMap[el] == null)
            modeMap[el] = 1;
        else
            modeMap[el]++;  
        if(modeMap[el] > maxCount)
        {
            maxEl = el;
            maxCount = modeMap[el];
        }
    }
    return maxEl;
}

1
いいですね...しかし、それは文字列に対してのみ機能します-必ずしも制限ではなく、考慮すべきことがあります。
James

どうもありがとう、私は完全な解決策を期待していませんでした。文字列と数字の両方で、1回のパスで機​​能します。これは非常に便利です。
バイス

1
同点を処理するために、このアルゴリズムのバージョンを追加しました。
samandmoore 2010

3
modeMap [el]がnullではなく未定義であったため、[2、3、3]を渡すときに奇妙な数が飛び込んできたため、 `f(modeMap [el] == null)をif(!modeMap [el])に置き換える必要がありました。
ナズ2014年

1
タイブレーカーを使用するのは合理的だと思います。この場合は、配列の最初に来る要素です。ただし、このアルゴリズムを簡単に変更して、それぞれをほとんどの場合に結び付けることができます。
Wylliam Judd 2017

62

2009年以降、JavaScriptにはいくつかの開発がありました。別のオプションを追加したいと思いました。それが実際に問題になるまで私は効率にあまり関心がないので、「エレガントな」コードの定義(OPで規定されている)は読みやすさを優先します-もちろん主観的です...

function mode(arr){
    return arr.sort((a,b) =>
          arr.filter(v => v===a).length
        - arr.filter(v => v===b).length
    ).pop();
}

mode(['pear', 'apple', 'orange', 'apple']); // apple

この特定の例では、セットの2つ以上の要素が同じオカレンスである場合、配列の最後に表示される要素が返されます。また、元の配列が変更されることも指摘しておく価値があります。これは、Array.slice事前に呼び出しを行うことで防ぐことができます 。


編集:2015年が発生したため、いくつかのES6 ファットアローで例を更新しました。それらはきれいに見えると思います...下位互換性に関心がある場合は、リビジョン履歴でこれを見つけることができます。


これは素晴らしい!配列内に別の項目と同じように発生する項目が複数ある場合、どのようにして複数の回答を返しますか?
クリスタル

モードが一意であると想定しているという点でナイーブです。複数を返す必要がある場合は、見た目が美しくない各アイテムの合計数を追跡する必要があります...次のようなものを試すことができます、プリミティブ値でのみ機能するという点で単純です(ただし、必要に応じてさらに調整できます)。
使者2015年

14
これがエレガントなコードでない場合、私は何であるかわかりません。関数型プログラミングの宣伝のようなものです。
サムH.

1
@GoranJakovljevic具体的に教えてください。私はそれがだ想像の機能矢印ES6 -あなたは試してみましたが、後方互換性の例を改定履歴に?
使者2016

あなたは正しいです、その矢印は機能します。はい、後方は問題なく動作します。
Goran Jakovljevic 2016

37

George Jempty'sアルゴリズムに同点を考慮させるという要求に従って、私はMatthew Flaschen'sアルゴリズムの修正バージョンを提案します。

function modeString(array) {
  if (array.length == 0) return null;

  var modeMap = {},
    maxEl = array[0],
    maxCount = 1;

  for (var i = 0; i < array.length; i++) {
    var el = array[i];

    if (modeMap[el] == null) modeMap[el] = 1;
    else modeMap[el]++;

    if (modeMap[el] > maxCount) {
      maxEl = el;
      maxCount = modeMap[el];
    } else if (modeMap[el] == maxCount) {
      maxEl += "&" + el;
      maxCount = modeMap[el];
    }
  }
  return maxEl;
}

これにより、モード要素が&記号で区切られた文字列が返されます。結果が受信されると、その&要素で分割でき、モードがあります。

別のオプションは、次のようにモード要素の配列を返すことです。

function modeArray(array) {
  if (array.length == 0) return null;
  var modeMap = {},
    maxCount = 1,
    modes = [];

  for (var i = 0; i < array.length; i++) {
    var el = array[i];

    if (modeMap[el] == null) modeMap[el] = 1;
    else modeMap[el]++;

    if (modeMap[el] > maxCount) {
      modes = [el];
      maxCount = modeMap[el];
    } else if (modeMap[el] == maxCount) {
      modes.push(el);
      maxCount = modeMap[el];
    }
  }
  return modes;
}

上記の例では、関数の結果をモードの配列として処理できるようになります。


1
2番目の例(配列1)では; 初期値として設定modesする必要はありません[array[0]]。これにより、に重複があることが確認されますmodes。これでうまくいくはずですvar modes = []
vdclouis 2015

1
これは素晴らしい!ただし、2つの異なる値を持つ配列でこれをテストすると、配列の最初の項目が2回返されます。わからないことが...起こっている理由
クリスタル

@xgriouxは、vdclouisがこのバグに対抗するために推奨する変更を加えます。つまり、[array [0]]を[]に変更します。
Dave Haigh 2016

厳密な平等を強制==する===ためにのインスタンスを変更することをお勧めします
Len Joseph

16

EmissaryのES6 +の回答に基づいて、Array.prototype.reduce(配列を並べ替えたり、ポップしたり、変更したりするのではなく)比較を行うことができます。これは非常に洗練されているように見えます。

const mode = (myArray) =>
  myArray.reduce(
    (a,b,i,arr)=>
     (arr.filter(v=>v===a).length>=arr.filter(v=>v===b).length?a:b),
    null)

私はデフォルトでnullに設定しています。これは、nullがフィルタリング対象の可能なオプションである場合、常に真実の応答を返すとは限りません。おそらく、それはオプションの2番目の引数である可能性があります。

欠点は、他のさまざまなソリューションと同様に、「描画状態」を処理しないことですが、これは、もう少し複雑なreduce関数を使用して実現できます。


14
a=['pear', 'apple', 'orange', 'apple'];
b={};
max='', maxi=0;
for(let k of a) {
  if(b[k]) b[k]++; else b[k]=1;
  if(maxi < b[k]) { max=k; maxi=b[k] }
}

これはまだO(n)ですが、不必要に2つのパスを使用します。
Matthew Flaschen 2009年

2
JavaScriptが送信されるので、小さなソリューションを見るのは常に興味深いことです。
nosredna 2009年

笑2マイナス適切な解決策;]私は不必要に2つのパスを修正し、それを速くしていましたが、それでも機能し、それでも最短の解決策です。
思想家

bへの各アクセスには少なくともlog(len(b))が必要なので、O(n)は少し楽観的かもしれません
Nicolas78 2010

nicolas78:配列が小さい場合、それは問題ではありません。したがって、プロジェクトによって異なります。
思想家

7

この関数をインタビュアーのクイズとして使用しているので、解決策を投稿します。

const highest = arr => (arr || []).reduce( ( acc, el ) => {
  acc.k[el] = acc.k[el] ? acc.k[el] + 1 : 1
  acc.max = acc.max ? acc.max < acc.k[el] ? el : acc.max : el
  return acc  
}, { k:{} }).max

const test = [0,1,2,3,4,2,3,1,0,3,2,2,2,3,3,2]
console.log(highest(test))

5

ここで宣言型アプローチを試してみてください。このソリューションは、各単語の出現を集計するためのオブジェクトを構築します。次に、各単語の出現総数をオブジェクトで見つかった最大値と比較することにより、オブジェクトを配列にフィルターします。

const arr = ['hello', 'world', 'hello', 'again'];

const tally = (acc, x) => { 

  if (! acc[x]) { 
    acc[x] = 1;
    return acc;
  } 

  acc[x] += 1;
  return acc;
};

const totals = arr.reduce(tally, {});

const keys = Object.keys(totals);

const values = keys.map(x => totals[x]);

const results = keys.filter(x => totals[x] === Math.max(...values));

あなたの答えを説明してください
ハリス2016

フィルタループで最大値を計算することを避け、keys-to-valuesマップステートメントを削除します。この答えは最もパフォーマンスが高いわけではありませんが、レデューサーでのフィルタリングほど悪くはなく、見栄えがよく読みやすいです。const maxValue = Math.max(... Object.values(totals)); const results = keys.filter(x => totals [x] === maxValue);
マイルサロン

3

別の解決策の時間:

function getMaxOccurrence(arr) {
    var o = {}, maxCount = 0, maxValue, m;
    for (var i=0, iLen=arr.length; i<iLen; i++) {
        m = arr[i];

        if (!o.hasOwnProperty(m)) {
            o[m] = 0;
        }
        ++o[m];

        if (o[m] > maxCount) {
            maxCount = o[m];
            maxValue = m;
        }
    }
    return maxValue;
}

簡潔さが重要な場合(重要ではない)、次のようになります。

function getMaxOccurrence(a) {
    var o = {}, mC = 0, mV, m;
    for (var i=0, iL=a.length; i<iL; i++) {
        m = a[i];
        o.hasOwnProperty(m)? ++o[m] : o[m] = 1;
        if (o[m] > mC) mC = o[m], mV = m;
    }
    return mV;
}

存在しないメンバー(スパース配列など)を回避する場合は、追加のhasOwnPropertyテストが必要です。

function getMaxOccurrence(a) {
    var o = {}, mC = 0, mV, m;
    for (var i=0, iL=a.length; i<iL; i++) {
        if (a.hasOwnProperty(i)) {
            m = a[i];
            o.hasOwnProperty(m)? ++o[m] : o[m] = 1;
            if (o[m] > mC) mC = o[m], mV = m;
        }
    }
    return mV;
}

getMaxOccurrence([,,,,,1,1]); // 1

ここでの他の回答は未定義を返します。


@ Jonah —簡潔さは、それ自体のために無意味であり、通常、コードの読み取りと保守を困難にします。もちろん、より冗長なコードは、長くなるだけで必ずしも良いとは限りません。しかし、これらの基準自体は、明快さや保守性などのはるかに重要な手段によって不要になります。
RobG 2015年

明らかに密集した、不可解な簡潔さは決して目標ではありません。ただし、一般に、密度がほぼ等しい同じコードの2つのバージョンを考えると、通常、短い方がより明確で優れています。それがルールだと言っているわけではありませんが、相関関係は強いです。実際、読みやすさと相関性の高い単一の指標は他にないと思います。これが、すべてのプログラマーがコードを削除するのが好きな理由です。コードレビューのほとんどの書き直しが元の書き直しよりも短いのはそのためです。
ジョナ

3

https://www.w3resource.com/javascript-exercises/javascript-array-exercise-8.phpからの別のJSソリューション

これも試すことができます:

let arr =['pear', 'apple', 'orange', 'apple'];

function findMostFrequent(arr) {
  let mf = 1;
  let m = 0;
  let item;

  for (let i = 0; i < arr.length; i++) {
    for (let j = i; j < arr.length; j++) {
      if (arr[i] == arr[j]) {
        m++;
        if (m > mf) {
          mf = m;
          item = arr[i];
        }
      }
    }
    m = 0;
  }

  return item;
}

findMostFrequent(arr); // apple

3

これは、O(n)の複雑さでそれを行う別のES6の方法です

const result = Object.entries(
    ['pear', 'apple', 'orange', 'apple'].reduce((previous, current) => {
        if (previous[current] === undefined) previous[current] = 1;
        else previous[current]++;
        return previous;
    }, {})).reduce((previous, current) => (current[1] >= previous[1] ? current : previous))[0];
console.log("Max value : " + result);

2
function mode(arr){
  return arr.reduce(function(counts,key){
    var curCount = (counts[key+''] || 0) + 1;
    counts[key+''] = curCount;
    if (curCount > counts.max) { counts.max = curCount; counts.mode = key; }
    return counts;
  }, {max:0, mode: null}).mode
}

このソリューションの問題は、「max」と「mode」という単語がマップのロジックの一部としてカウントされないことです...
Pablo

2

これがこの問題に対する私の解決策ですが、数字と新しい「設定」機能を使用しています。あまりパフォーマンスは良くありませんが、これを書くのは間違いなくとても楽しかったし、複数の最大値をサポートしています。

const mode = (arr) => [...new Set(arr)]
  .map((value) => [value, arr.filter((v) => v === value).length])
  .sort((a,b) => a[1]-b[1])
  .reverse()
  .filter((value, i, a) => a.indexOf(value) === i)
  .filter((v, i, a) => v[1] === a[0][1])
  .map((v) => v[0])

mode([1,2,3,3]) // [3]
mode([1,1,1,1,2,2,2,2,3,3,3]) // [1,2]

ちなみに、これを本番環境に使用しないでください。これは、ES6および配列関数のみを使用して問題を解決する方法を示したものにすぎません。


2

これが私の解決策です:-

function frequent(number){
    var count = 0;
    var sortedNumber = number.sort();
    var start = number[0], item;
    for(var i = 0 ;  i < sortedNumber.length; i++){
      if(start === sortedNumber[i] || sortedNumber[i] === sortedNumber[i+1]){
         item = sortedNumber[i]
      }
    }
    return item
  
}

   console.log( frequent(['pear', 'apple', 'orange', 'apple']))


2

本当に読みやすく、保守しやすいコードのために、私はこれを共有します:

function getMaxOcurrences(arr = []) {
  let item = arr[0];
  let ocurrencesMap = {};

  for (let i in arr) {
    const current = arr[i];

    if (ocurrencesMap[current]) ocurrencesMap[current]++;
    else ocurrencesMap[current] = 1;

    if (ocurrencesMap[item] < ocurrencesMap[current]) item = current;
  }

  return { 
    item: item, 
    ocurrences: ocurrencesMap[item]
  };
}

それが誰かを助けることを願っています;)!


2

このソリューションは、同点の場合に配列の複数の要素を返すことができます。たとえば、配列

arr = [ 3, 4, 3, 6, 4, ];

2つのモード値があります:36

これが解決策です。

function find_mode(arr) {
    var max = 0;
    var maxarr = [];
    var counter = [];
    var maxarr = [];

    arr.forEach(function(){
       counter.push(0);
    });

    for(var i = 0;i<arr.length;i++){
       for(var j=0;j<arr.length;j++){
            if(arr[i]==arr[j])counter[i]++; 
       }
    } 


    max=this.arrayMax(counter);   
  
    for(var i = 0;i<arr.length;i++){
         if(counter[i]==max)maxarr.push(arr[i]);
    }

    var unique = maxarr.filter( this.onlyUnique );
    return unique;

  };


function arrayMax(arr) {
      var len = arr.length, max = -Infinity;
      while (len--) {
              if (arr[len] > max) {
              max = arr[len];
              }
      }
  return max;
 };

 function onlyUnique(value, index, self) {
       return self.indexOf(value) === index;
 }

1
var mode = 0;
var c = 0;
var num = new Array();
var value = 0;
var greatest = 0;
var ct = 0;

注:ctは配列の長さです。

function getMode()
{
    for (var i = 0; i < ct; i++)
    {
        value = num[i];
        if (i != ct)
        {
            while (value == num[i + 1])
            {
                c = c + 1;
                i = i + 1;
            }
        }
        if (c > greatest)
        {
            greatest = c;
            mode = value;
        }
        c = 0;
    }
}

1
const mode = (str) => {
  return str
    .split(' ')
    .reduce((data, key) => {
      let counter = data.map[key] + 1 || 1
      data.map[key] = counter

      if (counter > data.counter) {
        data.counter = counter
        data.mode = key
      }

      return data
    }, {
      counter: 0,
      mode: null,
      map: {}
    })
    .mode
}

console.log(mode('the t-rex is the greatest of them all'))

1
function mode(array){
    var set = Array.from(new Set(array));
    var counts = set.map(a=>array.filter(b=>b==a).length);
    var indices = counts.map((a,b)=>Math.max(...counts)===a?b:0).filter(b=>b!==0);
    var mode = indices.map(a=>set[a]);
    return mode;
}

1

それも試してみてください。これはブラウザのバージョンを考慮していません。

function mode(arr){
var a = [],b = 0,occurrence;
    for(var i = 0; i < arr.length;i++){
    if(a[arr[i]] != undefined){
        a[arr[i]]++;
    }else{
        a[arr[i]] = 1;
    }
    }
    for(var key in a){
    if(a[key] > b){
        b = a[key];
        occurrence = key;
    }
    }
return occurrence;
}
alert(mode(['segunda','terça','terca','segunda','terça','segunda']));

この関数は、2つ以上のエントリが同じ回数出現した場合に、配列内の最新のオカレンスを返すことに注意してください。


1
// O(n)
var arr = [1, 2, 3, 2, 3, 3, 5, 6];
var duplicates = {};
max = '';
maxi = 0;
arr.forEach((el) => {
    duplicates[el] = duplicates[el] + 1 || 1;
  if (maxi < duplicates[el]) {
    max = el;
    maxi = duplicates[el];
  }
});
console.log(max);

1

組み込みのマップを使用した最新バージョンは次のとおりです(したがって、一意の文字列に変換できるもの以上のもので機能します)。

'use strict';

const histogram = iterable => {
    const result = new Map();

    for (const x of iterable) {
        result.set(x, (result.get(x) || 0) + 1);
    }

    return result;
};

const mostCommon = iterable => {
    let maxCount = 0;
    let maxKey;

    for (const [key, count] of histogram(iterable)) {
        if (count > maxCount) {
            maxCount = count;
            maxKey = key;
        }
    }

    return maxKey;
};

console.log(mostCommon(['pear', 'apple', 'orange', 'apple']));


0

2つのアプローチがあると思います。どちらにも利点があります。

並べ替えてからカウントまたはループスルーし、ハッシュテーブルを使用してカウントを実行します。

処理が完了すると、すべての個別の要素も含まれるため、ハッシュテーブルは便利です。ただし、数百万のアイテムがある場合、複製率が低いと、ハッシュテーブルが大量のメモリを使用することになります。ソートしてからカウントするアプローチでは、メモリフットプリントをはるかに制御できます。


0
var array = [1, 3, 6, 6, 6, 6, 7, 7, 12, 12, 17],
    c = {}, // counters
    s = []; // sortable array

for (var i=0; i<array.length; i++) {
    c[array[i]] = c[array[i]] || 0; // initialize
    c[array[i]]++;
} // count occurrences

for (var key in c) {
    s.push([key, c[key]])
} // build sortable array from counters

s.sort(function(a, b) {return b[1]-a[1];});

var firstMode = s[0][0];
console.log(firstMode);

0

あなたはこれを試すことができます:

 // using splice()   
 // get the element with the highest occurence in an array
    function mc(a) {
      var us = [], l;
      // find all the unique elements in the array
      a.forEach(function (v) {
        if (us.indexOf(v) === -1) {
          us.push(v);
        }
      });
      l = us.length;
      while (true) {
        for (var i = 0; i < l; i ++) {
          if (a.indexOf(us[i]) === -1) {
            continue;
          } else if (a.indexOf(us[i]) != -1 && a.length > 1) {
            // just delete it once at a time
            a.splice(a.indexOf(us[i]), 1);
          } else {
            // default to last one
            return a[0];
          }
        }
      }
    }

// using string.match method
function su(a) {
    var s = a.join(),
            uelms = [],
            r = {},
            l,
            i,
            m;

    a.forEach(function (v) {
        if (uelms.indexOf(v) === -1) {
            uelms.push(v);
        }
    });

    l = uelms.length;

    // use match to calculate occurance times
    for (i = 0; i < l; i ++) {
        r[uelms[i]] = s.match(new RegExp(uelms[i], 'g')).length;
    }

    m = uelms[0];
    for (var p in r) {
        if (r[p] > r[m]) {
            m = p;
        } else {
            continue;
        }
    }

    return m;
}

0

あなたはそれをO(n)の複雑さで解くことができます

var arr = [1,3,54,56,6,6,1,6];
var obj = {};

/* first convert the array in to object with unique elements and number of times each element is repeated */
for(var i = 0; i < arr.length; i++)
{
   var x = arr[i];
   if(!obj[x])
     obj[x] = 1;
   else 
     obj[x]++;
}

console.log(obj);//just for reference

/* now traverse the object to get the element */
var index = 0;
var max = 0;

for(var obIndex in obj)
{
  if(obj[obIndex] > max)
  {
    max = obj[obIndex];
    index = obIndex;
  }
}
console.log(index+" got maximum time repeated, with "+ max +" times" );

上記のコードを実行するには、Chromeコンソールにコピーして貼り付けるだけです。


0

この関数は、あらゆる種類の情報の汎用関数です。要素の出現をカウントし、出現する要素が最大の配列を返します。

function mode () {
  var arr = [].slice.call(arguments);
  if ((args.length == 1) && (typeof args[0] === "object")) {
    args = args[0].mode();
  }

  var obj = {};
  for(var i = 0; i < arr.length; i++) {
    if(obj[arr[i]] === undefined) obj[arr[i]] = 1;
    else obj[arr[i]]++;
  }

  var max = 0;
  for (w in obj) {
    if (obj[w] > max) max = obj[w];
  }

  ret_val = [];
  for (w in obj) {
    if (obj[w] == max) ret_val.push(w);
  }

  return ret_val;
}

0
function mode(){
  var input = $("input").val().split(",");
  var mode = [];
  var m = [];
  var p = [];
    for(var x = 0;x< input.length;x++){
      if(m.indexOf(input[x])==-1){
        m[m.length]=input[x];
    }}
  for(var x = 0; x< m.length;x++){
    p[x]=0;
    for(var y = 0; y<input.length;y++){
      if(input[y]==m[x]){
      p[x]++; 
 }}}
 for(var x = 0;x< p.length;x++){
   if(p[x] ==(Math.max.apply(null, p))){
     mode.push(m[x]);
 }} 
$("#output").text(mode);}

0

これが私のやり方です。私はデータの拳をグループ化しようとします。

const _ = require("underscore")

var test  = [ 1, 1, 2, 1 ];
var groupResult = _.groupBy(test, (e)=> e);

groupResultは次のようになります

{
  1: [1, 1, 1]
  2: [2] 
}

次に、配列が最も長いプロパティを見つけます

function findMax(groupResult){
   var maxArr = []
   var max;
   for(var item in groupResult){
     if(!max) { 
        max = { value:item, count: groupResult[item].length } ; 
        maxArr.push(max); 
        continue;
     }
     if(max.count < groupResult[item].length){ 
        maxArr = [];
        max = { value:item, count: groupResult[item].length }
        maxArr.push(max)
     } else if(max === groupResult[item].length)
        maxArr.push({ value:item, count: groupResult[item].length })
   }
   return maxArr;
}

完全なコードは次のようになります

const _ = require("underscore")

var test  = [ 1, 1, 2, 1 ];
var groupResult= _.groupBy(test, (e)=> e);
console.log(findMax(groupResult)[0].value);

function findMax(groupResult){
   var maxArr = []
   var max;
   for(var item in groupResult){
     if(!max) { 
        max = { value:item, count: groupResult[item].length } ; 
        maxArr.push(max); 
        continue;
     }
     if(max.count < groupResult[item].length){ 
        maxArr = [];
        max = { value:item, count: groupResult[item].length }
        maxArr.push(max)
     } else if(max === groupResult[item].length)
        maxArr.push({ value:item, count: groupResult[item].length })
   }
   return maxArr;
}

0
var cats = ['Tom','Fluffy','Tom','Bella','Chloe','Tom','Chloe'];
var counts = {};
var compare = 0;
var mostFrequent;
(function(array){
   for(var i = 0, len = array.length; i < len; i++){
       var word = array[i];

       if(counts[word] === undefined){
           counts[word] = 1;
       }else{
           counts[word] = counts[word] + 1;
       }
       if(counts[word] > compare){
             compare = counts[word];
             mostFrequent = cats[i];
       }
    }
  return mostFrequent;
})(cats);

0

ES6では、次のようにメソッドをチェーンできます。

    function findMostFrequent(arr) {
      return arr
        .reduce((acc, cur, ind, arr) => {
          if (arr.indexOf(cur) === ind) {
            return [...acc, [cur, 1]];
          } else {
            acc[acc.indexOf(acc.find(e => e[0] === cur))] = [
              cur,
              acc[acc.indexOf(acc.find(e => e[0] === cur))][1] + 1
            ];
            return acc;
          }
        }, [])
        .sort((a, b) => b[1] - a[1])
        .filter((cur, ind, arr) => cur[1] === arr[0][1])
        .map(cur => cur[0]);
    }
    
    console.log(findMostFrequent(['pear', 'apple', 'orange', 'apple']));
    console.log(findMostFrequent(['pear', 'apple', 'orange', 'apple', 'pear']));

2つの要素が同じオカレンスを持っている場合、両方を返します。そして、それはあらゆるタイプの要素で機能します。


arr変数がすでにパラメーターとして定義されているスコープ内で変数を使用しないでください。使用しているブラウザによっては、バグが発生する可能性があります。
mesqueeb

どちらarrが参照されarr.indexOf(cur)ますか?一番上のパラメータ、またはreduce内のパラメータ??
mesqueeb
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.