javascript-tutorial / javascript-tutorial/en.javascript.info
Bug in solution of this task: https://javascript.info/array-methods#create-keyed-object-from-array
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
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- 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