Liczenie wystąpień wartości w tabeli

Question

Cześć, mam taki kod z stacka:

function f(a1, a2) {
    let arr = [];
    for(let i = 0; i < a1.length; i++) {
      let count = 0;
      for(let z = 0; z < a2.length; z++) {
        if (a2[z] === a1[i]) count++;
      }
      arr.push({id:a1[i], q:count, u:"+"});
      //console.log(a1[i] + ' count is ' + count) 
    }
    return arr;
}

Dodałem to arr.push. w tablicy arr chce przechowywać dane w taki sposób
{liczba: 0, ile_powtorzen:3}

W tym momencie użycie funkcji z takimi danymi:
[0,0,0,1,0,0,0]
zwraca
0: {id: 0, q: 6, u: "+"}
1: {id: 0, q: 6, u: "+"}
2: {id: 1, q: 1, u: "+"}
3: {id: 0, q: 6, u: "+"}
4: {id: 0, q: 6, u: "+"}
5: {id: 0, q: 6, u: "+"}
6: {id: 0, q: 6, u: "+"}

zamiast
0: {id: 0, q: 6, u: "+"}
1: {id: 1, q: 1, u: "+"}

Wiem czemu tak się dzieje, ale nie umiem poprawić tej funkcji. Może ktoś pomóc?

Comments

Funkcja przyjmuje dwa argumenty: a1 i a2. Ty podałeś jeden parametr: [0,0,0,1,0,0,0], czym jest drugi?

Na pierwszy rzut oka brakuje jakiegoś if sprawdzającego, czy obiekt ze statystyką dla konkretnego id już jest w wynikowej tablicy.

---

Podaje dwa razy te same dane
f([0,0,0,1,0,0,0], [0,0,0,1,0,0,0]);

Chodzi o to, żeby ta funkcja mi wypisała powtórzenia z tej tablicy

Pierwszy argument to liczby, które chce szukać, a drugi to tablica, która ma zostać przeszukana

---

Ta funkcja jest stąd:
https://stackoverflow.com/a/5668116/11755974

---

Okej, przerobiłem inną funkcję z tego wątku i teraz działa
 

function foo(arr) {
    var newArr = [];
    var a = [], b = [], prev;
    
    arr.sort();
    for (let i=0;i<arr.length;i++) {
        if ( arr[i] !== prev ) {
            a.push(arr[i]);
            b.push(1);
        } else {
            b[b.length-1]++;
        }
        prev = arr[i];
    }

    for(let i=0;i<a.length;i++) {
        newArr.push({
            id: a[i],
            q: b[i],
            u: "+"
        });
    }
    
    return newArr;
}

Z pewnością dałoby się to lepiej napisać, ale niestety nie mam już chęci z tym walczyć, a jest to raczej mało znaczący fragment aplikacji

Answer 1

var getObjectIndex = function (output, input, i) {
    return output.findIndex((obj) => obj.id === input[i]);
}

function f(a1, a2) {
    let arr = [];
    for(let i = 0; i < a1.length; i++) {
      let count = 0;
      for(let z = 0; z < a2.length; z++) {
        if (a2[z] === a1[i]) count++;
      }
      
      var stat = {id:a1[i], q:count, u:"+"};

      if (getObjectIndex(arr, a1, i) === -1) {
		arr.push(stat);
      } else {
		arr[getObjectIndex(arr, a1, i)].q = count
      }
    }

    return arr;
}

Przed wrzuceniem nowego obiektu do tablicy powinieneś sprawdzić, czy znajduje się już tam obiekt z konkretnym id i w takim przypadku tylko zaktualizować property q. W przeciwnym razie dodajesz cały obiekt do tablicy.

Comments

Dzięki działa :)