javascript-tutorial / javascript-tutorial/en.javascript.info
Bug in solution of this task: https://javascript.info/array-methods#create-keyed-object-from-array
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- HTML
- Sterne
- 25.5k
- Forks
- 4k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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;
}, {})
}
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Öffne die Aufgabe zu Array-Methoden unter javascript.info/array-methods#create-keyed-object-from-array und vergleiche die dort angegebene Anforderung zur Gruppierung mit der veröffentlichten reduce-Lösung. Aktualisiere die dokumentierte Lösung so, dass wiederholte IDs alle gruppierten Objekte beibehalten, und überprüfe anschließend, dass Beispiel und Erklärung weiterhin zum korrigierten Ergebnis passen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- documentation
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 50/100