javascript-tutorial / javascript-tutorial/en.javascript.info

Bug in solution of this task: https://javascript.info/array-methods#create-keyed-object-from-array

Ouverte
#3,765 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
HTML
Étoiles
25.5k
Forks
4k
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

There's a bug in the solution posted in this task.
The task requires us to group objects by id and put the grouped objects in an array keyed by id.
The solution posted is:

function groupById(array) {
  return array.reduce((obj, value) => {
    obj[value.id] = value;
    return obj;
  }, {})
}

However, this does not work when there are items in array with same id.
To accomplish the task correctly, the solution should be:

function groupById(array) {
  return array.reduce((obj, value) => {
    obj[value.id] == undefined ? obj[value.id] = [value] : obj[value.id].push(value);
    return obj;
  }, {})
}

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Ouvrez la tâche sur les méthodes de tableau à l’adresse javascript.info/array-methods#create-keyed-object-from-array et comparez l’exigence de regroupement indiquée avec la solution publiée utilisant reduce. Mettez à jour la solution documentée afin que les IDs répétés conservent tous les objets regroupés, puis vérifiez que l’exemple et l’explication correspondent toujours au résultat corrigé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript
Domaine
documentation
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
50/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.